"Show DOM Properties" panel should appear as LTR even on RTL builds

VERIFIED FIXED in Firefox 53

Status

()

Firefox
Developer Tools: DOM
VERIFIED FIXED
a year ago
4 months ago

People

(Reporter: Itiel, Assigned: tomer)

Tracking

(Blocks: 1 bug, {rtl})

unspecified
Firefox 53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

(URL)

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Reporter)

Description

a year ago
See bug 704017 & bug 668960, the same idea is for this bug.
STR is to go to any webpage, open inspector, right click on any line, show DOM properties.
This panel's contents (except the "Filter Properties" bar) should appear as LTR always, even on RTL environment.

The same should possibly be applied to the DOM tab in the DevTools, except the "Filter DOM panel" and "Refresh" bar (?).
(Assignee)

Updated

11 months ago
Assignee: nobody → tomer.moz.bugs
Status: NEW → ASSIGNED
(Assignee)

Comment 1

11 months ago
Created attachment 8824697 [details]
Screenshot
(Assignee)

Updated

11 months ago
Comment hidden (mozreview-request)

Comment 3

10 months ago
mozreview-review
Comment on attachment 8824699 [details]
Bug 1322855 - "Show DOM Properties" panel should appear as LTR even on RTL builds

https://reviewboard.mozilla.org/r/103060/#review104940

Looks good and works for me.

R+

Thanks for the patch!

Honza
Attachment #8824699 - Flags: review?(odvarko) → review+
(Assignee)

Updated

10 months ago
Keywords: checkin-needed

Comment 4

10 months ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/b351c04d0dc8
"Show DOM Properties" panel should appear as LTR even on RTL builds r=Honza
Keywords: checkin-needed

Comment 5

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b351c04d0dc8
Status: ASSIGNED → RESOLVED
Last Resolved: 10 months ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
(Reporter)

Updated

4 months ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.