Open Bug 1453913 Opened 6 years ago Updated 8 months ago

[Accessibility] Unlabeled button to the left of the security info graphic in the address bar.

Categories

(Firefox for iOS :: Menu and Toolbar, defect, P3)

Other
iOS
defect

Tracking

()

REOPENED
Tracking Status
fxios 12.0 ---

People

(Reporter: MarcoZ, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(1 file)

55 bytes, text/x-github-pull-request
mohitagarwal92
: review?
farhan
Details | Review
STR:
1. Run Firefox 11 for iOS.
2. Turn on VoiceOver.
3. Touch the address bar, and hear VoiceOver announce it.
4. Swipe left. If this is a secure site, we will land on the lock icon, which will be spoken.
5. Swipe left again.

Expected: I should either be at the beginning, or hear a label, followed by Button.

Actual: I only hear the word Button being spoken.

This button is always the first thing that gets focus when a new page loads. When I double-tap it, I am just being taken to the address bar.

So, since this does the same as the address bar itself, this should be either IsAccessibilityElement: No, or get a proper label, and its actual function be performed when double-tapped.
Priority: -- → P2
Priority: P2 → P3
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

I did some investigation around the issue, The unlabeled button is actually the button for "scrollToTop" behavior and it only scrolls to top if we are not showing the home page. So on home page, this button is kind of a no-op button, but on other web pages it works as intended.
We should add a "Scroll to top" label to this button in my opinion.

(In reply to Mohit Agarwal from comment #1)

I did some investigation around the issue, The unlabeled button is actually the button for "scrollToTop" behavior

Is this the same scroll to top behaviour as is triggered when you tap the status bar? If so, why the additional button?

We should add a "Scroll to top" label to this button in my opinion.

That would be great (assuming we really need it as per above question).

(In reply to James Teh [:Jamie] from comment #2)

(In reply to Mohit Agarwal from comment #1)
Is this the same scroll to top behaviour as is triggered when you tap the status bar? If so, why the additional button?

Good question. I looked into git history and found that it was added to support scrolling in landscape mode. https://bugzilla.mozilla.org/show_bug.cgi?id=1162539

I think we should definitely add the label to this button so that it is useful in landscape mode and we should also think about making this button non accessible in portrait mode because status bar is already there for scroll to top functionality.

Blocks: iosa11y
Attached file GitHub Pull Request
Attachment #9067620 - Flags: review?(fpatel)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.