Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#52374 closed defect (bug) (fixed)

Twenty Twenty-One: Menu button has the wrong colors

Reported by: poena's profile poena Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.7 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: css Cc:

Description (last modified by SergeyBiryukov)

An unintended side effect of the updated button styles in #51927 was that the menu button changed color.

In trunk, the current menu button has a dark grey background color
var(--global--color-primary); and a transparent text.

As per design, the background should be transparent and the text should be dark grey.

Attachments (3)

menu-button-trunk.png (6.1 KB) - added by poena 3 years ago.
Menu button with incorrect colors
52374.patch (8.4 KB) - added by poena 3 years ago.
Override specificity from default button styles by using the button ID.
with-patch.png (30.6 KB) - added by jeroenrotty 3 years ago.
Mobile menu button with correct styling

Download all attachments as: .zip

Change History (9)

@poena
3 years ago

Menu button with incorrect colors

@poena
3 years ago

Override specificity from default button styles by using the button ID.

#1 @poena
3 years ago

  • Keywords needs-testing added

#2 @SergeyBiryukov
3 years ago

  • Description modified (diff)

#3 @SergeyBiryukov
3 years ago

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

In 50026:

Twenty Twenty-One: Correct colors for the Menu button.

As per design, the background should be transparent and the text should be dark grey, not vice versa.

Follow-up to [49987].

Props poena.
Fixes #52374. See #51927.

@jeroenrotty
3 years ago

Mobile menu button with correct styling

#4 @jeroenrotty
3 years ago

Tested the patch with @rolfsiebers and the patch worked as intended, see screenshot.

This ticket was mentioned in Slack in #core by francina. View the logs.


3 years ago

#6 @hellofromTonya
3 years ago

  • Keywords has-patch added; needs-testing removed
Note: See TracTickets for help on using tickets.