Fieldset '#collapsed' => TRUE Bug? w/ fix

paradigmshifter - January 7, 2009 - 10:06
Project:jQuery Update
Version:5.x-2.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Issue tags:accessibility
Description

After installing jQuery update my fieldsets could still expand and collapse but setting the default to collapsed wouldn't work. When looking at the code via FireBug I noticed that initially the div.fieldset-wrapper class styling is set to "Display:inline;" when default collapsed rather than "Display:none;" To fix the problem was a super minor change in the collapse.js.

~line 76: Originally:

.css({height: 'auto', display: 'inline'});

~line 76: Changed to:

.css({height: 'auto', display: 'none'});

Hope this helps someone. Cheers.

#1

mgifford - February 4, 2009 - 18:26

Needed to note here that display: none has accessibility issues - http://juicystudio.com/article/screen-readers-display-none.php

We need to have better approaches to deal with collapsing/expanding sections so that blind folks can even reach the advanced search page for instance.

Mike

#2

mgifford - April 4, 2009 - 14:51

More on display:none; here: http://drupal.org/node/58941

 
 

Drupal is a registered trademark of Dries Buytaert.