-
Notifications
You must be signed in to change notification settings - Fork 9.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
report: fix flicker in normal case report #9955
Conversation
Thanks for the PR! This needs to be sticky for devtools (#9023). Could you add a sticky declaration for
So we can get this nice fix for the regular report, but further work will be needed to fix this stutter in DevTools. For why this is different in DevTools: the report is within a containing element. Normally, the report is the entire document, so https://stackoverflow.com/a/48355127/2788187
|
@connorjclark Thanks! i changed it. :) |
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.
thanks!
Summary
Bugfix
change
.lh-sticky-header
's positionsticky
tofixed
.Related Issues/PRs
#9141