Account UI in settings should match the mockups

RESOLVED FIXED

Status

()

Firefox for iOS
Theme & Visual Design
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: darrin, Assigned: darrin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

3 years ago
Created attachment 8614684 [details]
settings-reorg.png

Detailed notes in the spec png, mostly moving things around. Robin + Ryan, can you take a peek and see if this makes sense and if there are any states I'm not thinking of? (Logout will appear if signed in, hidden otherwise... top row in bold will show appropriate action i.e. 'Sign In')

Should use the same copy for errors/states that Android does.
Flags: needinfo?(rfeeley)
Flags: needinfo?(randersen)
tracking-fennec: ? → +
tracking-fxios: ? → +
States are:
[Sign in]
[account email|Verify your email address] currently in grey, should be in orange
[account email|Enter your password to connect] in orange
[account email|Unable to verify email address] currently in grey, should be in orange
[account email|Disconnect]


It looks good, though the version number isn't in the default fontface ;-)
Flags: needinfo?(randersen)
(Assignee)

Comment 2

3 years ago
Version number and 'mozilla' are in Fira, as a hint towards our brand. Its subtle, but if we're shipping Fira Sans for RV, lets use it ;)

Can you specify the 'orange' to use?
Assignee: nobody → jhugman
Note that "Logout" is a noun like "login". As an action, the verb "Log Out" (much like "Log In") is more appropriate.
Flags: needinfo?(rfeeley)
Not blocking on polish at this point. Scream if this offends.
tracking-fennec: + → ---
tracking-fxios: + → ---
(Assignee)

Updated

2 years ago
Assignee: jhugman → dhenein
(Assignee)

Comment 5

2 years ago
Created attachment 8628923 [details] [review]
https://github.com/mozilla/firefox-ios/pull/665
Attachment #8628923 - Flags: review?(sarentz)
Comment on attachment 8628923 [details] [review]
https://github.com/mozilla/firefox-ios/pull/665

LGTM
Attachment #8628923 - Flags: review?(sarentz) → review+
(Assignee)

Comment 7

2 years ago
Merged https://github.com/mozilla/firefox-ios/pull/665
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.