Closed Bug 1700993 Opened 4 years ago Closed 4 years ago

Ensure fxa button in app menu has correct labels when disconnected or finishing account setup

Categories

(Firefox :: Menus, defect, P2)

defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox89 --- verified

People

(Reporter: emmamalysz, Assigned: emmamalysz)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-hamburger-menu] )

Attachments

(2 files)

When the account has been disconnected and we restart the browser, we need to update the fxa app menu button. I've attached an image of what it currently displays. We will run into the same issue for "finish account setup".

The disconnected prompt should:

  1. Introduce a new string: "You've been disconnected"
  2. Bold the email underneath
  3. Have a yellow warning icon
  4. Have the subviewbutton-nav class
    Without a browser restart, we are just shown "Account disconnected" without the "Sync and save data" label. Either way, we're going to need to adjust it.

The finish account setup prompt should:

  1. Use the "Finish Account setup" string, unbolded
  2. Bold the email underneath
  3. Have a yellow warning icon
  4. Have the subviewbutton-nav class
    We're going to need to make sure to hide "Sync and save data" label here too.

I suggest P1, but feel free to adjust.

Status: NEW → ASSIGNED

This patch:
-displays the correct fxa button in the app menu when
unverified or disconnected
-shows the "sign out" button in the fxa panel for the above states
-introduces a new string for a disconnected state
-strengthens our test coverage for the proton fxa panel

Attachment #9211621 - Attachment description: WIP: Bug 1700993, adjust app menu fxa button for different states → WIP: Bug 1700993, adjust app menu fxa button for different states.
Attachment #9211621 - Attachment description: WIP: Bug 1700993, adjust app menu fxa button for different states. → Bug 1700993, adjust app menu fxa button for different states.
Priority: P1 → --
Pushed by emalysz@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f2c9db311901 adjust app menu fxa button for different states. r=fluent-reviewers,flod,markh

@rtestard, can you please prioritize this bug.

Flags: needinfo?(rtestard)
Severity: -- → S3
Flags: needinfo?(rtestard)
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

Verified - Fixed in latest Nightly build 89.0a1 (build id: 20210330215136) using Windows 10, Windows 7 (with low resolution), MacOS 10.15 and Ubuntu 18.04.

Status: RESOLVED → VERIFIED
Regressions: 1706460
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: