Closed
Bug 1546599
Opened 6 years ago
Closed 6 years ago
FxA Name is not switched to default state after the user has disconnected from the account
Categories
(Firefox :: Firefox Accounts, defect)
Firefox
Firefox Accounts
Tracking
()
VERIFIED
FIXED
Firefox 68
People
(Reporter: mberlinger, Assigned: vbudhram)
References
Details
Attachments
(3 files)
(deleted),
image/gif
|
Details | |
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
(deleted),
patch
|
pascalc
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
Affected versions
- Firefox Nightly 68.0a1 (Build ID 20190423221610)
- Firefox Beta 67.0b13 (Build ID 20190422163745)
Affected Platforms
- Windows 10x64
- Ubuntu 18.04x64
- Windows 7x64
- mac Os 10.14
Prerequisites
- Firefox browser is opened.
- User is authenticated with a Firefox Account.
- FxA icon is pinned to the Overflow Menu.
Steps to reproduce
- Click on "More Tools..." menu button from the browser toolbar.
- Click on the "Firefox Account" button.
- Click on the "Manage Account..." button.
- Click on "Add..." butt for "Display name".
- Write a name and click on [Change] button.
- Verify that the name is displayed at the top of the FxA doorhanger in the Overflow Menu.
- Go to about:preferences#sync and click on "Disconnect" -> "Just Disconnect" button.
- Click on "More Tools..." menu button from the browser toolbar.
- Click on the "Firefox Account" button.
- Observe the name displayed at the top of the FxA doorhanger.
Expected results
- The name displayed is "Firefox Account".
Actual results
- The name displayed is the one defined by the user.
Regression range
- This issue is not a regression.
Notes
- This issue is reproducible only when FxA is in the Overflow menu.
- After browser restart the name is changed to "Firefox Account".
Comment 1•6 years ago
|
||
Vijay, could you look into this? I expect we need to remove the header and/or clear a title attribute somewhere.
Flags: needinfo?(vbudhram)
Assignee | ||
Comment 2•6 years ago
|
||
Hey :gijs, looked at this and removing the title attribute fixes this issue. Will post a patch soon.
Flags: needinfo?(vbudhram)
Assignee | ||
Comment 3•6 years ago
|
||
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/542aa9819121
Reset FxA avatar menu title on logout r=Gijs
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Comment 6•6 years ago
|
||
Vijay, this looks safe for beta, do you want to request an uplift?
Assignee: nobody → vbudhram
Flags: needinfo?(vbudhram)
Assignee | ||
Comment 7•6 years ago
|
||
Beta/Release Uplift Approval Request
- User impact if declined: Incorrect header in FxA toolbar menu when user logs out of FxA
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This isn't a risky change because a user needs to have set a display name for FxA, have the toolbar menu be in the overflow view and then log out.
- String changes made/needed:
Flags: needinfo?(vbudhram)
Attachment #9061380 -
Flags: approval-mozilla-beta?
Assignee | ||
Updated•6 years ago
|
Attachment #9060493 -
Flags: approval-mozilla-beta?
Comment 8•6 years ago
|
||
Comment on attachment 9061380 [details] [diff] [review]
Bug1546599.diff
Low-risk polish for fxa in 67, uplift accepted for 67 beta 16, thanks.
Attachment #9061380 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•6 years ago
|
Attachment #9060493 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•6 years ago
|
||
bugherder uplift |
Reporter | ||
Comment 10•6 years ago
|
||
This issue is verified fixed using Firefox 67.0b16 and Firefox 68.0a1 (20190502220333) on the following OSes: Windows 10 x64, Ubuntu 18.04 x64, Windows 7 x64, macOS 10.14.
You need to log in
before you can comment on or make changes to this bug.
Description
•