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

Only shows one row of days #1

Open
shri opened this issue Mar 3, 2013 · 4 comments · May be fixed by #8
Open

Only shows one row of days #1

shri opened this issue Mar 3, 2013 · 4 comments · May be fixed by #8

Comments

@shri
Copy link

shri commented Mar 3, 2013

When trying to implement the calendar, only one row of days is visible. This makes the plugin not usable at the moment.

@najlepsiwebdesigner
Copy link
Owner

I was not able to reproduce this issue, can you please provide more specific feedback?
Thanks!

@mbrakken
Copy link

The behavior is happening here: http://foundation-datetimepicker.peterbeno.com/example/example.html

I'll compare with foundation-datepicker to see if I can resolve it.

Thanks!

@clockwork189
Copy link

Is there any update on this issue as I am experiencing the same problem as well :S

@funkju
Copy link

funkju commented Feb 5, 2014

I am still seeing this issue as well.

As far as I can tell the issue is in Datetimepicker.prototype.fill starting on line 439.

I replaced it with code from the parallel foundation-datepicker package and resolved the problem.

The relevant portion of my fill function now looks like (starting at line 438):

this.fillMonths();
var prevMonth = UTCDate(year, month, 0,0,0,0,0),
    day = DPGlobal.getDaysInMonth(prevMonth.getUTCFullYear(), prevMonth.getUTCMonth());
prevMonth.setUTCDate(day);
prevMonth.setUTCDate(day - (prevMonth.getUTCDay() - this.weekStart + 7)%7);
var nextMonth = new Date(prevMonth.valueOf());
nextMonth.setUTCDate(nextMonth.getUTCDate() + 42);
nextMonth = nextMonth.valueOf();
var html = [];
var clsName;

@dsandstrom dsandstrom linked a pull request Jun 6, 2014 that will close this issue
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 a pull request may close this issue.

5 participants