gloda: chromedir! philor: You keep using that word. I do not think it means what you think it means.

RESOLVED FIXED in Thunderbird 3.1a1

Status

Thunderbird
Search
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: philor, Assigned: philor)

Tracking

Trunk
Thunderbird 3.1a1
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 415343 [details] [diff] [review]
rm

You only include global.dtd when you are going to use &locale.dir;, that's all it gives you.

You only use &locale.dir; for two things: to set the dir attribute on the body element in XHTML, and to set the chromedir attribute on an XUL element when you then plan on using that in CSS, in 1.9.1 or earlier where :-moz-locale-dir didn't exist, to do things differently for ltr and rtl.

Since nothing is trying to style searchInput or PopupGlodaAutocomplete based on their chromedir, I'm going to guess that they just sort of wandered into the glodasearch branch of their own accord, without anyone ever really feeling any actual need for them.
Attachment #415343 - Flags: review?(bugmail)
Comment on attachment 415343 [details] [diff] [review]
rm

This was likely the nefarious work of copy and paste.
Attachment #415343 - Flags: review?(bugmail) → review+
Also, thank you very much for explaining why we don't want it.
http://hg.mozilla.org/comm-central/rev/ae769efac6ed
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Blocks: 511682
You need to log in before you can comment on or make changes to this bug.