Followup from #3191682: Ensure that Olivero's focus states are visible and accessible in Windows High Contrast mode

The <summary> element within the <details> element's focus state is not visible in Windows high contrast mode.

In IE, there is a slight dotted outline, but if Edge and Firefox, there is no focus indicator at all.

A transparent outline should easily resolve these issues.

Testing instructions

  1. Test with Windows High Contrast mode
  2. Load patch or visit Tugboat link at https://3227427-details-hc-focus-infih8knbtdehncurbd6mmu2qxqwdmzq.tugboa...
  3. Look at a <details> element and tab to it.
  4. Verify the focus state is visible in HC mode.
CommentFileSizeAuthor
#5 Screen Recording 2021-08-16 at 5.23.17 PM.mov2.47 MBgauravvvv
#2 details-focus-ie.gif119.32 KBmherchel
#2 details-focus-edge.gif56.61 KBmherchel
#2 details-focus-ff.gif71.23 KBmherchel
#2 3227427.patch1.44 KBmherchel
summary.png79.63 KBmherchel

Comments

mherchel created an issue. See original summary.

mherchel’s picture

mherchel
he/him
Primary language English
Location Gainesville, FL, US
commented
Status: Active » Needs review
StatusFileSize
new 3227427.patch1.44 KB
new details-focus-ff.gif71.23 KB
new details-focus-edge.gif56.61 KB
new details-focus-ie.gif119.32 KB

Patch along with screenshots!

Firefox:

Edge:

IE:

mherchel’s picture

mherchel
he/him
Primary language English
Location Gainesville, FL, US
commented
Issue summary: View changes

Adding testing instructions and Tugboat link: https://3227427-details-hc-focus-infih8knbtdehncurbd6mmu2qxqwdmzq.tugboa...

mherchel’s picture

mherchel
he/him
Primary language English
Location Gainesville, FL, US
commented
Issue summary: View changes
gauravvvv’s picture

gauravvvv commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screen Recording 2021-08-16 at 5.23.17 PM.mov2.47 MB

Moving to RTBC.

Adding screen recoding for Chrome browser. Focus state is visible in high contrast mode.

  • lauriii committed d51a1db on 9.3.x 
    Issue #3227427 by mherchel, Gauravmahlawat: Olivero: <summary> focus...

  • lauriii committed 8e07ae3 on 9.2.x 
    Issue #3227427 by mherchel, Gauravmahlawat: Olivero: <summary> focus...
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Version: 9.3.x-dev » 9.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed d51a1db and pushed to 9.3.x. Also cherry-picked to 9.2.x because Olivero is experimental. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.