-
Notifications
You must be signed in to change notification settings - Fork 38
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
Graphs are not themeable #141
Comments
The issue with these graphs are that they are rendered server side. Any idea how we would handle 'System' theme preference with an extension point? I migrated the junit plugin to use echarts (https://github.com/jenkinsci/echarts-api-plugin) instead of trying to fight with this as it's rendered client side and can lookup CSS variables. #127 is a similar issue, although slightly easier as we can probably bundle a custom theme which has the extra CCS required for 'system' preference. |
Would try to wrap the graphs in a client side wrapper that passes the appropriate colors as query parameters, not unlike the JS that handles dynamic graph width today (but less amateurish). Probably won't handle color scheme changes while the page is being viewed unless we're spending real time on this, but probably good enough? |
To clarify, I currently have no specific plans to implement this myself. I think I'm out of my depth here with the CSS/JS required to make this work properly. |
There's at least two approaches I think,
I have no intention in the short term of working on it, possibly later on though |
Limited query parameter support exists. I think this can potentially be done more simply, as described in #141 (comment), which wouldn't require a new extension point. It is based on the assumption that CSS variable values can be accessed by JS. |
hudson.util.Graph
is used around Jenkins and not (really) themeable. While it can consume some limited query parameters for background color, they're neither set nor work in a way that suits the existing CSS variable approach AFAICT.On the plus side, if the color definitions are never set we can probably get away with adding an extension point to override them.
Examples
The text was updated successfully, but these errors were encountered: