289934 - set style "direction: ltr" to file and url paths in textbox

Status: NEW

(Firefox :: General, defect, P3)

Description

[Reuven Gonen]

currently, the file and url paths have "direction: rtl;" inherited from the
interface. we should make the direction LTR and align the text to the right.

my suggestion is to add the following to the intl.css:

#link-url-text, #source, #path, #url
{
	direction: ltr !important;
	text-align: right !important;
}

if there are other places please let me know.

Comment 1

[Tsahi Asher]

see Bug 219070 . in particular, bug 157607 added something like that to xul.css.
what is left is adding the uri-element class to all the relevant places.

i would suggest duping that to 219070.

Comment 2

[Reuven Gonen]

(In reply to comment #1)
> see Bug 219070 . in particular, bug 157607 added something like that to xul.css.
> what is left is adding the uri-element class to all the relevant places.
> 
> i would suggest duping that to 219070.

i see that bug 157607 is marked as fixed, while at least in Firefox the urlbar
shows /http://www.mozilla.org. second, why is Bug 219070 component set as MailNews?

Comment 3

[Tsahi Asher]

(In reply to comment #2)
> i see that bug 157607 is marked as fixed, while at least in Firefox the urlbar
> shows /http://www.mozilla.org. second, why is Bug 219070 component set as
MailNews?

bug 157607 is fixed in seamonkey. following our phone talk, it's possible that
the class in seamonkey's xul.css wasn't transferred to firefox (to toolkit?). if
that is the case, then a similar approach should be implemented for firefox and
thunderbird as well, IMHO.

bug 219070 is at the MailNews component because most of it's dependencies are
there. perhaps L10n is better.

Comment 4

[Asaf Romano (gone)]

The current intl.css is a disaster, we have a style rule for those cases in
xul.css (uri-element IIRC).

Comment 5

[(no longer active)]

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

Comment 6

[(no longer active)]

mano, I'm going to take this over if you don't mind.  :-)

Comment 7

[(no longer active)]

Note to self:

<http://mxr.mozilla.org/mozilla-central/source/browser/base/content/metaData.xul#79>
<http://mxr.mozilla.org/mozilla-central/source/embedding/components/ui/progressDlg/nsProgressDialog.xul#115>
<http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/downloads/content/unknownContentType.xul#81>
<http://mxr.mozilla.org/mozilla-central/source/browser/components/preferences/cookies.xul#111>
<http://mxr.mozilla.org/mozilla-central/source/extensions/reporter/resources/content/reporter/reportWizard.xul#85>
<http://mxr.mozilla.org/mozilla-central/source/browser/components/preferences/permissions.xul#71>
<http://mxr.mozilla.org/mozilla-central/source/browser/components/preferences/main.xul#163>
<http://mxr.mozilla.org/mozilla-central/source/browser/base/content/pageinfo/pageInfo.xul#150>
<http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#574>

Comment 8

[Phil Ringnalda (:philor)]

http://mxr.mozilla.org/mozilla-central/source/browser/components/preferences/connection.xul