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

closeOnSelect doesn't work on version 0.9.0 #112

Closed
chompunut opened this issue Nov 5, 2015 · 14 comments
Closed

closeOnSelect doesn't work on version 0.9.0 #112

chompunut opened this issue Nov 5, 2015 · 14 comments

Comments

@chompunut
Copy link

I installed version 0.9.0 the closeOnSelect doesn't work. I set it to true but when I select a date, the popup doesn't close and no error message.

$scope.datepickerObject = {
    closeOnSelect: true, 
    templateType: 'popup',
    to: new Date(),
    callback: function (date) { 
          console.log('callback');
    }
};
@renekorss
Copy link
Contributor

closeOnSelect was added by me onyl to days ago. So it's not in 0.9.0 release. Update you version to use master branch version and it will work.

You can install master branch version with bower install ionic-datepicker#master.

This issue should be closed, since there is no real issue here.

@chompunut
Copy link
Author

@renekorss I've installed the master branch but the issue still happen. Please see #111

@rajeshwarpatlolla
Copy link
Owner

In v0.9.0, this feature is not there.
This feature will make the user feel strange. Let's say when a user clicks on a date but don't want to select that date, even in that case the popup will close, again the user have to open it again. So i don't think, this feature will be used by so many. I suggest this feature to be implemented those who wish, but not as part of this component. Suggestions are welcome.

@chompunut
Copy link
Author

After I sent app to my users, they prefer the close on select function because it does like date picker in many web sites are using and it doesn't need many clicks which is suitable for mobile device. For example, the app use for planing many tasks in a day.

@rajeshwarpatlolla
Copy link
Owner

So, it's a kind of on demand feature. I was discussing with few people about this feature, most of them are not preferring this way, as it makes most of the users confuse. The user may wish to select day first and then month and then year, in this case the user should be abel to do all in one go. If we have this feature, then the user has to open the datepicker for 3 times.

@chompunut
Copy link
Author

But the version 0.9.0 doesn't support the way to select date first and then month and then year. When I select date first and then month, the picker doesn't highlight date on the selected month.

For example, I select the date 3nd on November then change month to April and click on Set button. The output date still be November 3 not April 3. This is why my user confuse about.

So, I think the close on select should be an option for developer to customize its behavior.

@renekorss
Copy link
Contributor

I can confirm what @chompunut said. In ver 0.9.0. If i check date and then change month, then the date wont be that months. Chaning month dosen't change selected date.

@harishanandan
Copy link

Hi can Any one plz help me with this datepicker I have installed everything but I could not get modal .can any one plz suggest

@djoks
Copy link

djoks commented Dec 25, 2015

First of all, great plugin, really handy :) however I am in agreement that closeOnSelect should be an option for the developer (doesn't work for me currently), most of my users prefer that option as well, considering how in my case a user can only select a date within a period of 7 days after tomorrow. In which case there should also be options to hide all the buttons. Also can you please tell me if there is a way to generally change the theme color of the plugin from one central setting. Thank you in advance. :)

@pharindoko
Copy link

I can confirm....
closeOnSelect
doesn't work in the current released bower version as it is not implemented and
does not work with #master version because of errors - it is implemented.

Hope that this will be fixed - I like this datepicker - and yeah I would need this option too.

@mwagena
Copy link

mwagena commented Feb 16, 2016

Hi,
This issue also occurs in the master branch, could you give an estimate on when the fix will be made?
Thanks in advance!

@rajeshwarpatlolla
Copy link
Owner

I am trying to make few changes and once they are done, i will release a new version with this feature.

@ghost
Copy link

ghost commented Feb 29, 2016

+1

@rajeshwarpatlolla
Copy link
Owner

Please check the new release v1.0.0. This issue has been addressed.

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

No branches or pull requests

7 participants