470123 - [qmo2][Discoverability] Add a logout link to the head bar beneath "My Accout" or "Create"

Status: VERIFIED FIXED

(quality.mozilla.org :: Website, defect, P2)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

With the current situation it will be hard for new users to find the logout button. No-one will expect to see it in the left column a long way down. We should also place a link into the head bar which gives easy access to the logout function. It could be in-front of "My Account" or behind "Create".

At least, why we need the "Create" link for logged-in users? We could replace it with "Logout".

Comment 1

[Carsten Book [:Tomcat]]

thanks Henrik, marking p2

Comment 2

[Marcia Knous [:marcia]]

Mass move of bugs to new product

Comment 3

[Aakash Desai [:aakashd]]

Per QMO Discoverability, the new implementation will not only create a logout button, but also a create drop down menu and removal of the account admin seciton at the bottom left portion of the page.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Please leave [qmo2] in the whiteboard for tracking purposes. Thanks.

Comment 5

[Carsten Book [:Tomcat]]

(In reply to comment #3)
> Per QMO Discoverability, the new implementation will not only create a logout
> button, but also a create drop down menu and removal of the account admin
> seciton at the bottom left portion of the page.

paul: is this possible to do with the current customizations of qmo ?

Comment 6

[Paul Craciunoiu [:paulc]]

(In reply to comment #5)
> (In reply to comment #3)
> paul: is this possible to do with the current customizations of qmo ?
I believe so, probably through horizontal (as opposed to vertical) nice_menus:
http://drupal.org/project/nice_menus
Depending on how it's currently done, it might take more or less time.

Comment 7

[Aakash Desai [:aakashd]]

Implementation page: 

https://wiki.mozilla.org/QA/Community/QMO_Discoverability/Logout

Comment 8

[Paul Craciunoiu [:paulc]]

Yay, an easy change! Updated block code on both stage and production. This also fixes the spaces in names issue with the "my account" link (mentioned on friday; if there's a bug for that, please cc me)

Comment 9

[Aakash Desai [:aakashd]]

Actually, there's a little bit more to the bug than that. We still need the following:

* account admin section on the bottom left of QMO removed
* on-hover menu for "create" instead of the current page  

It's all on https://wiki.mozilla.org/QA/Community/QMO_Discoverability/Logout

Comment 10

[Paul Craciunoiu [:paulc]]

(In reply to comment #9)
> * on-hover menu for "create" instead of the current page  
What do you mean instead of the current page?

Comment 11

[Aakash Desai [:aakashd]]

http://quality.mozilla.org/node/add

Comment 12

[Paul Craciunoiu [:paulc]]

Ah, it could be both, but I can make it a "#" to avoid people clicking on it and going somewhere else.

Comment 13

[Aakash Desai [:aakashd]]

that works too ! i like it

Comment 14

[Paul Craciunoiu [:paulc]]

(In reply to comment #9)
> Actually, there's a little bit more to the bug than that. We still need the
> following:
> 
> * account admin section on the bottom left of QMO removed
> * on-hover menu for "create" instead of the current page  
> 
> It's all on https://wiki.mozilla.org/QA/Community/QMO_Discoverability/Logout

I'll do both on Friday, hopefully.

Comment 15

[Paul Craciunoiu [:paulc]]

r26122 / r26126
Committed on stage only.

The block structure needs to be updated on push. I'll attach the text that needs to go into the block on production, after push (it's rather large to paste, I'd say).

https://quality.authstage.mozilla.com/

Comment 16

[Paul Craciunoiu [:paulc]]

Attachment: block text for top menu

Comment 17

[Paul Craciunoiu [:paulc]]

r26141 / r26142
Some IE only Javascript to fix some issues with the dropdown. I took it all from here:
http://www.alistapart.com/articles/dropdowns/
But waited first to see if they fixed it in IE8. Guess not :)

(In reply to comment #15)
> r26122 / r26126
Oops here I committed on both eventually. I was first waiting to see it work on stage.

Comment 18

[Aakash Desai [:aakashd]]

hm, a look-see at the QMO staging website, using the "test" account, shows now drop down menus. It works fine for an admin account though.

Comment 19

[Aakash Desai [:aakashd]]

Attachment: Create Off-Center Screenshot

Paul, the menus look shifted to the right from their correct placement on IE7

Comment 20

[Aakash Desai [:aakashd]]

Attachment: My Acct Off-Center Screenshot

Paul, the menu doesn't seem to be wide enough on IE 7 either. I'm going to move this to Re-Opened for a fix.

Comment 21

[Paul Craciunoiu [:paulc]]

I'll need an IE7 box to test this. I'll be in the office in MV this week, can I get some QA help on this?

Comment 22

[Paul Craciunoiu [:paulc]]

r27824
Try it in IE7 on stage.

Comment 23

[Paul Craciunoiu [:paulc]]

I think this is ready for commit on production. Confirm?

Comment 24

[Paul Craciunoiu [:paulc]]

r27942 expanded width to drop down menu. Check in IE7 and confirm.

Comment 25

[Aakash Desai [:aakashd]]

Looks good on IE 7 and Fx3.5 on XP

Comment 26

[Paul Craciunoiu [:paulc]]

Filed push bug 498966

Comment 27

[Aakash Desai [:aakashd]]

verified FIXED