Opened 3 weeks ago
Last modified 9 days ago
#59924 new defect (bug)
Twenty Nineteen: The navigation block submenu button is unreadable
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | css | Cc: |
Description
The navigation block has an option called "Open on click" that is available when there is a submenu.
When the option is enabled, the parent menu item is a <button>
element.
On the front of the website, this button inherits the background styles from the themes button CSS:
button:hover, button:hover, input[type="button"]:hover, input[type="reset"]:hover, input[type="submit"]:hover { background: #111; cursor: pointer; }
So when the themes default colors are used and someone hovers over the button, the background and text has the same colors, and the text is not readable.
Testing instructions:
Activate Twenty Nineteen
Make sure that your WordPress installation has some content that you can place in the navigation block, because you will need to create a submenu.
Create a new post or page.
Add a navigation block. In the block, select the inserter and add a link: this will be your parent menu item. Click on the link and select the option "Add submenu". Add a link.
Save.
Go to the front of the website.
Locate the navigation and hover over the link item that has the submenu.
Confirm that both the text and the background are dark grey, close to black.
Change History (7)
This ticket was mentioned in PR #5703 on WordPress/wordpress-develop by TalhaQuddoos.
11 days ago
#2
- Keywords has-patch added; needs-patch removed
In src/wp-content/themes/twentynineteen/style.css
, I added the following lines in the end:
.wp-block-navigation-item.open-on-click .wp-block-navigation-submenu__toggle:hover, .wp-block-navigation-item.open-on-click .wp-block-navigation-submenu__toggle:hover+.wp-block-navigation__submenu-icon, .wp-block-navigation-item.open-on-click .wp-block-navigation-submenu__toggle:focus, .wp-block-navigation-item.open-on-click .wp-block-navigation-submenu__toggle:focus+.wp-block-navigation__submenu-icon { color: #fff; }
Trac ticket: https://core.trac.wordpress.org/ticket/59924
This ticket was mentioned in PR #5704 on WordPress/wordpress-develop by @sabernhardt.
10 days ago
#5
This keeps the background color transparent when hovering or focusing on the toggle button.
It also removes the padding and resets the focus outline offset and border radius.
Below are images showing the focus outline for both the arrow button and the toggle button that includes the text.
#6
@
10 days ago
- Focuses css added
If changing the text color is better than making the background consistently transparent, that patch would need to edit styles in SCSS files and compile to style.css
and style-rtl.css
.
However, I think these buttons should adjust padding and the outline offset, so I made another option.
#59925 was marked as a duplicate.