[TYPO3-core] FYI24: #11124: Validation issue: End tag for "br" omitted, but OMITTAG NO was specified in typo3/mod/tools/em/class.em_index.php
Steffen Kamper
info at sk-typo3.de
Wed May 20 17:50:34 CEST 2009
Hi,
Ernesto Baschny [cron IT] schrieb:
> Steffen Kamper wrote: on 20.05.2009 17:39:
>
>> Ernesto Baschny [cron IT] schrieb:
>>> seem not needed. Both are valid XHTML. The space before the "/" is AFAIK
>>> optional. Same goes for the <hr/> => <hr /> corrections.
>>>
>> that's true, both are valide, but some validators complain and some
>> purists too. I'm in favor having the space (like good style, and
>> compares to our CGL having the spaces too)
>> So it doesn't hurt.
>
> Which validator complains about that? That must be a bug. I thought the
> space is needed for "older browsers" that cannot cope with XHTML to
> still understand the tag (interpret it as "HTML"). So while I also
> prefer the space before the "/", I think fixing that issue with 40
> FYI24s is a bit too much. :)
>
yeah, validators are don't tell the truth all the time. i think
w3validator does a good job.
I talked to Patrick and he was curious about doing rhis in one patch
changing so much files.
> Any why in this context:
>
> + $content.= '<label for="_lookUp">Look up:</label> <input type="text"
> id="_lookUp" name="_lookUp" value="' .
> htmlspecialchars($this->lookUpStr) . '" /><input type="submit"
> value="Search"/><br /><br />';
>
>
> isn't the <input/> also fixed? With that logic, wouldn't that
> require a space before the "/" also?
>
he concentrated on br and hr this round, but he has a lot of others like
this in the backhand ;-)
vg Steffen
More information about the TYPO3-team-core
mailing list