Re: [bug] AllowedCharacters + PreserveSpace - minor issues

Allowed characters and preserve space have been updated to match your
output. I am going to start work on the Id value data category now.

Leroy




On 5 December 2012 22:08, Fredrik Liden <fliden@enlaso.com> wrote:

> These looks good, few minor things...****
>
> * *
>
> *AllowedCharacters *
>
> ** **
>
> Missing the “Pointer”****
>
>
> File: allowedcharacters\xml\allowedcharacters2xmloutput.txt   ****
>
> 6****
>
> /res/head[1]/its:rules[1]/its:allowedCharactersRule[1]/@allowedCharacters
> Pointer****
>
> <>** **
>
> 6****
>
> /res/head[1]/its:rules[1]/its:allowedCharactersRule[1]/@allowedCharacters*
> ***
>
> ** **
>
> ** **
>
> ** **
>
> *PreserveSpace*
>
> ** **
>
> I think we should add space=”default” to all elements and attributes since
> it’s listed in the default column.
> http://www.w3.org/International/multilingualweb/lt/drafts/its20/its20.html#datacategories-defaults-etc
> ****
>
> I just says “default” though not “space=default” so it could be
> interpreted differently. But listing it for each node will show that
> inheritance and override works.****
>
> ** **
>
> If anything, since space applies to the whole element and attributes I
> think we should at least display it for all the attributes as well. See
> preservespace2xmloutput below.****
>
>
> Left base: Suggested****
>
> Right base: Current
>
>
> File: preservespace\xml\preservespace1xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
> 3****
>
> /book/info[1]        space="default"****
>
>  ****
>
> 3****
>
> /book/info[1]****
>
> 4****
>
> /book/info[1]/its:rules[1]        space="default"****
>
>  ****
>
> 4****
>
> /book/info[1]/its:rules[1]****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version        space="default"****
>
>  ****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version****
>
> 6****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]        space="default"
> ****
>
>  ****
>
> 6****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]****
>
> 7****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@selector
> space="default"****
>
>  ****
>
> 7****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@selector****
>
> 8****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@space
> space="default"****
>
>  ****
>
> 8****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@space****
>
>
>
> File: preservespace\xml\preservespace2xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
>  ****
>
> 4****
>
> /book/verse[1]/@xml:space        space="preserve"****
>
> <>** **
>
> 4****
>
> /book/verse[1]/@xml:space****
>
>
>
> File: preservespace\xml\preservespace3xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
> 3****
>
> /book/info[1]        space="default"****
>
>  ****
>
> 3****
>
> /book/info[1]****
>
> 4****
>
> /book/info[1]/its:rules[1]        space="default"****
>
>  ****
>
> 4****
>
> /book/info[1]/its:rules[1]****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version        space="default"****
>
>  ****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version****
>
> 6****
>
> /book/info[1]/its:rules[1]/@xlink:href        space="default"****
>
>  ****
>
> 6****
>
> /book/info[1]/its:rules[1]/@xlink:href****
>
>
>
> File: preservespace\xml\preservespace4xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
>  ****
>
> 4****
>
> /book/its:span[1]/@xml:space        space="preserve"****
>
> <>** **
>
> 4****
>
> /book/its:span[1]/@xml:space****
>
>
>
> File: preservespace\xml\preservespace5xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
> 3****
>
> /book/info[1]        space="default"****
>
>  ****
>
> 3****
>
> /book/info[1]****
>
> 4****
>
> /book/info[1]/its:rules[1]        space="default"****
>
>  ****
>
> 4****
>
> /book/info[1]/its:rules[1]****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version        space="default"****
>
>  ****
>
> 5****
>
> /book/info[1]/its:rules[1]/@version****
>
> 6****
>
> /book/info[1]/its:rules[1]/its:param[1]        space="default"****
>
>  ****
>
> 6****
>
> /book/info[1]/its:rules[1]/its:param[1]****
>
> 7****
>
> /book/info[1]/its:rules[1]/its:param[1]/@name        space="default"****
>
>  ****
>
> 7****
>
> /book/info[1]/its:rules[1]/its:param[1]/@name****
>
> 8****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]        space="default"
> ****
>
>  ****
>
> 8****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]****
>
> 9****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@selector
> space="default"****
>
>  ****
>
> 9****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@selector****
>
> 10****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@space
> space="default"****
>
>  ****
>
> 10****
>
> /book/info[1]/its:rules[1]/its:preserveSpaceRule[1]/@space****
>
>  ****
>
> 12****
>
> /book/verse[1]/@id        space="preserve"****
>
> <>** **
>
> 12****
>
> /book/verse[1]/@id****
>
>
>
> File: preservespace\xml\preservespace6xmloutput.txt   ****
>
> 1****
>
> /book        space="default"****
>
> <>** **
>
> 1****
>
> /book****
>
> 2****
>
> /book/@its:version        space="default"****
>
>  ****
>
> 2****
>
> /book/@its:version****
>
> 3****
>
> /book/its:rules[1]        space="default"****
>
>  ****
>
> 3****
>
> /book/its:rules[1]****
>
> 4****
>
> /book/its:rules[1]/@version        space="default"****
>
>  ****
>
> 4****
>
> /book/its:rules[1]/@version****
>
> 5****
>
> /book/its:rules[1]/@xlink:href        space="default"****
>
>  ****
>
> 5****
>
> /book/its:rules[1]/@xlink:href****
>
> 6****
>
> /book/its:rules[1]/@xlink:type        space="default"****
>
>  ****
>
> 6****
>
> /book/its:rules[1]/@xlink:type****
>
>  ****
>
> 8****
>
> /book/verse[1]/@id        space="preserve"****
>
> <>** **
>
> 8****
>
> /book/verse[1]/@id****
>
> ** **
>