427026 - [RTL] Places URL field should be LTR even when using RTL UI

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Tomer Cohen :tomer]

Attachment: Places with RTL URL field

The RTL Places show the bookmark url. While we use RTL UI, we probably prefers to get that field LTR, so we won't end up with URLs that looks like "/http://", see screenshot.

Comment 1

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).

Comment 2

[(no longer active)]

Attachment: Patch (v1)

Trivial patch.

Comment 3

[Dão Gottwald [:dao]]

Comment on attachment 359980 [details] [diff] [review]
Patch (v1)

>+                   class="locationField"

use class="uri-element"

Comment 4

[(no longer active)]

Attachment: Patch (v2)

Comment 5

[(no longer active)]

Attachment: Patch (v2)

Oops, wrong patch...

Comment 6

[(no longer active)]

Comment on attachment 359985 [details] [diff] [review]
Patch (v2)

This patch was wrong...

Comment 7

[(no longer active)]

<http://hg.mozilla.org/mozilla-central/rev/0eabef9f1e36>

Comment 8

[(no longer active)]

Comment on attachment 359986 [details] [diff] [review]
Patch (v2)

This is a very simple patch which fixes an RTL problem, and is pretty low-risk.

Comment 9

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 359986 [details] [diff] [review]
Patch (v2)

a191=beltzner

Comment 10

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/4a57c94fe383