Add "My account"

gaele - September 30, 2008 - 16:37
Project:Administration menu
Version:6.x-3.x-dev
Component:User interface
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

From the Navigation menu Create content, Administer and Log out are all shown in admin_menu. Only My account is missing. Add the latter, and the whole Navigation menu can go.

The username is already shown. How about displaying /user (username), /admin/user/user and /logout ("Log out") on the right?

#1

sun - September 30, 2008 - 19:07

The user status counter already links to /admin/user/user. Since the logout link already contains the username, I'm unsure where this link could be placed.

#2

gaele - September 30, 2008 - 21:08

Sorry, I wasn't clear. My suggestion is to add a my account link, labeled with the username, on the left side of this block. The username can then be removed from the logout link. So (in my case):

gaele  0/1  Log out

#3

sun - September 30, 2008 - 21:30

ok, if we would turn that around into

0/1  sun  Logout

the current username and logout link would be closely tied together. If we change this, we could also move Devel's switch user links from the logout link to the username, resulting in a more intuitive UI (for developers), because those links neither logout the current user nor the linked user, but change the current user instead.

So, yes, +1 from me. However, I'm unsure when I'll be able to code this.

#4

wulff - November 1, 2008 - 12:49

The attached patch separates the username from the logout link and moves the switch links to the username.

The downside to this solution is that users without access to admin/user/user get links to both user and user/<uid>.

AttachmentSize
admin_menu.patch 1.36 KB

#5

sun - November 1, 2008 - 15:18
Status:active» needs review

Looks good! I didn't understand your point about admin/user/user though. Do you mean the link on the user counter icon? Shall we remove the link on it if the user has no 'administer users' permission? I think that would make sense.

...or did anyone ever use this link on the user counter icon? Maybe we should output only the counter without a link...?

#6

wulff - November 1, 2008 - 18:20

My point about the link from the counter icon is, that a user without the 'administer users' permission will see the following links if she's using the patched version of the module:

Counter icon: user
Username: user/<uid>

While the admin will see:

Counter icon: admin/user/user
Username: user/<uid>

Mostly a cosmetic problem, I guess.

Speaking from my own experience, I have never used the link on the counter icon to get to the user list, so I'm in favor of removing it. I've been trying a couple of options, but haven't found a clean way to include the counter in the menu without it being a link to an existing page.

#7

gaele - November 3, 2008 - 12:41

Shall we remove the link on it if the user has no 'administer users' permission? I think that would make sense.

...or did anyone ever use this link on the user counter icon? Maybe we should output only the counter without a link...?

I've never used this link. It is an administration link, while the other two are personal links, so perhaps it may fit better on the left side (if at all).

#8

sun - February 12, 2009 - 18:45
Version:6.x-1.1» 6.x-3.x-dev

#9

sun - March 7, 2009 - 01:12
Status:needs review» fixed

Actually, a very nice patch! Committed to 3.x :)

#10

System Message - March 21, 2009 - 01:20
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.