12 Jan
2015
12 Jan
'15
5:01 p.m.
2015-01-05 13:35 GMT+01:00 Ecaterina Moraru (Valica) <valicac(a)gmail.com>om>:
+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