Add experimental Plugin Manager UI#11332
Conversation
| }, 5000); | ||
| } | ||
|
|
||
| window.scrollTo(0, 10000); |
There was a problem hiding this comment.
This results in weird scrolling behaviour every time the page loads, don't see a great reason to keep it for either legacy/experimental view.
| <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" | ||
| xmlns:t="/lib/hudson" xmlns:f="/lib/form"> | ||
| <l:layout title="${%Advanced settings} - ${%Plugins}" permission="${app.SYSTEM_READ}"> | ||
| <l:settings-subpage permission="${app.SYSTEM_READ}" header="${null}" includeSidepanel="true"> |
There was a problem hiding this comment.
Recommend ignoring whitespace when reviewing this.
timja
left a comment
There was a problem hiding this comment.
I've given this a good test, nothing major
feedback below, looks great though
|
Please take a moment and address the merge conflicts of your pull request. Thanks! |
timja
left a comment
There was a problem hiding this comment.
LGTM, breadcrumb can come later
|
Hey @mawinter69, any thoughts on this? |
|
/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 PR adds support for the new experimental Manage Jenkins UI for Plugins and its subpages.
Screenshots
More screenshots
Changes
Testing done
Proposed changelog entries
Proposed changelog category
/label web-ui,rfe
Proposed upgrade guidelines
N/A
Submitter checklist
@Restrictedor have@since TODOJavadocs, as appropriate.@Deprecated(since = "TODO")or@Deprecated(forRemoval = true, since = "TODO"), if applicable.evalto ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@jenkinsci/sig-ux
Before the changes are marked as
ready-for-merge:Maintainer checklist
upgrade-guide-neededlabel is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidateto be considered (see query).