Problem/Motivation
During last weeks accessibility walkthrough, @mgifford and @mherchel helped identify several contrast issues and how to test for them in Chrome - regardless of OS! No more Virtualbox/Parallels (for this at least).
Steps to reproduce
In the Chrome developer tools render config, test with just forced colors

Then test with forced colors + light/dark schemes

Here's an example of a problem that surfaces with a dark scheme

Here's an example of a problem that surfaces with a light scheme

Bugs addressed in this issue:
Icons:
- Advanced filters icon
- Close icon on applied filter pills
- List and Grid icons
- Project usage icon on module page for dark mode
- Project usage icon in list view for light color scheme
- Copy icons on instructional modal in light color scheme
Borders:
- Filter button
- Applied filter pills
- List and Grid buttons
- Categories on module page
- Categories on module cards
- Commands in instructional modal
Proposed resolution
- Assess all the forced colors related issues and determine if it's better to address them all here, or turn this issue into a meta and move problems to dedicated issues
- If this doesn't become a meta issue, each find should be added to the issue summary so we don't need to sift through comments to find the contrast bugs that need addressing.
- Look for missing/obscured borders, text icons and other image use essential to understanding the UI.
- Use the forced colors media query for changes that should only be present in forced colors mode
- Check out how the forced color media query is used in Claro and Olivero for ways of addressing the issue
Comments
Comment #4
srishtiiee commentedComment #5
srishtiiee commentedFixed a couple of forced color contrast bugs:
Comment #6
narendrarChanges looks good to me.
Found some more places where this can be implemented:
Comment #7
srishtiiee commentedAddressed all the remaining bugs.
Comment #8
narendrarChanges looks good to me. Marked issue as RTBC.
Comment #9
bnjmnmLeft a few comments in the MR.
Also, lets do the
filter: invert(1);treatment to the "Covered by Drupal Security Team" icons on the projects and in the advanced filter area.It would be nice to do something similar with the magnifying glass icon in the search input, but it would require a different solution due to the img src being svg. If a solution isn't quickly apparent, file a follow up and mention it here as I don't want to delay the good additions here.
Comment #10
srishtiiee commentedComment #11
bnjmnmI took care of the search and security coverage icons.

This means I'm not able to RTBC/commit, but due to Chrome's recent improvements with forced colors testing (documented in the issue summary), this should be a pretty easy review for anyone. All the CSS changes from @srishtiiee are something I'd commit, so a reviewer could largely focus on the dark color scheme detection and the image swaps that result from it.
Comment #12
narendrarFound couple of issues/suggestions:

1. Text size is different
2. Buttons can have borders to give better visibility.
Comment #13
narendrarComment #14
bnjmnmComment #15
srishtiiee commentedComment #16
narendrarChanges looks good to me. Marking as RTBC.
Comment #19
tim.plunkettMerged, thanks!