- π¨π¦Canada mgifford Ottawa, Ontario
My mistake on tagging this 131. I should have gotten back to Andrew on that sooner.
- Assigned to kentr
- πΊπΈUnited States kentr Durango, CO
I'm going to try to move the MR forward a little.
Adding Needs issue summary update because the IS mentions IE11 and AFAIK Drupal dropped support for IE11.
- Merge request !11498Higher contrast shortcut icons for Claro and shortcut module, 11.x β (Open) created by kentr
- πΊπΈUnited States kentr Durango, CO
There were merge conflicts when updating the existing MR branch, so here's a new MR against
11.x
.Some comments
- I went back to the non-concentric fill for the "remove" states as specified by @ckrina's comment #16.
- For the "add, hover" state, the previous MR had a solid fill for
forced-colors: active
.
That wasn't in @ckrina's screenshots, and due to the+
above the star in the Claro icons I wondered if it is necessary. So I left the fill empty for now as shown in the screenshots. - Inside the CSS
url()
function, the SVG with aforced-colors: active
media query embedded in astyle
element did not display theLinkText
color as expected.
However, there are many instances in core of usingmask-image
forforced-colors: active
, so I went with that method. - For the time being, I've put the same icons into both Claro and the shortcut module.
I suggest creating a lower-priority followup to refine the shortcut module's icons since they'll appear on Olivero (which has different blues).
These screenshots are from Mac Firefox in standard mode and with
browser.display.document_color_use
set to2
.Needs manual testing with & without forced-colors / WHCM. I will update the IS with details.
- πΊπΈUnited States smustgrave
Have not reviewed but previously tagged for issue summary update
Tried to cleanup what I assume are old tags.
- πΊπΈUnited States kentr Durango, CO
Updated the IS, included testing notes, and attached the screenshots that didn't save for #57.
- πΊπΈUnited States kentr Durango, CO
Forgot to change status / remove IS update.