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

Update UI to use new Jenkins Design Library #171

Merged

Conversation

grogersxyz
Copy link
Contributor

The claims plugin was using older table formatting and the original Jenkins styles. This pull request includes changes to UI styles using elements from the Jenkins Design Library.

This is my first time making UI changes to a Jenkins plugin so happy to recieve any feedback you have.

Screenshots of the new UI have been provided in the docs folder. The major changes are to the way the correct build result icon is retrieved, and a margin was added to the badge to create space between test names and the claim badge. This was tested against Jenkins 2.361.1

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

Geordie Rogers added 2 commits September 19, 2022 21:19
The original claim icon is now replaced by the built in Jenkins Design
Library user icon.

Build Resuly icons are now rendered using the same flow as in the
jenkinsci/jenkins repository.
The images in the README have now been updated to show the new UI styles
from the Jenkins Design Library.
@Greybird Greybird self-requested a review September 20, 2022 06:56
@Greybird
Copy link
Contributor

Greybird commented Oct 2, 2022

Thanks for this work on the UI.
I just updated the PR to use a symbol from font awesome plugin, which I think is better than the blank user one from core.
Your proposal made this quite easy!

@Greybird Greybird merged commit 3a4f047 into jenkinsci:master Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants