Skip to content
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

Merged
merged 7 commits into from
Jan 31, 2025

Conversation

janfaracik
Copy link
Contributor

@janfaracik janfaracik commented Jan 22, 2025

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.

image

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
image

After
image


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.

View comparison

View comparison

View comparison


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

  • Compared each page on Design Library, changes are expected/no breakages

Proposed changelog entries

  • Improve accessibility and clean up components

Proposed upgrade guidelines

N/A

Submitter checklist

Preview Give feedback

Desired reviewers

@jenkinsci/sig-ux

Before the changes are marked as ready-for-merge:

Maintainer checklist

Preview Give feedback

@janfaracik janfaracik marked this pull request as ready for review January 22, 2025 15:41
@janfaracik janfaracik requested a review from a team January 22, 2025 15:41
@janfaracik janfaracik added web-ui The PR includes WebUI changes which may need special expertise rfe For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted labels Jan 22, 2025
@krisstern krisstern requested a review from a team January 23, 2025 01:52
Copy link

@A1exKH A1exKH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Member

@timja timja left a 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!

@comment-ops-bot comment-ops-bot bot added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Jan 30, 2025
Copy link
Member

@timja timja left a 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!

@krisstern krisstern merged commit d03a2e1 into jenkinsci:master Jan 31, 2025
16 checks passed
@janfaracik janfaracik deleted the improve-accessibility branch February 1, 2025 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback rfe For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted web-ui The PR includes WebUI changes which may need special expertise
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants