-
Notifications
You must be signed in to change notification settings - Fork 274
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 configurable (with Bootstrap 5) #889
Conversation
Codecov Report
@@ Coverage Diff @@
## master #889 +/- ##
============================================
- Coverage 79.79% 79.57% -0.22%
- Complexity 1425 1427 +2
============================================
Files 238 238
Lines 5271 5283 +12
Branches 411 414 +3
============================================
- Hits 4206 4204 -2
- Misses 911 925 +14
Partials 154 154
Continue to review full report at Codecov.
|
# Conflicts: # plugin/pom.xml
# Conflicts: # plugin/pom.xml
Now the general trend options are changed for *all* charts. So individual trend chart dialogs change the global configuration and the local one.
Now configuration of trend charts in job page and trend charts in details pages use a different dialog.
Now configuration of trend charts in job page and trend charts in details pages use a different dialog.
# Conflicts: # plugin/pom.xml
@@ -1,5 +1,11 @@ | |||
/* global jQuery3, view, echartsJenkinsApi */ | |||
/* global jQuery3, view, echartsJenkinsApi, bootstrap5, trendDefaultStorageId */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trendDefaultStorageId doesn't seem to be referenced elsewhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was searching for "storage" because it was not clear to me whether the configuration is stored in the browser, in the Jenkins job, or in the Jenkins user profile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently the configuration is stored in the Browser local storage only. If required this can be changed later on.
Rework UI components to use the upcoming Bootstrap 5 release.
Make build trend charts configurable:
Downstream of jenkinsci/echarts-api-plugin#124