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

Use new messages dialog and improve associated icon link #1119

Merged
merged 5 commits into from
Nov 21, 2021

Conversation

uhafner
Copy link
Member

@uhafner uhafner commented Nov 20, 2021

Use the new messages dialog of the bootstrap plugin that has been introduced with jenkinsci/bootstrap5-api-plugin#65. Additionally use the new icon button introduced with jenkinsci/font-awesome-api-plugin#118. This new button uses the restyled tooltips that have been introduced in Jenkins 2.318 (jenkinsci/jenkins#5763).

@uhafner uhafner added the enhancement Enhancement of existing functionality label Nov 20, 2021
@codecov
Copy link

codecov bot commented Nov 20, 2021

Codecov Report

Merging #1119 (ba01d6b) into master (d90b48c) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1119      +/-   ##
============================================
+ Coverage     79.74%   79.78%   +0.04%     
- Complexity     1477     1479       +2     
============================================
  Files           244      244              
  Lines          5376     5377       +1     
  Branches        420      420              
============================================
+ Hits           4287     4290       +3     
+ Misses          933      931       -2     
  Partials        156      156              
Impacted Files Coverage Δ
...s/plugins/analysis/core/model/InfoErrorDetail.java 100.00% <ø> (+20.00%) ⬆️
...ins/plugins/analysis/core/model/DetailFactory.java 94.44% <100.00%> (+0.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d90b48c...ba01d6b. Read the comment docs.

Due to changed CSS classes the existing logic needed to be replaced.
Now we can detect the error or warnings status by inspecting a CSS class.
@uhafner uhafner merged commit a396d74 into master Nov 21, 2021
@uhafner uhafner deleted the info-messaages branch November 21, 2021 23:01
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
1 participant