Closed Bug 1370858 Opened 8 years ago Closed 8 years ago

<input type=date> datepicker doesn't fire change event

Categories

(Core :: DOM: Core & HTML, defect, P3)

x86
macOS
defect

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: jeroen.pulles, Assigned: jessica)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(2 files)

Attached file datepicker.html
The new datepicker widget for <input type=date> doesn't fire change events. I expected to get change events when I click on a date in the picker panel, and I expect to get change events when typing a date in the text field and then blurring the input. The value property of the input does show the edited value when polled (see attached example). I'm running Firefox Nightly: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:55.0) Gecko/20100101 Firefox/55.0
Priority: -- → P3
Works on <input type="time">, let me see what's going on.
Assignee: nobody → jjong
NS_FORM_INPUT_DATE missing in HTMLInputElement::MayFireChangeOnBlur() [1], fixing it now. [1] http://searchfox.org/mozilla-central/rev/d840ebd5858a61dbc1622487c1fab74ecf235e03/dom/html/HTMLInputElement.h#1612-1617
Attached patch patch, v1.Splinter Review
Hi Olli, details are in the patch comment. I've added tests only for changing value from the keyboard, since we do not have a way for testing picker yet.
Attachment #8876647 - Flags: review?(bugs)
Attachment #8876647 - Flags: review?(bugs) → review+
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/20882c387b4e Fire change/input events after selecting a date/time from picker. r=smaug
Keywords: checkin-needed
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
on removing the value in an input of type date by clicking the (x) will not yet fire the change-event v59.0.2 & v60.0b11 Developer
I´ll disable my account, because i don´t want to publish my mail for other users/bots, very disappointing. hope the comment remains.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: