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

Add tooltip showing the time in UTC and hinting at the timezone of the timestamp in the page #18

Open
darkk opened this issue Nov 4, 2016 · 4 comments

Comments

@darkk
Copy link

darkk commented Nov 4, 2016

The screenshot is done in UTC+3.
One page of wui shows measurement time in local time and another in UTC.
Looks like a bug :)

tz mismatch 2016-11-04 23-07-20

@darkk darkk changed the title TZ mismatch in /measurements and actual measurement TZ mismatch in /measurements and actual measurement page Nov 4, 2016
@hellais
Copy link
Contributor

hellais commented Nov 4, 2016

Oh yeah you are right, this is for sure a bug. The desired behaviour is that measurements from the wui should always be shown in the local time of the user as is shown in the first of the two pages.

Thanks for finding this!

@hellais hellais added the bug label Nov 4, 2016
@hellais
Copy link
Contributor

hellais commented Dec 12, 2016

@irl suggested on IRC that we add a tooltip specifying in which timezone a certain time is indicated in to make it clearer.

@irl
Copy link

irl commented Dec 12, 2016

Adding a tooltip to times/dates that gives an ISO8601 representation of the time/date including timezone to avoid any confusion.

@hellais
Copy link
Contributor

hellais commented Feb 5, 2017

The timezone does not mismatch anymore, however the tooltip is not currently implemented.

I am going to leave this open and rename it to add support for the tooltip.

@hellais hellais changed the title TZ mismatch in /measurements and actual measurement page Add tooltip showing the time in UTC and hinting at the timezone of the timestamp in the page Feb 5, 2017
@hellais hellais added enhancement and removed bug labels Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants