1171028 - Account UI in settings should match the mockups

Status: RESOLVED FIXED

(Firefox for iOS :: Theme & Visual Design, defect)

Description

[Darrin Henein [:darrin]]

Attachment: 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.

Comment 1

[Robin Andersen [:tecgirl]]

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 ;-)

Comment 2

[Darrin Henein [:darrin]]

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?

Comment 3

[Ryan Feeley [:rfeeley]]

Note that "Logout" is a noun like "login". As an action, the verb "Log Out" (much like "Log In") is more appropriate.

Comment 4

[Richard Newman [:rnewman]]

Not blocking on polish at this point. Scream if this offends.

Comment 5

[Darrin Henein [:darrin]]

Attachment: https://github.com/mozilla/firefox-ios/pull/665

Comment 6

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Comment on attachment 8628923 [details] [review]
https://github.com/mozilla/firefox-ios/pull/665

LGTM

Comment 7

[Darrin Henein [:darrin]]

Merged https://github.com/mozilla/firefox-ios/pull/665