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

freeRange selection #46

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

freeRange selection #46

wants to merge 7 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 25, 2013

I'm not really a javascripter, so my solution probably isn't that elegant (and made use of a bit of copy/paste from the code above). Implements the wished feature in this issue: #44

Mo Abrahams added 2 commits January 25, 2013 12:41
@didijo
Copy link

didijo commented Feb 21, 2013

It seems that this doesnt work in IE (tested version 8 and 9 and different jquery versions). After selecting the first date i always get this error:
Conversion from "string" format not allowed on jQuery.multiDatesPicker (jquery.min.js - row 2 - char 13143)

@ghost
Copy link
Author

ghost commented Feb 21, 2013

Ah. I haven't got IE on any of my computers. I'll eventually need to test this project in IE, so when I get to that point I'll have a look if I can see what is wrong. If anyone else knows what is wrong with it, then great ;) :).

@didijo
Copy link

didijo commented Feb 21, 2013

the error appears when i preselect dates or disable dates ... the free-range selection doesnt work at all. Besides it looks like the IE doesent support any of the types for pre-selecting / disabling dates.

@didijo
Copy link

didijo commented Feb 21, 2013

here is the solution for the general IE problem: #12 (robfuscatory)

But the free range selection still doesnt work

UPDATE:
after repalcing this code-snippet the free range selektion worked in IE, too
(Row 413 in "freeRange" case)
//var end = new Date(date);
var end = $.datepicker.parseDate('yy-mm-dd', date);

@dubrox dubrox self-assigned this Sep 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants