-
-
Notifications
You must be signed in to change notification settings - Fork 1.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 sticky-file-header
on commits
#5717
Conversation
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.
Seems right. Can you also hotfix it?
That component has no shadow indeed. The shadow on PRs comes from a standalone component .pr-toolbar.is-stuck {
box-shadow: 0 2px 2px rgba(0, 0, 0, 0.075);
border-bottom: 1px solid rgba(0,0,0,.15)
} |
This is still not fixed for me on 22.7.11. |
Never mind, I'm still on 22.6.25. Firefox was displaying the version as 22.7.11 before restarting, something wrong with hotfixing maybe? |
Broken by GitHub's Pull Request File Tree feature added a while ago.
Test URLs
1d9dd9f
Screenshot
Screen.Recording.2022-06-23.at.01.45.58.mov
I would really like to add kidonng/cherry@997ccdf as well, since that's pretty much the only place left with no sticky file header (and technically speaking, GitHub already enabled it for markup i.e. Markdown files).
P.S. it's funny GitHUb pushed a bug when I'm working on this