8 Apr
2015
8 Apr
'15
5 p.m.
Ok thanks for the example.
I need to think about it a bit more but I don’t like that we break existing skins with
more recent versions of apps. We also need to start preparing for supporting several skins
in the future.
Also using the HTML macro with clean=true should be prohibited in the code we provide IMO.
Also right now the HTML Cleaner generates valid XHTML, not valid HTML5 AFAIK so there’s an
issue there already, no?
If there’s no way to produce HTML code that is valid both in HTML5 and XHTML in our wiki
pages then we need to decide what to do:
* break XHTML compat for all apps from now on
* break XHTML compat for some apps only (decide which one we break and which ones we don’t
break)
* introduce some API to generate HTML (as Denis suggested at some point)
Right now I’m not sure I agree to break XHTML validity for existing xwiki instances when
they upgrade to XWiki 7.1+ and when they upgrade the apps they use.
I need a bit more time to think about this (especially the consequences) and probably do a
brainstorming session with you.
Thanks
-Vincent
On 8 Apr 2015 at 16:40:50, Guillaume Louis-Marie Delhumeau
(gdelhumeau@xwiki.com(mailto:gdelhumeau@xwiki.com)) wrote:
And the worse case is the XWiki Syntax help page,
where we explicitly encourage our users to use the invalid rel="_blank"
attribute:
https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwi…
2015-04-08 16:38 GMT+02:00 Guillaume "Louis-Marie" Delhumeau :
But Vincent, the problem is that these kind of
links are present in wiki pages too, which do not depend on skins.
Example, in the watchlist:
https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwi…
2015-04-08 16:31 GMT+02:00 vincent@massol.net(mailto:vincent@massol.net) :
--
Guillaume Delhumeau (gdelhumeau@xwiki.com(mailto:gdelhumeau@xwiki.com))
Research & Development Engineer at XWiki SAS
Committer on the XWiki.org project Hi,
What we need to ensure is that people can still continue to write XHTML-valid skins or
use existing XHTML skins without problem.
+1 to support only HMTL5 **for the Flamingo skin**.
Thanks
-Vincent
On 8 Apr 2015 at 16:12:02, Guillaume Louis-Marie Delhumeau
(gdelhumeau@xwiki.com(mailto:gdelhumeau@xwiki.com)(mailto:gdelhumeau@xwiki.com)) wrote:
> Hello developers.
>
> I am working on making the HTML 5 code of XWiki valid. But in some places,
> I cannot do it without breaking the XHTML validity (which Colibri still
> uses, but this skin is going to be deprecated).
>
> Example:
> in XHTML 1.0, the attribute target="_blank", for links, is forbidden. In
> XWiki, we have "fixed" this by using the attribute rel="_blank"
instead,
> and then we have a javascript snippet that dynamically adds the
> target="_blank" attribute to these links [1]. So the code is valid until
> the javascript is loaded, which is enough to pass our validation tests.
> This is not very clean, since the target="_blank" was forbidden by the
w3c
> because they think it should be the user choice to open a link in a new
> window or not. But nobody respects this, and that is why it is no longer
> forbidden in HTML5.
>
> The problem we have, is that the value "_blank" is not authorized anymore
> for the "rel" attribute of the links in HTML5.
>
> So I propose to remove our workaround, which is useless and prohibited for
> HTML5. As a consequence, we would lose the XHTML validity (which is
> actually the case when javascript is loaded anyway).
>
> WDYT?
>
> Guillaume
>
> [1] The code that looks for ref="_blank":
>
https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwi…
> _______________________________________________
> devs mailing list
> devs@xwiki.org(mailto:devs@xwiki.org)
> http://lists.xwiki.org/mailman/listinfo/devs
--
Guillaume Delhumeau (gdelhumeau@xwiki.com(mailto:gdelhumeau@xwiki.com))
Research & Development Engineer at XWiki SAS
Committer on the XWiki.org project