Closed
Bug 1029787
Opened 12 years ago
Closed 11 years ago
Keep profile information in sticky nav instead of language
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cassie, Unassigned)
Details
(Whiteboard: designQA, nav)
Attachments
(2 files, 1 obsolete file)
When you scroll down on a page like https://webmaker.org/en-US/resources the subnav sticks to the top.
On the top righthand side, the language picker stays while the profile information disappears. I think it would be better if it were the other way around, if the profile information stayed and the language picker disappeared – there is more you can do with the profile dropdown, whereas the language picker is a set-and-forget type of functionality.
Comment 1•12 years ago
|
||
Attachment #8456434 -
Flags: review?(kate)
Comment 2•12 years ago
|
||
Comment on attachment 8456434 [details] [review]
https://github.com/mozilla/webmaker.org/pull/877
Looks good!
If this is using the exact same html as the other user bar area, I would suggest abstracting this into a partial with ngInclude, or a creating a custom directive.
If you decide to do that, put it back on me for R
Attachment #8456434 -
Flags: review?(kate) → review+
Comment 3•12 years ago
|
||
Using a partial now
Attachment #8456434 -
Attachment is obsolete: true
Attachment #8456963 -
Flags: review?(kate)
Comment 4•12 years ago
|
||
Comment on attachment 8456963 [details] [review]
https://github.com/mozilla/webmaker.org/pull/877
Nice!
Attachment #8456963 -
Flags: review?(kate) → review+
Comment 5•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/webmaker.org
https://github.com/mozilla/webmaker.org/commit/6b1c61fc4720a539e8831983f33d210e671e63f1
Bug 1029787 Keep profile information in sticky nav
https://github.com/mozilla/webmaker.org/commit/8f8eacc0e0e46d2f793030edba10d58e153e0e36
Merge pull request #877 from errietta/bug1029787
Bug 1029787 Keep profile information in sticky nav
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•