@wtfalch/email 0.6.0 → 0.6.1

package/dist/mailbox/react/ThreadList.js

@@ -10,14 +10,27 @@ import { useCallback, useEffect, useRef } from 'react';
  *  time is the thing they do not know. */
 export function shortTime(iso, now = new Date()) {
     const then = new Date(iso);
+    /* Before the elapsed-time branches, not after them, and that ordering is
+       the whole of this function's only sharp edge.
+  
+       Elapsed time and calendar day disagree for one hour out of every
+       twenty-four. Just after midnight a message sent forty-five minutes ago
+       was sent yesterday, so the row said "45m" under a heading that said
+       "Yesterday" -- both true, and together they read as a fault. Checking
+       elapsed time first is what let that through; the version before this one
+       only reached the clock after the hour was up, so the hour where it
+       mattered was the hour it never covered.
+  
+       The heading wins, because the heading is the thing the eye has already
+       read. A row under "Yesterday" says a time of day, always. */
+    if (dayGroup(iso, now) === 'Yesterday') {
+        return then.toLocaleTimeString('en-GB', { hour: '2-digit', minute: '2-digit' });
+    }
     const minutes = Math.max(0, Math.round((now.getTime() - then.getTime()) / 60_000));
     if (minutes < 1)
         return 'now';
     if (minutes < 60)
         return `${minutes}m`;
-    if (dayGroup(iso, now) === 'Yesterday') {
-        return then.toLocaleTimeString('en-GB', { hour: '2-digit', minute: '2-digit' });
-    }
     const hours = Math.round(minutes / 60);
     if (hours < 24)
         return `${hours}h`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wtfalch/email",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "The wtfalch estate: reading a mailbox over JMAP, and administering the Stalwart server it lives on. Two entries with no code in common.",
   "license": "MIT",
   "type": "module",