Fix shutdown banner visibility on Plugin Manager pages#26234
Merged
MarkEWaite merged 3 commits intojenkinsci:masterfrom Feb 2, 2026
Merged
Conversation
The shutdown banner was being obscured by the sticky app bar's backdrop blur effect on the Installed, Available, and Updates pages. Root cause: The sticky app bar (introduced in c42ab43) uses pseudo-elements with backdrop-filter: blur(10px) that extend upwards (negative top inset). These pseudo-elements have z-index: -1 relative to the sticky bar (which has z-index: 2), but the shutdown banner had no explicit z-index (defaulting to 0). Fix: Add position: relative and z-index: 5 to #shutdown-msg and #safe-restart-msg to ensure they render above the sticky app bar's backdrop effect. This is a minimal fix that preserves the header redesign while ensuring critical system messages remain visible. Fixes jenkinsci#26022
timja
approved these changes
Jan 31, 2026
Member
timja
left a comment
There was a problem hiding this comment.
Thanks! I tested and also pushed a fix for it not working on the experimental dashboard page (which had a z-index of 102).
janfaracik
approved these changes
Jan 31, 2026
Member
janfaracik
left a comment
There was a problem hiding this comment.
Tested, working fine for me. Thanks!
Member
|
/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! |
14 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Fixes the shutdown banner being obscured by the sticky app bar's backdrop blur effect on the Installed, Available, and Updates pages of the Plugin Manager.
Root Cause
The sticky app bar (introduced in c42ab43) uses pseudo-elements with
backdrop-filter: blur(10px)that extend upwards (negative top inset). These pseudo-elements havez-index: -1relative to the sticky bar (which hasz-index: 2), but the shutdown banner had no explicit z-index (defaulting to 0), causing it to be visually obscured.Solution
Added
position: relativeandz-index: 5to#shutdown-msgand#safe-restart-msginsrc/main/scss/base/_style.scssto ensure they render above the sticky app bar's backdrop effect.This is a minimal fix (4 lines of CSS) that preserves the header redesign while ensuring critical system messages remain visible.
Fixes #26022
Closes #26024
Fixes #25949
Closes #26234
Testing done
git bisect(c42ab43)mvn -am -pl war,bom -Pquick-build clean installmvn -pl war jetty:runScreenshots (UI changes only)
Before
The shutdown banner is obscured by a gradient/blur effect on the Installed, Available, and Updates pages (as shown in issue #26022).

After
The shutdown banner is now clearly visible on all 4 Plugin Manager pages, appearing above the sticky search bar.

Proposed changelog entries
Proposed changelog category
/label bug
Proposed upgrade guidelines
N/A
Submitter checklist
@Restrictedor have@since TODOJavadocs, as appropriate. (N/A - CSS only change)@Deprecated(since = "TODO")or@Deprecated(forRemoval = true, since = "TODO"), if applicable. (N/A)evalto ease future introduction of Content Security Policy (CSP) directives. (N/A - CSS only change)Desired reviewers
@MarkEWaite @timja