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)
Core
Internationalization: Localization
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.
Reporter | ||
Updated•16 years ago
|
Alias: intl.css-cleanup
Reporter | ||
Updated•16 years ago
|
Comment 1•16 years ago
|
||
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.
Comment 2•16 years ago
|
||
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.
Reporter | ||
Comment 3•16 years ago
|
||
(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?
Updated•8 years ago
|
Component: Tracking → Localization
Comment 4•8 years ago
|
||
No idea why this wouldn't be a tracking bug.
Component: Localization → Tracking
Comment 5•8 years ago
|
||
We are eliminating the Core:Tracking component: useful tracking bugs should now live in the relevant components.
Component: Tracking → Localization
Comment 6•8 years ago
|
||
All Locales: https://dxr.mozilla.org/l10n-central/search?q=file%3Aintl.css&redirect=false
Arabic https://dxr.mozilla.org/l10n-central/source/ar/toolkit/chrome/global/intl.css
Farsi https://dxr.mozilla.org/l10n-central/source/fa/toolkit/chrome/global/intl.css
Hebrew https://dxr.mozilla.org/l10n-central/source/he/toolkit/chrome/global/intl.css
Urdu https://dxr.mozilla.org/l10n-mozilla-aurora/source/ur/toolkit/chrome/global/intl.css
Comment 7•4 years ago
|
||
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.
Description
•