date-prototype icon indicating copy to clipboard operation
date-prototype copied to clipboard

Published 20 hours ago • verified publisher icon kensnyder

Added daylight saving time support to add/diff

Open toolstack opened this issue 11 years ago • 2 comments

toolstack avatar Nov 05 '14 21:11 toolstack

This is great. I'm not sure where the proxy property is defined, but I like the approach you took. I'll take a closer look and add some unit tests.

kensnyder avatar Nov 07 '14 22:11 kensnyder

I have to admit I actually fixed a different javascript charting library that included your code but a slightly older version (looks like version 2.0.2 if that makes sense) and the proxy property was defined at the top of the class as a javascript Date object. It looks like the library underwent a pretty major rewrite since then.

It looks like in the new code the proxy object was dropped and just "this." is used instead.

toolstack avatar Nov 08 '14 00:11 toolstack