Review Request 123079: digitalclock: Don't reverse js timezone offset when using to generate utc from local time.

Jeremy Whiting jpwhiting at kde.org
Fri Mar 20 11:51:55 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123079/
-----------------------------------------------------------

(Updated March 20, 2015, 11:51 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Martin Klapetek.


Changes
-------

Submitted with commit 0b82fc7e455bdc2c4b419c26e1f5349205c25a65 by Jeremy Whiting to branch Plasma/5.2.


Repository: plasma-workspace


Description
-------

digitalclock: Don't reverse js timezone offset when using to generate utc from local time.


Diffs
-----

  applets/digital-clock/package/contents/ui/DigitalClock.qml 1292f730467d43a331d1253a3872d41ebe2d59c1 

Diff: https://git.reviewboard.kde.org/r/123079/diff/


Testing
-------

In this place we are using the offset to calculate utc from the local time. My timezone is -6 and my getTimezoneOffset is 360. To take my local time and get utc, we need to add 6 hours, not subtract 6 hours. We could either subtract the reverse of the js value, or just add it directly.

My local time in the tooltip has the right time now. All others are also calculated correctly now.


Thanks,

Jeremy Whiting

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150320/1fb1eb57/attachment.html>


More information about the Plasma-devel mailing list