[TYPO3-core] Accessible PopUp links

[Martin Kutschker]

Michael Stucki <michael at typo3.org> writes on 
Tue, 22 Nov 2005 14:18:33 +0100 (MET):

> Hi Andreas,
> 
> 
> >> If the above is true, then I'm supporting the target attribute.
> > > Just make
> >> sure that there won't be two additional windows...
> > 
> > There won't be two windows.
> > 
> > My question is related to the fact that the TARGET-attribute is not
> > part
> > of the xhtml-transitional or xhtml-strict.
> 
> Aha. Sorry, didn't remember this because I didn't care about it yet.
> 
> I support Masis suggestion to make it depending of the doctype, but I
> wouldn't make it configurable. If someone wants XHTML compliant web
> pages,
> then he is not willing to make an exception for the target attribute.
> So it should be fine to simply act according to the doctype.


I was thinking more about the other case. Someone using transitional, but wants no target. If users really care about this we can implement a target-disable switch later on.

As for the patch:

Is $finalTagParts['url'] htmlspecialchars'ed already? I notice that $onclick is hsc'ed within the string concatenation.

Please check, otherwise +1

Masi