2015-01-05 13:35 GMT+01:00 Ecaterina Moraru (Valica) <[email protected]>:
+0 if it's really needed.
Well, it is not really needed. We can also decide that we support only one syntax and set it in xwiki.properties. But it is not the way we usually do in XWiki.
From an user perspective it's a shame that we are polluting the Skin class with a setting that will be rarely used and that we cannot auto-determine the doctype from the skin's htmlheader.vm
We can actually render htmlheaders.vm, detect the doctype (string detection on "<!DOCTYPE") and put it in memory to not do this detection at every request. But what if htmlheaders.vm is not correctly rendered (because of an #if() somewhere)? I think this solution is relatively complicated for very few benefit. An other idea is to put this setting in skin.properties only. The reasoning is that if you create a skin on the wiki, unless you overrides all the templates, you will need to use the same syntax than the parent skin, in the FS. This way, we don't add a field in the Skin Class, and things looks easier. This is the solution I will put in place internally in order to have valid HTML5 in XWiki 6.4, but this discussion still remains for 7.x. WDYT? Thanks, Guillaume