On Drupal sites with plenty of modules installed, some sub-level menus can become so large that these item lists are longer than the actual HTML page. For example, vertical browser window scrollbars are displayed on pages that contain almost no content and also no blocks in sidebars.

We are using left: -999em of the original Suckerfish implementation, because:

/* using left instead of display to hide menus because display: none isn't read by screen readers */

But who administers Drupal with a screen reader?

Attached patch adds display: none to the stylesheet, so invisible menu items in sub-level menus are actually hidden.

Needs thorough testing on different platforms and browsers.

Done: Windows: Firefox 2, IE6, Safari 3, Opera 9.

CommentFileSizeAuthor
admin_menu-DRUPAL-5--2.css-display.patch1.3 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SamRose’s picture

SamRose CreditAttribution: SamRose commented

tested on Mac OS X Firefox 2.0.0.14: works.

Tested IE7 Windows XP: works.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Fixed

Committed to HEAD, 6.x, and 5.x-2.x.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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