Closed
Bug 427026
Opened 17 years ago
Closed 16 years ago
[RTL] Places URL field should be LTR even when using RTL UI
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 3.6a1
People
(Reporter: tomer, Assigned: ehsan.akhgari)
References
(Blocks 2 open bugs)
Details
(Keywords: fixed1.9.1, polish, rtl)
Attachments
(2 files, 2 obsolete files)
|
(deleted),
image/png
|
Details | |
|
(deleted),
patch
|
dao
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
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.
|
Assignee | |
Updated•17 years ago
|
Blocks: Persian-Fx3.5, fx35-l10n-fa
|
|
Assignee | |
Updated•17 years ago
|
No longer blocks: Persian-Fx3.5
|
Reporter | |
Updated•17 years ago
|
Component: he-IL / Hebrew → Theme
Product: Mozilla Localizations → Firefox
QA Contact: hebrew.he → theme
|
|
Assignee | |
Comment 1•17 years ago
|
||
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
|
||
Updated•16 years ago
|
Blocks: Persian-Fx3.5
|
|
Assignee | |
Updated•16 years ago
|
|
|
Assignee | |
Comment 2•16 years ago
|
||
Trivial patch.
|
||
Comment 3•16 years ago
|
||
Comment on attachment 359980 [details] [diff] [review]
Patch (v1)
>+ class="locationField"
use class="uri-element"
Attachment #359980 -
Flags: review?(dao) → review-
|
|
Assignee | |
Comment 4•16 years ago
|
||
Attachment #359980 -
Attachment is obsolete: true
Attachment #359985 -
Flags: review?(dao)
|
|
||
Updated•16 years ago
|
Attachment #359985 -
Flags: review?(dao) → review+
|
|
Assignee | |
Comment 5•16 years ago
|
||
Oops, wrong patch...
Attachment #359985 -
Attachment is obsolete: true
Attachment #359986 -
Flags: review?(dao)
|
|
Assignee | |
Comment 6•16 years ago
|
||
Comment on attachment 359985 [details] [diff] [review]
Patch (v2)
This patch was wrong...
Attachment #359985 -
Flags: review+ → review-
|
|
||
Updated•16 years ago
|
Attachment #359986 -
Flags: review?(dao) → review+
|
|
Assignee | |
Comment 7•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-litmus?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.2a1
|
|
Assignee | |
Updated•16 years ago
|
Attachment #359986 -
Flags: approval1.9.1?
|
|
Assignee | |
Comment 8•16 years ago
|
||
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.
|
||
Updated•16 years ago
|
Attachment #359986 -
Flags: approval1.9.1? → approval1.9.1+
|
|
||
Comment 9•16 years ago
|
||
Comment on attachment 359986 [details] [diff] [review]
Patch (v2)
a191=beltzner
|
|
Assignee | |
Comment 10•16 years ago
|
||
Keywords: fixed1.9.1
|
|
Assignee | |
Updated•16 years ago
|
No longer blocks: fx35-l10n-fa
|
|
Assignee | |
Updated•12 years ago
|
Flags: in-litmus?
You need to log in
before you can comment on or make changes to this bug.
Description
•