1344642 - Redefine date/time input related preferences

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P2)

Description

[Jessica Jong [:jessica] (inactive)]

We now have the following prefs:
- "dom.forms.datetime": for all five types date time input types, that is, date, time, week, month and datetime-local on desktop
- "dom.forms.datepicker": for picker for input type=date on desktop, but using OS native date picker
- "dom.experimental_forms": used to be for new experimental html forms, but now we only use it for input type=date and input=time on mobile (Fennec and B2G). Note that date and time input types show as a single line text box and uses native OS picker.

I think "dom.forms.datepicker" and related code can be removed, since after our DateTime Input project, we're using our custom date/time picker across all platforms.

We may need a pref for only input type=date and type=time on desktop though, as we plan to ship only type=date and type=time first in our DateTime Input project.

In conclusion, we need:
- a pref for input date and time on mobile: We can use the existing "dom.experimental_forms" or maybe rename it to "dom.forms.mobile.experimental"
- a pref for input date and time on desktop: maybe "dom.forms.datetime"?
- a pref for input week, month and datetime-local on desktop: "dom.forms.datetime.others"?

Comment 1

[Jessica Jong [:jessica] (inactive)]

Attachment: Part 1: cleanup dom.forms.datepicker related code.

This basically reverts Bug 825294.

Comment 2

[Jessica Jong [:jessica] (inactive)]

Attachment: Part 2: add a new pref for input type=week, month a datetime-local.

Now, `dom.forms.datetime` is for input type=date and time only, and `dom.forms.datetime.others` is for input type=week, month and datetime-local.

This is because we plan to ship input type=date and time first, so we need a separate pref.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8847885 [details] [diff] [review]
Part 2: add a new pref for input type=week, month a datetime-local.

>         newType = aResult.GetEnumValue();
>-        if ((IsExperimentalMobileType(newType) &&
>-             !IsExperimentalFormsEnabled()) ||
>-            (newType == NS_FORM_INPUT_NUMBER && !IsInputNumberEnabled()) ||
>+        if ((newType == NS_FORM_INPUT_NUMBER && !IsInputNumberEnabled()) ||
>             (newType == NS_FORM_INPUT_COLOR && !IsInputColorEnabled()) ||
>             (IsDateTimeInputType(newType) &&
>              !IsDateTimeTypeSupported(newType))) {

ok, you can remove the IsExperimental* since IsDateTime* covers that already.

Comment 4

[Jessica Jong [:jessica] (inactive)]

(In reply to Olli Pettay [:smaug] from comment #3)
> Comment on attachment 8847885 [details] [diff] [review]
> Part 2: add a new pref for input type=week, month a datetime-local.
> 
> >         newType = aResult.GetEnumValue();
> >-        if ((IsExperimentalMobileType(newType) &&
> >-             !IsExperimentalFormsEnabled()) ||
> >-            (newType == NS_FORM_INPUT_NUMBER && !IsInputNumberEnabled()) ||
> >+        if ((newType == NS_FORM_INPUT_NUMBER && !IsInputNumberEnabled()) ||
> >             (newType == NS_FORM_INPUT_COLOR && !IsInputColorEnabled()) ||
> >             (IsDateTimeInputType(newType) &&
> >              !IsDateTimeTypeSupported(newType))) {
> 
> ok, you can remove the IsExperimental* since IsDateTime* covers that already.

Exactly, thanks!

Comment 5

[Jessica Jong [:jessica] (inactive)]

Attachment: Part 1: cleanup dom.forms.datepicker related code.

rebased on top of bug 1347640, carrying r+.

Comment 6

[Jessica Jong [:jessica] (inactive)]

Attachment: Part 2: add a new pref for input type=week, month a datetime-local.

rebased on top of bug 1347640, carrying r+.

Comment 7

[Jessica Jong [:jessica] (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d459b0581ac90826c6dbc8ae6b52f0a7d4066bbd&group_state=expanded

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ece6f82b4de3
Part 1: Cleanup dom.forms.datepicker related code. r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/8603d4d348a4
Part 2: Add a new pref for input type=week, month and datetime-local. r=smaug

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/ece6f82b4de3
https://hg.mozilla.org/mozilla-central/rev/8603d4d348a4