Closed
Bug 1121681
Opened 10 years ago
Closed 10 years ago
Add "Passwords" entry point in menu
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(firefox39 verified)
RESOLVED
FIXED
Firefox 39
Tracking | Status | |
---|---|---|
firefox39 | --- | verified |
People
(Reporter: antlam, Assigned: liuche)
References
Details
Attachments
(4 files, 1 obsolete file)
With all this work that we're doing around Passwords management, we should expose a "Passwords" item in the menu to access about:passwords easier.
Option 1:
- 3-dot menu -> Settings -> Privacy -> Manage passwords -> Opens "about:passwords"
Option 2:
- 3-dot menu -> Tools -> Passwords -> Opens "about:passwords"
Comment 1•10 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #0)
> With all this work that we're doing around Passwords management, we should
> expose a "Passwords" item in the menu to access about:passwords easier.
>
> Option 1:
> - 3-dot menu -> Settings -> Privacy -> Manage passwords -> Opens
> "about:passwords"
>
> Option 2:
> - 3-dot menu -> Tools -> Passwords -> Opens "about:passwords"
On first blush, I would go with the Tools menu approach, but then I remembered that Robin had ideas to reduce the menu size a bit and use other entry points. I think having a Settings > Privacy entry point makes some sense, and eases our Menu load.
Ryan is working on other "fast acccess to passwords" approaches too, for those situations where we want to make it as easy as possible. It's a different use case than just wanting to get to the Password Manager. For that case, I think we should try the Settings approach first. Maybe we'll find ways to migrate some lesser used menus to Settings. We have talked about doing that for Add-ons in the past.
Comment 2•10 years ago
|
||
Might I suggest a long-press handler in a password field as another "expectable" entry point?
Perhaps "Passwords for this site…"?
Comment 3•10 years ago
|
||
These "fast access" entry points are good ideas, but I still think that we should have a canonical way to get to your password manager, such as an item buried in settings.
To be consistent with our similar about: pages (add-ons, downloads), I would vote for an item in the Tools menu. If we don't like the Tools menu as a spot for things like this, we should revive that menu reorganization bug.
Comment 4•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #3)
> To be consistent with our similar about: pages (add-ons, downloads), I would
> vote for an item in the Tools menu. If we don't like the Tools menu as a
> spot for things like this, we should revive that menu reorganization bug.
I would like to use this bug to revive the Menu re-org a bit. Maybe not do the entire re-org, but we could start with moving some items, like Tools > Add-ons and (yet to be added) Passwords to the Setting system.
Menu Reorg - Settings is bug 1073294
Menu Reorg is bug 1069035
Comment 5•10 years ago
|
||
Bear in mind that the following activity stack is not possible:
BrowserApp > GeckoPreferences > BrowserApp
so there might be some gymnastics here if password management is a content page that's accessed via settings, particularly if you want to be able to hit Back in about:passwords and go back to the settings UI.
Comment 6•10 years ago
|
||
Right. This is already the case for Settings > Mozilla > About Firefox
Updated•10 years ago
|
Blocks: passwords-2015-Q1
Updated•10 years ago
|
Priority: -- → P1
Updated•10 years ago
|
Assignee: nobody → liuche
Reporter | ||
Comment 7•10 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #0)
> Option 2:
> - 3-dot menu -> Tools -> Passwords -> Opens "about:passwords"
This looks like the best choice right now. As Mark pointed out, we could surface some of the settings under Settings as well. But we still need a "Passwords" item in Tools.
Comment 8•10 years ago
|
||
I'm all for surfacing as much as possible but not so sure users would know to look in "Tools" for Password Management. I would press for Option 1.
Assignee | ||
Comment 9•10 years ago
|
||
/r/4283 - Bug 1121681 - Add "Passwords" entry point in menu. r=mhaigh
Pull down this commit:
hg pull review -r 50f5c97e537141de3d5e369d35684ffa2cfc251e
Attachment #8568876 -
Flags: review?(mhaigh)
Assignee | ||
Comment 10•10 years ago
|
||
Added entry points in both the "Tools" menu and in the "Settings > Privacy" preferences. Also added nightly flags for new code + tests because about:passwords is behind the Nightly flag.
Assignee | ||
Comment 11•10 years ago
|
||
Assignee | ||
Comment 12•10 years ago
|
||
Also considering putting a category separator because we now have 3 passwords preference items.
Attachment #8568888 -
Flags: review?(alam)
Assignee | ||
Comment 13•10 years ago
|
||
Comment on attachment 8568876 [details]
MozReview Request: bz://1121681/liuche
/r/4283 - Bug 1121681 - Add "Passwords" entry point in menu. r=mhaigh
/r/4289 - Passwords category.
Pull down these commits:
hg pull review -r eb1b74f093833f79283819e4edbe894d97d05eeb
Attachment #8568876 -
Flags: review?(mhaigh) → review?(margaret.leibovic)
Assignee | ||
Comment 14•10 years ago
|
||
Comment on attachment 8568876 [details]
MozReview Request: bz://1121681/liuche
/r/4283 - Bug 1121681 - Add "Passwords" entry point in menu. r=margaret
/r/4289 - Passwords category.
Pull down these commits:
hg pull review -r d33a297c73f80afc5cee0e472c193fc437a67794
Reporter | ||
Comment 15•10 years ago
|
||
Comment on attachment 8568888 [details]
Screenshot: Category
Yeah this is probably where I would put it. + for that!
Two things though :)
- could we change this ordering so that "Manage" is first under "LOGINS"?
- We should probably say "Manage Logins" rather than "Manage passwords"
Attachment #8568888 -
Flags: review?(alam) → review+
Reporter | ||
Comment 16•10 years ago
|
||
Likewise it should probably be Tools > Logins rather than Tools > Passwords I think
Comment 17•10 years ago
|
||
https://reviewboard.mozilla.org/r/4283/#review3525
r+ if you update the strings to be what antlam wants :)
::: mobile/android/base/resources/menu-xlarge-v11/browser_app_menu.xml
(Diff revision 2)
> + <item android:id="@+id/passwords"
Ugh, we have 4 versions of this file.
::: mobile/android/base/tests/testSettingsMenuItems.java
(Diff revision 2)
> + final String[] MANAGE_PASSWORDS_ARR = { StringHelper.MANAGE_PASSWORDS_LABEL };
Good job remembering to update the tests.
Comment 18•10 years ago
|
||
Comment 19•10 years ago
|
||
Comment on attachment 8568876 [details]
MozReview Request: bz://1121681/liuche
https://reviewboard.mozilla.org/r/4281/#review3529
Ship It!
Attachment #8568876 -
Flags: review?(margaret.leibovic) → review+
Assignee | ||
Comment 20•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/14486598bf2d
https://hg.mozilla.org/integration/fx-team/rev/b726620f6dc2
Target Milestone: --- → Firefox 39
Comment 21•10 years ago
|
||
Comment 22•10 years ago
|
||
Tested with:
Device: Nexus 4 (Android 4.4)
Build: Firefox for Android 39.0a1 (2015-03-04)
Tapping the three-dot-menu -> Settings -> Privacy -> Manage logins will open "about:passwords"
Also a category separator "Logins" is displayed containing the following 3 passwords preferences: "Manage logins", "Remember passwords", "Use master password".
Tapping the three-dot-menu -> Tools -> Logins will also open "about:passwords"
Updated•10 years ago
|
Assignee | ||
Comment 23•9 years ago
|
||
Attachment #8568876 -
Attachment is obsolete: true
Attachment #8619132 -
Flags: review+
Attachment #8619133 -
Flags: review+
Assignee | ||
Comment 24•9 years ago
|
||
Assignee | ||
Comment 25•9 years ago
|
||
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
•