Make WordPress Core

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: wildworks's profile wildworks Owned by: rcreators's profile rcreators
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)

actual.png (51.0 KB) - added by wildworks 12 months ago.
Current view
expected.png (53.7 KB) - added by wildworks 12 months ago.
The view I want to look
patch-61616.patch (775 bytes) - added by hbhalodia 11 months ago.
Path for issue - 61616
patch-61616-2.patch (940 bytes) - added by hbhalodia 11 months ago.
Updated Patch
new-content-icon-too-tall.png (5.7 KB) - added by sabernhardt 10 months ago.
the New content link icon is too tall on smaller screens
61616.outline-submenus.diff (2.5 KB) - added by sabernhardt 10 months ago.
option: add outlines for submenus
admin-toolbar-submenu-outlines.png (73.3 KB) - added by sabernhardt 10 months ago.
possible outlines for the toolbar submenus
admin-sidebar-submenu-outlines.png (155.9 KB) - added by sabernhardt 10 months ago.
possible outlines/borders for the admin menu's submenus
61616.submenus.2.diff (2.6 KB) - added by joedolson 9 months ago.
Adjusts line height of add new icon

Download all attachments as: .zip

Change History (24)

@wildworks
12 months ago

Current view

@wildworks
12 months ago

The view I want to look

#1 @wildworks
12 months ago

  • Focuses accessibility added

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


11 months ago

#3 @joedolson
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 @joedolson
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 @hbhalodia
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.

@hbhalodia
11 months ago

Path for issue - 61616

@hbhalodia
11 months ago

Updated Patch

#7 @wildworks
11 months ago

@hbhalodia Thanks for the update. patch-61616-2.patch works fine for me.

#8 @hbhalodia
11 months ago

Thanks for the confirmation @wildworks.

#9 @rcreators
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

@sabernhardt
10 months ago

the New content link icon is too tall on smaller screens

@sabernhardt
10 months ago

option: add outlines for submenus

@sabernhardt
10 months ago

possible outlines for the toolbar submenus

@sabernhardt
10 months ago

possible outlines/borders for the admin menu's submenus

#10 @sabernhardt
10 months ago

  • Keywords 2nd-opinion added

PR 7163 is a good start but maybe not complete.

  1. 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.
  2. 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 to 100000 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 @joedolson
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

@joedolson
9 months ago

Adjusts line height of add new icon

#14 @joedolson
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.

#15 @joedolson
9 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 59041:

Accessibility: Add border around menus and submenus in high contrast mode.

Add outlines and borders to mark the boundaries between the admin navigation menu and content and around adminbar submenus that are visible when Windows High Contrast Mode is enabled. This clarifies the page structure and makes high contrast mode easier to use.

Props wildworks, hbhalodia, sabernhardt, joedolson, rcreators.
Fixes #61616.

Note: See TracTickets for help on using tickets.