-
-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
Improve accessibility and clean up components #10198
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.
LGTM.
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.
Tested locally LGTM
/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!
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.
Tested locally LGTM
/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!
Starting this with an apology for the amount of SCSS changes, it's a little painful to read, the majority of the changes are swapping to variables or cleaning things up. If splitting this out further is necessary (such as I've done for #10182 #10176 #10175 #10174) I'm happy to do so.
This PR aims to improve accessibility of our components and also tidies them up a little. This is quite vague so I'll do my best to demonstrate with pictures:
Controls now have a subtle border to improve contrast:
This isn't major, but it should help contrast a little bit across Jenkins. Components now lift off the page a little more than they used to, and this is applied consistently across buttons, menus and more.
The 'Refresh' button on the 'Plugin updates' page, note the button and the banners consistent borders.
Control heights are now consistent at 38px, focus states are more compact:
For example, a side panel task link is the same height as a button, which is the same height as a search bar. This keeps things nice and tidy, and keeps app bars consistent in height across pages.
Before
After
Prefers increase contrast is now respected in Jenkins:
When enabled the vast majority of components will show a hard, contrasty border that separates them from their backgrounds.
Button classes have been cleaned up, making use of relative color syntax and variables:
Inspired by recent works in the buttons file, this change aims to make such changes easier in the future and greatly simplify one of our most used components.
Testing done
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@jenkinsci/sig-ux
Before the changes are marked as
ready-for-merge
:Maintainer checklist