Closed
Bug 637881
Opened 14 years ago
Closed 14 years ago
Remove 'Reflow on Zoom' preference for Fennec 4
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
Tracking
(fennec2.0+)
VERIFIED
FIXED
| Tracking | Status | |
|---|---|---|
| fennec | 2.0+ | --- |
People
(Reporter: mfinkle, Assigned: mfinkle)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
We are removing this for Fennec 4. The code stays, but the visible preference goes.
|
Assignee | |
Comment 1•14 years ago
|
||
Removes the preference, leaves the rest.
An add-on can re-enable the behavior.
Attachment #516044 -
Flags: review?(mbrubeck)
|
||
Comment 2•14 years ago
|
||
If we remove the pref we should remove the string (&reflowZoom.title;) right?
|
||
Comment 3•14 years ago
|
||
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+
|
|
||
Updated•14 years ago
|
OS: Linux → All
Hardware: x86_64 → All
|
|
Assignee | |
Comment 4•14 years ago
|
||
> (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
|
|
Assignee | |
Comment 5•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Comment 6•14 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
You need to log in
before you can comment on or make changes to this bug.
Description
•