Bug 93888

Summary: Web Inspector: Unhide hidden characters
Product: WebKit Reporter: Ebrahim Byagowi <ebrahim>
Component: Web Inspector (Deprecated)Assignee: Alexander Pavlov (apavlov) <apavlov>
Status: RESOLVED FIXED    
Severity: Enhancement CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 95548    
Bug Blocks:    
Attachments:
Description Flags
Patch pfeldman: review+

Description Ebrahim Byagowi 2012-08-13 13:20:07 PDT 
It would be nice if inspector could make hidden characters (like &lrm; &rlm; &zwnj; &zwj; &nbsp; ...) visible and show them to web developer for better debugging.
You can see list of hidden characters on http://getfirebug.com/developer/api/firebug1.6X/symbols/src/content_firebug_lib.js.html line 2186 to 2194.
Comment 1 Alexander Pavlov (apavlov) 2012-09-03 04:15:24 PDT 
Created attachment 161897 [details]
Patch
Comment 2 Alexander Pavlov (apavlov) 2012-09-03 06:22:22 PDT 
Committed r127418: <http://trac.webkit.org/changeset/127418>
Comment 3 Ebrahim Byagowi 2012-09-03 09:58:09 PDT 
Thanks a lot! :)
I think other directional control characters can also added like lrm and rlm that already added.

LRE	U+202A
RLE	U+202B
LRO	U+202D
RLO	U+202E
PDF	U+202C
Reference: http://unicode.org/reports/tr9/
Comment 4 Ebrahim Byagowi 2012-09-03 10:32:06 PDT 
For example in Wikipedia, we have some pages that using rle instead of zwnj. It would be great if with inspector we can detect this misuses.
Comment 5 Alexander Pavlov (apavlov) 2012-09-03 10:40:32 PDT 
(In reply to comment #3)
> Thanks a lot! :)
> I think other directional control characters can also added like lrm and rlm that already added.
> 
> LRE    U+202A
> RLE    U+202B
> LRO    U+202D
> RLO    U+202E
> PDF    U+202C
> Reference: http://unicode.org/reports/tr9/

Since WebKit requires one patch per issue, please file these as a separate request. It won't be hard to fix :)
Comment 6 Ebrahim Byagowi 2012-09-03 10:48:13 PDT 
(In reply to comment #5)
> (In reply to comment #3)
> > Thanks a lot! :)
> > I think other directional control characters can also added like lrm and rlm that already added.
> > 
> > LRE    U+202A
> > RLE    U+202B
> > LRO    U+202D
> > RLO    U+202E
> > PDF    U+202C
> > Reference: http://unicode.org/reports/tr9/
> 
> Since WebKit requires one patch per issue, please file these as a separate request. It won't be hard to fix :)

Thanks! Bug 95696 :)