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:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

95548

Bug Blocks:

Attachments:

Description	Flags
Patch	pfeldman: review+

Ebrahim Byagowi

Reported 2012-08-13 13:20:07 PDT

It would be nice if inspector could make hidden characters (like &lrm; &rlm; &zwnj; &zwj;   ...) 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.

Attachments
Patch (7.10 KB, patch) 2012-09-03 04:15 PDT, Alexander Pavlov (apavlov)	pfeldman: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Alexander Pavlov (apavlov)

Comment 1 2012-09-03 04:15:24 PDT

Created attachment 161897 [details] Patch

Alexander Pavlov (apavlov)

Comment 2 2012-09-03 06:22:22 PDT

Committed r127418: <http://trac.webkit.org/changeset/127418>

Ebrahim Byagowi

Comment 3 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/

Ebrahim Byagowi

Comment 4 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.

Alexander Pavlov (apavlov)

Comment 5 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 :)

Ebrahim Byagowi

Comment 6 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 :)

Note You need to log in before you can comment on or make changes to this bug.