WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#45689 closed defect (bug) (fixed)

Twenty Nineteen: Incorrect opacity and text shadow on submenus when there's a featured image.

Reported by: kjellr Owned by: laurelfulford
Priority: normal Milestone: 5.0.3
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch fixed-major
Cc: Focuses:

Description

On pages that have a featured image, submenu item labels are incorrectly inheriting some styles from the top-level menu text:

  • The text have a small text-shadow
  • The menu items are at 0.6 opacity on hover

This hurts readability, and results in the menu items appearing different from their counterparts on pages with no featured image.

https://cldup.com/kpZmnkUcrj.png

The patch attached restores the expected styles:

https://cldup.com/RKFUmxjdnH.thumb.jpg

(Ported from ​https://github.com/WordPress/twentynineteen/pull/730 in the Twenty Nineteen GitHub repository)

Attachments (2)

45689.patch​ (2.5 KB ) - added by kjellr 8 years ago.
45689.2.patch​ (2.5 KB ) - added by kjellr 8 years ago.
Corrected patch to run from the src directory.

Download all attachments as: .zip

Change History (7)

@kjellr
8 years ago

@kjellr
8 years ago

Corrected patch to run from the src directory.

#1 @laurelfulford
8 years ago

  • Milestone Awaiting Review → 5.0.3
  • Owner set to laurelfulford
  • Status new → assigned

#2 @laurelfulford
8 years ago

Thanks @kjellr -- this looks good to me! I'll commit, and add a comment to the related GitHub PR to close the loop.

#3 @laurelfulford
8 years ago

  • Resolution → fixed
  • Status assigned → closed

In 44368:

Twenty Nineteen: Remove transparency from submenus.

On pages and posts with featured images, the top level menu items have a slight transparency on hover, which was being inherited by their submenus. This update removes that inheritance, improving readability and consistency in the menu's appearance.

Props kjellr.
Fixes #45689.

#4 @SergeyBiryukov
8 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

Reopening for merging to the 5.0 branch.

#5 @SergeyBiryukov
8 years ago

  • Resolution → fixed
  • Status reopened → closed

In 44427:

Twenty Nineteen: Remove transparency from submenus.

On pages and posts with featured images, the top level menu items have a slight transparency on hover, which was being inherited by their submenus. This update removes that inheritance, improving readability and consistency in the menu's appearance.

Props kjellr.
Merges [44368] to the 5.0 branch.
Fixes #45689.

Note: See TracTickets for help on using tickets.