[AccessFu] Zooming doesn't work with TalkBack enabled

RESOLVED FIXED in Firefox 43

Status

()

Core
Disability Access APIs
P1
normal
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: Kostadin Kolev, Assigned: eeejay)

Tracking

(Blocks: 1 bug)

unspecified
mozilla43
Points:
---

Firefox Tracking Flags

(firefox43 fixed, fennec+)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release)
Build ID: 20140506152807

Steps to reproduce:

1. Enable TalkBack from Settings -> Accessibility -> TalkBack.
2. Start Firefox and load a page in it (a one that could be zoomed in).
3. Try zooming in and out the loaded page, using 2 (or 3 if necessary) fingers.


Actual results:

Zooming doesn't work.


Expected results:

Zooming should work, even when TalkBack is enabled. If necessary - 3 finger gestures should be used to make it work.

Updated

3 years ago
tracking-fennec: --- → ?
Component: General → Graphics, Panning and Zooming
OS: Windows 8.1 → Android
Hardware: x86_64 → ARM
Any thoughts on whether you want this supported, and what the user interaction would be like? IMO if you want to support zooming with accessibility enabled then it might be better to have desktop-style zoom (i.e. fixed zoom amounts) rather than the current mobile pinch-zoom.
Flags: needinfo?(eitan)
(Assignee)

Comment 2

3 years ago
(In reply to Kostadin Kolev from comment #0)
> Zooming should work, even when TalkBack is enabled. If necessary - 3 finger
> gestures should be used to make it work.

Is this how you zoom in Chrome?
Flags: needinfo?(eitan)
(Assignee)

Comment 3

3 years ago
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)
> Any thoughts on whether you want this supported, and what the user
> interaction would be like? IMO if you want to support zooming with
> accessibility enabled then it might be better to have desktop-style zoom
> (i.e. fixed zoom amounts) rather than the current mobile pinch-zoom.

Yeah, we capture all touch events, so we may need to do something ourselves.
(Reporter)

Comment 4

3 years ago
Yes, I want zooming to be supported even with TalkBack enabled. And YES - in Chrome, I place 3 fingers on the screen and move 1 of them away or towards the other 2 and that is how zooming in and out works.
(Assignee)

Comment 5

3 years ago
Great, thanks Kostadin. I agree this would be a great improvement, and also needed in Firefox OS.

I am refiling this as an AccessFu bug. We currently capture all touch events, and we will probably be implementing this on our end as opposed to passing this through in Android.

Thanks for putting this on our radar!
Component: Graphics, Panning and Zooming → Disability Access APIs
OS: Android → All
Product: Firefox for Android → Core
Hardware: ARM → All
Summary: Zooming doesn't work with TalkBack enabled → [AccessFu] Zooming doesn't work with TalkBack enabled
Version: Firefox 29 → unspecified
Do we have any idea about how we want to track this bug? Just a +? Or do we want this to land sooner?
Status: UNCONFIRMED → NEW
Ever confirmed: true
tracking-fennec: ? → +
Assignee: nobody → wjohnston
filter on [mass-p5]
Priority: -- → P5
I have no idea why I assigned this to myself. From comment 5 it sounds like the accessibility people wanted it.
Assignee: wjohnston → nobody
Seems important – I'm going to renom.
tracking-fennec: + → ?
Still happens?
Flags: needinfo?(kbrosnan)
This still happens in nightly. Chrome allows zooming using the 2 finger pinch/pull gesture.
Flags: needinfo?(kbrosnan)
(Assignee)

Updated

2 years ago
Blocks: 1182162
Mike - Can you take a look at how the Android Gestures are being handled? Might be a Java side issue, but could be an APZ issue too?
Assignee: nobody → michael.l.comella
Flags: needinfo?(michael.l.comella)
tracking-e10s: --- → +
Priority: P5 → P1
tracking-fennec: ? → +
tracking-e10s: + → ---
(Assignee)

Comment 13

2 years ago
Created attachment 8658454 [details] [diff] [review]
[AccessFu] Only capture explore by touch events in Android.
Attachment #8658454 - Flags: review?(yzenevich)
(Assignee)

Comment 14

2 years ago
We currently capture all touch/mouse events in order to support things like swipe navigation with TalkBack in ICS. The ICS share is shrinking, and it is not worth having that extra functionality for a small segment.

Note, removing this does not completely remove accessibility support for ICS, it just removes the extra features we had in order to make it suck a bit less than the rest of the platform..
Assignee: michael.l.comella → eitan
Thanks for grabbing this Eitan. If you need me to jump in again, NI me.
Flags: needinfo?(michael.l.comella)
Comment on attachment 8658454 [details] [diff] [review]
[AccessFu] Only capture explore by touch events in Android.

Review of attachment 8658454 [details] [diff] [review]:
-----------------------------------------------------------------

This is good stuff :)
Attachment #8658454 - Flags: review?(yzenevich) → review+

Comment 17

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/bb14c6b7a30b
https://hg.mozilla.org/mozilla-central/rev/bb14c6b7a30b
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Release Note Request (optional, but appreciated)
[Why is this notable]:  User facing improvements! Great. 
[Suggested wording]: Accessibility improvements for scrolling, zooming, and context menus in Talkback 
[Links (documentation, blog post, etc)]: http://blog.monotonous.org/2015/09/15/firefox-for-android-accessibility-refresh/

It seems good to me to describe this and several other bugs mentioned in the blog post, rolled up into one release note. 

If you have improvements for the release note wording, please comment and needinfo me. Thanks!
relnote-firefox: --- → 43+
Flags: needinfo?(eitan)
(Assignee)

Comment 20

2 years ago
The wording looks good, I would add braille:
Accessibility improvements for scrolling, zooming, braille, and context menus in Talkback 

Also, this relnote tag should probably be on the dependent bug 1182162 instead of this one.
Flags: needinfo?(eitan)
relnote-firefox: 43+ → ---
You need to log in before you can comment on or make changes to this bug.