[TYPO3-core] RFC #14461: TCEforms: Use fieldsets for palettes instead of tables

Rens Admiraal renst3 at rensnel.nl
Tue May 25 00:27:46 CEST 2010


Just reviewed the patch. Pallettes are still working, with the 'show 
secundary options' checkbox, and with the icon on the right site of the 
field. HTML looks valid, and patch looks clean.

I vote a + 1

Greetz,
Rens

Op 24-5-2010 23:31, Steffen Gebert schreef:
> Am 22.05.2010, 18:03 Uhr, schrieb Benjamin Mack <benni at typo3.org>:
>
>> Type: cleanup
>>
>> Branches: trunk
>>
>> BT reference: http://bugs.typo3.org/view.php?id=14461
>>
>> Problem/Solution:
>> In order to be semantically correct, have less HTML-tables for the
>> layout, and to have more flexibility, we should use a fieldset for a
>> palette. This patch also introduces CSS classes according to the new
>> naming scheme of the fields in this palette.
>>
>> Notes:
>> #1 I'm quite unsure about the impact on this one, maybe someone might
>> improve the CSS here to make it more consistent? Doesn't feel 100% yet,
>> but I want your comments :)
>>
>> #2 This patch was done by me, but initially developed by Joey Hasenau on
>> the T3UXW09, I merely updated it, did some minor adjustments to fit to
>> current trunk.
>
> Would be nice, if sb. could review this.
> I need it.
>
> Thanks
> Steffen



More information about the TYPO3-team-core mailing list