Closed Bug 486822 (intl.css-cleanup) Opened 16 years ago Closed 4 years ago

Tracking bug for getting rid of intl.css hacks for RTL locales

Categories

(Core :: Internationalization: Localization, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: ehsan.akhgari, Unassigned)

References

Details

(Keywords: rtl)

As suggested by Tomer, I'm filing this bug to act as a tracker for bugs which lead to changes (hopefully simplifications) to intl.css hacks for RTL locales.
Alias: intl.css-cleanup
Ehsan, I have seen you have removed the resizer hack (http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/fa/diff/ecf05a12cca5/toolkit/chrome/global/intl.css, and bug 436190 comment 20). Is it safe to remove it from the intl.css as well?

In the future, please report RTL intl.css hacks here, so we will be able to follow. Thanks.
No longer depends on: 240536, 417489, 437797, 477696, 479588, 479598
Ehsan, I have seen you have removed the resizer hack (http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/fa/diff/ecf05a12cca5/toolkit/chrome/global/intl.css, and bug 436190 comment 20). Is it safe to remove it from the intl.css as well?

In the future, please report RTL intl.css hacks here, so we will be able to follow. Thanks.
(In reply to comment #2)
> Ehsan, I have seen you have removed the resizer hack
> (http://hg.mozilla.org/releases/l10n-mozilla-1.9.1/fa/diff/ecf05a12cca5/toolkit/chrome/global/intl.css,
> and bug 436190 comment 20). Is it safe to remove it from the intl.css as well?

Yes, it is.

> In the future, please report RTL intl.css hacks here, so we will be able to
> follow. Thanks.

Oh, sure.  I was under the impression that you'll check out the dependencies and CC yourself on them but I will try to post a notice about the changes here as well.

Have you checked the rest of the dependencies here?
Depends on: 493440
Depends on: 601092
Component: Tracking → Localization
No idea why this wouldn't be a tracking bug.
Component: Localization → Tracking
We are eliminating the Core:Tracking component: useful tracking bugs should now live in the relevant components.
Component: Tracking → Localization
Depends on: 1332782
Depends on: 1332832
Depends on: 1332833

Looks like we have been landing fixes in bugs over time, not sure how scalable is to have them depend on a tracking bug.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.