Closed Bug 532040 Opened 12 years ago Closed 12 years ago

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

Categories

(Thunderbird :: Search, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.1a1

People

(Reporter: philor, Assigned: philor)

References

Details

Attachments

(1 file)

Attached patch rmSplinter Review
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
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.