-
-
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
Refine content and appearance of the project 'Configure' screen #9734
Conversation
Co-authored-by: Tim Jacomb <[email protected]>
/label web-ui,rfe /reviewer @jenkinsci/sig-ux |
This comment was marked as outdated.
This comment was marked as outdated.
Done :) |
Please take a moment and address the merge conflicts of your pull request. 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.
LGTM - could you file a corresponding PR to pipeline please for consistency?
@janfaracik ^^ |
Heya, I'm on it. |
/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! |
This pull request updates the content and minor styling of the project 'Configure' screen to provide descriptions for each section, fix the sidebar for Pipeline projects, and decrease section padding a little. This is part of an ongoing effort to simplify Jenkins' many forms.
Changes
Before
After
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