698593 - Ux Designs for Preferences

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect, P1)

Description

[Erin Lancaster [:elan]]

meta bug.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Not a meta bug

This bug is for deciding the master list of preferences and the grouping/ordering of the preferences.

Comment 2

[Madhava Enros [:madhava]]

It should include one for picking a default search engine, incidentally.

Comment 3

[Madhava Enros [:madhava]]

(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

Comment 4

[Madhava Enros [:madhava]]

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            [ ] |
+-------------------------------------+

Comment 5

[Madhava Enros [:madhava]]

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"

Comment 6

[Alex Pakhotin (:alexp)]

(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).

Comment 8

[Matt Brubeck (:mbrubeck)]

Attachment: 1/3: reorganize and clean up preferences

This just removes or regroups some preferences to more closely match the design in comment 4.

Comment 9

[Matt Brubeck (:mbrubeck)]

Attachment: 2/3: Add the "About Firefox" pref

Comment 10

[Matt Brubeck (:mbrubeck)]

Attachment: 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.

Comment 11

[Matt Brubeck (:mbrubeck)]

Attachment: 1/3: reorganize and clean up preferences

Rebased to birch tip, and removed an unneeded line that was left in by mistake.

Comment 12

[Matt Brubeck (:mbrubeck)]

Attachment: 2/3: Add the "About Firefox" pref

Rebased and removed an unneeded line.

Comment 13

[Sriram Ramasubramanian [: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.

Comment 14

[Sriram Ramasubramanian [:sriram]]

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.

Comment 15

[Mark Finkle (:mfinkle) (use needinfo?)]

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.

Comment 16

[Matt Brubeck (:mbrubeck)]

(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

Comment 17

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/9c586fb69711
https://hg.mozilla.org/mozilla-central/rev/bc9ef9db0bd1
https://hg.mozilla.org/mozilla-central/rev/d38a6eac43a7