Closed Bug 1340384 Opened 8 years ago Closed 8 years ago

[DateTimePicker] Use mozILocaleService for getting UI locale

Categories

(Core :: Layout: Form Controls, defect)

Product:
Core
Component:
Layout: Form Controls
Version:
54 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID
Tracking Flags:
Tracking Status
firefox54 --- affected

People

(Reporter: scottwu, Unassigned)

References

(Blocks 1 open bug)

Details

Replace nsIXULChromeRegistry with mozILocaleService (Bug 1336281) for getting UI locale.
bug 1334234 may also solve it for you.
(In reply to Zibi Braniecki [:gandalf][:zibi] from comment #1) > bug 1334234 may also solve it for you. Does it mean I can omit the locale argument when using mozIntl APIs? I think perhaps passing the locale explicitly is better here because I also use Intl.DateTimeFormat and Intl.NumberFormat in the picker, and they default to follow OS locale if omitted.
> Does it mean I can omit the locale argument when using mozIntl APIs? I think perhaps passing the locale explicitly is better here because I also use Intl.DateTimeFormat and Intl.NumberFormat in the picker, and they default to follow OS locale if omitted. Sure, you can! I'd like to get to the point where in chrome context Intl formatters use app locale by default as well, but for now using LocaleService sounds good.
This bug has been fixed in Bug 1347314. (Thanks Zibi!)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.