On this page, "radio" is listed as an option for a skinr type. It should be "radios".
http://skinr.org/skin-property-reference
(Also, please turn off the auto-rotation on the home tab, if that's an easy change! It's hard to follow the instructions with the pages turning.)
If an invalid type is used (such as radio), then skin options will duplicate.
For example, the definition below has one error: an invalid type for the second skin, "asdfasdf". The effect of the faulty code is General Styles will display twice. See screenshot.
$skins['acquia_marina_general_styles'] = array(
'title' => t('General Styles'),
'type' => 'select',
'attached' => array('css' => array('css/skins.css')),
'default status' => TRUE,
'group' => 'general',
'options' => array(
'rounded_corner' => array(
'title' => t('Rounded Corner'),
'class' => array('marina-rounded-corners')
),
),
);
$skins['acquia_marina_my_lists'] = array(
'title' => t('Lists Styles'),
'type' => 'asdfsadf',
'attached' => array('css' => array('css/skins.css')),
'default status' => TRUE,
'group' => 'general',
'options' => array(
'option_1' => array(
'title' => t('Apply equal heights'),
'class' => array('equal-heights'),
),
),
);
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | patch_commit_a25f829f6136.patch | 4.91 KB | moonray |
| duplicate skin | 19.97 KB | aquariumtap |
Comments
Comment #1
moonray commentedDocs fixed, but the behavior you describe above sounds like a bug.
Comment #2
vrajak@gmail.com commentedI tested this also using Acquia Marina, and can confirm the behavior reported. "General Styles" does appear twice.
I didn't try using another theme yet but I can if you think that's necessary.
It would seem kind of moot since proper skin 'type' needs to be used for things to really work anyway, but the bug or whatever it is does exist.
Comment #3
moonray commentedAnd here's a patch with tests and a fix.
Comment #4
moonray commentedCommitted the patch with some slight modifications to the test output text to make it more brief.