[TYPO3-core] RFC: #7396: hardcoded fieldset and legend around radio buttons

Ingo Renner ingo at typo3.org
Sun Mar 9 11:01:33 CET 2008


Martin Kutschker wrote:

Hi Masi,

> The patch does not do what the docs say. 

well, then the docs should be changed ;-)

> allWrap is only applied in 
> accessibility mode. The docs do not mention this. I stead they do talk 
> of compatibility. Why? The fieldset has been added deliberately. Please 
> rephrase to "default is ...." (or fix the spelling of "backwards").

will, do when commited, before adding to Pending Docs.

> But as the wrap is not apllied always I suggest to name it either 
> accessibilityWrap or do some TS wonders to be of effect only in this 

I think this is not needed as accessibilty mode is automatically enabled 
by setting compatMode to 3.9.0. It's not available in other situations 
anyways.

> mode. BTW, you made it a pure wrap. I'm pretty sure somebody will want 
> it to be a stdWrap :-)

I think we do not need a stdWrap everywhere, especially not here. Other 
than that a wrap is faster then a stdWrap. However, if someone ffels 
like we need a stdWrap, he can come up with a separae patch.


Ingo

-- 
Ingo Renner
TYPO3 Core Developer, Release Manager TYPO3 4.2


More information about the TYPO3-team-core mailing list