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>&nbsp;</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>
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs
