form() should respect block level elements. [#23387]

This patch uses a blank div to allow block level ellements (like input> to go inside the form element per xhtml 1.0 specs. This patch works and validates, and is required for validation.

Robin

Comment	File	Size	Author
#5	empty__div_form__item_fix.patch	5.06 KB	Robin Monks
#5
#3	empty div form item fix_0.patch	5.06 KB	Robin Monks
#3
	empty div form item fix.patch	682 bytes	Robin Monks

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Dries CreditAttribution: Dries commented 23 May 2005 at 20:30

Yes/no? Please advise. If this patch is required, we might want to revise the call sites of form(). I believe some are adding divs to solve exactly this.

Comment #2

Robin Monks CreditAttribution: Robin Monks commented 23 May 2005 at 20:43

You're right, some calls are using <div class="container-inline"> to do this. I'll find these instances and correct them, and then attach a new patch for this.

Robin

Comment #3

Robin Monks CreditAttribution: Robin Monks commented 24 May 2005 at 12:55

File	Size
empty div form item fix_0.patch	5.06 KB

Here is the new patch, which also removes unneeded (AFAIK) <div>'s from form's.

Robin

Comment #4

Stefan Nagtegaal CreditAttribution: Stefan Nagtegaal commented 24 May 2005 at 12:59

Could you please remove the spaces from your patch name? they break the link in e-mail clients quite often.. I would prefer not to use spaces at all inside patch filenames at all, I would encourage you to use something like 'patch_name.patch'..

Stefan