tdesign-react icon indicating copy to clipboard operation
tdesign-react copied to clipboard

fix(Table): 修复展开收起斑马纹样式错误

Open Ali-ovo opened this issue 1 year ago • 6 comments

🤔 这个 PR 的性质是?

  • [x] 日常 bug 修复
  • [ ] 新特性提交
  • [ ] 文档改进
  • [ ] 演示代码改进
  • [x] 组件样式/交互改进
  • [ ] CI/CD 改进
  • [ ] 重构
  • [ ] 代码风格优化
  • [ ] 测试用例
  • [ ] 分支合并
  • [ ] 其他

🔗 相关 Issue

fix: #2669

💡 需求背景和解决方案

lock className and style 添加了一个默认的 tr className 更方便开发时进行样式覆盖

📝 更新日志

  • fix(Table): 修复斑马纹效果的样式跳动问题

  • [ ] 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

  • [x] 文档已补充或无须补充
  • [x] 代码演示已提供或无须提供
  • [x] TypeScript 定义已补充或无须补充
  • [x] Changelog 已提供或无须提供

Ali-ovo avatar Mar 29 '24 06:03 Ali-ovo

失败

github-actions[bot] avatar Mar 29 '24 06:03 github-actions[bot]

@uyarn @carolin913 请问 src/_common里面的less文件要怎么提交了 我看好像普通提交不会提上来里面改动的代码

Ali-ovo avatar Apr 08 '24 02:04 Ali-ovo

@uyarn @carolin913 请问 src/_common里面的less文件要怎么提交了 我看好像普通提交不会提上来里面改动的代码

需要到https://github.com/Tencent/tdesign-common 提交PR

uyarn avatar Apr 08 '24 02:04 uyarn

@uyarn @carolin913 请问 src/_common里面的less文件要怎么提交了 我看好像普通提交不会提上来里面改动的代码

需要到https://github.com/Tencent/tdesign-common 提交PR

看了一下这个bug 我的方案是根据 data 的 index 来给每行 table 的 tr 添加 even\odd 的 class 类名 但是如果改的话需要涉及到 common 里面的样式 但是这个样式又是公共的 可能 vue 或者其他用到公共样式的组件都要更改

Ali-ovo avatar Apr 08 '24 03:04 Ali-ovo

@uyarn @carolin913 请问 src/_common里面的less文件要怎么提交了 我看好像普通提交不会提上来里面改动的代码

需要到https://github.com/Tencent/tdesign-common 提交PR

看了一下这个bug 我的方案是根据 data 的 index 来给每行 table 的 tr 添加 even\odd 的 class 类名 但是如果改的话需要涉及到 common 里面的样式 但是这个样式又是公共的 可能 vue 或者其他用到公共样式的组件都要更改

这周review下你这个PR再跟你确认要不要改~

uyarn avatar Apr 08 '24 03:04 uyarn

@uyarn @carolin913 请问 src/_common里面的less文件要怎么提交了 我看好像普通提交不会提上来里面改动的代码

需要到https://github.com/Tencent/tdesign-common 提交PR

看了一下这个bug 我的方案是根据 data 的 index 来给每行 table 的 tr 添加 even\odd 的 class 类名 但是如果改的话需要涉及到 common 里面的样式 但是这个样式又是公共的 可能 vue 或者其他用到公共样式的组件都要更改

这周review下你这个PR再跟你确认要不要改~

嗯嗯 我先把 common 的样式文件改动提交一下 我看 vue 那边的样式也是有这个问题的,但是如果兼容我新加的这个样式名字的话就可以一起修复

Ali-ovo avatar Apr 08 '24 03:04 Ali-ovo