Closed (fixed)
Project:
Administration menu
Version:
6.x-3.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
30 Sep 2008 at 16:37 UTC
Updated:
21 Mar 2009 at 01:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
sunThe 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.
Comment #2
gaele commentedSorry, 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):
Comment #3
sunok, if we would turn that around into
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.
Comment #4
wulff commentedThe 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>.
Comment #5
sunLooks 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...?
Comment #6
wulff commentedMy 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.
Comment #7
gaele commentedI'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).
Comment #8
sunComment #9
sunActually, a very nice patch! Committed to 3.x :)