Closed Bug 1578334 Opened 7 months ago Closed 6 months ago

NVDA reports confusing status to user when enabling or disabling ETP from the protections panel

Categories

(Firefox :: Protections UI, defect, P1)

Unspecified
Windows
defect

Tracking

()

RESOLVED FIXED
Firefox 71
Tracking Status
firefox70 + fixed
firefox71 --- fixed

People

(Reporter: asa, Assigned: timhuang)

Details

(Keywords: access, Whiteboard: [privacy-panel][skyline][access-p1])

Attachments

(1 file)

When disabling or enabling ETP, the screen reader user gets confusing announcements that suggest the operation didn't work, at least at first. A moment later the user gets a second announcement that the operation did work.

@jteh said:

  1. In the panel, you toggle protection.
  2. Focus returns to the protection button on the toolbar. However, the tooltip on the button hasn't been updated yet.
  3. The tooltip updates. NVDA reports the updated tooltip.

Fixing this is going to require some kind of timing dance. We need to ensure the tooltip gets updated before the button gets focus after the panel is dismissed. That means either:

  1. Updating the tooltip earlier.
  2. Closing the panel when we do now and somehow delaying focus of the button.
  3. Delaying closing of the panel until the tooltip updates, thus delaying return of focus to the button.
Whiteboard: [skyline] → [privacy-panel][skyline]

Seems like we could maybe artificially change the tooltip before the 500ms wait - Tim, could you comment on this?

Flags: needinfo?(tihuang)
Priority: -- → P1

I think we can update the tooltip before the 500ms wait. But, I got a little confused regarding the tooltip of the protection button which Asa mentioned. Did you mean the tooltip of the tracking protection icon or something else?

Flags: needinfo?(tihuang) → needinfo?(asa)

If you meant the tooltip of the tracking protection icon, I think we can immediately change it to the tooltip of the disabled protections when toggling to the off state before the 500ms wait. And same for toggling to ON state, we can change it to the tooltip of the 'No trackers detected' and it will get updated when the first onContentBlockingEvent happens.

Flags: needinfo?(asa)
Whiteboard: [privacy-panel][skyline] → [privacy-panel][skyline][access-p1]

I don't quite understand what's happening here and filed this bug with information from Jamie. Jamie, can you answer Tim's question?

Flags: needinfo?(jteh)

Yes, we're referring to the tooltip of the tracking protection icon; i.e. the tooltip of #tracking-protection-icon-container.

Flags: needinfo?(jteh)

Tracking for 70 release - we can still accept a patch for this issue in 70, preferably next week for beta 11 or 12.

Flags: needinfo?(jhofmann)

Tim, did you intend to work on this? :)

Flags: needinfo?(jhofmann) → needinfo?(tihuang)

I will work on this. I'll try to fix this if possible before next week.

Assignee: nobody → tihuang
Status: NEW → ASSIGNED
Flags: needinfo?(tihuang)

In order to show a correct tooltip after toggling the TP switch. We
update the tooltip of the tracking protection icon when toggling the TP
switch right before the 500 ms waiting.

Pushed by tihuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8602fcecc75
Update the tooltip of the tracking protection icon when toggling the TP switch. r=nhnt11
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

Hi James,

Would you be able to verify if this issue been fixed? Thanks.

Flags: needinfo?(jteh)

I can confirm that this is no longer confusing like it was. With today's Nightly build, now the screen reader communicates the proper state after switching from on to off and off to on.

Can you request beta uplift? Thanks!

Flags: needinfo?(jteh)

Comment on attachment 9097322 [details]
Bug 1578334 - Update the tooltip of the tracking protection icon when toggling the TP switch. r?nhnt11!

Beta/Release Uplift Approval Request

  • User impact if declined: Confusing feedback to screen-reader users when ETP is toggled.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): The patch mainly moves some code to execute before some other code. Not too risky, verified by Asa in comment 13.
  • String changes made/needed: None
Attachment #9097322 - Flags: approval-mozilla-beta?

Comment on attachment 9097322 [details]
Bug 1578334 - Update the tooltip of the tracking protection icon when toggling the TP switch. r?nhnt11!

Fix for a11y issue for skyline feature, OK for uplift for beta 12.

Attachment #9097322 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: needinfo?(tihuang)
You need to log in before you can comment on or make changes to this bug.