Bug 95548 – Some whitespace characters and joiners are not rendered as entities in MarkupAccumulator

• Status: RESOLVED
• Resolution: INVALID
• Component: HTML Editing

[Alexander Pavlov (apavlov)]

http://getfirebug.com/developer/api/firebug1.6X/symbols/src/content_firebug_lib.js.html, lines 2186-2194 list a few "invisible" characters that have corresponding HTML entities. Some of these entities, however, are never rendered in the markup generated for text nodes. This is the list of Unicode characters along with entities that should be generated for them:

0x2002 - 'ensp'
0x2003 - 'emsp'
0x2009 - 'thinsp'
0x200c - 'zwnj'
0x200d - 'zwj'
0x200e - 'lrm'
0x200f - 'rlm'
0x200b - '#8203'

[Alexander Pavlov (apavlov)]

Created attachment 161676 [details]
Patch

[Alexey Proskuryakov]

Please elaborate on why these characters should be encoded, and whether that matches other browsers.

Also, please test what happens with XML - it's not OK to use these named entities in raw XML documents.

[Darin Adler]

Comment on attachment 161676 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=161676&action=review

> Source/WebCore/ChangeLog:16
> +        Add the following entities to be rendered in the generated markup:
> +        -  
> +        -  
> +        -  
> +        - ‌
> +        - ‍
> +        - ‎
> +        - ‏
> +        - ​ (zero-width space, '\u200b')

Why? Does the specification say that these need to be entities and not just characters?

[Alexander Pavlov (apavlov)]

(In reply to comment #3)
> (From update of attachment 161676 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=161676&action=review

[...]

> Why? Does the specification say that these need to be entities and not just characters?

Aha, I finally found the spec and take my words back! Sorry for the noise.

[Benjamin Poulain]

Comment on attachment 161676 [details]
Patch

> > Why? Does the specification say that these need to be entities and not just characters?
> 
> Aha, I finally found the spec and take my words back! Sorry for the noise.

I assume you also meant to clear the review flag?