Opened 12 months ago
Closed 9 months ago
#61616 closed enhancement (fixed)
Add border to admin sidebar/bar for Windows high contrast mode
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch has-screenshots commit |
Focuses: | accessibility | Cc: |
Description
When you enable high contrast mode on Windows, no border is drawn below the admin bar or to the right of the admin sidebar.
This makes the boundary between the content area and the navigation area unclear.
To address this issue in high contrast mode, both core and Gutenberg use an approach that adds styles like this one: outline: 1px solid transparent;
Attachments (9)
Change History (24)
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
11 months ago
#3
@
11 months ago
- Milestone changed from Awaiting Review to 6.7
- Owner set to joedolson
- Status changed from new to accepted
Makes sense to me!
#4
@
11 months ago
- Owner changed from joedolson to rcreators
- Status changed from accepted to assigned
This ticket was mentioned in PR #7127 on WordPress/wordpress-develop by @hbhalodia.
11 months ago
#5
- Keywords has-patch added
Trac ticket: https://core.trac.wordpress.org/ticket/61616
## Issue
- When you enable high contrast mode on Windows, no border is drawn below the admin bar or to the right of the admin sidebar. This makes the boundary between the content area and the navigation area unclear.
#6
@
11 months ago
Hi @wildworks @joedolson, I have rasied the PR as per the suggestion - https://github.com/WordPress/wordpress-develop/pull/7127
Can you please review the patch and let me know if that needs some more work, I am happy to update the PR with requested changes.
Thank You.
#7
@
11 months ago
@hbhalodia Thanks for the update. patch-61616-2.patch works fine for me.
#9
@
11 months ago
- Keywords has-screenshots added
Hello @hbhalodia,
Thanks for the PR. It works as expected.
Before Patch: https://prnt.sc/7Ne0fZuMhufG
After Patch: https://prnt.sc/C6EwzGx9xaaR
This one looks ready to commit @joedolson
#10
@
10 months ago
- Keywords 2nd-opinion added
PR 7163 is a good start but maybe not complete.
- The toolbar's outline breaks under the New content link icon on smaller screens because the
line-height
is too tall on#wpadminbar #wp-admin-bar-new-content .ab-icon:before
. - I would like to consider adding an outline (or border) for submenus. 61616.outline-submenus.diff keeps the 5px left border from r49961 and adds 1px for the other three sides, and it makes the current and non-current submenu styles more consistent. In the toolbar, if the incomplete outline around the Network Admin submenu is a significant problem, raising the second submenu's
z-index
to100000
might help.
Also, the border around the mobile menu toggle appears when the page loads, and it does not change when focused. (border added in r26134)
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
10 months ago
#12
@
10 months ago
I'm in favor of adding the outline on submenus as well. It's clearly related to this topic, and it would greatly enhance the usability of high contrast mode.
The border on the mobile menu toggle maybe should be an independent ticket; although I could easily be convinced otherwise. Menus and their associated toggles are really just one component, after all.
This ticket was mentioned in Slack in #core by stoyangeorgiev. View the logs.
10 months ago
#14
@
9 months ago
- Keywords commit added; 2nd-opinion removed
Updated patch adjusts the line-height on the add new icon to 1.27, which evens out the outlines. It doesn't appear to cause any problems; the visual alignment still works to my eye.
I'm inclined to commit this, even without resolving the multisite menu issue. I don't really see that as a significant problem. When resolving it, it results in a visible overlap between the two outlines, and I'm not sure that's an improvement.
I'm marking this for commit.
Current view