Problem/Motivation

The following issues made a positive difference towards Navigation being usable in Windows High Contrast mode, but there are still a few outstanding issues to rectify.

#3423277: Icons are invisible in high contrast mode
#3483209: Navigation leverage icon core API

WCAG success criteria

Steps to reproduce

First set up your environment to test the latest 11.x branch (or dev release) on forced-colors mode. There are a couple of ways to do this, eg.

The following issues are present (description followed by screenshot).

1. The carets are only barely visible:
*retested in 11.x, not an issue anymore

Screenshot pointing out the carets barely visible in Windows high contrast

2. The right border of the border is invisible, that way it is impossible to distinguish between drawer and background of the main content area:

Screenshot of where a border is needed

3. The caret inside the expand and collapse button at the bottom of the sidebar is also close to invisible:
*retested in 11.x, not an issue anymore

Screenshot of caret inside expand / collapse button

4. With the sidebar expanded and the focus on configuration in this case, hovering the mouse cursor over appearance the outline is also hardly noticeable: (*not an issue - the screenshot was taken with automatic dark mode enabled in edge, and that setting is causing inaccurate display of forced colors, use "emulate css media feature prefers-color-scheme instead for accurate display)

Screenshot of expanded sidebar of the new navigation with focus outline around configuration and hover highlight around appearance

5. With the sidebar collapsed hovering over a top level item without sub menu items a dark badge is shown in the non high contrast mode. With the high contrast mode on, the dark background of that badge fusions into one with the dark background of the main content area and the sidebar:

Screenshot of collapsed sidebar of the new navigation and the mouse cursor hovering over the people top level menu item showing a badge called people

6. On desktop viewport the top bar is missing the bottom border, on mobile viewport is bottom border is visible

Screenshot of collapsed drawer top bar on small mobile viewport

7. On the mobile breakpoint, the close button is barely visible:

Screenshot of mobile close button barely visible

8. On mobile the hamburger in the top bar is barely visible:

Screenshot of mobile hamburger top bar icon barely visible

9. On mobile, the right border of the navigation is oddly interrupted:

Screenshot of interrupted border on mobile

10. the icon used in the top bar is not visible:

the topbar and header of article node edit form called a blog post in forced color mode

11. the icons of the top level menu items have the identical color even though the labels differ depending if the menu item has a sub menu or not:
expanded navigation sidebar in forced color mode with the night sky theme

some context: the problem is not reproducible in 11.x. the problem was either fixed indirectly on an other issue in 11.x or it got introduced within this MR - the latter is more likely. the problem is reproducible by checking out git checkout 4e7784d9fae9a87861ef8ee3b447606fa286d0e5. The last commit on the issue (d3f4484b56b9558e1a3b62804d9c8131a4777045) fixed the problem again.

12. The current page is shaded with a light colorized background in regular mode (#3426468: Make the active menu item visually stand out more is about improving the contrast and make it stand out more - but that is out of the scope for this issue)

section in the navigation sidebar in regular mode

but the problem in the context of forced color mode is that the information about current state is missing entirely

section in the navigation sidebar in forced colors mode

Proposed Resolution

User interface changes

2.
Before
collapsed navigation sidebar with the structure submenu expanded in forced color mode
After
collapsed navigation sidebar with the structure submenu expanded in forced color mode

5.
Before
collapsed navigation sidebar with the people top level menu item hovered in forced color mode showing the people toggle
After
collapsed navigation sidebar with the people top level menu item hovered in forced color mode showing the people toggle

6.
Before
the top of the node article node edit form showing the top of the navigation sidebar, the topbar and the header in forced color mode
After
the top of the node article node edit form showing the top of the navigation sidebar, the topbar and the header in forced color mode

7.
Before
the top of the navigation sidebar in mobile viewport expanded in force color mode
After
the top of the navigation sidebar in mobile viewport expanded in force color mode

8.
Before
the collapsed navigation sidebar in mobile viewport in forced color mode showing the expand sidebar button
After
the collapsed navigation sidebar in mobile viewport in forced color mode showing the expand sidebar button

9.
Before
the expanded navigation sidebar in mobile viewport in forced color mode with a gap in the right border
After
the expanded navigation sidebar in mobile viewport in forced color mode with a gap in the right border

10.
Before
the navigation sidebar and topbar with just the height of the topbar in forced contrast mode
the navigation sidebar and topbar in forced contrast mode with the more actions menu expanded
After
the navigation sidebar and topbar with just the height of the topbar in forced contrast mode
the navigation sidebar and topbar in forced contrast mode with the more actions menu expanded

11.
Before
the upper half of the expanded navigation sidebar in forced color mode
the upper half of the collapsed navigation sidebar in forced color mode
After
the upper half of the expanded navigation sidebar in forced color mode
the upper half of the collapsed navigation sidebar in forced color mode

12.
Before
section in the navigation sidebar in regular mode
section in the navigation sidebar in regular mode

After
section in the navigation sidebar in forced colors mode
section in the navigation sidebar in forced colors mode

Testing instructions

Walkthough of testing at https://www.youtube.com/watch?v=v4dTY4__F6U

Setup

  1. Install navigation module
  2. Create a node (the top bar exists on the node page)
  3. Use Chromium based browser such as Chrome or Edge

Testing criteria

  1. Ensure all icons visible
  2. Ensure all borders are visible
  3. Ensure text is visible (this wasn’t a problem before)

Testing steps: mobile

  1. Emulate forced colors in dev tools https://developer.chrome.com/docs/devtools/rendering/emulate-css
  2. Be logged in
  3. Note that the top navigation should have a bottom border and icon should be high contrast
  4. Expand the button
  5. The navigation will expand.
  6. Note the right border is visible. All icons will be high contrast.
  7. Open a submenu
  8. All text and icons should be visible
  9. Navigate to your node.
  10. Icons and text should be visible in top bar.
  11. Badge should have border
  12. Contextual dropdown should have border

Testing steps: wide

  1. Emulate forced colors in dev tools https://developer.chrome.com/docs/devtools/rendering/emulate-css
  2. Be logged in
  3. Note the left collapsed sidebar. All icons should be visible, have hover and focus states
  4. Expand a submenu. Border and any chevrons should be visible
  5. Expand the sidebar.
  6. Check that all icons and text are visible as expected
  7. Expand submenus. Everything should be visible as expected
  8. Navigate to your node, and check the top bar. All icons should be visible
  9. Dropdowns should have borders
CommentFileSizeAuthor
#51 3438477-nr-bot_ncxblslt.txt91 bytesneeds-review-queue-bot
#50 current_page_fc_col.jpg8.08 KBrkoller
#50 12-after-col.jpg8.44 KBrkoller
#50 12-after-exp.jpg17.98 KBrkoller
#50 current_page_fc_exp.jpg17.09 KBrkoller
#50 current_page_reg_col.jpg7.26 KBrkoller
#50 current_page_reg_exp.jpg15.43 KBrkoller
#48 Content_types___Drush_Site-Install_and_toolbar-button_pcss_css_—_drupal.png77.25 KBmherchel
#45 11.2-after.jpg13.15 KBrkoller
#45 11.1-after.jpg29.14 KBrkoller
#45 10.2-after.jpg36.32 KBrkoller
#45 10.1-after.jpg25.52 KBrkoller
#45 9-after.jpg42.4 KBrkoller
#45 8-after.jpg8.2 KBrkoller
#45 7-after.jpg8.76 KBrkoller
#45 6-after.jpg25.37 KBrkoller
#45 5-after.jpg7.96 KBrkoller
#45 2-after.jpg54.63 KBrkoller
#45 11.2-before.jpg12.84 KBrkoller
#45 11.1-before.jpg29.01 KBrkoller
#45 10.2-before.jpg29.03 KBrkoller
#45 10.1-before.jpg9.05 KBrkoller
#45 9-before.jpg42.61 KBrkoller
#45 8-before.jpg7.87 KBrkoller
#45 7-before.jpg8.53 KBrkoller
#45 6-before.jpg24.29 KBrkoller
#45 5-before.jpg7.55 KBrkoller
#45 2-before.jpg37.09 KBrkoller
#44 coloring.jpg30.55 KBrkoller
#43 collapsed_after2.jpg13.31 KBrkoller
#43 expanded_after2.jpg29.27 KBrkoller
#42 collapsed_after.jpg12.31 KBrkoller
#42 expanded_after.jpg28.53 KBrkoller
#42 collapsed_before.jpg13.93 KBrkoller
#42 expanded_before.jpg27.86 KBrkoller
#42 desert_icon.jpg11.97 KBrkoller
#42 rendering_icon.jpg12.47 KBrkoller
#38 tooltip_fixed.jpg8.92 KBrkoller
#36 test___Drush_Site-Install.png830.54 KBmherchel
#35 focus.mp4134.77 KBrkoller
#35 skip.jpg37.1 KBrkoller
#35 tooltip_regular.jpg8.42 KBrkoller
#35 tooltip.jpg8.47 KBrkoller
#35 dark_mode.jpg14.15 KBrkoller
#35 white.jpg13.93 KBrkoller
#35 aquatic.jpg12.92 KBrkoller
#35 night_sky.jpg14.59 KBrkoller
#35 desert.jpg12.01 KBrkoller
#35 dusk.jpg14.15 KBrkoller
#27 workspaces.jpg34.61 KBrkoller
#27 top_bar_icon.jpg21.78 KBrkoller
#25 3438477-25-activetext.firefox-mac-light.png22.28 KBkentr
#25 3438477-25-activetext.firefox-mac-dark.png22.12 KBkentr
#25 3438477-25-activetext.page-colors-dusk.png174.71 KBkentr
#23 3438477-23-menu-close-and-chevron-icons.png47.17 KBkentr
#22 3438477-22-menu-border-and-user-icon.png100.97 KBkentr
#18 mobile_border_interrupted.png203.04 KBm4olivei
#18 mobile_hamburger_barely_visible.png829.73 KBm4olivei
#18 mobile_close_button_barely_visible.png202.81 KBm4olivei
#16 carent_expand_collapse_button.png15.19 KBm4olivei
#16 drawer_border_needed.png404.7 KBm4olivei
#16 carets_barely_visible.png440.47 KBm4olivei
#12 Screenshot 2024-06-06 at 9.47.50 AM.png200.36 KBgauravvvv
#6 render.jpg251.51 KBrkoller
#6 quickview.jpg51.37 KBrkoller
#3 mobile_expanded.jpg11.21 KBrkoller
#3 mobile_collapsed.jpg11.28 KBrkoller
hover_collapsed.jpg5.63 KBrkoller
hover_expanded.jpg22.26 KBrkoller
expanded_drawer.jpg53.02 KBrkoller

Issue fork drupal-3438477

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

rkoller created an issue. See original summary.

rkoller’s picture

Issue summary: View changes
rkoller’s picture

Issue summary: View changes
StatusFileSize
new11.28 KB
new11.21 KB

forgot the mobile viewport. made a few additions to the issue summary

kostyashupenko’s picture

mmm, i can't reproduce it @rkoller (i mean i can't reproduce almost invisible icons)

so you said:

enable automatic dark mode ticked, forced colors active and preer contrast set to more

In the system OS settings on Mac OS i have enabled dark mode
in MS Edge browser i have enabled forced-colors: active
and this preer contrast set to more i think you were talking about system OS settings again? I see range bar in the settings to increase contrast (or this setting somehow i can apply inside of MS Edge)?

ckrina’s picture

rkoller’s picture

StatusFileSize
new51.37 KB
new251.51 KB

no all those three settings were set within ms edge (@mherchel provided the instructions for this handy feature testing high contrast mode a while ago on slack if i remember correctly). you have to add the Rendering tab either to the quick view or move it to the activity bar on top in the ms edge devtools:

expanded context menu showing all the available tabs to add to the quickview in ms edge on mac

and i then only set and adjust those three settings highlighted:

on the left the conten blocks page in high contrast mode on the right the dev tools with the render tab highlighting in red rectangels the three settings to activate high contrast mode - automatice dark mode enabled, forced color active and prefers contrast more

katannshaw’s picture

Issue summary: View changes

Added WCAG 2.2 success criteria and moved steps to reproduce into its own section for easier readability.

rkoller’s picture

Issue tags: +WCAG 143, +wcag1411
ckrina’s picture

Project: Navigation » Drupal core
Version: 1.x-dev » 11.x-dev
Component: User interface » navigation.module
ckrina’s picture

Issue tags: +Portland2024
kostyashupenko’s picture

I can't find solution for this, maybe some hints?

Maybe we can replace CSS mask-image with embedded svg icons ? Or to use iconic font and content css property (but embedded technology is better)

gauravvvv’s picture

Issue summary: View changes
StatusFileSize
new200.36 KB

I have updated the latest screenshot in the issue summary

gauravvvv’s picture

Issue summary: View changes
mgifford’s picture

For folks testing in high contrast, here is a guide.
https://www.lambdatest.com/blog/windows-high-contrast-mode/

m4olivei’s picture

We recently landed #3483209: Navigation leverage icon core API, which had a positive impact on Windows High Contrast mode. I'll be doing some testing on the latest 11.x with the goal of updating the issue description to reflect the outstanding issues.

m4olivei’s picture

Issue summary: View changes
StatusFileSize
new440.47 KB
new404.7 KB
new15.19 KB
m4olivei’s picture

Issue summary: View changes
m4olivei’s picture

Added a few more issues that I noticed.

m4olivei’s picture

Issue summary: View changes
rinku jacob 13’s picture

kentr’s picture

Issue summary: View changes

I believe #4 in the IS (regarding the outline of the hovered menu entry) is caused by automatic dark mode. I can't reproduce it in basic forced colors emulation or MacOS Edge with Page Colors (dark themes on both).

I suggest removing that item. @rkoller, does this sound right, and do you agree that item can be removed?

Is #6 in the IS referring to the hamburger icon that #8 refers to?

I'm also adding Page Colors to the IS for seeing forced-colors mode. It's a little nicer DX than forced-colors emulation in DevTools.

kentr’s picture

Issue summary: View changes
StatusFileSize
new100.97 KB

For IS #9 (gap in menu right border) I'm only seeing the issue in Firefox (Mac).

Also noticed another issue: The icon next to the username at top is hard to see. It may not be a UX / accessibility problem, though.

Screenshot attached from Chromium.

screenshot of open menu in a chromium browser, with menu border and user icon highlighted

Tweaking the STR a little more.

kentr’s picture

StatusFileSize
new47.17 KB

I retract my previous comment about the interrupted menu right border. I now see it in Edge with Page Colors.

Suggestion for the remaining hard-to-see inline SVG icons:

  • For the color issue, use currentColor in some form.
  • For the chevrons, give them a stroke with a larger width.

Here's POC CSS:

@media (forced-colors: active) {
    svg[class*="toolbar"] path[stroke] {
        stroke: currentColor
    }
    
    svg[class*="toolbar"] path[fill] {
        fill: currentColor
    }

    :is(
        .toolbar-button__chevron, 
        .admin-toolbar__expand-button-chevron
    ) path {
        stroke-width: 5%;
        stroke: currentColor;
    }
}

Screenshot:

screenshot of icons with suggested CSS applied.

rkoller’s picture

thanks for getting some momentum back into this issue @kentr!

re #21 yep in regard to point 4 i agree, good catch. just retested without the automatic dark mode and the hover looks totally fine and clearly visible to me. on the other hand the active menu item for the current site is close to invisible, but that has already an issue for the none forced color mode (#3426468: Make the active menu item visually stand out more). so i wonder about two things, should the forced color mode with the active menu item be fixed in the aforementioned issue or within the scope of this issue? and in regard to updating the issue summary, instead of removing point 4 i wonder would it be the better choice to strike it through and leave a note. that way the references of the others points after the current point 4 wouldnt have to be updated throughout the comments?

in regard to point 6 in the issue summary. i have to admit writing up that issue was a while ago and the issue summary also got updated by other people. not sure i did point 8 back then, that was probably added afterwards by someone else? in regard to point 6, that screenshot was taken with the automatic dark mode checkbox ticked and as outlined it was about the barely visible hamburger icon AND the missing top bar border. you were unable to distinguish what is the page and what is the top bar. but looking at the site without the automatic forced color mode active the issue about distinguishing the top bar due to a not visible border is not an issue without the automatic forced color mode. or wait i was too hasty. the missing top bar border is still an issue for desktop viewports! there it is missing while it is visible for the mobile view port. maybe point 6 should be rescoped to the not visible bottom border in desktop view port and point 8 should be solely about the hamburger. that way the two images used are more appropriate? if you agree i can update both points in a follow up comment.

re #22 yep i've noticed that as well now. in your example you are referring to the icon in the top bar on a user profile page but the invisible icon also applies to node edit forms. they are using the same image there as well. and the gap i am also still experiencing for the mobile view port in edge.

kentr’s picture

@rkoller

in regard to updating the issue summary, instead of removing point 4 i wonder would it be the better choice to strike it through and leave a note

👍 Good idea.

on the other hand the active menu item for the current site is close to invisible, but that has already an issue for the none forced color mode (#3426468: Make the active menu item visually stand out more). so i wonder about two things, should the forced color mode with the active menu item be fixed in the aforementioned issue or within the scope of this issue?

I don't know. It may need more than a system-color change.

I tested ActiveText in all Edge Page Colors themes and using the emulation in Chromium DevTools. The color is indistinguishable from LinkText. Shouldn't ActiveText be a distinct color?

There was a Chromium bug related to this.

Here's a screenshot in Page Colors Dusk theme. The "Appearance" item is current and has the following CSS.

@media (forced-colors: active) {
    .toolbar-button.current {
        color: ActiveText;
    }
}

screenshot of menu item colored with activetext system color, in page colors dusk theme

Firefox uses a distinct color for ActiveText.

screenshot of menu item colored with activetext system color, in firefox mac dark theme

screenshot of menu item colored with activetext system color, in firefox mac light theme

maybe point 6 should be rescoped to the not visible bottom border in desktop view port and point 8 should be solely about the hamburger. that way the two images used are more appropriate? if you agree i can update both points in a follow up comment.

👍 Sounds good.

Adding a new issue: More actions button should have a border (related Slack discussion).

This may be fixed by #3540337: Improve visibility of More actions menu button, but does the menu sidebar "Close" icon button also need a border?

kentr’s picture

Did a little research, and I believe I misunderstood ActiveText. Apparently it's for links that are being activated (pressed), not for the "active" link that corresponds to the current page.

rkoller’s picture

Issue summary: View changes
StatusFileSize
new21.78 KB
new34.61 KB

made the following changes to the issue summary

  • striked point 4 as outdated and added a note
  • rescoped point 6, making it about the missing bottom border on the desktop view port
  • added point 10, about the invisible icon @kentr raised in #22

and there is one more potential detail that could be added to the issue summary, the background color for the workspace is not displayed in forced color mode. in "regular" mode you have a green background for the workspace that is live while all the other none live workspaces have an orange background (SC1.4.1 might apply here?). in addition to that the yellow font color used on the live workspace implies that it would lead to a page but strictly speaking just an area on top of the current page is displayed.

the top of the user profile page in forced color mode with the live menu item on top in the navigation sidebar missing the green background

kentr’s picture

in "regular" mode you have a green background for the workspace that is live while all the other none live workspaces have an orange background (SC1.4.1 might apply here?).

Looks like that is mentioned in #2910673: Workspace UI usability review.

in addition to that the yellow font color used on the live workspace implies that it would lead to a page but strictly speaking just an area on top of the current page is displayed.

I'd vote for making that element a progressively-enhanced button like in the MR for #3090120: Improve accessibility semantics for Toolbar buttons with trays.

Then the style can be something like this:

@media (forced-colors: active) {
  .toolbar-button[role="button"] {
    color: buttontext;
  }  
}

I'm uncertain if this is a Navigation issue or a Workspaces issue. It would be nice if Navigation handled it for all menu components.

I'd also really like to see this pattern be global, with a policy of progressively-enhancing all cases like this. Then it could be:

@media (forced-colors: active) {
  a[role="button"] {
    color: buttontext;
  }  
}

meeni_dhobale made their first commit to this issue’s fork.

mherchel changed the visibility of the branch 3438477-high-contrast-mode to hidden.

mherchel’s picture

Status: Active » Needs review

Merry Christmas and Happy Holidays!

Changes for forced-colors are made. This mainly involves adding transparent borders (which are made visible in forced-colors), and adding proper fills for SVGs.

mherchel’s picture

Issue summary: View changes
Issue tags: -Needs issue summary update +🎅

Adding testing instructions to IS

mherchel’s picture

Issue summary: View changes

Added a walk though of testing at https://www.youtube.com/watch?v=v4dTY4__F6U

rkoller’s picture

Status: Needs review » Needs work
StatusFileSize
new14.15 KB
new12.01 KB
new14.59 KB
new12.92 KB
new13.93 KB
new14.15 KB
new8.47 KB
new8.42 KB
new37.1 KB
new134.77 KB

that looks fab and really close! as already mentioned on slack i really liked the additional testing instructions you've provided as well as the youtube video as a linkable resource to others to quickly illustrate how to test forced color mode. i've then went ahead and manually tested in edge. I've noticed a few minor details that might require some more work (some might be out of scope) but overall this look really close:

point 5 from the issue summary still requires work. (using dark color scheme with forced color active and prefers contrast set to more)
the collapsed navigation sidebar only showing icons and the appearance top level menu item hovered showing the appearance tooltip in forced color mode
i guess the easiest would be by simply adding an outline to the tooltip in forced color mode - the inverting background color pattern wouldn't be viable nor solvable across the possible different forced color choices imho. for regular mode the tooltip uses a dark dark background inverting the white background for example.
the collapsed navigation sidebar only showing icons and the appearance top level menu item hovered showing the appearance tooltip

point 10 from the issue summary also requires work. the icon in the top bar is not visible for every default page color theme. for some light color themes it is visible but for the dark variants it is completely invisible:

Dusk (settings -> accessibility - page colors)
the topbar in forced color mode with dusk page color theme

Desert (settings -> accessibility - page colors)
the topbar in forced color mode with desert page color theme

Night Sky (settings -> accessibility - page colors)
the topbar in forced color mode with night sky page color theme

Aquatic (settings -> accessibility - page colors)
the topbar in forced color mode with aquatic page color theme

White (settings -> accessibility - page colors)
the topbar in forced color mode with white page color theme

dark color scheme with forced colors active and prefers contrast more in devtools (page colors are off in the accessibility settings)
the topbar in dark color scheme with forced colors active and prefers contrast set to more

Out of scope for this issue, but one detail that is highlighted again in the examples for point 10 in forced color, is the need to increase at least the size of the dots on the more actions button ( see #3540337: Improve visibility of More actions menu button). The dots are shown in the correct color across the different page color choices but they are barely visible nor noticeable.

Also out of the scope for this issue, and I am uncertain if there is already an issue for that (i "thiiiiink" there might be, but was unable to find it yet), with the navigation sidebar collapsed you are able to tab through the top level menu items of the sidebar. so you have several tab presses with no visible focus outline (see focus.mp4)

One additional detail I've noticed, the Skip to main content link is missing a visible focus outline while in regular mode the focus outline is visible:
a node edit page in mobile vuw with the top bar collapsed and the skip to main content link on top of the expand sidebar button

mherchel’s picture

StatusFileSize
new830.54 KB

The three dots in the top bar's contextual menu is responding to the forced colors. It's set to buttonText which is a system color in CSS that responds to forced colors. I think it's just small (as you said), so it's not highly visible.

Pushing a fix for the tooltip in a sec!

mherchel’s picture

Status: Needs work » Needs review

Tooltip border added. Hopefully good to go!

rkoller’s picture

StatusFileSize
new8.92 KB

tooltip looks good now:
collapsed navigation sidebar in forced colors mode with the appearance tooltip shown for the appearance  top level menu item
thank you!

mherchel’s picture

Pushed another commit ensuring that the title icon is visible in high contrast. Hopefully good to go!

kentr’s picture

Also out of the scope for this issue, and I am uncertain if there is already an issue for that (i "thiiiiink" there might be, but was unable to find it yet), with the navigation sidebar collapsed you are able to tab through the top level menu items of the sidebar. so you have several tab presses with no visible focus outline

That sounds like #3541910: Elements in closed sidebar are focusable.

mherchel’s picture

Pushed another commit to ensure that the toolbar-button component uses the proper system color within forced colors (e.g. linkText for links and buttonText for buttons)

rkoller’s picture

StatusFileSize
new12.47 KB
new11.97 KB
new27.86 KB
new13.93 KB
new28.53 KB
new12.31 KB

i can confirm that the last open issue, point 10 from the issue summary, is fixed now as well:

the topbar in forced color mode
the topbar in forced color mode with the desert theme

and while testing with the navigation sidebar collapsed by accident in forced color mode i've noticed one other issue i've raised in the dicussion with @mherchel on slack. in the expanded sidebar the labels of top level menu item with a submenu and those without have different colors while the all the icons employ the same color.
the expanded navigation sidebar in forced color mode

So color is conveying additional meaning that is lost in the collapsed state:
the collapsed navigation sidebar in forced color mode

with the recent changes by @mherchel icons are using the appropriate colors:
the expanded navigation sidebar in forced color mode
the collapsed navigation sidebar in forced color mode

In regard to the focus related issues i've agreed with mike on the following. the "skip to main content" link is not styled in the navigation module but the admin theme and should therefore be fixed in claro in a followup issue. same for the dotted focus outlines, those should also be fixed in a followup issue in the claro queue.

and thanks to @kentr for digging up the correct issue! i knew that there was one. was away from the issue queue for too long the last few months. so for the tabable top-level menu items in the collapsed mobile menu the followup is #3541910: Elements in closed sidebar are focusable. so the navigation sidebar and the top level sidebar look good to go in forced color mode. i will update the issue summary tomorrow.

rkoller’s picture

StatusFileSize
new29.27 KB
new13.31 KB
rkoller’s picture

Issue summary: View changes
StatusFileSize
new30.55 KB

updated the steps to reproduce section and striked every issue that isn't reproducible in 11.x anymore and add the additional problem that surfaced in #42. will add before and after screenshots for the remaining issue in my next comment.

rkoller’s picture

Issue summary: View changes
StatusFileSize
new37.09 KB
new7.55 KB
new24.29 KB
new8.53 KB
new7.87 KB
new42.61 KB
new9.05 KB
new29.03 KB
new29.01 KB
new12.84 KB
new54.63 KB
new7.96 KB
new25.37 KB
new8.76 KB
new8.2 KB
new42.4 KB
new25.52 KB
new36.32 KB
new29.14 KB
new13.15 KB

i was over eager, reverted the striking of point 6, it still applies on the desktop viewport. aside that i'Ve added a user interface changes section to the issue summary with before and after screenshots for easier review.

mherchel’s picture

Is there anything left to do?

rkoller’s picture

Finally finished writing up the two followup issues.

#3565271: The skip to main content link is missing a visible focus outline in forced color mode
#3565294: Increase the contrasting area of focus outlines in forced color mode

plus the issue @kentr linked:

#3541910: Elements in closed sidebar are focusable

not sure if it is enough to link them as related followups or if all three should be also added to the issue summary? the only open question as discussed on slack is the question how to label the current page in the navigation sidebar.

mherchel’s picture

Pushed a new commit adding a left border for the active menu item.

mgifford’s picture

This looks good to me. Thanks for all the work improving this!

rkoller’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new15.43 KB
new7.26 KB
new17.09 KB
new17.98 KB
new8.44 KB
new8.08 KB

i've asked for feedback by @kentr and @mgifford in regard of the latest changes about how to label the current page in forced color mode and if it is a reasonable approach to solve this problem - everyone agreed. So I've updated the issue summary, and added how to label the current page as point 12. Overall this issue looks ready to go. Per the discussion with @mherchel i'll set it to RTBC. and also thanks again to @mherchel for all the work over last days pushing this issue to the finish line!

needs-review-queue-bot’s picture

Status: Reviewed & tested by the community » Needs work
StatusFileSize
new91 bytes

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

kentr’s picture

It looks like performance tests need to be updated for the CSS changes. That should make the pipeline pass:

https://git.drupalcode.org/issue/drupal-3438477/-/pipelines/702218/test_...

kentr’s picture

I don't know what caused #51, though.

mherchel’s picture

Status: Needs work » Reviewed & tested by the community

Woohoo! Tests passed!

Setting back to RTBC per #50.

The only changes are changing target of MR to main, fix tests, and resolve merge conflict (which was minor).

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

  • lauriii committed 18202df4 on main
    feat: #3438477 High contrast mode needs some more refinement
    
    By:...

  • lauriii committed 64451de0 on 11.x
    feat: #3438477 High contrast mode needs some more refinement
    
    By:...

lauriii’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for working on this! Committed 🚀

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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