Hi

would you please add own class for each fieldset in create quiz form?
I would like to change color of backgrounds for various fieldset and hide whole "Multiple takes" fieldset (I didn't need this), but now I can't target some concrete fieldset, because all of them are without css class attribute.

thanks
Igor

Comments

falcon’s picture

falcon commented
Status: Active » Closed (fixed)

I though Drupal already did this, but obviously it didn't. I added id attributes to all the fieldsets. (Not classes because each fieldset will always be unique AFAIK. id's have better javascript and css performance)

This kinda is a feature, but there is no way this can cause any other bugs AFAIK.

Thanks for reporting!

igorik’s picture

igorik commented
Status: Closed (fixed) » Active

Hi

Fieldset "Multiple takes" is still without id.

It is in quiz.module on the line 936

$form['taking']['multiple_takes'] = array(
'#type' => 'fieldset',
'#title' => t('Multiple takes'),
'#collapsible' => FALSE,
'#collapsed' => FALSE,
);

It could be

$form['taking']['multiple_takes'] = array(
'#type' => 'fieldset',
'#title' => t('Multiple takes'),
'#collapsible' => FALSE,
'#collapsed' => FALSE,
'#attributes' => array('id' => 'multiple-takes'),
);

thanks
Igor

falcon’s picture

falcon commented
Status: Active » Closed (fixed)

Sorry about that. Fixed it now. Thanks!