Field zoom is not toggleable (formhelper.* prefs need hooking up)

RESOLVED FIXED in Firefox 25

Status

()

Firefox for Android
Graphics, Panning and Zooming
RESOLVED FIXED
5 years ago
a year ago

People

(Reporter: vinceying113, Assigned: botond)

Tracking

21 Branch
Firefox 25
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [parity-xul])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Android; Linux i686; rv:21.0) Gecko/21.0 Firefox/21.0
Build ID: 20130124162801

Steps to reproduce:

Tried to disable form zooming with formhelper.zoom in about:config.  The zoom feature makes filling out long forms tedious due to contant zooming in and out.


Actual results:

Nothing.


Expected results:

Should have disabled the formhelper.zoom functionality.
The pref is 'formhelper.autozoom' but I believe that is not what you want. Are you referring to the recently added functionality in bug 725018? I don't believe the patches added a toggle state preference.
Blocks: 725018
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

5 years ago
Severity: normal → enhancement
OS: Linux → Android
Hardware: x86_64 → ARM
Summary: formhelper.zoom can not be disabled → Field zoom is not toggleable
(In reply to vinceying113 from comment #0)
> Tried to disable form zooming with formhelper.zoom in about:config.  The
> zoom feature makes filling out long forms tedious due to contant zooming in
> and out.

If you switch between field, why would this cause a zoom out?

Do you have a URL you've been seeing this on?
We should probably make the behaviour from bug 725018 depend on these prefs. The prefs look like they were used in XUL fennec.
(Reporter)

Comment 4

5 years ago
Aaron, the reason I want to disable the zoom is Firefox zooms all the way in to the field - so descriptive text next to the field is not visible.  So, on a form with lots of fields, I have to zoom out to view the descriptor on the next field, then click on the field which zooms all the way in, and so on.  Thanks.
I see, thanks; I was thinking that you were jumping directly between fields and it would first zoom out.
Severity: enhancement → normal
Whiteboard: [parity-xul]
Duplicate of this bug: 872266
Summary: Field zoom is not toggleable → Field zoom is not toggleable (formhelper.* prefs need hooking up)
This worth tracking for recent browser preference changes and additions?

Comment 8

4 years ago
Probably.  IMO, the autozoom feature is broken (jumping to max zoom on tiny form fields, where you can't even see next one-line field without scrolling), and I figure I'm probably not the only one with that opinion.  The WebOS browser had a zoom disable, and I turned it off even though that zoom wasn't as frustrating as Firefox's.
Can you file a bug with a link to an example? I'd like to try and fix any issues people find (but I'm not opposed to making this pref work either).
(Assignee)

Updated

4 years ago
Assignee: nobody → botond
(Assignee)

Comment 10

4 years ago
Created attachment 774912 [details] [diff] [review]
patch

The attached patch hooks up formhelper.mode, and removes the other formhelper.* options (autozoom, autozoom.caret, and restore). The OP's desire (disabling auto-zooming) can now be accomplished by setting formhelper.mode to 0.

Here is the rationale, based on my understanding of the code (I am new to this code so please forgive me and correct me if I've gotten something wrong):

  - The rationale for removing formhelper.autozoom.caret and formhelper.restore 
    is that the underlying functionality is not implemented in native fennec. 
    If it's later implemented, they can be added again.

  - The rationale for removing formhelper.autozoom is that in native fennec, 
    the formhelper doesn't currently do anything besides zooming, and so
    formhelper.mode and formhelper.autozoom would mean the same thing
    (except that formhelper.mode allows discriminating between tablets and
    phones, a flexibility that I think is desirable).
Attachment #774912 - Flags: review?(bugmail.mozilla)

Comment 11

4 years ago
I don't know enough about the codebase to review the patch, but the described solution sounds practical as long as the prefs' meanings are consistent across whatever platforms they're implemented on (I know I carry preference bundles with me across machines).  The doc for Form Assistant isn't clear but suggests that formhelper.mode should also control the autocomplete popup, which *is* particularly helpful on mobile, and it seems that autozoom was intended to be (is?) controlled separately.  (Also, of course, I found the pref by searching about:config for "zoom"!  "formhelper.mode" is not at all obvious and should have an option in the UI if that's where this control lives.)
(Assignee)

Comment 12

4 years ago
(In reply to Christopher Smith from comment #11)
> The doc for Form Assistant isn't clear

Where can I find this doc?

Comment 13

4 years ago
https://wiki.mozilla.org/Mobile/Fennec/FormAssistant is the only Google result for "formhelper.mode" and appears to be the only description of the preference's settings.
(Assignee)

Comment 14

4 years ago
(In reply to Christopher Smith from comment #11)
> The doc for Form Assistant
> isn't clear but suggests that formhelper.mode should also control the
> autocomplete popup, which *is* particularly helpful on mobile, and it seems
> that autozoom was intended to be (is?) controlled separately.

If I'm reading the code right, xul fennec used auto-complete regardless of formhelper.mode (and current fennec certainly does). Do you think that should change?

Either way, I will update the wiki once we settle on a solution.

Comment 15

4 years ago
My experience with autocomplete on Android has been... erratic.  I think the option to disable it should be a privacy setting, and I don't know enough to suggest where in the namespace it should go.
Comment on attachment 774912 [details] [diff] [review]
patch

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

Looks good to me. Please update the commit message though, so that it reflects what the patch is doing rather than just copying the bug summary.
Attachment #774912 - Flags: review?(bugmail.mozilla) → review+
(Assignee)

Comment 17

4 years ago
Created attachment 775806 [details] [diff] [review]
updated patch

Upon a more careful reading of the code, the current form helper performs two tasks that deserve to be differentiated: scrolling to bring the focused input element into view, and zooming in on it.

The updated patch:

  - Modifies the default behaviour (formhelper.mode = 2) so that 
    scrolling and zooming happens on phones and only srolling happens 
    on tablets. Setting formhelper.mode = 1 enables zooming on
    tablets as well. Setting formhelper.mode = 0 disables both
    scrolling and zooming everywhere.

  - Restore the formhelper.autozoom preferenece (defaults to true),
    which can be set to false to disable zooming for all devices.
Attachment #774912 - Attachment is obsolete: true
Attachment #775806 - Flags: review?(bugmail.mozilla)
(Assignee)

Comment 18

4 years ago
I updated https://wiki.mozilla.org/Mobile/Fennec/FormAssistant to reflect the changes in the updated patch.
(Assignee)

Comment 19

4 years ago
Try results for the updated patch: https://tbpl.mozilla.org/?tree=Try&rev=6aa969de4c85
(Assignee)

Comment 20

4 years ago
(In reply to Christopher Smith from comment #11)
> (Also, of
> course, I found the pref by searching about:config for "zoom"! 
> "formhelper.mode" is not at all obvious and should have an option in the UI
> if that's where this control lives.)

This concern is addressed by restoring the formhelper.autozoom preference in the updated patch.
(Assignee)

Comment 21

4 years ago
(In reply to Christopher Smith from comment #15)
> My experience with autocomplete on Android has been... erratic.  I think the
> option to disable it should be a privacy setting, and I don't know enough to
> suggest where in the namespace it should go.

I filed bug 893979 for this.
Attachment #775806 - Flags: review?(bugmail.mozilla) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e3d601a9ab2
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/3e3d601a9ab2
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
You need to log in before you can comment on or make changes to this bug.