--panel-description-color is currently using system color GrayText reserved for disabled items only
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: jules, Assigned: dao)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
global-shared.css
currently sets --panel-description-color
to GrayText
. Accessibility recommends we reserve GrayText
for disabled or placeholder items only.
Updated•2 years ago
|
Comment 1•2 years ago
|
||
Hey Jules, do you have a recommendation for what color we should use here instead?
Reporter | ||
Comment 2•2 years ago
|
||
Hey Amy, last time I spoke to Dao about this he recommended inherit
since panels already have a color set. Thanks for asking, I forgot to include that.
Comment 3•2 years ago
|
||
Looks like we use GreyText
and a hsl grey hsl(0,0%,50%);
for MacOS. I assume we want to update both? (By removing the MacOS override)
Assigning P2 to this - its a simple patch and a11y issue we should get to sooner rather than later.
Reporter | ||
Comment 4•2 years ago
|
||
Thanks, Sam. Yes, we should update both places.
Assignee | ||
Comment 5•2 years ago
|
||
(In reply to Sam Foster [:sfoster] (he/him) from comment #3)
Looks like we use
GreyText
and a hsl greyhsl(0,0%,50%);
for MacOS. I assume we want to update both? (By removing the MacOS override)
Assigning P2 to this - its a simple patch and a11y issue we should get to sooner rather than later.
Yeah, we should use something like color-mix(in srgb, currentColor 70%, transparent)
instead of GrayText
on all platforms when not in HCM, and inherit
in HCM.
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 6•2 years ago
|
||
Updated•2 years ago
|
Comment 8•2 years ago
|
||
bugherder |
Updated•1 years ago
|
Description
•