search

praveenpuglia / vuetify-daterange-picker

The missing date range picker for Vuetify JS you have been looking for.
https://vuetify-daterange-picker.gitbook.io/docs
MIT License
197 stars 59 forks source link

Input triggers twice #29

Closed outoppie closed 6 years ago

outoppie commented 6 years ago

It appears that every date selection triggers the input event twice. You can see this in the sandbox by adding console.log

praveenpuglia commented 6 years ago

Hi! This is intended. Each time a date is selected, the combination of start and end date is sent out. What's expected?

outoppie commented 6 years ago

Hi Praveen

Many thanks for responding and thanks for creating the date range picker.

Yes, I receive start and end date in the range field. Not sure if it is intended, but my query is that each time I select either start or end date, is it correct that the input event fires twice ?

Regards

Mike Moncrieff

Computer Leverage

From: Praveen Puglia [mailto:notifications@github.com] Sent: 17 September 2018 08:59 AM To: praveenpuglia/vuetify-daterange-picker Cc: outoppie; Author Subject: Re: [praveenpuglia/vuetify-daterange-picker] Input triggers twice (#29)

Hi! This is intended. Each time a date is selected, the combination of start and end date is sent out. What's expected?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/praveenpuglia/vuetify-daterange-picker/issues/29#issuecomment-421908504 , or mute the thread https://github.com/notifications/unsubscribe-auth/AMEblVq8SS0RK2ogjYnlR75CXSaOgf-Lks5ub0hBgaJpZM4Wq-4A .Image removed by sender.

praveenpuglia commented 6 years ago

Yes! that's how I designed it. I should probably also add an option to give Apply and Cancel buttons which will ensure the event gets fired only when Apply is clicked.

outoppie commented 6 years ago

Thanks once again for the quick response. I have adjusted my code accordingly.