OK, I feel brave today, and although I could get hate (love?) mail for this, I wanted to give it a go, see what the community thinks.
This is something that has bothered me (and others) for some time in CCK, and I see it stil in d7 fields. The current situation is such that for all fields, no matter whether they have a single value, or several, the markup is like this:
<div class="field field-name-body field-type-text-with-summary field-label-hidden"> <div class="field-items"> <div class="field-item even" property="content:encoded"> .... </div> </div> </div>
That gives us two levels of unnecessary divs. What I propose would get you:
<div class="field field-name-body field-type-text-with-summary field-label-hidden"> .... </div>
It makes more sense to me, anyway, and I feel a lot of themers who care probably already do something similar in their theme implementations.
I've taken the initiative to draft up the patch, please let me know what you think if this is something to consider at all or not, and if we should approach it a different way.
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 935878-field_single_markup-7.patch. Unable to apply patch. See the log in the details link for more information. View
FAILED: [[SimpleTest]]: [MySQL] 40,370 pass(es), 6 fail(s), and 8 exception(s). View
|field-single-markup.patch||1.4 KB||Manuel Garcia|
FAILED: [[SimpleTest]]: [MySQL] 26,014 pass(es), 7 fail(s), and 9 exception(es). View