-
-
Notifications
You must be signed in to change notification settings - Fork 79k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add .sticky-thead
for sticky table headers on scroll
#36125
base: main
Are you sure you want to change the base?
Conversation
mdo
commented
Apr 7, 2022
- Replaces Added a class for a new feature to fix table header on top #29456
- Fixes Thead Fixed #35998
- Fixes Feature request: Scrollable table body with fixed header #13058, fixes Consider adding class table-fixedheader #12780, fixes Static <thead> on Long Tables #1804 (lol)
22a0ece
to
6d47d6d
Compare
.thead-sticky { | ||
position: sticky; | ||
top: var(--#{$prefix}table-thead-sticky-top, 0); | ||
z-index: $table-thead-sticky-zindex; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We tried to add:
> * > th {
$border-bottom-thead: subtract(100%, ($table-border-width * 2));
background: linear-gradient(transparent 0%, transparent $border-bottom-thead, var(--#{$prefix}table-border-color) $border-bottom-thead, var(--#{$prefix}table-border-color) 100%);
background-color: var(--#{$prefix}table-bg);
}
to "redraw" the separator between thead
and tbody
, I don't know if this could be useful here (instead of having a utility to change bg)(preview)?
What speaks against merging this? |
@mdo When can this merge be approved? |
85f0fff
to
65d976c
Compare
Just FYI, I've rebased and resolved the conflicts for this branch so that it's ready to be reviewed.
For now, it's planned for the v5.4 as it's a new feature, and we're currently working on some patches for v5.3.x. |
Kindly add this feature, i need it for my project, |