admin/access hover help - little patch, big help
jjeff - August 5, 2005 - 19:59
| Project: | Drupal |
| Version: | x.y.z |
| Component: | user.module |
| Category: | feature request |
| Priority: | normal |
| Assigned: | jjeff |
| Status: | duplicate |
Description
Here's a patch that adds a 'title' tag to all of the checkboxes in the admin/access grid. When you roll over the checkboxes in most browsers a tool-tip box will pop up with information about what each checkbox is.
Examples:
"anonymous user : edit own blog"
"authenticated user : post comments"
"admin : administer filters"
On a site with many roles and many modules, it becomes very easy to scroll away from the table headers, so this is essential for figuring out what you're clicking. It doesn't solve all of the admin/access page usability problems, but it does make the page a lot more usable.
Little patch. Big help.
-Jeff
| Attachment | Size |
|---|---|
| admin.access.hover.help.patch | 799 bytes |
#1
Coding style needs work (incorrect use of spaces). Shouldn't these strings be translatable?
#2
This should be better.
By 'incorrect use of spaces', do you mean spacing around the array keys and values? I've fixed that and the $perm (permission) value now gets translated. Role names aren't translated elsewhere on the admin/access page, so I've left $name (role name) untranslated.
-Jeff
#3
#4
Anyone want to review my code?
Thanks,
Jeff
#5
Rerolled patch against current HEAD. Looks good to me and seems to be pretty useful :)
#6
changed spaces. And created http://drupal.org/node/30785.
#7
lets push this one line beauty to 'ready for commit'
#8
Thank you Moshe!
#9
Committed by HEAD.
#10
#11
#12
Looks like this functionality got destroyed during the formapi upgrade.
#13
i put this feature into my patch at http://drupal.org/node/44379. if mine fails, then resubmit this one alone. required a tiny change to theme_user_admin_perm(). here is the changed line (see end):
$row[] = array('data' => form_render($form['checkboxes'][$rid][$key]), 'align' => 'center', 'title' => $key);