Test-infra: prow: deck: time hover is backwards
Right now you get the full timestamp in the UI and the "N minutes ago" on hover -- seems like that should be reversed. Show the nice human-focused time and hover to get the nitty gritty timestamp.
/kind bug
/area prow
/assign @qhuynh96
/cc @BenTheElder
stevekuznetsov
All 10 comments
You can't hover on mobile or copy the timestamp and "x minutes ago" goes
stale. 🤷♂️
On Fri, Mar 2, 2018, 07:47 Steve Kuznetsov notifications@github.com wrote:
Right now you get the full timestamp in the UI and the "N minutes ago" on
hover -- seems like that should be reversed. Show the nice human-focused
time and hover to get the nitty gritty timestamp./kind bug
/area prow
/assign @qhuynh96 https://github.com/qhuynh96
/cc @BenTheElder https://github.com/bentheelder—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/test-infra/issues/7098, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4Bq7HSqx48TsVqxJCzyQ_blh9Zf3_1ks5taWl8gaJpZM4SaFef
.
BenTheElder
on 2 Mar 2018
We reversed this after a discussion about people may want to interact with the time directly (copy/paste) also you cant see full time on mobile. WDYT
qhuynh96
on 2 Mar 2018
What about making the timestamp format more concise and show a more detailed one / timezone etc on hover?
OTOH I could be OK with "x minutes ago" if we EG had some worker updating that every so often so it was accurate or something... As it stood "x minutes ago" didn't seem that helpful and it was difficult to make out the timestamp from hover even on fancy giant monitors 🤷♂️
BenTheElder
on 2 Mar 2018
Hmmm. I guess in the majority of cases displays will have the "x minutes ago" and hover to get the real time, that was just most common in what I saw elsewhere and what I expected from that background. If it's a static thing that doesn't get refreshed that isn't the end of the world but agreed if it ticks up that would be better.
stevekuznetsov
on 2 Mar 2018
Also agreed that the formatting could help -- right now I just want to know roughly when a job ran. Neither the old format (wrong timezone) nor the current one (huge block of text) is helping me there -- a concise version might help a lot there.
stevekuznetsov
on 2 Mar 2018
right now we have 'MMM DD YYYY, HH:MM:SS [GMT]ZZ', perhaps use that one for hover and something like 'HH:MM:SS' for display which is more concise even than two minutes ago. We could also try to fix hover / mobile and make the hover larger instead.
BenTheElder
on 2 Mar 2018
The time is already updated every 1 min so you dont need to refresh. I dont use this much so I could not tell which way is better, defer to you, @BenTheElder, @ixdy
qhuynh96
on 2 Mar 2018
I agree that it seems a bit backwards right now, at least compared to most sites which show relative times (with a click or hover to get absolute times). To solve the mobile issue, can we also add a tap target or something which reveals the real time?
Side note, every job is showing the exact same started time right now (Mar 02 2018, 10:03:00 GMT-0800), which seems like a bug.
ixdy
on 2 Mar 2018
(bikeshed/soapbox: I personally do not like the "N minutes ago" timestamps, and much prefer absolute times always.)
ixdy
on 2 Mar 2018
@ixdy @BenTheElder the bug 10:03 is due to wrong time format, should be HH:mm:ss not HH:MM:SS
MM is month
qhuynh96
on 2 Mar 2018
Related issues
xiangpengzhao
·
3Comments
cjwagner
·
3Comments
fen4o
·
4Comments
MrHohn
·
4Comments
stevekuznetsov
·
3Comments
Most helpful comment
right now we have
'MMM DD YYYY, HH:MM:SS [GMT]ZZ', perhaps use that one for hover and something like'HH:MM:SS'for display which is more concise even thantwo minutes ago. We could also try to fix hover / mobile and make the hover larger instead.