Closed Bug 1180811 Opened 6 years ago Closed 6 years ago

Dismiss "Zoomed View" when user taps outside the zoomed area

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(firefox43 verified)

RESOLVED FIXED
Firefox 42
Tracking Status
firefox43 --- verified

People

(Reporter: antlam, Assigned: domivinc, Mentored)

References

Details

Attachments

(1 file)

After some discussion in https://bugzilla.mozilla.org/show_bug.cgi?id=1165127#c5 and some User studies we ran, I think we should implement this behaviour now.
Flags: needinfo?(michael.l.comella)
Flags: needinfo?(domivinc)
Dominique?
Assignee: nobody → domivinc
Mentor: michael.l.comella
Flags: needinfo?(michael.l.comella)
I'm going to work on it.
Flags: needinfo?(domivinc)
Just one point to take into account during your tests: if the tap occurs outside the zoomed view but on a cluster area, the zoomed view will be displayed on the new cluster area.
To close the zoomed view, the tap must be done in an area without any cluster of links.
Attachment #8631058 - Flags: review?(michael.l.comella)
(In reply to Dominique Vincent [:domivinc] from comment #3)
> Just one point to take into account during your tests: if the tap occurs
> outside the zoomed view but on a cluster area, the zoomed view will be
> displayed on the new cluster area.
> To close the zoomed view, the tap must be done in an area without any
> cluster of links.

I wonder how intuitive this functionality would be.
Comment on attachment 8631058 [details] [diff] [review]
patch-08072015 1-Bug_1180811___Dismiss_Zoomed_View_when_user_taps_outside_the_zoomed_area__r_mcomella.patch

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

Seems reasonable to me.
Attachment #8631058 - Flags: review?(michael.l.comella) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/00d51ff47b84
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Verified as fixed using:
Device: LG Nexus 4 (Android 5.0)
Build: Firefox for Android 43.0a1 (2015-08-17)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.