Closed Bug 1830850 Opened 1 year ago Closed 1 year ago

Annotations popups and icons must have a style for HCM

Categories

(Firefox :: PDF Viewer, defect, P3)

Firefox 114
Desktop
Windows
defect

Tracking

()

VERIFIED FIXED
117 Branch
Accessibility Severity s3
Tracking Status
firefox-esr115 --- wontfix
firefox114 --- wontfix
firefox115 --- wontfix
firefox116 --- wontfix
firefox117 --- verified
firefox118 --- verified

People

(Reporter: calixte, Assigned: calixte)

References

Details

(Keywords: access)

Attachments

(4 files)

Attached file annotation-fileattachment.pdf (deleted) —

STR:

  • switch to HCM
  • open the attached pdf
  • click on the pin

The popup isn't clearly visible:

  • there is no border
  • the background should probably be in a different color

The icon for the pin isn't really visible.

(In reply to Calixte Denizet (:calixte) from comment #0)

The icon for the pin isn't really visible.

Please note that this icon is actually specified by the PDF document itself, via the /AP dictionary of the annotation.

Attached file annotation-link-text-popup.pdf (deleted) —

That's true and it means that we must apply the filter to it when it's drawn.
I thought it was an icon we embed.
Anyway we should have one for the hcm case: right now, the icon isn't drawn at all (see annotation-link-text-popup.pdf) or draw it with a filter.

Assignee: nobody → cdenizet
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch

The pop-up hasn't been done for the moment, hence I reopen this bug.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 115 Branch → ---

Looks to me like an a11y-s3 - the note appears as a custom tooltip, so this element is:

  1. missing border that should use CanvasText. The text and the background appear to be using proper Canvas colors.
  2. On the second example, annotation-link-text-popup.pdf, The image mark (sticker icon) itself now uses static/Canvas+CanvasText colors. In the stable 113 and 114.0.1 releases the icon that has the tooltip/note attached is created using ButtonFace+ButtonText color pair which is a good thing, IMO, as it signals to the user that it is actionable and they can hover over it. But Fx 115 uses static Canvas+CanvasText colors for it, which does miss the point of the image control.
  3. The image mark does not have a hover HCM styling by itself, but the tooltip with the note appears on hover so I’d say it’s not a blocker issue. Expected hover styles for regular HCM button can be found in the Acorns' Components > Button > States > Hover section
Accessibility Severity: --- → s3
Status: REOPENED → RESOLVED
Closed: 1 year ago1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
Flags: qe-verify+

The mentioned changes have been introduced in 3 steps: in branch 115 the link hover effect as a different background color, in branch 116 the tooltip border, in branch 117 the icons hover effect and the correct color for the link hover effect. The tooltip's background color remains the same as the page's background color, but I deem it unnecessary considering the border is visible enough.
Based on the above, I consider branches 117 and 118 as verified (Nightly v118.0a1 and Beta v117.0b2) in Windows 10 with 4 different HCM profiles.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
OS: Unspecified → Windows
Hardware: Unspecified → Desktop
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: