Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

组件【Table 表格】样式优化 #189

Closed
eugenewoody opened this issue Aug 22, 2022 · 4 comments
Closed

组件【Table 表格】样式优化 #189

eugenewoody opened this issue Aug 22, 2022 · 4 comments
Labels
help wanted Extra attention is needed issueshoot

Comments

@eugenewoody
Copy link

eugenewoody commented Aug 22, 2022

这个功能解决了什么问题

解决开发样式和设计样式没有对齐的问题(备注:本次修改建议在【comment 评论】修改完成之后进行)

你建议的方案是什么

一、【组件Table 表格】codesign地址:https://codesign.qq.com/s/ALwE9VKLvA0X1Dp
访问密码:CUOA
二、【组件Table 表格】figma地址源文件:https://www.figma.com/file/KInDIZzoxifaKVQ3n5KB8W/TDesign-for-web?node-id=46868%3A353120
三、具体修改细节如下(详见图片):
table

@xiaosansiji xiaosansiji added help wanted Extra attention is needed issueshoot labels Aug 23, 2022
@github-actions
Copy link

任何人都可以处理此问题。
请务必在您的 pull request 中引用此问题。
感谢你的贡献! ✨

@zhangpaopao0609
Copy link
Collaborator

处理中

@zhangpaopao0609
Copy link
Collaborator

zhangpaopao0609 commented Aug 24, 2022

  1. 第一个已修改,第二个 padding 8 是由 size small 导致的

  2. 这个圆角是不是考虑一下,感觉有个圆角没啥毛病

  3. 想确认一下,现在是用阴影来实现的,确定要修改成 border 吗

  4. 已修改

  5. 已修改

  6. 首先这里的按钮使用的是 button 组件,所以无法更改,同时,这不是组件应该提供的能力

  7. 这里因为button 直接使用的是 dialog 的默认 button,如果要修改的话,需要动到 useColumnController,请确认是否要更改
    image

  8. 已修改

  9. 不是很明白意思,待确认

  10. 第一个是用户设置的,由用户控制,第二个已修改

  11. 宽度为用户自己设置的

  12. 已修改

  13. 不易修复,最好交给熟悉 table 的大佬来修改

  14. 已修改

  15. 已修改

@zhangpaopao0609
Copy link
Collaborator

经确认
2. 保留圆角
3. 修改成 border
7. 不修改
9. 按照设计稿修复即可

@xiaosansiji xiaosansiji moved this to 完成开发 in TDesign Sep 2, 2022
Repository owner moved this from 完成开发 to 已发布 in TDesign Nov 30, 2022
@uyarn uyarn removed this from TDesign Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed issueshoot
Projects
None yet
Development

No branches or pull requests

4 participants