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

User-defined themes no longer work in Jenkins 2.346.1 #83

Closed
pcarenza opened this issue Jun 24, 2022 · 3 comments
Closed

User-defined themes no longer work in Jenkins 2.346.1 #83

pcarenza opened this issue Jun 24, 2022 · 3 comments
Labels

Comments

@pcarenza
Copy link

Jenkins and plugins versions report

Environment
Paste the output here

What Operating System are you using (both controller, and any agents involved in the problem)?

Windows Server 2019

Reproduction steps

  • Write CSS using material theme website, download css file, place in web-accessible location. In my case I had an embedded SVG and light blue color
  • point theme at CSS and refresh
  • Title shows correctly, but none of the colors are correct and icon doesn't show up
  • This worked in Jenkins 2.332.X

Expected Results

Theme properly displays with blue and icon

Actual Results

Theme is entirely black, no icon

Anything else?

No response

@pcarenza pcarenza added the bug label Jun 24, 2022
@breakingspell
Copy link

breakingspell commented Jun 24, 2022

It seems some CSS has changed with 2.346.1. We use a small Extra CSS entry to theme the headers on the two Jenkins instances we use to tell them apart by glance.
After adjusting the CSS to match the new version, the header is fixed, but the breadcrumb bar colors are not correct:

.page-header, .jenkins-breadcrumbs { background-color:darkred; }

The breadcrumb bar takes on a barely-pink tint, same with our other instance's green being a very light green, almost white. It's enough to tell them apart as intended, but not the previous behavior in 2.332.x (breadcrumb bar fully matching header and much easier to see).

I haven't had a chance to drill further yet, I suspect background-color is no longer the right property to set for the breadcrumb bar. Many people may need to update their themes for this release

cossons added a commit to cossons/jenkins-material-theme that referenced this issue Jun 27, 2022
fix(css): Fix css id for the jenkins page. Keeping old id for backward compatibility.
See: Issue jenkinsci/simple-theme-plugin#83 on Jenkins plugins
@cossons
Copy link

cossons commented Jun 27, 2022

Hi, I have created a PR on the jenkins-material-theme to add the new header id. A kept the old on for backward compatibility.

offa pushed a commit to offa/jenkins-modern-themes that referenced this issue Jun 30, 2022
fix(css): Fix css id for the jenkins page. Keeping old id for backward compatibility.
See: Issue jenkinsci/simple-theme-plugin#83 on Jenkins plugins

Signed-off-by: offa <bm-dev@yandex.com>
@TobiX
Copy link
Contributor

TobiX commented Jun 30, 2022

This is not an issue in this plugin, but with the individual theme(s) instead (Hint: The material theme is dead "upstream", you might want to switch and help improve the material-theme-plugin instead). Jenkins Core has started some more breaking UI improvements in recent months, so breaking themes are almost guaranteed.

@TobiX TobiX closed this as not planned Won't fix, can't repro, duplicate, stale Jun 30, 2022
Jorg3Lucas pushed a commit to Jorg3Lucas/jenkins-modern-themes that referenced this issue Jul 11, 2022
* fix(css): Fix css id for the jenkins page.

fix(css): Fix css id for the jenkins page. Keeping old id for backward compatibility.
See: Issue jenkinsci/simple-theme-plugin#83 on Jenkins plugins

Signed-off-by: offa <bm-dev@yandex.com>

* Modern-Themes: New Update from commit d048528

* Fix search box padding

* Modern-Themes: New Update from commit 3f6a2b1

Co-authored-by: cossons <sebastien.cosson01@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants