Remove 'Reflow on Zoom' preference for Fennec 4

VERIFIED FIXED

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: mfinkle, Assigned: mfinkle)

Tracking

Details

Attachments

(1 attachment)

We are removing this for Fennec 4. The code stays, but the visible preference goes.
Created attachment 516044 [details] [diff] [review]
patch

Removes the preference, leaves the rest.

An add-on can re-enable the behavior.
Attachment #516044 - Flags: review?(mbrubeck)
If we remove the pref we should remove the string (&reflowZoom.title;) right?
Comment on attachment 516044 [details] [diff] [review]
patch

Code change is fine, though I honestly think we are better off shipping this than not, even in its current state.

(In reply to comment #2)
> If we remove the pref we should remove the string (&reflowZoom.title;) right?

If it's already translated, maybe we should leave it in place for use by the add-on that re-enables this?
Attachment #516044 - Flags: review?(mbrubeck) → review+
OS: Linux → All
Hardware: x86_64 → All
> (In reply to comment #2)
> > If we remove the pref we should remove the string (&reflowZoom.title;) right?
> 
> If it's already translated, maybe we should leave it in place for use by the
> add-on that re-enables this?

Exactly
pushed:
http://hg.mozilla.org/mobile-browser/rev/c779e14f9594
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 6

7 years ago
verified fixed on build id: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b13pre) Gecko/20110302
Firefox/4.0b13pre Fennec /4.0b6pre
Status: RESOLVED → VERIFIED
This change was reverted in bug 640069.
Blocks: 640069
You need to log in before you can comment on or make changes to this bug.