Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Double event trigger (second one is invalid) #170

Closed
devoto13 opened this issue Feb 6, 2014 · 1 comment
Closed

Double event trigger (second one is invalid) #170

devoto13 opened this issue Feb 6, 2014 · 1 comment
Milestone

Comments

@devoto13
Copy link

devoto13 commented Feb 6, 2014

Please, see this jsfiddle to complete example of what is going on: http://jsfiddle.net/devoto13/fEgK9/1/

I subscribed for change.dp event, but when date is changed widget triggers event two times. Additionally in second event there is not date field.

According to docs this is intended behavior:

Fires when the datepicker changes or updates the date. Note that change may also fire for knockout support.

I think it will be good to have option to disable double-triggerring change event. Or any other solution, better than checking for e.date field in event handler.

This was referenced Feb 27, 2014
@Eonasdan Eonasdan modified the milestone: 3.0.0 Mar 3, 2014
Eonasdan added a commit that referenced this issue Mar 3, 2014
## 3.0.0
## **Breaking changes in this update! Pay attention!!**
* Fix for #170, #179, #183: Changed event to `dp.-` options load as
`data-date-OPTION` instead of `data-OPTION`. This should fix the double
change event firing.
* Fix for #192: `setDate` now fires `dp.change`
* Fix for #182: Picker will **not** set the default date if the input
field has a value
* Fix for #169: Seconds doesn't get reset when changing the date (Thanks
to PR #174)
* Fix for #168 z-index fix for BS modal
* Fix for #155 Picker properly displays the active year and month
* Fix for #154 CSS update to fix the collapse jump
* Fix for #150 and #75 `minViewMode` and `viewMode` work properly
* Fix for #44 Finally! It's here!! Thanks to @ruiwei and his code on
#210 picker will adjust the positioning of the widget.

#### **Manually merged PR**
* #178 When using `minuteStepping` the minute select grid will only show
available steppings
* #195, #197 Using the `data-OPTION` has been changed to
`data-date-OPTION`. These options are expected to be on the
`input-group` if you're using the `form-group` **or** the input field
* #184 The option `sideBySide` change be used to display both the d and
the timepicker side by side

#### **Other Changes**
* Changed picker width to 300px if using seconds and am/pm
* Added option `useCurrent`, thanks to @ruiwei. When true, picker will
set the value to the current date/time (respects picker's format)
* Added option `showToday`, thanks to @ruiwei. When true, picker will
display a small arrow to indicate today's date.
* Changed `startDate` to `minDate` and `endDate` to `maxDate` to make it
more clear what these options do.
@Eonasdan
Copy link
Owner

Eonasdan commented Mar 3, 2014

Please check the Staging branch to verify your issue(s) have been resolved.

@ghost ghost locked and limited conversation to collaborators Jun 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants