Skip to content
This repository has been archived by the owner. It is now read-only.

Remove avatar section and improve avatar hover state #4520

Closed
ryanfeeley opened this issue Dec 16, 2016 · 4 comments
Closed

Remove avatar section and improve avatar hover state #4520

ryanfeeley opened this issue Dec 16, 2016 · 4 comments
Assignees

Comments

@ryanfeeley
Copy link
Contributor

@ryanfeeley ryanfeeley commented Dec 16, 2016

Let's simplify the settings section, but getting rid of the Account picture section. An improved hover effect with inner white border should help. Also add link title "Change account picture"
screenshot 2016-12-16 12 55 25

Most editing the avatar don't come from here anyway:
screenshot 2016-12-16 12 56 49

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Dec 16, 2016

Yea!

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Dec 19, 2016

@ryanfeeley - I am a strong -1 on this suggestion. Using only a hover state will make it almost impossible for mobile or screen reader users to discover the change avatar functionality.

I have this suspicion the linked to graph does not show what you think it shows. My hunch is you are interpreting the graph as showing users that open /settings/avatar/change directly from preferences or the menupanel. If so, that's a misinterpretation. Rather, the graph shows users that somehow get to the avatar section from preferences or the menupanel. It could be they opened settings from the "manage account" link and then clicked on "Account picture".

A suggestion was floated in HI that I liked was to alter the settings page layout to be more hierarchical. A "profile" section and a "security" section I think the suggestion was.

@ryanfeeley
Copy link
Contributor Author

@ryanfeeley ryanfeeley commented Dec 21, 2016

@shane-tomlinson Good point. We are one of the few internet properties that has a responsive profile section. Most have an entirely different view. I'll take a look and see what's the latest on this issue.

@shane-tomlinson shane-tomlinson added this to the FxA-0: quality milestone Dec 21, 2016
@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Dec 22, 2016

closing for now until we needed ...

@vladikoff vladikoff closed this Dec 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants