-
Notifications
You must be signed in to change notification settings - Fork 304
Fix Daylight Saving Time #41
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's probably better to have it as is, than 'sort of' fixed. http://stackoverflow.com/questions/2532729/daylight-saving-time-and-timezone-best-practices touches on some of the things you need to think about. Adding 2 hours isn't really an appropriate fix - in all likelyhood it should probably be addressed in the date labelling code.
|
I can't seem to reproduce the problem, though I don't know if it requires changing the system date & time/time zone to see it. @gabrielgaliaso , can you check to see if this is still an issue in the current version of the plugin? If so, please make a jsfiddle to show the bug in action, or at least list the steps to reproduce the behavior you describe. If it's no longer an issue, we can close this issue. Thanks! |
|
Still an issue -- see #95 and linked fiddle (set system time zone to Brasilia time to reproduce the bug). |
|
Hello Gentlemen, okay with you? [
{
"name":" ",
"desc":"\"INTEGRAÇÃO COM FORPONTO\"",
"values":[{
"id":"\"\"",
"from":"\"2013-10-10 00:00:00.000\"",
"to":"\"2013-08-21 00:00:00.000\"",
"label":"\"INTEGRAÇÃO COM FORPONTO\"",
"customClass":"ganttGreen"
}]
},
{
"name":" ",
"desc":"\"TESTE\"",
"values":[{
"id":"\"\"",
"from":"\"2013-10-20 00:00:00.000\"",
"to":"\"2013-08-22 00:00:00.000\"",
"label":"\"TESTE\"",
"customClass":"ganttGreen"
}]
}
]
|
|
@maxanalista , you can start by looking at the |
|
Did not quite understand the topic ebf84ca. There is a new version that understands the Brasilia time zone that I can receive email [email protected] or download here? The code you see here is a bit different from what I downloaded. I made some changes and stopped working. |
|
So, I downloaded the current version and the problem persisted in the rendering of the activities that have the time passing by October 19. |
|
@maxanalista , there is no fix yet in the current version, the commit line and comment that I linked to is where at least part of the bug (or a related one) is. |
|
@usmonster , I have not had time to study all the code but I will. What I have in mind as a partial solution to the problem is to verify through a condition that the activity belongs to a period which goes through October 19. If so, it adds some extra pixels in the width of the activity. I'll try to do it here. |
|
@maxanalista , in this case, I would suggest the simpler workaround of changing your input's end times to 23:59 (instead of 00:00). This should work without any modifications to the code. |
|
Has anyone found solution for this issue? |
Hopefully fixes taitems#95 & closes taitems#41.
|
Hello from 2019! :) This pull request is unfortunately outdated, but the issue should be fixed in a cleaner way in #226. Please feel free to review and/or test out the code before it's merged and report back if you see any regressions or other problems. (This PR will be auto-closed once the other one is merged.) |


This fix Daylight Saving Time.
On 2012-10-20 was rendering twice.