I'm not sure I agree with this patch (even though it might be right from a semantic POV), at least not before we discuss it and decide voluntarily that we want to change our existing terminology. We call them headers and not headings at the moment. This is true both in the language we use and in the technical code (HeaderBlock, etc). So if we want to change that, we'll need to : - modify the documentation (xwiki.org and javadoc) - modify the class names in the code (HeadingBlock, HeadingLevel, etc) Since it's a big change I'm not sure we want it. Also note that this has nothing to do with HTML so even though HTML might call them headings it doesn't matter for us (our syntax is independant or the renderer syntax). WDYT? Thanks -Vincent On Apr 22, 2010, at 12:56 AM, sdumitriu (SVN) wrote:
Author: sdumitriu Date: 2010-04-22 00:56:11 +0200 (Thu, 22 Apr 2010) New Revision: 28505
Modified: enterprise/trunk/wiki/src/main/resources/XWiki/XWikiSyntax.xml Log: XE-647: Misuse of the "header" term instead of "heading" when referring to the html h1-6 Fixed. Patch from Dan Bozaru applied without changes.
Modified: enterprise/trunk/wiki/src/main/resources/XWiki/XWikiSyntax.xml =================================================================== --- enterprise/trunk/wiki/src/main/resources/XWiki/XWikiSyntax.xml 2010-04-21 22:55:34 UTC (rev 28504) +++ enterprise/trunk/wiki/src/main/resources/XWiki/XWikiSyntax.xml 2010-04-21 22:56:11 UTC (rev 28505) @@ -85,7 +85,7 @@ |Two standalone elements not separated by 2 new lines|{{{| table cell}}}\\{{{* list item}}}|{{{| table cell}}}\\\\{{{* list item}}} |Two standalone elements not separated by 2 new lines|{{{paragraph}}}\\{{{----}}}\\{{{paragraph}}}|{{{paragraph}}}\\\\{{{----}}}\\\\{{{paragraph}}} |Ignored new line at beginning of document|<new line at beginning of document>\\paragraph|paragraph -|Not closed header syntax|{{{=== header}}}|{{{=== header ===}}} +|Not closed heading syntax|{{{=== heading}}}|{{{=== heading ===}}}
= Paragraphs =
@@ -102,11 +102,11 @@ |Two paragraphs|Paragraph one\\\\Paragraph two|Paragraph one\\\\Paragraph two|Paragraph one\\\\Paragraph two|Paragraph one\\\\Paragraph two |Parametrized paragraph|<p style="text-align:center;color:blue">Centered and blue paragraph</p>|{{{(% style="text-align:center;color:blue" %)}}}\\Centered and blue paragraph|(% style="text-align:center;color:blue" %)\\Centered and blue paragraph|(% style="text-align:center;color:blue" %)\\Centered and blue paragraph
-= Headers = += Headings =
(% style="width:100%" %) |=Feature|=XWiki Syntax 1.0|=XWiki Syntax 2.0|=Result -|Standard headers|((( +|Standard headings|((( 1 level 1 1.1 level 2 1.1.1 level 3 @@ -125,11 +125,11 @@ <!-- Note: replace this with wiki syntax when http://jira.xwiki.org/jira/browse/XWIKI-4358 is implemented --> <h1 id="Hlevel1"><span>level 1</span></h1><p> </p><h2 id="Hlevel2"><span>level 2</span></h2><h3 id="Hlevel3"><span>level 3</span></h3><h4 id="Hlevel4"><span>level 4</span></h4><h5 id="Hlevel5"><span>level 5</span></h5><h6 id="Hlevel6"><span>level 6</span></h6> {{/html}}))) -|Parametrized headers|<h1><span style="color:blue">header</span></h1>|{{{(% style="color:blue" %)}}}\\= header =|((( +|Parametrized headings|<h1><span style="color:blue">heading</span></h1>|{{{(% style="color:blue" %)}}}\\= heading =|((( {{html}} -<h1 id="Hheader" style="color:blue"><span>header</span></h1> +<h1 id="Hheading" style="color:blue"><span>heading</span></h1> {{/html}}))) -|Headers with wiki syntax|1.1.1 Header with *bold*|{{{=== Header with **bold** ===}}}|(((=== Header with **bold** ===))) +|Headings with wiki syntax|1.1.1 Heading with *bold*|{{{=== Heading with **bold** ===}}}|(((=== Heading with **bold** ===)))
= Text Formatting =
@@ -829,12 +829,12 @@ (% style="width:100%" %) |=XWiki Syntax 1.0|=XWiki Syntax 2.0|=Generated XHTML |((({{{ -<h1 class="myClass" style="myStyle" id="myId">header</h1> +<h1 class="myClass" style="myStyle" id="myId">heading</h1> }}})))|((({{{ (% class="myClass" style="myStyle" id="myId" %) -= header = += heading = }}})))|((({{{ -<h1 class="myClass" style="myStyle" id="myId">header</h1> +<h1 class="myClass" style="myStyle" id="myId">heading</h1> }}})))
{{id name="syntaxes"/}} @@ -848,4 +848,4 @@ * [[Creole 1.0>>http://www.wikicreole.org/wiki/Creole1.0]] * [[XHTML 1.0>>http://www.w3.org/TR/xhtml1/]]
-{{warning}}The implementation for these syntaxes is not fully finished yet. For example support for links is not working perfectly yet. We also need to define if we want to extend the original syntaxes to support XWiki-specific features like ability to link to another sub-wiki.{{/warning}}</content></xwikidoc> \ No newline at end of file +{{warning}}The implementation for these syntaxes is not fully finished yet. For example support for links is not working perfectly yet. We also need to define if we want to extend the original syntaxes to support XWiki-specific features like ability to link to another sub-wiki.{{/warning}}</content></xwikidoc>