Ux Designs for Preferences

RESOLVED FIXED in Firefox 11

Status

()

Firefox for Android
General
P1
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: elan, Assigned: mbrubeck)

Tracking

unspecified
Firefox 11
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(3 attachments, 2 obsolete attachments)

(Reporter)

Description

6 years ago
meta bug.
(Reporter)

Updated

6 years ago
Assignee: nobody → madhava
Not a meta bug

This bug is for deciding the master list of preferences and the grouping/ordering of the preferences.
Priority: -- → P1
Keywords: uiwanted
It should include one for picking a default search engine, incidentally.
(In reply to Madhava Enros [:madhava] from comment #2)
> It should include one for picking a default search engine, incidentally.

Ah, that's mostly covered over in bug 695198
This is the set and groupings that I'd suggest:

+-------------------------------------+
|General                              |
|-------------------------------------|
|About Firefox                        | Goes to about:firefox
|Default search engine            (v) | Select the default search (there's a bug for this)
|Language                         (v) | Choose the Firefox language
|Sync                                 | ** when we get sync ** goes to Sync setup or settings
+-------------------------------------+
|Content                              |
+-------------------------------------+
|Character encoding                   | Sorting out the designs here
|Font size                        (v) | There's a bug and a design for this
|Start page                       (v) | Choose a start page - already built; working on page design
+-------------------------------------+
|Privacy and Security                 |
+-------------------------------------+
|Clear history                        |
|Clear private data                   |
|Tell sites not to track me       [x] |
|Remember passwords               [x] |
|Enable Cookies                   [x] |
|Send performance data            [ ] |
+-------------------------------------+
There's some discussion of dropping "Enable Cookies" -- do we need it?

Also - are we going to have a "tap to enable" option on flash? If so, we should have an

"Enable Flash      (v)"

item that can be "Yes / No / Tap to Enable"
(In reply to Madhava Enros [:madhava] from comment #4)
> This is the set and groupings that I'd suggest:
> 
> +-------------------------------------+
> |General                              |
> |-------------------------------------|
> |About Firefox                        | Goes to about:firefox

Just a thought: the apps on Android, and the system itself usually have the About item at the very end of the list, if there is one on the Settings screen, and it makes sense because it's less important that the actual preferences.

We may have a special section at the end named "Info" for this one item, and may even add some other information there if needed, like "Firefox Help", "FAQ" or "Support" (by the way, it might actually be useful to have a link there similar to the "Get Help" on the home page).

Updated

6 years ago
Depends on: 705684

Updated

6 years ago
Depends on: 696950
Assignee: madhava → mark.finkle
Assignee: mark.finkle → mbrubeck
No longer depends on: 705684
Duplicate of this bug: 705684
(Assignee)

Comment 8

6 years ago
Created attachment 579436 [details] [diff] [review]
1/3: reorganize and clean up preferences

This just removes or regroups some preferences to more closely match the design in comment 4.
Attachment #579436 - Flags: review?(sriram)
(Assignee)

Comment 9

6 years ago
Created attachment 579437 [details] [diff] [review]
2/3: Add the "About Firefox" pref
Attachment #579437 - Flags: review?(sriram)
(Assignee)

Comment 10

6 years ago
Created attachment 579440 [details] [diff] [review]
3/3: Override "about:" to redirect to the mobile about page

Instead of separate "about:fennec" and "about:firefox" URIs with logic to switch between them based on branding (as we had in XUL Fennec), we can use just plain old "about:" like desktop Firefox.
Attachment #579440 - Flags: review?(mark.finkle)
(Assignee)

Updated

6 years ago
Blocks: 708076
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
Depends on: 703029
Keywords: uiwanted
(Assignee)

Comment 11

6 years ago
Created attachment 579461 [details] [diff] [review]
1/3: reorganize and clean up preferences

Rebased to birch tip, and removed an unneeded line that was left in by mistake.
Attachment #579436 - Attachment is obsolete: true
Attachment #579436 - Flags: review?(sriram)
Attachment #579461 - Flags: review?(sriram)
(Assignee)

Comment 12

6 years ago
Created attachment 579463 [details] [diff] [review]
2/3: Add the "About Firefox" pref

Rebased and removed an unneeded line.
Attachment #579437 - Attachment is obsolete: true
Attachment #579437 - Flags: review?(sriram)
Attachment #579463 - Flags: review?(sriram)
Comment on attachment 579461 [details] [diff] [review]
1/3: reorganize and clean up preferences

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

This looks fine to me.
Attachment #579461 - Flags: review?(sriram) → review+
Comment on attachment 579463 [details] [diff] [review]
2/3: Add the "About Firefox" pref

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

This looks fine to me.
Attachment #579463 - Flags: review?(sriram) → review+
Comment on attachment 579440 [details] [diff] [review]
3/3: Override "about:" to redirect to the mobile about page

As silly as this sounds, can we keep the redundant about:fennec and about:firefox ? We have bookmarks, from XUL profiles, and we might use those again for local history DBs.

I guess you won't be able to collapse about:fennec like we do about:firefox, but your other cleanup patch makes removes a bunch of cruft.
Attachment #579440 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 16

6 years ago
(In reply to Mark Finkle (:mfinkle) from comment #15)
> As silly as this sounds, can we keep the redundant about:fennec and
> about:firefox ?

Sure.  Pushed with that change:

https://hg.mozilla.org/integration/mozilla-inbound/rev/9c586fb69711
https://hg.mozilla.org/integration/mozilla-inbound/rev/bc9ef9db0bd1
https://hg.mozilla.org/integration/mozilla-inbound/rev/d38a6eac43a7
https://hg.mozilla.org/mozilla-central/rev/9c586fb69711
https://hg.mozilla.org/mozilla-central/rev/bc9ef9db0bd1
https://hg.mozilla.org/mozilla-central/rev/d38a6eac43a7
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
tracking-fennec: --- → 11+
(Assignee)

Updated

6 years ago
status-firefox11: --- → fixed
Target Milestone: --- → Firefox 11
You need to log in before you can comment on or make changes to this bug.