Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Added "link" property #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added "link" property #51

wants to merge 1 commit into from

Conversation

kaliyev
Copy link

@kaliyev kaliyev commented Nov 9, 2017

Added property link, to show link instead of input
Добавлено свойство ссылка, для отображения ссылки вместо инпута

Added link property, to show link instead of input
Добавлено свойство ссылка, для отображения ссылки вместо инпута
@tonydspaniard
Copy link
Member

Thanks @kaliyev But I am not sure this is the best way to do it as DatePicker is extending from InputWidget it should indeed render as an input as that is the expected result for such widget.

I think would be better to actually create a new object on this case like DatePickerLink but still, what would be the use case scenario? Simply showing a link that will popup the calendar? What is going to happen after? Do we need to add functions that will set some hidden inputs on our form? If that is the case, then why not forcing on the DatePickerLink those events to be properly configured?

Simply adding the link property is not enough. We need to elaborate further. Don't you think?

Thanks again for your time.

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

Successfully merging this pull request may close these issues.

2 participants