这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Scribilicious
Copy link

Added a close button functionality... However had to convert your code for my project to UTF-8... Thats the reason for the changes in your comment lines :-)

Added the close button trigger feature...
Added the close button feature.
Added the close button styles...
Add an extra setting "default", in this setting a default date can be parsed into the datepicker using the setting default : new Date(...);
added a extra save button + outerClickSave functionality. If set true the date will be saved when clicked outside the date picker.
See update picker.date.js
Added styles for the save button
@amsul amsul added the todo label Jul 3, 2014
@amsul amsul added this to the 3.5.3 milestone Jul 3, 2014
@amsul
Copy link
Owner

amsul commented Jul 3, 2014

Nice, I appreciate the PR! But seems like there’s a lot of changes in there that aren’t relevant to adding a close button (such as settings.default?). There’s been some discussion started here as well: #345 with some development done.

If you can reduce this PR to the relevant changes so it’s easier to digest, I can take a look at it again.

Cheers! :)

@amsul amsul closed this Jul 3, 2014
@amsul amsul removed the todo label Jul 12, 2014
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

Successfully merging this pull request may close these issues.

2 participants