Unnecessary margin div generated for multiple fields when they're hidden
Developer Preview
7 years ago
When I generate a form field like so:
$sections[0][] = array(
'title' => 'set_label',
'desc' => 'set_label_help',
'fields' => array(
'set_id' => array(
'type' => 'hidden',
'value' => $set_id
),
'set_label' => array(
'required' => TRUE,
'type' => 'text',
'value' => $set['set_label']
)
)
);
…I get a div.add-mrg-top around the second input field, resulting in unnecessary spacing. The div shouldn’t be added when there’s hidden fields there.
7 years ago
Confirmed and fixed, thanks Low, the next build will only add that margin for visible sequential fields. If you want to patch yours up now so you can be sure your forms are rendering properly, open system/ee/EllisLab/ExpressionEngine/View/_shared/form/fieldset.php and replace the $count++ on line 130 with:
if ($field['type'] != 'hidden')
{
$count++;
}