[HCM] Resize annotation icon difficult to see
Categories
(Core :: Layout, defect)
Tracking
()
Accessibility Severity | s3 |
People
(Reporter: morgan, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
4.61 KB,
image/png
|
Details |
STR:
- Open a PDF w/
pdfjs.annotationEditorEnabled
enabled - Using the pencil tool, scribble on the PDF 😀
- Select the annotation
Expected:
The resize icon in the lower right of the annotation uses colors from the HCM color palette
Actual:
It stays grey/white regardless of user's chosen colors
We should either use Canvas
and CanvasText
here or ButtonFace
and ButtonText
.
This is the default FF resize icon, so filing in general themeing instead of PDF specifically. Maybe this belongs in layout?
Updated•3 years ago
|
Comment 1•3 years ago
|
||
The severity field is not set for this bug.
:dao, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•2 years ago
|
Updated•2 years ago
|
Comment 2•2 years ago
|
||
The pdf editor is using the default resizer so I think this bug isn't specific to the pdf editor.
For example, I screenshoted a textarea on Bugzilla with a visible resizer in HCM (Aquatic theme) on Windows 11 and I don't think that it's really visible for people with visual disabilities.
Comment 3•2 years ago
|
||
Dao, should this be in Firefox::Theme or somewhere in Layout?
Updated•2 years ago
|
Comment 4•2 years ago
|
||
No answer from Dao, I'll move this back to Firefox::Theme for now.
Since https://github.com/mozilla/pdf.js/pull/16732 we are no longer using the resize property in PDF.js, so the STRs are no longer valid, but the bug is reproducible on https://developer.mozilla.org/docs/Web/CSS/resize.
Comment 5•2 years ago
|
||
The severity field is not set for this bug.
:dao, could you have a look please?
For more information, please visit BugBot documentation.
Comment 6•2 years ago
|
||
Moving this to Core::Layout where the resizer
styling lives: https://searchfox.org/mozilla-central/rev/dc8348b3730c0d29dafd01c653d9151eaa9bc30f/layout/style/res/scrollbars.css#86
Not sure if context-fill
could be used there, the fill color is currently hardcoded in the image: https://searchfox.org/mozilla-central/source/toolkit/themes/shared/icons/resizer.svg
Comment 7•1 years ago
|
||
The severity field is not set for this bug.
:TYLin, could you have a look please?
For more information, please visit BugBot documentation.
Updated•1 years ago
|
Description
•