Closed Bug 1527561 Opened 6 years ago Closed 4 years ago

Attach the text icon at the right bottom edge of the visual viewport with a way other than position:fixed for the case user sets 'Always enable zoom' preference

Categories

(Firefox for Android Graveyard :: Reader View, defect, P5)

Unspecified
Android
defect

Tracking

(firefox65 unaffected, firefox66 unaffected, firefox67 fix-optional, firefox68 fix-optional)

RESOLVED INCOMPLETE
Tracking Status
firefox65 --- unaffected
firefox66 --- unaffected
firefox67 --- fix-optional
firefox68 --- fix-optional

People

(Reporter: hiro, Unassigned)

References

Details

(Keywords: regression)

No description provided.
OS: Unspecified → Android

Hiroyuki, can you please provide STR for this?

Flags: needinfo?(hikezoe)

Hiroyuki, since this is marked as regression, I'm wondering if you also know of a regression-window here?

I can provide steps:

  1. In Settings -> Accessibility, turn on "Always enable zoom"
  2. Load any website that supports reader mode (e.g. a news article)
  3. Enter reader mode via the button in the toolbar
  4. Pinch-zoom in

Expected results:

The "Aa" icon remains on screen even when zoomed in.

Actual results:

The "Aa" icon scrolls out of view when zoomed in.

(In reply to Neha Kochar [:neha] from comment #2)

since this is marked as regression, I'm wondering if you also know of a regression-window here?

The bug that would have changed the behaviour here is bug 656036.

Flags: needinfo?(hikezoe)

That said, it's not entirely clear to me that the behaviour prior to bug 656036 was preferable. Even though the "Aa" icon remained on screen when zoomed, it would also get bigger, obscuring more and more of the page as you zoom in.

Perhaps the current behaviour of the icon scrolling out of view is acceptable in the case when "Always enable zoom" is set. It's a bit of an edge case anyways.

Blocks: 656036
No longer blocks: 1520077
Priority: -- → P5

P5 and edge case as per comment #4, setting as fix-optional for 67 so as to not remove it from our weekly triage meeting.

Bulk change for all regression bugs with status-firefox67 as 'fix-optional' to be marked 'affected' for status-firefox68.

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.