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?

CommentFileSizeAuthor
#4 admin_menu.patch1.36 KBwulff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

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.

gaele’s picture

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
sun’s picture

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.

wulff’s picture

FileSize
1.36 KB

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

sun’s picture

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...?

wulff’s picture

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.

gaele’s picture

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

sun’s picture

Version: 6.x-1.1 » 6.x-3.x-dev
sun’s picture

Status: Needs review » Fixed

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

Status: Fixed » Closed (fixed)

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