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 第一行 hover 时 background 带圆角 #4373

Closed
jaredleechn opened this issue Dec 27, 2016 · 8 comments
Closed

table 第一行 hover 时 background 带圆角 #4373

jaredleechn opened this issue Dec 27, 2016 · 8 comments
Assignees

Comments

@jaredleechn
Copy link
Contributor

jaredleechn commented Dec 27, 2016

重新方法:<Table style={{ background: 'pink'}} columns={columns} title={data => data.length} dataSource={data} showHeader={false} />

如图,圆角应该是属于 title 的,但是分配了个第一行记录

期望:hover 时第一行 bg 没有圆角

其他信息:如果设置了 bordered 就不会有这个问题

image

BTW,能否考虑把 padding 开放出来,现在的设计稿上基本上头部和实际内容都有 padding 来包裹,分别用 render 来做总感觉很麻烦,每一个用 table 的地方都需要定义一次

@RaoHai RaoHai self-assigned this Dec 27, 2016
@RaoHai
Copy link
Contributor

RaoHai commented Dec 27, 2016

这个比较纠结。。圆角是属于 table 的,而 title 并不在 table 里。。

@RaoHai
Copy link
Contributor

RaoHai commented Dec 27, 2016

好像就是这么设计的:4b2e9bb

@jaredleechn
Copy link
Contributor Author

@afc163 这个是 by design 的吗?

bordered 和 非 border 在使用场景上有什么差异吗?

@afc163
Copy link
Member

afc163 commented Dec 27, 2016

目前 bordered 也是第一列圆角。

image

@afc163 afc163 assigned afc163 and unassigned RaoHai Dec 27, 2016
@jaredleechn
Copy link
Contributor Author

在有 title 和 bordered 的情况下,是 title 圆角

image

@afc163
Copy link
Member

afc163 commented Dec 27, 2016

哦是的,因为可视部分需要是圆角。bordered={false} 时,title 可以看做是表格外了。

@afc163 afc163 closed this as completed in 668c481 Dec 27, 2016
@afc163
Copy link
Member

afc163 commented Dec 27, 2016

没 header 时,第一列不再是圆角。

@lock
Copy link

lock bot commented May 2, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators May 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants