search

jekor / drcal

A Minimalist JavaScript Calendar (not a date picker)
http://www.minjs.com/#drcal
MIT License
26 stars 5 forks source link

Changing months events dont go through #1

Closed JZL closed 12 years ago

JZL commented 12 years ago

If you select an event or add a class to a extra table cell(when the table shows the next few days of the next/previous month to complete the table square) then it doesnt go through to the next arrows

jekor commented 12 years ago

Nice catch. I hadn't thought of that.

I'll work on making it so that the specific day cells are re-used between months. It might take a bit to figure out a good way to do this.

JZL commented 12 years ago

thanks, im trying to make a copy of seinfeld's dont break the chain. this is invaluable.

On Sun, Jul 15, 2012 at 2:40 PM, Chris Forno < reply@reply.github.com

wrote:

Nice catch. I hadn't thought of that.

I'll work on making it so that the specific day cells are re-used between months. It might take a bit to figure out a good way to do this.

Reply to this email directly or view it on GitHub: https://github.com/jekor/drcal/issues/1#issuecomment-6993850

jekor commented 12 years ago

OK, now drcal stores each week in the cache instead of entire months. This means that days are no longer duplicated. You can test it by looking at the demo on minjs, select a "extra" day, and watch it remain selected as you move to the next/previous month.

I bumped the version to 1 because monthRender was replaced with weekRender.

JZL commented 12 years ago

thank you so much