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

<thead />ではなく<th />にスタイルを当てる #48

Merged
merged 1 commit into from
Jun 1, 2020

Conversation

youchann
Copy link
Contributor

@youchann youchann commented Jun 1, 2020

position: stickyを当てる事ができる<th />にbackgroundやborderを当てる事で、利用者側が見出し行の固定スクロールを実装できるようにする。

@youchann youchann self-assigned this Jun 1, 2020
Copy link
Contributor Author

@youchann youchann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#34 (comment)

あたりをみつつstorybookで確認すると良いかもです。

width: ${({ width }) => width};
padding: ${({ theme }) => theme.spacing * 2}px
${({ theme }) => theme.spacing * 3}px;
border-bottom: ${Size.Border.Small} solid
${({ theme }) => theme.palette.gray.light};
`;

const HeaderCell = styled.th<CellProps>`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この部分は使われていないっぽいのでけしました。

@youchann youchann requested a review from kinokoruumu June 1, 2020 01:50
@kinokoruumu
Copy link
Contributor

position: stickyを当てる事ができる<th />にbackgroundやborderを当てる事で、利用者側が見出し行の固定スクロールを実装できるようにする。

これそもそも<tr/>の一つ目だけを<th/>にする場合とかもあるので、thでスタイリングしないとダメだね

Copy link
Contributor

@kinokoruumu kinokoruumu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです!

@youchann youchann merged commit 4ac33f5 into master Jun 1, 2020
@youchann youchann deleted the tweak-table branch June 1, 2020 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants