402637 - Make uri-element class work on any element

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[Phil Ringnalda (:philor)]

Attachment: Fix v.1

The "uri-element" class that bug 157607 added to xul.css currently only works for textboxes and menulists, which are most, but not all, of the things where we display URIs that ought to remain ltr in an rtl locale - bug 400882 adds an hbox around textbox+label that together make up a URI, bug 213727 wants to add a listbox of URIs, and there are probably more I haven't spotted.

Currently doesn't affect Firefox, which doesn't use uri-element, but might make it possible to clean up its rather confusing current state where it's per-theme, and who knows what you might get with a third-party theme.

Comment 1

[Phil Ringnalda (:philor)]

Attachment: Fix v.2

Maybe this alternate approach will work better :)

Note that I don't give a rat's ass whether I get r+ or WONTFIX, but I've been blocked on making the account manager not look stupid in RTL for a few days shy of four months now waiting for a simple decision on this.

Comment 2

[Phil Ringnalda (:philor)]

Attachment: Really fix v.2

Apparently, if you notice you left a stray comma and take it out, then you need to diff *after* you do.

Comment 3

[Asaf Romano (gone)]

Comment on attachment 307156 [details] [diff] [review]
Really fix v.2

PLEASE GET YOUR CHANGES REVIEWED BY [e-mail] AS WELL.

Comment 4

[Phil Ringnalda (:philor)]

Comment on attachment 307156 [details] [diff] [review]
Really fix v.2

SINCE [e-mail] IGNORED MY REVIEW REQUEST FOR FOUR MONTHS, I DON'T SEE HOW THAT'S ANY DIFFERENT FROM WONTFIX.

Comment 5

[Mike Connor [:mconnor]]

Comment on attachment 307156 [details] [diff] [review]
Really fix v.2

Huh, so this is the bug you were talking about.  If I wasn't clearly lazy and mean I'd have emailed you instead of waiting for you to come back on IRC before I forgot again. :)

Was going to change this to Neil Deakin going forward, been meaning to do that for like three months, so instead of removing that, please just change to [e-mail] rather than removing it and r=me.  The point of the requirement is that bad rules here can break XUL and cause performance issues, etc, so someone who knows XUL layout internals is best to own this...

Comment 6

[Phil Ringnalda (:philor)]

Attachment: For checkin

With requested change; thanks.

Drivers: zero direct risk to Firefox, which doesn't use this class. The only potential risk is to extensions which are using the same class on things other than textboxes or menulists, but since they would have to use the same classname with nothing else in the selector to not be more specific, to have this rule override their rule they would have to be applying their stylesheet *before* xul.css, which as far as I know isn't possible. Reward is limiting the proliferation of classes for directionality of URI bits and bobs, which will make life easier in the future when we have to deal with the issue of multiple text directions in a URI.

Comment 7

[Damon Sicore (:damons)]

Comment on attachment 308104 [details] [diff] [review]
For checkin

a1.9+=damons

Comment 8

[Phil Ringnalda (:philor)]

toolkit/content/xul.css 1.118