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)
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)
408 bytes,
text/html
|
Details | |
11.42 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
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
Updated•8 years ago
|
Blocks: datetime-bugs
Priority: -- → P3
Assignee | ||
Comment 1•8 years ago
|
||
Works on <input type="time">, let me see what's going on.
Assignee: nobody → jjong
Assignee | ||
Comment 2•8 years ago
|
||
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
Assignee | ||
Comment 3•8 years ago
|
||
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)
Updated•8 years ago
|
Attachment #8876647 -
Flags: review?(bugs) → review+
Assignee | ||
Comment 4•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e7b28fe42f9109b912eb4c53c55f53fef3d7be82&group_state=expanded
There some failures but seems not related to this patch.
Keywords: checkin-needed
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
![]() |
||
Comment 6•8 years ago
|
||
bugherder |
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
status-firefox56:
--- → fixed
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.
Description
•