RESOLVED FIXED 207214
Web Inspector RTL: Elements closing tag is reversed
https://bugs.webkit.org/show_bug.cgi?id=207214
Summary Web Inspector RTL: Elements closing tag is reversed
Lukas Möller
Reported 2020-02-04 11:14:31 PST
Created attachment 389681 [details] Screenshot of Safari 13 dev tools. If a HTML DOM node contains a Unicode right-to-left override character (U+202E) the tag name of the closing tag is reversed as well. This could possibly be intended behavior, but I would prefer opening and closing tag to match. Only the content of the tag should change when including rtl or ltr overrides / embeddings. This might be a duplicate of #200601.
Attachments
Screenshot of Safari 13 dev tools. (365.81 KB, image/png)
2020-02-04 11:14 PST, Lukas Möller
no flags
Reduction (47 bytes, text/html)
2020-02-04 16:22 PST, Nikita Vasilyev
no flags
Patch (3.54 KB, patch)
2020-02-04 16:34 PST, Nikita Vasilyev
no flags
Patch (1.48 KB, patch)
2020-02-04 17:31 PST, Nikita Vasilyev
no flags
Nikita Vasilyev
Comment 1 2020-02-04 15:30:07 PST
Thank you for reporting, Lukas! This isn't a duplicate of bug 200601.
Nikita Vasilyev
Comment 2 2020-02-04 16:22:44 PST
Created attachment 389735 [details] Reduction
Nikita Vasilyev
Comment 3 2020-02-04 16:34:20 PST
Nikita Vasilyev
Comment 4 2020-02-04 16:50:44 PST
Ultimately, we should display all Unicode BiDi characters as HTML entities, e.g. ‮.
Nikita Vasilyev
Comment 5 2020-02-04 17:31:50 PST
Created attachment 389752 [details] Patch A simpler CSS-only solution.
WebKit Commit Bot
Comment 6 2020-02-11 14:38:28 PST
Comment on attachment 389752 [details] Patch Clearing flags on attachment: 389752 Committed r256374: <https://trac.webkit.org/changeset/256374>
WebKit Commit Bot
Comment 7 2020-02-11 14:38:29 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2020-02-11 14:39:15 PST
Ebrahim Byagowi
Comment 9 2021-01-21 15:07:21 PST
> Ultimately, we should display all Unicode BiDi characters as HTML entities, e.g. &#8238;. Fun fact, the reason Chrome has the feature is the request I made on Bug 93888 and unfortunately the change removed while Web Inspector UI revamp landed after Blink's fork, so I filed Bug 220831 and will try to fix it myself this time :)
Note You need to log in before you can comment on or make changes to this bug.