-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
[JENKINS-73695] Prevent unnecessary horizontal scrollbar in Firefox #9695
Conversation
Signed-off-by: Thorsten Scherler <[email protected]>
Follow up for #9667 which produced a horizontal scrollbar in FF |
You could remove the top margin from .jenkins-icon-size {
display: flex;
align-items: center;
justify-content: space-between;
margin: 0 0 var(--section-padding) 0;
...
} |
@janfaracik just tried that but it did not work in FF |
Seems like it's the |
Signed-off-by: Thorsten Scherler <[email protected]>
@janfaracik added the media query now instead of reducing the width |
Co-authored-by: Jan Faracik <[email protected]>
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 PR is now ready for merge. We will merge it after approximately 24 hours if there is no negative feedback.
/label ready-for-merge
This should have a different Jira issue than the original issue so this can be backported into LTS. |
See JENKINS-73695.
Testing done
Interactive tests with Firefox
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
N/A
Before the changes are marked as
ready-for-merge
:Maintainer checklist