Make WordPress Core

Opened 6 weeks ago

Closed 4 weeks ago

#64780 closed defect (bug) (fixed)

[Admin Reskin] WordPress Events widget list background and borders still use old colors.

Reported by: r1k0's profile r1k0 Owned by: audrasjb's profile audrasjb
Milestone: 7.0 Priority: normal
Severity: normal Version:
Component: Administration Keywords: admin-reskin has-test-info has-screenshots has-patch
Focuses: css Cc:

Description

The no events container (left border and background color) still uses the old color scheme.

Steps to Reproduce

  1. Head over to the admin dashboard.
  2. Look under the "WordPress Events and News" section while there are no events.
  3. Observe the border left color and background color of the "There are no events scheduled near you at the moment. Would you like to organize a WordPress event?" container.

Expected Behavior

The colors should match the current admin color scheme.

Screenshot

https://i.ibb.co/KxfFzxr4/no-events-colors.png

Change History (17)

This ticket was mentioned in PR #11116 on WordPress/wordpress-develop by @r1k0.


6 weeks ago
#1

  • Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/64780#ticket

## Use of AI Tools
None

#2 @huzaifaalmesbah
6 weeks ago

Related to #64756 (WordPress Events widget still using old hardcoded color scheme).

@mukesh27 commented on PR #11116:


6 weeks ago
#3

The new BG color looks good for event List but the border didn't looks good after changes.

https://github.com/user-attachments/assets/bb32c990-4980-4e57-9736-0276ea341496

#4 @ocean90
6 weeks ago

  • Keywords admin-reskin removed
  • Milestone changed from Awaiting Review to 7.0

#5 @manhar
6 weeks ago

Tested. It is working as expected after patch.

before patch :
https://postimg.cc/nXP4TvQ1

after-patch :
https://postimg.cc/T5fgh0ZT

@r1k0 commented on PR #11116:


6 weeks ago
#6

@mukeshpanchal27 Nice catch, let me update that.

@r1k0 commented on PR #11116:


6 weeks ago
#7

@mukeshpanchal27 Changed the border color, from f0f0f1 to e9e9ed. Have a look at it and let me know what you think.

https://github.com/user-attachments/assets/5d1aa895-842e-4fb4-ae33-b188ba969535

#8 @huzaifaalmesbah
5 weeks ago

  • Keywords admin-reskin added

#9 follow-up: @amin7
5 weeks ago

Hi,

I tried applying the patch using npm run grunt patch but it failed due to a conflict in src/wp-admin/css/dashboard.css.

It looks like both the patch and the current codebase have changes to the same part of that file, so they're clashing. Could you take a look at the PR, resolve the conflict in dashboard.css, and push an updated version so the patch applies cleanly?

Thanks

@r1k0 commented on PR #11116:


5 weeks ago
#10

Merged latest trunk and resolved conflict in dashboard.css.

#11 in reply to: ↑ 9 @r1k0
5 weeks ago

Replying to amin7:

Hi,

I tried applying the patch using npm run grunt patch but it failed due to a conflict in src/wp-admin/css/dashboard.css.

It looks like both the patch and the current codebase have changes to the same part of that file, so they're clashing. Could you take a look at the PR, resolve the conflict in dashboard.css, and push an updated version so the patch applies cleanly?

Thanks

PR updated, feel free to test it now

@huzaifaalmesbah commented on PR #11116:


5 weeks ago
#12

Looks good to me.
Before:
https://github.com/user-attachments/assets/e91f8e64-5a61-4568-bf20-a62cdcaa0ee9

After:
https://github.com/user-attachments/assets/2e3025c3-9666-4593-997f-98cdd95e2b2c

#13 @huzaifaalmesbah
5 weeks ago

  • Summary changed from [Admin Reskin] Dashboard: Update no events colors. to [Admin Reskin] WordPress Events widget background and list item borders still use old colors.

#14 @huzaifaalmesbah
5 weeks ago

  • Summary changed from [Admin Reskin] WordPress Events widget background and list item borders still use old colors. to [Admin Reskin] WordPress Events widget list background and borders still use old colors.

#15 @amin7
5 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested:https://github.com/WordPress/wordpress-develop/pull/11116

Environment

WordPress: 7.0-beta3
PHP: 8.2.7
Server: ngnix
Browser: Chrome 145.0.0.0
OS: macOS
Theme: Twenty Twenty-Five
MU Plugins: None activated

Result

✅ Issue resolved with this patch.

@shailu25 commented on PR #11116:


5 weeks ago
#16

WordPress Events widget list background color and border color look good after the update.

BeforeAfter
https://github.com/user-attachments/assets/da0102b3-f56d-4a0c-bc82-9197f9f7bd01https://github.com/user-attachments/assets/0e1f971d-ba8e-4d3b-a1e6-4148f3098a56

#17 @audrasjb
4 weeks ago

  • Owner set to audrasjb
  • Resolution set to fixed
  • Status changed from new to closed

In 62031:

Administration: Update Events widget background and border colors.

This changeset ensures the Events dashboard widget uses the new default admin colors.

Props r1k0, huzaifaalmesbah, mukesh27, audrasjb, amin7, shailu25, rishavdutta.
Fixes #64780.

Note: See TracTickets for help on using tickets.