Closed (fixed)
Project:
Drupal core
Version:
9.4.x-dev
Component:
Claro theme
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
21 Mar 2022 at 18:14 UTC
Updated:
6 Jun 2022 at 09:24 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #3
ckrinaComment #4
ckrinaComment #5
ckrinaComment #6
ckrinaComment #7
javi-er commentedSeems like CSS needs to be recompiled for tests to pass and also the deleted image needs to be re added in a different folder, in my local it's saying that core/themes/claro/images/core/dc2323/error.svg is missing when there is an error in the status report.
Comment #8
javi-er commentedSorry, I uploaded the wrong screenshot in the previous comment.
Comment #9
javi-er commentedUpdated patch for 9.4.x, this is passing all tests in my local.
Comment #10
javi-er commentedThe color in error.svg needed to be updated as well.
Comment #11
sharayurajput commentedComment #14
sharayurajput commentedComment #15
javi-er commentedAdjusted the D9 patch to account for the new changes in 9.4.x ( blue variations)
Also, I moved the error icon to
/core/misc/icons/dc2323/error.svgfollowing " Refactor references to images in Claro's images/core directory "Comment #16
javi-er commentedUpdated D10 patch.
Comment #17
javi-er commentedComment #18
WagnerMelo commentedI reviewed this issue, and all the changes asked looks like ok for me, all the codes was changed correctly, and i checked the color on drupal site.
Comment #20
alexpottCommitted a2ce4d4 and pushed to 10.0.x. Thanks!
Committed and pushed 304c695aee to 9.5.x and 20d4f84621 to 9.4.x. Thanks!