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

Make trend charts theme-aware #287

Merged
merged 7 commits into from
May 19, 2023
Merged

Make trend charts theme-aware #287

merged 7 commits into from
May 19, 2023

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented May 15, 2023

Add a JenkinsPalette that can be used to obtain colors that are supported by the current Jenkins theme. All line charts can now handle fixed colors or CSS color variables that will be resolved on the client side using the actual values of the selected theme.

  • Use theme-aware CSS variables for all background colors
  • Use theme-aware CSS variables for all foreground colors

Downstream PRs

Light Theme

Bildschirmfoto 2023-05-15 um 11 40 29
Bildschirmfoto 2023-05-15 um 11 40 42
Bildschirmfoto 2023-05-15 um 11 49 56

Dark Theme

Bildschirmfoto 2023-05-15 um 11 41 30
Bildschirmfoto 2023-05-15 um 11 41 25

Add a `JenkinsPalette` that can be used to obtain colors that are
supported by the current Jenkins theme. All line charts can now handle
fixed colors or CSS color variables that will be resolved on the client
side using the actual values of the selected theme.
@uhafner uhafner added the enhancement Enhancement of existing functionality label May 15, 2023
@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Merging #287 (fc60555) into master (b136199) will increase coverage by 16.81%.
The diff coverage is 100.00%.

❗ Current head fc60555 differs from pull request most recent head 9070226. Consider uploading reports for the commit 9070226 to get more accurate results

@@              Coverage Diff              @@
##             master     #287       +/-   ##
=============================================
+ Coverage     34.61%   51.42%   +16.81%     
- Complexity        9       17        +8     
=============================================
  Files             6        7        +1     
  Lines            78      105       +27     
  Branches          3        3               
=============================================
+ Hits             27       54       +27     
  Misses           51       51               
Impacted Files Coverage Δ
...ava/io/jenkins/plugins/echarts/JenkinsPalette.java 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

uhafner added 3 commits May 15, 2023 16:32
All pie charts can now handle fixed colors or CSS color variables that
will be resolved on the client side using the actual values of the
selected theme.

import static org.assertj.core.api.Assertions.*;

class JenkinsPaletteTest {

Check notice

Code scanning / CodeQL

Unused classes and interfaces

Unused class: JenkinsPaletteTest is not referenced within this codebase. If not used as an external API it should be removed.
@uhafner uhafner marked this pull request as ready for review May 19, 2023 21:46
@uhafner uhafner merged commit 771d261 into master May 19, 2023
@uhafner uhafner deleted the theme-colors branch May 19, 2023 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant