From 9cf8ee21628b24422fafa4b836c60dc213bcf938 Mon Sep 17 00:00:00 2001 From: Mechiel Lukkien Date: Tue, 23 Jan 2024 17:01:34 +0100 Subject: [PATCH] webmail: don't who an age of "-<1min", drop the - if a browser is ahead just a few seconds, we would show "-<1min", not great. just show "<1min" in that case. we'll still show negative age if drift is more than 1 minute, which seems like a good hint to get time fixed on either client or server. --- webmail/webmail.js | 2 ++ webmail/webmail.ts | 2 ++ 2 files changed, 4 insertions(+) diff --git a/webmail/webmail.js b/webmail/webmail.js index 4f3b4a7..4dedbb8 100644 --- a/webmail/webmail.js +++ b/webmail/webmail.js @@ -2648,6 +2648,8 @@ const newMsgitemView = (mi, msglistView, otherMailbox, listMailboxes, receivedTi if (t < 60) { s = '<1min'; nextSecs = 60 - t; + // Prevent showing '-<1min' when browser and server have relatively small time drift of max 1 minute. + negative = ''; } dom._kids(r, negative + s); // note: Cannot have delays longer than 24.8 days due to storage as 32 bit in diff --git a/webmail/webmail.ts b/webmail/webmail.ts index 717f6de..5505898 100644 --- a/webmail/webmail.ts +++ b/webmail/webmail.ts @@ -1939,6 +1939,8 @@ const newMsgitemView = (mi: api.MessageItem, msglistView: MsglistView, otherMail if (t < 60) { s = '<1min' nextSecs = 60-t + // Prevent showing '-<1min' when browser and server have relatively small time drift of max 1 minute. + negative = '' } dom._kids(r, negative+s)