If a module adds:

$form['favicon']['#collapsible'] = true;
$form['favicon']['#collapsed'] = true;

The favicon fieldset won’t collapse properly because the fieldset’s description is placed as a separate, un-wrapped 'markup' instead of a '#description' of the fieldset. This patch only changes $form['favicon']['text'] to $form['favicon']['#description'].

CommentFileSizeAuthor
#2 favicon-fieldset.patch1.07 KBZen
system_theme_settings.patch948 bytesjohnalbin

Comments

johnalbin’s picture

Title: 'Shortcut icon settings' not collapsible in system_theme_settings() » 'Shortcut icon settings' not using proper FAPI value

Fixed title to better represent what the patch does.

Zen’s picture

Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new1.07 KB

Well spotted :P Patch rerolled to sync with HEAD + form array split into multiple lines.

Setting to RTBC.
-K

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

drumm’s picture

COmmitted to 5.

Anonymous’s picture

Status: Fixed » Closed (fixed)