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
Avoid scrollbar in dropdown popups (page footer, log recorder) #8704
Conversation
Obviously it's OK to not have this in the changelog as it doesn't affect functionality, I filled the "proposed changelog" entry just in case. |
The same problem can be seen in the menu of log recorders: |
I think the reason for the negative margin is to extend the separator to the edges when it is wrapped in a |
b133438
to
148d8a8
Compare
@mawinter69 thanks, I changed the code to use the same wrapping div in all usages of |
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.
/label ready-for-merge
This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback.
Thanks!
Adds a wrapping
<div>
to have consistent paddingwith other dropdowns (e.g. the ones in sidebar) -- that avoids extra scrollbar when a separator is present. The diff makes more sense without whitespace changes.Before:
After:
Footer popup introduced in this PR: #7989
Testing done
Screenshot above.
Proposed changelog entries
Avoid scrollbar in the version popup (page footer)
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@janfaracik
Before the changes are marked as
ready-for-merge
:Maintainer checklist
upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).