-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(files): selected files actions position on scroll/with readme block #43172
fix(files): selected files actions position on scroll/with readme block #43172
Conversation
/backport 7a63327 to stable28 |
|
||
&.files-list__table--with-thead-overlay { | ||
// Hide the table header below the overlay | ||
margin-top: calc(-1 * var(--row-height)); |
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.
clever!
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.
This is the place I dislike the most in mi solution 🙈
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.
Code make sense! Thanks!!
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
513d329
to
e90e136
Compare
Rebased onto master, recompiled assets |
/backport f01e169 to stable28 |
Summary
absolute
->sticky
- Cannot say what correcting position via negative margin is a beautiful solution, but I have no more ideas...
Screenshots
🏚️ Before
before-actions.mp4
🏡 After
after-actions.mp4
Also works on small screen (a11y size)
Checklist