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

q-time doesn't behave well when combined with date model #6279

Closed
pikil opened this issue Feb 1, 2020 · 5 comments
Closed

q-time doesn't behave well when combined with date model #6279

pikil opened this issue Feb 1, 2020 · 5 comments

Comments

@pikil
Copy link

pikil commented Feb 1, 2020

When dragging minutes arrow in <q-time /> which has mask="YYYY-MM-DD HH:mm" the element emits undefined value. When just selecting a minute, that works perfectly fine.

You can view that in the documentation (open codepen) by selecting the date and then by starting dragging arrows in <q-time /> element.
https://quasar.dev/vue-components/time#Example--QDate-and-QTime-with-QInput

Many thanks!

@metalsadman
Copy link
Member

@hawkeye64
Copy link
Member

@metalsadman That never popped up the QDate for me:
image

@metalsadman
Copy link
Member

metalsadman commented Feb 4, 2020

what browser and os are you using? i could swear it was working in the office, but yeah not working in my home win 10 pc on chrome. just tried now on edge and i got undefined :s.

edit. after further investigation, it's based on the dimensions of the qpopupproxy serving the qtime that the values not emitting properly since the component seem to re-render each time you drag the clock hands.
image

reproducible on the op's link at the docs when adjusting the browser's width or height (proxy going into dialog mode), then clicking the q-input's time icon to open the q-time popup.
image

pdanpdan added a commit to pdanpdan/quasar that referenced this issue Feb 4, 2020
rstoenescu added a commit that referenced this issue Feb 4, 2020
feat(QTime): Add defaultDate prop (defaults to current day) to fill the date part if not present #6279
@rstoenescu
Copy link
Member

Enhancement will be available in "quasar" v1.9.0.

@pikil
Copy link
Author

pikil commented Feb 6, 2020

Great, thank you guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants