#540282: Accessibility improvements for the toolbar made the toolbar drawer collapsing work without JavaScript, but the code for that seems to have gotten out of date and no longer works at all.

The attached patch mostly seems to make it work. The only issue I still see is that when collapsing/expanding with Javascript disabled, the padding top (that makes the toolbar correctly displace the page content vertically) isn't consistent between Garland and Seven. I think there might be work at #668280: Toolbar font size keeps jumping around or #698976: Fix toolbar height with CSS and make text zoom usable that would help with that though.

CommentFileSizeAuthor
#1 toolbar-collapse-no-javascript.patch1.7 KBDavid_Rothstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
FileSize
toolbar-collapse-no-javascript.patch1.7 KB

Swear I attached it the first time.

james.elliott’s picture

james.elliott CreditAttribution: james.elliott commented
Status: Needs review » Reviewed & tested by the community

Tested this in IE7, IE8, Chrome 4, FF 3.5, FF 3.6, Safari 4 and setting to RTBC

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

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