Skip to content

Fix minimum position of sticky sidebar in run tab - #831

Merged
uhafner merged 1 commit into
jenkinsci:mainfrom
janfaracik:individual-scrollers
Sep 29, 2026
Merged

uhafner merged 1 commit into
jenkinsci:mainfrom
janfaracik:individual-scrollers

Conversation

@janfaracik

@janfaracik janfaracik commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Testing done

Previously the sticky sidepanel would stick under the top bar, this PR fixes that. This isn't the cleanest solution, but for the time being works, I'd like to redo how we do sidepanels in these views.

See jenkinsci/jenkins#27380

Before

Screenshot 2026-09-29 at 08 19 41

After

Screenshot 2026-09-29 at 08 19 24

Submitter checklist

  • 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 demonstrate the feature works or the issue is fixed

@github-actions
github-actions Bot requested a review from uhafner September 29, 2026 07:19
@github-actions

Copy link
Copy Markdown

☀️   Quality Monitor

Tests

   JUnit   Unit Tests: ✅ successful — 325 passed $\textsf{(±0)}$
   🚀   Integration Tests: ✅ successful — 176 passed $\textsf{(±0)}$, 7 skipped $\textsf{(±0)}$
   ⛔   Architecture Tests: ✅ successful — 15 passed $\textsf{(±0)}$

Coverage for New Code

   〰️   Line Coverage: 100.00% — perfect 🎉
   ➰   Branch Coverage: 100.00% — perfect 🎉

Coverage for Whole Project

   〰️   Line Coverage: 80.80% $\textsf{(±0)}$ — 676 missed lines
   ➰   Branch Coverage: 72.28% $\textsf{(±0)}$ — 273 missed branches

Style

   CheckStyle   CheckStyle: No warnings $\textsf{(±0)}$
   PMD   PMD: No warnings $\textsf{(±0)}$
   ☕   Java Compiler: No warnings $\textsf{(±0)}$

Bugs

   SpotBugs   SpotBugs: No bugs $\textsf{(±0)}$
   🐛   Error Prone: No bugs $\textsf{(±0)}$

Vulnerabilities

   🛡️   OWASP Dependency Check: 280 vulnerabilities $\color{green}{\textsf{(+280)}}$ — error: 25, high: 77, normal: 167, low: 11

Software Metrics

   🌀   Cyclomatic Complexity: 1208 (total)
   💭   Cognitive Complexity: 614 (total)
   ➿   N-Path Complexity: 1594 (total)
   📏   Lines of Code: 13953 (total)
   📝   Non Commenting Source Statements: 4634 (total)
   🔗   Class Cohesion: 100.00% (maximum)
   ⚖️   Weight of Class: 100.00% (maximum)

📌 Reference Results

Delta reports computed against the reference results of 5294182 in workflow run 36104731019.

🚦 Quality Gates

Overall Status: ✅ SUCCESS

✅ Passed Gates

  • ✅ Overall Tests Success Rate: 100.00 >= 100.00
  • ✅ Line Coverage in New Code: 100.00 >= 90.00
  • ✅ Branch Coverage in New Code: 100.00 >= 90.00
  • ✅ Potential Bugs in Whole Project: 0.00 <= 0.00
  • ✅ Style Violation in Whole Project: 0.00 <= 0.00

Created by Quality Monitor v4.15.0 (#82d77af). More details are shown in the GitHub Checks Result.

@janfaracik janfaracik changed the title Update styles.css Update for 'Standardise experimental Jenkins pages' Sep 29, 2026
@uhafner uhafner changed the title Update for 'Standardise experimental Jenkins pages' Fix minimum position of sticky sidebar in run tab Sep 29, 2026
@uhafner uhafner added the bug Bugs or performance problems label Sep 29, 2026
@uhafner
uhafner merged commit 60e9145 into jenkinsci:main Sep 29, 2026
31 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs or performance problems

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants