Closed
Bug 1216257
Opened 9 years ago
Closed 9 years ago
Create new settings categories based on user research
Categories
(Firefox for Android Graveyard :: Settings and Preferences, defect)
Tracking
(firefox45 fixed, b2g-v2.5 fixed, relnote-firefox 45+)
RESOLVED
FIXED
Firefox 45
People
(Reporter: Margaret, Assigned: Margaret)
References
Details
Attachments
(16 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
rnewman
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
sebastian
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
gbrown
:
review+
|
Details |
Anthony, can you provide a mock-up (or just a comment) listing the finalized spec for the new settings organization?
Flags: needinfo?(alam)
Comment 1•9 years ago
|
||
(Not including some of the renaming suggestions from the study for scope reasons)
Top-level
---
Sign in / user@email.com (bug 1189356)
General
Search
Privacy
Accessibility
Advanced
Mozilla
Clear private data
Second-level in
---
Sign in
General
- Home
- Language
- Title bar
Search
- Keep as is
Privacy
- Do not track
- Tracking Protection
- Cookies
- Manage logins
- Remember logins
- Use master password
- Clear private data on exit
- Data sharing choices
-- Telemetry
-- Crash reporter
-- Mozilla Location Service
-- Health report
Accessibility
- Always enable zoom
- Full-screen browsing
- Text size
Items not accounted for in the study but I think could belong in "Accessibility" as well:
*-------------------------*
| - Magnify small areas |
| - Voice input |
| - QR input |
*-------------------------*
Advanced
- Import from Android
- Download updates automatically
- Tabs (Can we call this "Restore tabs"?)
- Plugins
- Allow Autoplay
- Character encoding
- Paint flashing
- Remote debugging
Mozilla
- Keep as is
Clear private data now
Does this work Margaret?
Flags: needinfo?(alam) → needinfo?(margaret.leibovic)
Comment 2•9 years ago
|
||
Thanks Anthony! I'm updating this slightly to include the findings form both Card Sort #1 and Card Sort #2. Happy to discuss any inconsistencies. I agree that the new items work under Accessibility. Also, FYI - I did not test item order within categories. So what you see here is not necessarily prescriptive of order.
Top-level
---
Sign In / user@email.com (bug 1189356)
General & Accessibility (maybe make Accessibility a sub-heading?)
Search
Privacy
Advanced
About Firefox
Clear Private Data
Make Default Browser (if possible....)
Second-level in
---
Sign in
General & Accessibility
- Home
- Language
- Title bar
- Always Enable Zoom
- Full-screen Browsing
- Text Size
- Magnify small areas
- Voice input
- QR input
Search
- Add More Search Providers
- Installed Search Engines
- Show Search Suggestions
Privacy
- Do not track
- Tracking Protection
- Cookies
- Manage logins
- Remember logins
- Use master password
- Clear private data on exit
- Data sharing choices
-- Telemetry
-- Crash reporter
-- Mozilla Location Service
-- Health report
Advanced
- Import Options
- Automatic Updates
- Restore Tabs
- Plugins
- Allow Autoplay
- Character encoding
- Paint flashing
- Remote debugging
About Firefox
- About Mozilla
- FAQs
- Give Feedback
Clear private data now
Make Default Browser
Comment 3•9 years ago
|
||
Also, two comments for consideration:
Margaret - Character Encoding might be important enough in some regions to move to General & Accessibility.
Barbara - Import Options might need to be moved to General & Accessibility so it doesn’t seem intimidating to the average user.
Comment 4•9 years ago
|
||
Talked to Gemma and I'm gonna make a final pass at this. And I think it's ready to go Margaret :)
Top-level
---
Sign In / user@email.com (bug 1189356)
General
Search
Privacy
Accessibility
Advanced
About Firefox
Clear Private Data
Make Default Browser (if possible....)
Second-level in
---
Sign in
General
- Home
- Language
- URL bar
- Always enable zoom
- Full-screen browsing
Search
- Add more search providers
- Installed search engines
- Show search suggestions
Privacy
- Do not track
- Tracking protection
- Cookies
- Manage logins
- Remember logins
- Use master password
- Clear private data on exit
- Data sharing choices
-- Telemetry
-- Crash reporter
-- Mozilla Location Service
-- Health report
Accessibility
- Text size
- Magnify small areas
- Voice input
- QR input
Advanced
- Import options
- Automatic updates
- Restore Tabs
- Click-to-play images
- Plugins
- Allow Autoplay
- Character encoding
- Paint flashing
- Remote debugging
About Mozilla
- About Firefox
- FAQs
- Give Feedback
Clear private data now
Make Default Browser (Is this do-able? I forget now)
Comment 5•9 years ago
|
||
Quick update.
Talked to Gemma about "About Mozilla". The last item in this list.
It _was_ a bit awkward to see "Mozilla" and I'm in agreement that making it "Firefox" makes more sense here. Particularly because that's how most of our user facing messaging tends to label us. Being in the "Firefox" app just seems to reinforce this and confuse users when they see "Mozilla".
For reasons of familiarity, we think it best to go with:
Firefox
- About Firefox
- FAQs
- Give feedback
Gemma, feel free to correct me if I'm wrong!
Flags: needinfo?(gpetrie)
Comment 6•9 years ago
|
||
Anthony and I had another chat about this and we are revising our recommendation for the Firefox/About section :)
We propose:
Mozilla Firefox
- About Firefox
- FAQs
- Give feedback
Thanks!
Flags: needinfo?(gpetrie)
Assignee | ||
Comment 7•9 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #4)
> Top-level
> ---
> Sign In / user@email.com (bug 1189356)
> General
> Search
> Privacy
> Accessibility
> Advanced
> About Firefox
> Clear Private Data
> Make Default Browser (if possible....)
Another concern: on tablets, we have a split-pane view, so everything that's at the "top-level" there needs to actually be a category.
So I think we'll need to tuck "Clear Private Data" into one of them. Maybe we should put it under "General", since that's the first category that's automatically selected?
Flags: needinfo?(margaret.leibovic)
Comment 8•9 years ago
|
||
Perhaps under Privacy?
Would this be tucked only for Tablet's or for Mobile devices also?
Cause we _could_ also just create a "Clear private data" category with "clear now" in it..? Open to suggestions
Assignee | ||
Comment 9•9 years ago
|
||
I noticed that "Clear private data" is hidden below the fold on my N4 with the proposed configuration. I'm not sure what we could do to make it obvious that this screen scrolls, or how much we've customized the style of this list ourselves, but I think this is a discoverability problem.
Comment 10•9 years ago
|
||
Although this will be re-organized, I had a look at Telemetry UI and noticed the following top clicks in settings:
For release (in order of # of clicks):
preferences_privacy
sync_setup
preferences_customize
preferences_display
preferences_vendor
preferences_customize_tablet
preferences_home
preferences_devtools
preferences_search
preferences_locale
sync_settings
We might want to align ourselves with those
So here my suggestion for now for top level:
Sign In (we want people to sign in)
General
Privacy
Clear Private Data
Accessibility
Search
Advanced
Make Default Browser
About Firefox
Assignee | ||
Comment 11•9 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #8)
> Perhaps under Privacy?
That's the natural place to find it, but the reason we decided to surface it at the top level is that people do this all the time.
> Would this be tucked only for Tablet's or for Mobile devices also?
We can do whatever we want. The only constraint is that top-level items on tablet all need to have some content that they display on the right side of the screen.
> Cause we _could_ also just create a "Clear private data" category with
> "clear now" in it..? Open to suggestions
True. Although we decided not to do that for sync. But then again, sync fits well into "General", but "Clear private data" might be weird if there's a separate "Privacy" category.
New question: Where should "Open multiple links" go? My first instinct is "Advanced", but if this is a key differentiator we want to promote, should we put it in "General"?
Flags: needinfo?(gpetrie)
Flags: needinfo?(alam)
Assignee | ||
Comment 12•9 years ago
|
||
(In reply to Barbara Bermes [:barbara] from comment #10)
> Although this will be re-organized, I had a look at Telemetry UI and noticed
> the following top clicks in settings:
>
> For release (in order of # of clicks):
> preferences_privacy
> sync_setup
> preferences_customize
> preferences_display
> preferences_vendor
> preferences_customize_tablet
> preferences_home
> preferences_devtools
> preferences_search
> preferences_locale
> sync_settings
>
> We might want to align ourselves with those
>
> So here my suggestion for now for top level:
>
> Sign In (we want people to sign in)
> General
> Privacy
> Clear Private Data
> Accessibility
> Search
> Advanced
> Make Default Browser
> About Firefox
I'd like to hear Anthony and Gemma's opinions on this change. One tricky thing about telemetry is that they way things are organize might influence how often they're used. For example, search is pretty buried right now - maybe more people would use it if it's at the top level?
Although I do like the idea of placing "Clear Private Data" next to "Privacy".
Assignee | ||
Comment 13•9 years ago
|
||
Comment 14•9 years ago
|
||
(In reply to :Margaret Leibovic from comment #11)
> New question: Where should "Open multiple links" go? My first instinct is
> "Advanced", but if this is a key differentiator we want to promote, should
> we put it in "General"?
I think this falls squarely in "Advanced".
(In reply to :Margaret Leibovic from comment #12)
> (In reply to Barbara Bermes [:barbara] from comment #10)
> > Although this will be re-organized, I had a look at Telemetry UI and noticed
> > the following top clicks in settings:
> >
> > For release (in order of # of clicks):
> > preferences_privacy
> > sync_setup
> > preferences_customize
> > preferences_display
> > preferences_vendor
> > preferences_customize_tablet
> > preferences_home
> > preferences_devtools
> > preferences_search
> > preferences_locale
> > sync_settings
> >
> > We might want to align ourselves with those
> >
> > So here my suggestion for now for top level:
> >
> > Sign In (we want people to sign in)
> > General
> > Privacy
> > Clear Private Data
> > Accessibility
> > Search
> > Advanced
> > Make Default Browser
> > About Firefox
>
> I'd like to hear Anthony and Gemma's opinions on this change. One tricky
> thing about telemetry is that they way things are organize might influence
> how often they're used. For example, search is pretty buried right now -
> maybe more people would use it if it's at the top level?
>
> Although I do like the idea of placing "Clear Private Data" next to
> "Privacy".
From my experience and what we've observed, the current telemetry data is more of a reflection of how crazy our current ordering and hierarchy is.
Take "Customize" and "Display" for example, both are pretty confusing and they're placed next to each other. It's fairly common to just click into and back out of both categories until one finds what they're looking for.
Given that I'm pretty confident in Gemma's findings and her suggested top level also is derived from this research, I'd like to reorganize based on our suggestions. There will be a cost of change for sure, but in the long run, I think it's going to be worth it.
Flags: needinfo?(alam)
Comment 15•9 years ago
|
||
(In reply to :Margaret Leibovic from comment #9)
> Created attachment 8677100 [details]
> "Clear private data" hidden below the fold
>
> I noticed that "Clear private data" is hidden below the fold on my N4 with
> the proposed configuration. I'm not sure what we could do to make it obvious
> that this screen scrolls, or how much we've customized the style of this
> list ourselves, but I think this is a discoverability problem.
I'm not too concerned with the "fold" in this case. Android is really diverse and in the long run, I think it makes sense to make these decisions more from an informational hierarchy point of view. These were things Gemma and I considered too when placing this at the bottom of the list.
Gemma can chime in about this a bit more but I think placing this at the bottom made sense too because we were grouping the "actions" together. "Clear data" and "Make default" are more "action" orientated things that don't require "setting" anything. This "one and done" nature to them made more sense at the ends of the list rather than sandwiched between other prefs.
That being said, I'd be open to moving both "Clear Private data" and "Make default browser" to be above "Mozilla Firefox". That is the only other category I think that makes sense at the bottom of this list.
For clarity, this is what we currently have (barring feedback about comment 14)
> Top-level
> ---
> Sign In / user@email.com
> General
> Search
> Privacy
> Accessibility
> Advanced
> Mozilla Firefox
> Clear Private Data
> Make Default Browser
Assignee | ||
Comment 16•9 years ago
|
||
Assignee | ||
Comment 17•9 years ago
|
||
Assignee | ||
Comment 18•9 years ago
|
||
Comment 19•9 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #4)
> General
> - Home
> - Language
> - URL bar
> - Always enable zoom
> - Full-screen browsing
> Accessibility
> - Text size
> - Magnify small areas
> - Voice input
> - QR input
I'd like to push 'Always enable zoom' from 'General' to 'Accessibility', since it is definitely an a11y feature. Not having it in 'Accessibility' feels broken from an a11y standpoint.
Assignee | ||
Comment 20•9 years ago
|
||
Bug 1216257 - Rename "Customize" category to "General" r?sebastian
Attachment #8678966 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 21•9 years ago
|
||
Bug 1216257 - Move "Search" category to top level r?sebastian
Attachment #8678967 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 22•9 years ago
|
||
Bug 1216257 - Rename "Display" category to "Accessibility" r?sebastian
Attachment #8678968 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 23•9 years ago
|
||
Bug 1216257 - Rename "Developer Tools" category to "Advanced" r?sebastian
Attachment #8678969 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 24•9 years ago
|
||
Bug 1216257 - Move "Language" settting to "General" section r?rnewman
Attachment #8678970 -
Flags: review?(rnewman)
Assignee | ||
Comment 25•9 years ago
|
||
Bug 1216257 - Move clear private data setting to top level. r?sebastian
Attachment #8678971 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 26•9 years ago
|
||
Bug 1216257 - Move restore tabs pref from to "Advanced". r?sebastian
Attachment #8678972 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 27•9 years ago
|
||
Bug 1216257 - Move tap-to-load images pref from to "Advanced". r?sebastian
Attachment #8678973 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 28•9 years ago
|
||
Bug 1216257 - Move titlebar pref to "General". r?sebastian
Attachment #8678974 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 29•9 years ago
|
||
Bug 1216257 - Move zoom and full-screen browsing prefs to "General". r?sebastian
Attachment #8678975 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 30•9 years ago
|
||
Bug 1216257 - Rearrange search settings to match spec. r?sebastian
Attachment #8678976 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 31•9 years ago
|
||
Bug 1216257 - Rearrange privacy prefs to match spec. r?sebastian
Attachment #8678977 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 32•9 years ago
|
||
Bug 1216257 - Move data reporting prefs to "Privacy". r?sebastian
Attachment #8678978 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 33•9 years ago
|
||
Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r?sebastian
Attachment #8678979 -
Flags: review?(s.kaspari)
Assignee | ||
Comment 34•9 years ago
|
||
Bug 1216257 - Remove disabled tests and associated strings. r?gbrown
Attachment #8678980 -
Flags: review?(gbrown)
Assignee | ||
Comment 35•9 years ago
|
||
Here's a build to test:
http://people.mozilla.org/~mleibovic/fennec-settings.apk
Anthony, let me know what you think and what needs to be changed.
Let's try to stay focused on only the reorganization here, and we can deal with renaming things in future bugs.
One thing to note is that I just made up new summary text for the categories. We should think about what we actually want those to be.
Flags: needinfo?(alam)
Assignee | ||
Comment 36•9 years ago
|
||
(In reply to :Margaret Leibovic from comment #35)
> Here's a build to test:
> http://people.mozilla.org/~mleibovic/fennec-settings.apk
Actually, use this:
http://archive.mozilla.org/pub/mobile/try-builds/mleibovic@mozilla.com-24d9aa5169028a6d2f80202c044db3fa3e3fe7b3/try-android-api-11/fennec-44.0a1.en-US.android-arm.apk
Comment 37•9 years ago
|
||
Comment on attachment 8678980 [details]
MozReview Request: Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
https://reviewboard.mozilla.org/r/23319/#review20763
Nice cleanup - thanks!
I think it is a good idea to delete tests that are disabled unless we have realistic plans to revive them: I worry about the clutter and the possible mis-perception that we have test coverage when we actually do not.
::: mobile/android/tests/browser/robocop/robocop.ini:60
(Diff revision 1)
> # disabled on 2.3; bug 979603
> # disabled on 4.0; bug 1006242
> # skip-if = android_version == "10" || android_version == "15"
I think these 3 comments must be associated with testMasterPassword too.
Attachment #8678980 -
Flags: review?(gbrown) → review+
Comment 38•9 years ago
|
||
Comment on attachment 8678966 [details]
MozReview Request: Bug 1216257 - Rename "Customize" category to "General" r=sebastian
https://reviewboard.mozilla.org/r/23291/#review20769
Attachment #8678966 -
Flags: review?(s.kaspari) → review+
Comment 39•9 years ago
|
||
Comment on attachment 8678967 [details]
MozReview Request: Bug 1216257 - Move "Search" category to top level r=sebastian
https://reviewboard.mozilla.org/r/23293/#review20771
Attachment #8678967 -
Flags: review?(s.kaspari) → review+
Comment 40•9 years ago
|
||
Comment on attachment 8678968 [details]
MozReview Request: Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
https://reviewboard.mozilla.org/r/23295/#review20773
::: mobile/android/base/preferences/GeckoPreferences.java:18
(Diff revision 1)
> import org.mozilla.gecko.GeckoEvent;
> +import org.mozilla.gecko.GeckoEvent;
This import is duplicated.
Attachment #8678968 -
Flags: review?(s.kaspari) → review+
Comment 41•9 years ago
|
||
Comment on attachment 8678969 [details]
MozReview Request: Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
https://reviewboard.mozilla.org/r/23297/#review20775
Attachment #8678969 -
Flags: review?(s.kaspari) → review+
Comment 42•9 years ago
|
||
Comment on attachment 8678971 [details]
MozReview Request: Bug 1216257 - Move clear private data setting to top level. r=sebastian
https://reviewboard.mozilla.org/r/23301/#review20777
Follow-up?: This setting is currently visible in restricted profiles (with "Disable 'Clear browsing history'" enabled).
Attachment #8678971 -
Flags: review?(s.kaspari) → review+
Comment 43•9 years ago
|
||
Comment on attachment 8678972 [details]
MozReview Request: Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
https://reviewboard.mozilla.org/r/23303/#review20779
Attachment #8678972 -
Flags: review?(s.kaspari) → review+
Updated•9 years ago
|
Attachment #8678973 -
Flags: review?(s.kaspari) → review+
Comment 44•9 years ago
|
||
Comment on attachment 8678973 [details]
MozReview Request: Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
https://reviewboard.mozilla.org/r/23305/#review20781
Comment 45•9 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #19)
> (In reply to Anthony Lam (:antlam) from comment #4)
>
> > General
> > - Home
> > - Language
> > - URL bar
> > - Always enable zoom
> > - Full-screen browsing
>
> > Accessibility
> > - Text size
> > - Magnify small areas
> > - Voice input
> > - QR input
>
> I'd like to push 'Always enable zoom' from 'General' to 'Accessibility',
> since it is definitely an a11y feature. Not having it in 'Accessibility'
> feels broken from an a11y standpoint.
I'm OK with this too.
(In reply to :Margaret Leibovic from comment #36)
> (In reply to :Margaret Leibovic from comment #35)
> > Here's a build to test:
> > http://people.mozilla.org/~mleibovic/fennec-settings.apk
>
> Actually, use this:
> http://archive.mozilla.org/pub/mobile/try-builds/mleibovic@mozilla.com-
> 24d9aa5169028a6d2f80202c044db3fa3e3fe7b3/try-android-api-11/fennec-44.0a1.en-
> US.android-arm.apk
This is looking good! I'll file a bug for the first round of copy updates as per our research :)
Flags: needinfo?(alam)
Comment 46•9 years ago
|
||
^Thanks Margaret! I filed bug 1218557 for the copy nits. So between that and bug 1189356, we should be in good shape :D
Comment 47•9 years ago
|
||
Comment on attachment 8678974 [details]
MozReview Request: Bug 1216257 - Move titlebar pref to "General". r=sebastian
https://reviewboard.mozilla.org/r/23307/#review20869
Attachment #8678974 -
Flags: review?(s.kaspari) → review+
Comment 48•9 years ago
|
||
Comment on attachment 8678975 [details]
MozReview Request: Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
https://reviewboard.mozilla.org/r/23309/#review20871
Attachment #8678975 -
Flags: review?(s.kaspari) → review+
Comment 49•9 years ago
|
||
Comment on attachment 8678976 [details]
MozReview Request: Bug 1216257 - Rearrange search settings to match spec. r=sebastian
https://reviewboard.mozilla.org/r/23311/#review20873
Attachment #8678976 -
Flags: review?(s.kaspari) → review+
Comment 50•9 years ago
|
||
Comment on attachment 8678977 [details]
MozReview Request: Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
https://reviewboard.mozilla.org/r/23313/#review20877
Attachment #8678977 -
Flags: review?(s.kaspari) → review+
Comment 51•9 years ago
|
||
Comment on attachment 8678978 [details]
MozReview Request: Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
https://reviewboard.mozilla.org/r/23315/#review20879
Attachment #8678978 -
Flags: review?(s.kaspari) → review+
Updated•9 years ago
|
Attachment #8678979 -
Flags: review?(s.kaspari) → review+
Comment 52•9 years ago
|
||
Comment on attachment 8678979 [details]
MozReview Request: Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
https://reviewboard.mozilla.org/r/23317/#review20881
Comment 53•9 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #45)
> (In reply to Mark Finkle (:mfinkle) from comment #19)
> > (In reply to Anthony Lam (:antlam) from comment #4)
> >
> > > General
> > > - Home
> > > - Language
> > > - URL bar
> > > - Always enable zoom
> > > - Full-screen browsing
> >
> > > Accessibility
> > > - Text size
> > > - Magnify small areas
> > > - Voice input
> > > - QR input
> >
> > I'd like to push 'Always enable zoom' from 'General' to 'Accessibility',
> > since it is definitely an a11y feature. Not having it in 'Accessibility'
> > feels broken from an a11y standpoint.
>
> I'm OK with this too.
Good point. I agree.
Flags: needinfo?(gpetrie)
Comment 54•9 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #15)
> (In reply to :Margaret Leibovic from comment #9)
> > Created attachment 8677100 [details]
> > "Clear private data" hidden below the fold
> >
> > I noticed that "Clear private data" is hidden below the fold on my N4 with
> > the proposed configuration. I'm not sure what we could do to make it obvious
> > that this screen scrolls, or how much we've customized the style of this
> > list ourselves, but I think this is a discoverability problem.
>
> I'm not too concerned with the "fold" in this case. Android is really
> diverse and in the long run, I think it makes sense to make these decisions
> more from an informational hierarchy point of view. These were things Gemma
> and I considered too when placing this at the bottom of the list.
>
> Gemma can chime in about this a bit more but I think placing this at the
> bottom made sense too because we were grouping the "actions" together.
> "Clear data" and "Make default" are more "action" orientated things that
> don't require "setting" anything. This "one and done" nature to them made
> more sense at the ends of the list rather than sandwiched between other
> prefs.
>
> That being said, I'd be open to moving both "Clear Private data" and "Make
> default browser" to be above "Mozilla Firefox". That is the only other
> category I think that makes sense at the bottom of this list.
>
> For clarity, this is what we currently have (barring feedback about comment
> 14)
>
> > Top-level
> > ---
> > Sign In / user@email.com
> > General
> > Search
> > Privacy
> > Accessibility
> > Advanced
> > Mozilla Firefox
> > Clear Private Data
> > Make Default Browser
I think it would be fine to move Clear and Default above Mozilla Firefox. Mozilla Firefox feels like a natural final item to me.
Updated•9 years ago
|
Attachment #8678970 -
Flags: review?(rnewman)
Comment 55•9 years ago
|
||
Comment on attachment 8678970 [details]
MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r=rnewman
https://reviewboard.mozilla.org/r/23299/#review21043
::: mobile/android/base/preferences/GeckoPreferenceFragment.java:100
(Diff revision 1)
> +// }
Clean up commented-out code.
::: mobile/android/base/preferences/GeckoPreferences.java:289
(Diff revision 1)
> - updateTitle(R.string.pref_header_language);
> + updateTitle(R.string.pref_header_general);
Leave a comment for why we do this, now you know :)
::: mobile/android/base/preferences/GeckoPreferences.java:500
(Diff revision 1)
> - if (header.id == R.id.pref_header_language && !localeSwitchingIsEnabled) {
> + if (header.id == R.id.pref_header_advanced && !RestrictedProfiles.isAllowed(this, Restriction.DISALLOW_DEVELOPER_TOOLS)) {
Why did you get rid of this? We still need to hide this pref if local switching is disabled. That might mean moving this to a section check or the individual pref checks.
s/settting/setting
Assignee | ||
Comment 56•9 years ago
|
||
(In reply to Richard Newman [:rnewman] from comment #55)
> ::: mobile/android/base/preferences/GeckoPreferences.java:500
> (Diff revision 1)
> > - if (header.id == R.id.pref_header_language && !localeSwitchingIsEnabled) {
> > + if (header.id == R.id.pref_header_advanced && !RestrictedProfiles.isAllowed(this, Restriction.DISALLOW_DEVELOPER_TOOLS)) {
>
> Why did you get rid of this? We still need to hide this pref if local
> switching is disabled. That might mean moving this to a section check or the
> individual pref checks.
This just hides the header for tablets. We already have separate logic to remove the actual pref:
http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/preferences/GeckoPreferences.java#684
I tested with locale switching disabled, and the pref was hidden as expected.
Assignee | ||
Comment 57•9 years ago
|
||
Assignee | ||
Comment 58•9 years ago
|
||
Comment on attachment 8678966 [details]
MozReview Request: Bug 1216257 - Rename "Customize" category to "General" r=sebastian
Bug 1216257 - Rename "Customize" category to "General" r=sebastian
Attachment #8678966 -
Attachment description: MozReview Request: Bug 1216257 - Rename "Customize" category to "General" r?sebastian → MozReview Request: Bug 1216257 - Rename "Customize" category to "General" r=sebastian
Assignee | ||
Updated•9 years ago
|
Attachment #8678967 -
Attachment description: MozReview Request: Bug 1216257 - Move "Search" category to top level r?sebastian → MozReview Request: Bug 1216257 - Move "Search" category to top level r=sebastian
Assignee | ||
Comment 59•9 years ago
|
||
Comment on attachment 8678967 [details]
MozReview Request: Bug 1216257 - Move "Search" category to top level r=sebastian
Bug 1216257 - Move "Search" category to top level r=sebastian
Assignee | ||
Comment 60•9 years ago
|
||
Comment on attachment 8678968 [details]
MozReview Request: Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
Attachment #8678968 -
Attachment description: MozReview Request: Bug 1216257 - Rename "Display" category to "Accessibility" r?sebastian → MozReview Request: Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
Assignee | ||
Updated•9 years ago
|
Attachment #8678969 -
Attachment description: MozReview Request: Bug 1216257 - Rename "Developer Tools" category to "Advanced" r?sebastian → MozReview Request: Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
Assignee | ||
Comment 61•9 years ago
|
||
Comment on attachment 8678969 [details]
MozReview Request: Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
Assignee | ||
Comment 62•9 years ago
|
||
Comment on attachment 8678966 [details]
MozReview Request: Bug 1216257 - Rename "Customize" category to "General" r=sebastian
Bug 1216257 - Rename "Customize" category to "General" r=sebastian
Assignee | ||
Comment 63•9 years ago
|
||
Comment on attachment 8678970 [details]
MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r=rnewman
Bug 1216257 - Move "Language" settting to "General" section r=rnewman
Attachment #8678970 -
Attachment description: MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r?rnewman → MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r=rnewman
Attachment #8678970 -
Flags: review?(rnewman)
Assignee | ||
Comment 64•9 years ago
|
||
Comment on attachment 8678967 [details]
MozReview Request: Bug 1216257 - Move "Search" category to top level r=sebastian
Bug 1216257 - Move "Search" category to top level r=sebastian
Assignee | ||
Comment 65•9 years ago
|
||
Comment on attachment 8678971 [details]
MozReview Request: Bug 1216257 - Move clear private data setting to top level. r=sebastian
Bug 1216257 - Move clear private data setting to top level. r=sebastian
Attachment #8678971 -
Attachment description: MozReview Request: Bug 1216257 - Move clear private data setting to top level. r?sebastian → MozReview Request: Bug 1216257 - Move clear private data setting to top level. r=sebastian
Assignee | ||
Comment 66•9 years ago
|
||
Comment on attachment 8678968 [details]
MozReview Request: Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
Assignee | ||
Comment 67•9 years ago
|
||
Comment on attachment 8678969 [details]
MozReview Request: Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
Attachment #8678972 -
Attachment description: MozReview Request: Bug 1216257 - Move restore tabs pref from to "Advanced". r?sebastian → MozReview Request: Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
Assignee | ||
Comment 68•9 years ago
|
||
Comment on attachment 8678972 [details]
MozReview Request: Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
Assignee | ||
Comment 69•9 years ago
|
||
Comment on attachment 8678970 [details]
MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r=rnewman
Bug 1216257 - Move "Language" settting to "General" section r=rnewman
Assignee | ||
Comment 70•9 years ago
|
||
Comment on attachment 8678973 [details]
MozReview Request: Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
Attachment #8678973 -
Attachment description: MozReview Request: Bug 1216257 - Move tap-to-load images pref from to "Advanced". r?sebastian → MozReview Request: Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
Assignee | ||
Comment 71•9 years ago
|
||
Comment on attachment 8678971 [details]
MozReview Request: Bug 1216257 - Move clear private data setting to top level. r=sebastian
Bug 1216257 - Move clear private data setting to top level. r=sebastian
Attachment #8678974 -
Attachment description: MozReview Request: Bug 1216257 - Move titlebar pref to "General". r?sebastian → MozReview Request: Bug 1216257 - Move titlebar pref to "General". r=sebastian
Assignee | ||
Comment 72•9 years ago
|
||
Comment on attachment 8678974 [details]
MozReview Request: Bug 1216257 - Move titlebar pref to "General". r=sebastian
Bug 1216257 - Move titlebar pref to "General". r=sebastian
Assignee | ||
Comment 73•9 years ago
|
||
Comment on attachment 8678972 [details]
MozReview Request: Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
Assignee | ||
Comment 74•9 years ago
|
||
Comment on attachment 8678975 [details]
MozReview Request: Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
Attachment #8678975 -
Attachment description: MozReview Request: Bug 1216257 - Move zoom and full-screen browsing prefs to "General". r?sebastian → MozReview Request: Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
Assignee | ||
Comment 75•9 years ago
|
||
Comment on attachment 8678973 [details]
MozReview Request: Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
Assignee | ||
Updated•9 years ago
|
Attachment #8678976 -
Attachment description: MozReview Request: Bug 1216257 - Rearrange search settings to match spec. r?sebastian → MozReview Request: Bug 1216257 - Rearrange search settings to match spec. r=sebastian
Assignee | ||
Comment 76•9 years ago
|
||
Comment on attachment 8678976 [details]
MozReview Request: Bug 1216257 - Rearrange search settings to match spec. r=sebastian
Bug 1216257 - Rearrange search settings to match spec. r=sebastian
Assignee | ||
Comment 77•9 years ago
|
||
Comment on attachment 8678974 [details]
MozReview Request: Bug 1216257 - Move titlebar pref to "General". r=sebastian
Bug 1216257 - Move titlebar pref to "General". r=sebastian
Assignee | ||
Comment 78•9 years ago
|
||
Comment on attachment 8678977 [details]
MozReview Request: Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
Attachment #8678977 -
Attachment description: MozReview Request: Bug 1216257 - Rearrange privacy prefs to match spec. r?sebastian → MozReview Request: Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
Assignee | ||
Comment 79•9 years ago
|
||
Comment on attachment 8678975 [details]
MozReview Request: Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
Assignee | ||
Comment 80•9 years ago
|
||
Comment on attachment 8678978 [details]
MozReview Request: Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
Attachment #8678978 -
Attachment description: MozReview Request: Bug 1216257 - Move data reporting prefs to "Privacy". r?sebastian → MozReview Request: Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
Assignee | ||
Comment 81•9 years ago
|
||
Comment on attachment 8678976 [details]
MozReview Request: Bug 1216257 - Rearrange search settings to match spec. r=sebastian
Bug 1216257 - Rearrange search settings to match spec. r=sebastian
Assignee | ||
Updated•9 years ago
|
Attachment #8678979 -
Attachment description: MozReview Request: Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r?sebastian → MozReview Request: Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
Assignee | ||
Comment 82•9 years ago
|
||
Comment on attachment 8678979 [details]
MozReview Request: Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
Assignee | ||
Comment 83•9 years ago
|
||
Comment on attachment 8678977 [details]
MozReview Request: Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
Assignee | ||
Comment 84•9 years ago
|
||
Comment on attachment 8678980 [details]
MozReview Request: Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Attachment #8678980 -
Attachment description: MozReview Request: Bug 1216257 - Remove disabled tests and associated strings. r?gbrown → MozReview Request: Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Assignee | ||
Comment 85•9 years ago
|
||
Comment on attachment 8678978 [details]
MozReview Request: Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
Assignee | ||
Comment 86•9 years ago
|
||
Comment on attachment 8678979 [details]
MozReview Request: Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
Assignee | ||
Comment 87•9 years ago
|
||
Comment on attachment 8678980 [details]
MozReview Request: Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Assignee | ||
Comment 88•9 years ago
|
||
Comment 89•9 years ago
|
||
Comment on attachment 8678970 [details]
MozReview Request: Bug 1216257 - Move "Language" settting to "General" section r=rnewman
https://reviewboard.mozilla.org/r/23299/#review21539
Attachment #8678970 -
Flags: review?(rnewman) → review+
Assignee | ||
Comment 90•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/a7e5e29700525351e0ed526ffe7f5b3adda7417b
Bug 1216257 - Rename "Customize" category to "General" r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/d95025365c7bc9b242f4d99693b910da792c6171
Bug 1216257 - Move "Search" category to top level r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/c14c69c00c93bb3bcceeb94e4c2517f6d1af89ee
Bug 1216257 - Rename "Display" category to "Accessibility" r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/311d614a4f3945b4ac30e418ba5060058f0f6855
Bug 1216257 - Rename "Developer Tools" category to "Advanced" r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/337a67f70d857d36a64387f585454010934690dc
Bug 1216257 - Move "Language" settting to "General" section r=rnewman
https://hg.mozilla.org/integration/fx-team/rev/fdc1692ee653210fb32c3853479e9f9e26d2bd8e
Bug 1216257 - Move clear private data setting to top level. r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/0d0fbf8cc9846eb7e80f935fb3224b67ab5f2fe3
Bug 1216257 - Move restore tabs pref from to "Advanced". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/b6b88bfc3e6fea6a0b1ee39f9bc2e3941f035c83
Bug 1216257 - Move tap-to-load images pref from to "Advanced". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/bb0185d08c37ee7311eca0cb3cb7fd9bea9fdb6e
Bug 1216257 - Move titlebar pref to "General". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/9f452774df93aab94e145801acea3733440d826f
Bug 1216257 - Move full-screen browsing pref to "General". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/7b3d106493724deccb95693d6ae4dc0e003eb120
Bug 1216257 - Rearrange search settings to match spec. r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/7187870fbc9a8c36ec094e87ae4d503155acc291
Bug 1216257 - Rearrange privacy prefs to match spec. r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/464002006f1dc3f899fb8a8df3705212d040497e
Bug 1216257 - Move data reporting prefs to "Privacy". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/52b8491314efce1263eb6fa338ea333a63116452
Bug 1216257 - Move settings items that don't belong elsewhere to "Advanced". r=sebastian
https://hg.mozilla.org/integration/fx-team/rev/a2fd04686fd78e86337bda329dd8668f96c08387
Bug 1216257 - Remove disabled tests and associated strings. r=gbrown
Comment 91•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a7e5e2970052
https://hg.mozilla.org/mozilla-central/rev/d95025365c7b
https://hg.mozilla.org/mozilla-central/rev/c14c69c00c93
https://hg.mozilla.org/mozilla-central/rev/311d614a4f39
https://hg.mozilla.org/mozilla-central/rev/337a67f70d85
https://hg.mozilla.org/mozilla-central/rev/fdc1692ee653
https://hg.mozilla.org/mozilla-central/rev/0d0fbf8cc984
https://hg.mozilla.org/mozilla-central/rev/b6b88bfc3e6f
https://hg.mozilla.org/mozilla-central/rev/bb0185d08c37
https://hg.mozilla.org/mozilla-central/rev/9f452774df93
https://hg.mozilla.org/mozilla-central/rev/7b3d10649372
https://hg.mozilla.org/mozilla-central/rev/7187870fbc9a
https://hg.mozilla.org/mozilla-central/rev/464002006f1d
https://hg.mozilla.org/mozilla-central/rev/52b8491314ef
https://hg.mozilla.org/mozilla-central/rev/a2fd04686fd7
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Comment 92•9 years ago
|
||
Tested using Nexus 9 (Android 6.0) is it expected that on tablet "Import from Android" option is missing and "Clear Private data now" is under General submenu?
Assignee | ||
Comment 93•9 years ago
|
||
(In reply to Teodora Vermesan (:TeoVermesan) from comment #92)
> Tested using Nexus 9 (Android 6.0) is it expected that on tablet "Import
> from Android" option is missing and "Clear Private data now" is under
> General submenu?
Import from Android should be under Advanced... it's a bug if that's missing.
Yes, I decided to put "Clear private data now" under General, to match what we do with the Sync sign-in option, but there was some discussion about whether or not that's the best solution, and looking back at the comments here, I don't actually see a consensus on this issue.
Could you file bugs for these two issues and assign them to me?
Flags: needinfo?(teodora.vermesan)
Comment 94•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a7e5e2970052
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/d95025365c7b
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/c14c69c00c93
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/311d614a4f39
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/337a67f70d85
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/fdc1692ee653
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/0d0fbf8cc984
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/b6b88bfc3e6f
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/bb0185d08c37
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/9f452774df93
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/7b3d10649372
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/7187870fbc9a
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/464002006f1d
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/52b8491314ef
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/a2fd04686fd7
status-b2g-v2.5:
--- → fixed
Comment 95•9 years ago
|
||
Is it expected that "Advanced" sub-menu is missing from Guest Mode Session?
"Advanced" now has "import from android", "download updates automatically", "tabs", "tap-to-load images", "plugins", etc; options that were present in guest mode before this settings reorganization.
Assignee | ||
Comment 96•9 years ago
|
||
(In reply to Teodora Vermesan (:TeoVermesan) from comment #95)
> Is it expected that "Advanced" sub-menu is missing from Guest Mode Session?
> "Advanced" now has "import from android", "download updates automatically",
> "tabs", "tap-to-load images", "plugins", etc; options that were present in
> guest mode before this settings reorganization.
I filed bug 1217073 about restricted profiles, but I don't have a bug explicitly filed about guest profiles. I imagine a lot of the work will be the same, but could you file a follow-up bug for me to remember to fix that? Good find, thanks!
Comment 97•9 years ago
|
||
Release Note Request (optional, but appreciated)
[Why is this notable]:menu settings have changed/moved
[Suggested wording]: Optimized & re-organized "Settings" under "Menu"
[Links (documentation, blog post, etc)]:
relnote-firefox:
--- → ?
Updated•9 years ago
|
QA Contact: teodora.vermesan
Updated•4 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
•