Closed
Bug 803630
Opened 12 years ago
Closed 11 years ago
Remove the UI for reflow-on-zoom setting
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox18 verified, firefox19 affected)
RESOLVED
FIXED
People
(Reporter: mfinkle, Assigned: jwir3)
References
Details
Attachments
(1 file, 1 obsolete file)
4.03 KB,
patch
|
jwir3
:
review+
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Since we don't want to ship the reflow-on-zoom feature for Fx18, lets remove the UI for the preference and leave it defaulted off. This is a patch for Fx18 (Aurora)
Assignee | ||
Updated•12 years ago
|
OS: Linux → Android
Hardware: x86_64 → ARM
Assignee | ||
Updated•12 years ago
|
Version: Firefox 16 → Firefox 18
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #673455 -
Flags: review?(mark.finkle)
Reporter | ||
Comment 2•12 years ago
|
||
Comment on attachment 673455 [details] [diff] [review] b803630 looks good to me
Attachment #673455 -
Flags: review?(mark.finkle) → review+
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 673455 [details] [diff] [review] b803630 Ask for aurora approval. I think we only want to land this on aurora, which might mean a slightly different patch due to bitrot.
Assignee | ||
Comment 4•12 years ago
|
||
[Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 795932 User impact if declined: User-facing preference that is not internationalize-able, and also is not ideal from a UX perspective. The feature the preference controls should not be easily accessible in beta, since it's not yet stable, and so we're disabling the user-facing preference for it. Testing completed (on m-c, etc.): no testing necessary - this is simply the removal of a string that is currently non-l10n-ideal and the preference that uses it. Risk to taking this patch (and alternatives if risky): very low risk String or UUID changes made by this patch: removal of a string that would otherwise be difficult to translate/internationalize Same patch, but updated to reflect the state of the aurora tree.
Attachment #673455 -
Attachment is obsolete: true
Attachment #673477 -
Flags: review+
Attachment #673477 -
Flags: approval-mozilla-aurora?
Comment 5•12 years ago
|
||
Comment on attachment 673477 [details] [diff] [review] b803630-aurora Approving for aurora as this is low risk UI backout of a feature not ready for FF18 yet
Attachment #673477 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 6•12 years ago
|
||
Pushed to aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/ddd4b27cd64f
Updated•12 years ago
|
status-firefox18:
--- → fixed
Comment 8•12 years ago
|
||
Verified on: Build ID: 18.0 (2012-11-19) Aurora Channel Device: Samsung Galaxy R OS: Android 2.3.4
Updated•12 years ago
|
status-firefox19:
--- → affected
Assignee | ||
Comment 9•12 years ago
|
||
akeybl, you pinged me about this today, and I just realized that the string was changed from "Enable Reflow On Zoom" to "Pinch to reflow text". Are we still looking to remove this setting string? It is possible that this feature will be available in FF19, depending on tweaks that happen in the next couple of weeks (bug 800805 and bug 803719). I can remove it on aurora, I just want to make sure that this is what we want to do.
Flags: needinfo?(akeybl)
Reporter | ||
Comment 10•12 years ago
|
||
(In reply to Scott Johnson (:jwir3) from comment #9) > It is possible that this feature will be available in FF19, depending on > tweaks that happen in the next couple of weeks (bug 800805 and bug 803719). I thought that the feature would be "always on", eventually. That we would use it as a fallback for when font-inflation didn't get the results we wanted. I'm assuming that we would want to remove the UI setting at that time. Do we want this setting available in Aurora (Fx 19)? Or would we want the "always on" case to be the default in Fx 19?
Assignee | ||
Comment 11•12 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #10) > (In reply to Scott Johnson (:jwir3) from comment #9) > > > It is possible that this feature will be available in FF19, depending on > > tweaks that happen in the next couple of weeks (bug 800805 and bug 803719). > > I thought that the feature would be "always on", eventually. That we would > use it as a fallback for when font-inflation didn't get the results we > wanted. I'm assuming that we would want to remove the UI setting at that > time. Well, I think that we should have it on by default when we first ship it (I think the target is FF19), but still keep the option for users who may find it difficult to use and thus want to turn it off. Perhaps subsequent releases could then have it always on, but my recommendation would be to have it on by default with an option to disable it (perhaps only in about:config). > Do we want this setting available in Aurora (Fx 19)? Or would we want the > "always on" case to be the default in Fx 19? I think our target is FF19, yes, but it really depends on bug 803719 and 800805, which I'm currently working on.
Comment 12•12 years ago
|
||
I don't have an opinion here - just trying to follow up and make sure the product is in the shape that product/engineering expect. Please discuss with mfinkle/karen as necessary.
Flags: needinfo?(akeybl)
Comment 13•12 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #10) > Do we want this setting available in Aurora (Fx 19)? Or would we want the > "always on" case to be the default in Fx 19? Can we come to consensus here? Firefox 19 is now on beta.
Flags: needinfo?(mark.finkle)
Reporter | ||
Comment 14•11 years ago
|
||
Given the situation and where the trains are, I think we'll continue to let the visible setting stay in the product until the feature is working well enough to leave it "always on" and we can then remove the setting. For now, that would ride the trains, so no new action is needed. This only landed on Fx18 and that's fine.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(mark.finkle)
Resolution: --- → FIXED
Updated•11 years ago
|
tracking-firefox19:
+ → ---
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•