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

displayDate on the directive one doesn't work #254

Closed
miyolou opened this issue Oct 19, 2017 · 7 comments
Closed

displayDate on the directive one doesn't work #254

miyolou opened this issue Oct 19, 2017 · 7 comments
Assignees
Labels
Milestone

Comments

@miyolou
Copy link

miyolou commented Oct 19, 2017

Hi there,

I was using the directive datepicker and I couldn't get the displayDate working at all and turns out even on the demo it doesn't work when I populate the display date on the date time picker directive while the date time picker works ok.

Not working:
https://prnt.sc/gzchkx

Working:
https://prnt.sc/gzchyk

@vlio20
Copy link
Owner

vlio20 commented Oct 19, 2017

Hi. It would be great if you will post a stackblitz demo.

@miyolou
Copy link
Author

miyolou commented Oct 19, 2017

I'm sorry, been racking my head on this and realised it wasn't a config but an attribute to the component. The question now then is, is displayDate supported for the directive version?

@vlio20
Copy link
Owner

vlio20 commented Oct 19, 2017

yes, it should be. The displayDate is only for open the calendar on a specific date (default is today)

@miyolou
Copy link
Author

miyolou commented Oct 19, 2017

https://stackblitz.com/edit/angular-nf5gyb?file=app%2Fapp.component.html,
Could only think of binding [displayDate] on the input element but it just complains that it is not an attribute.

@vlio20
Copy link
Owner

vlio20 commented Oct 20, 2017

this is a bug! will address as soon as possible

@vlio20 vlio20 added this to the 2.6.0 milestone Oct 20, 2017
@miyolou
Copy link
Author

miyolou commented Oct 20, 2017

You're the best! Thank you!

@vlio20 vlio20 self-assigned this Oct 21, 2017
vlio20 added a commit that referenced this issue Oct 21, 2017
* fixing displayDate on the directive resolves - #254

* fixing ts error

* fixing e2e run
@vlio20
Copy link
Owner

vlio20 commented Oct 21, 2017

resolved in 984aab8

@vlio20 vlio20 closed this as completed Oct 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants