Bug 93888 - Web Inspector: Unhide hidden characters
Summary: Web Inspector: Unhide hidden characters
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Alexander Pavlov (apavlov)
URL:
Keywords:
Depends on: 95548
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-13 13:20 PDT by Ebrahim Byagowi
Modified: 2012-09-03 10:48 PDT (History)
10 users (show)

See Also:


Attachments
Patch (7.10 KB, patch)
2012-09-03 04:15 PDT, Alexander Pavlov (apavlov)
pfeldman: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ebrahim Byagowi 2012-08-13 13:20:07 PDT
It would be nice if inspector could make hidden characters (like ‎ ‏ ‌ ‍   ...) 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 :)