TCO Email Forms Extra Fieldsets

[strobley]

Hi Themeco,

Try the email forms extension, http://demo.theme.co/email-forms/

On version 2.0.3 in the current TF cdownload when I create any form of any style it seems to generate 2 extra

<fieldset>
</fieldset>

<fieldset>
</fieldset>

Tags inbetween the email input and submit button. Not really a problem, only that these extra tags create a space between the email input and submit when display: inline; is set.

This behaviour is not the Themeco demo, however I have installed it multiple times on a number of clean installs, including, excluding names, combing names, etc. and the same happens every time. Is the version in TF the latest, does the same occur on your test installs?

[paul.r]

Hi,

I was able to replicate the issue on my test site, but it shouldn’t add extra space.

You can try adding vertical-align:top; to your css code and remove top margin of your submit button

Please change your css code to this.

.tco-subscribe-form fieldset {
    vertical-align: top;
    display: inline-block;
}
.tco-subscribe-form input[type="submit"] {
    margin-top:0;
}

Hope the helps

[strobley]

Hi Paul,

I’ve already done this. Using display: inline-block it is displaying both email and submit inline as desired. I’ve set the margin-top: 0 on the submit and vertical-align: top; on fieldset.

This does not help wih the space as is to the right of input[type=“email”] and to the left of input[type=“submit”] - i.e. the space is the 2 additional fieldset tags. When I set display: none; or delete these fieldset elements in Developer Tools the space is removed.

I can target these 2 additional fieldset tags with nth-child(n) display:none; however this is hacky and assumes I’m always displaying only email and submit. If many months from now names are added, either separated or combined - both variations would generate exrta fieldsets to css needs updating. It is far cleaner to remove these on next release.

[christian]

Hey @strobley,

Thanks for reporting. I’ll post this in our issue tracker. Please stay tuned for updates.