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

dateRange: true on change is triggered twice #2927

Open
4 tasks done
itajackass opened this issue Jul 18, 2024 · 1 comment
Open
4 tasks done

dateRange: true on change is triggered twice #2927

itajackass opened this issue Jul 18, 2024 · 1 comment
Labels
State: Triage Tickets that need to be triaged. Type: Bug

Comments

@itajackass
Copy link

itajackass commented Jul 18, 2024

Prerequisites

  • I have searched for duplicate or closed issues.
  • I have validated my setup against the latest version on StackBlitz.
  • I have read the contributing guidelines.
  • I understand that only sponsored issues will be worked on and that if I wish to sponsor an issue, I will contact the owner FIRST.

Describe the issue

Using dateRange, the change.td events trigger different times.

I expeted that using dateRange: true is set, change.td event is fired only on both start and end date are selected... but actually change.td is fired on every press of a date (fired first time when start is selected, and fired another one when end is selected). Also changing dates manually (without calendar) it fires 2 times...
Using hide.td events reduces the "bug", but manual change of dates (instead by calendar) is not triggered...

You can see my example: first picker is bad..... in second picker i've implemented all checks to prevent false change.td events.

https://stackblitz.com/edit/tempus-dominus-v6-simple-setup-kc1hxu?file=index.js,index.html

StackBlitz fork

https://stackblitz.com/edit/tempus-dominus-v6-simple-setup-kc1hxu?file=index.js,index.html

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Firefox

What version of are you using? You can find this information from the sample StackBlitz.

latest

What your browser's locale? You can find this information from the sample StackBlitz.

it

@itajackass itajackass added State: Triage Tickets that need to be triaged. Type: Bug labels Jul 18, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Tempus Dominus Jul 18, 2024
@Eonasdan
Copy link
Owner

As I'm sure you noticed, since you checked the box indicating that you did on all the issues you've opened, I'm not working on the project anymore for free.

The source is available. If you want to contribute a PR to help resolve bugs you are facing, I'm happy to review them. Otherwise, there are various ways that you can get paid support. Thanks for your interest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
State: Triage Tickets that need to be triaged. Type: Bug
Projects
Status: 🆕 New
Development

No branches or pull requests

2 participants