#58096 closed defect (bug) (fixed)
Incorrect color for "Theme enabled." admin notice
Reported by: | ocean90 | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.2.1 | Priority: | normal |
Severity: | minor | Version: | 6.2 |
Component: | Networks and Sites | Keywords: | good-first-bug has-patch |
Focuses: | ui | Cc: |
Attachments (6)
Change History (20)
This ticket was mentioned in PR #4309 on WordPress/wordpress-develop by MarineEvain.
22 months ago
#2
- Keywords has-patch added; needs-patch removed
#3
@
22 months ago
Hi,
This is my first time patching WordPress code. If someone can review it and let me know if this helps...
Have a nice day!
#4
in reply to:
↑ 1
@
22 months ago
Replying to audrasjb:
I can reproduce the issue on my side.
Looks like classesnotice notice-updated is-dismissible
should be replaced withupdated notice is-dismissible
for this notice. Not CSS change needed.
The updated
class is legacy, see #44640, so we should just replace notice-updated
with notice-success
here:
notice notice-success is-dismissible
#5
@
22 months ago
Thanks, @dhrumilk for the patch.
The Patch looks good to me. but still need to update line 386 and 387
#10
@
22 months ago
- Keywords fixed-major added; commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for branch 6.2 backport.
@audrasjb commented on PR #4309:
22 months ago
#11
Note: See
TracTickets for help on using
tickets.
I can reproduce the issue on my side.
Looks like classes
notice notice-updated is-dismissible
shoulb be replaced withupdated notice is-dismissible
for this notice. Not CSS change needed.