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

CommentFileSizeAuthor
#6 user.module_6_0.patch805 bytesBèr Kessels
#5 user.module_6.patch803 bytesm3avrck
#2 admin.access.hover.help.v2.patch804 bytesjjeff
admin.access.hover.help.patch799 bytesjjeff

Comments

dries’s picture

dries commented
Status: Reviewed & tested by the community » Needs work

Coding style needs work (incorrect use of spaces). Shouldn't these strings be translatable?

jjeff’s picture

jjeff commented
StatusFileSize
new admin.access.hover.help.v2.patch804 bytes

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

morbus iff’s picture

morbus iff
they/them
commented
Status: Needs work » Needs review
jjeff’s picture

jjeff commented

Anyone want to review my code?

Thanks,
Jeff

m3avrck’s picture

m3avrck commented
StatusFileSize
new user.module_6.patch803 bytes

Rerolled patch against current HEAD. Looks good to me and seems to be pretty useful :)

Bèr Kessels’s picture

Bèr Kessels commented
StatusFileSize
new user.module_6_0.patch805 bytes

changed spaces. And created http://drupal.org/node/30785.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

lets push this one line beauty to 'ready for commit'

jjeff’s picture

jjeff commented

Thank you Moshe!

dries’s picture

dries commented

Committed by HEAD.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
jjeff’s picture

jjeff commented
Version: » 4.6.5
Status: Closed (fixed) » Active

Looks like this functionality got destroyed during the formapi upgrade.

moshe weitzman’s picture

moshe weitzman commented
Status: Active » Closed (duplicate)

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