History rows show the relative age without the ISO datetime #760

Open
opened 2026-11-10 20:54:19 +01:00 by clawbot · 1 comment
Collaborator

History rows on Home, AddressDetail and AddressToken show only the relative age of a transaction ("3 hours ago"); the datetime is only in the tooltip (src/popup/views/home.js, addressDetail.js, addressToken.js). A tooltip shows nothing until the pointer rests on the row.

The rule: RULES.md Display Consistency, "Timestamps: ISO datetime + relative age, everywhere they appear". sneak's README.md text: "Timestamps include both an ISO datetime and a humanized relative age wherever shown." RULES.md says the code changes to match it.

Found in the RULES.md read on #672 (comment). Choice taken: the rows change, not the rule. Reversing it is one line per view.

Fix

  • Each history row shows isoDate() and then timeAgo() in parentheses, as TransactionDetail does, through one helper in src/popup/views/helpers.js that all three views and TransactionDetail call. The tooltip goes.
  • The row's layout keeps its height whether the text is short or long; on the 360 px popup the datetime and the direction label fit on one line, or the datetime takes its own line. No layout shift when history arrives.
  • README.md: the sentence "A history row shows the age, with the datetime as its tooltip" states the new row.

Definition of done

  • A test, failing on current next, that a rendered history row contains both the ISO datetime and the relative age.
  • A screenshot check in the headless browser harness (make test-e2e) of Home with history at 360 px wide, described in the PR body, showing no overflow.
  • make check green. Gate runs: flock -w 3600 /srv/code/tmp/.autistmask-gate.lock script/cibuild.

Model: opus-5-5

History rows on Home, AddressDetail and AddressToken show only the relative age of a transaction ("3 hours ago"); the datetime is only in the tooltip (`src/popup/views/home.js`, `addressDetail.js`, `addressToken.js`). A tooltip shows nothing until the pointer rests on the row. The rule: `RULES.md` Display Consistency, "Timestamps: ISO datetime + relative age, everywhere they appear". sneak's `README.md` text: "Timestamps include both an ISO datetime and a humanized relative age wherever shown." `RULES.md` says the code changes to match it. Found in the `RULES.md` read on https://git.eeqj.de/sneak/AutistMask/issues/672#issuecomment-160312. Choice taken: the rows change, not the rule. Reversing it is one line per view. ## Fix - Each history row shows `isoDate()` and then `timeAgo()` in parentheses, as TransactionDetail does, through one helper in `src/popup/views/helpers.js` that all three views and TransactionDetail call. The tooltip goes. - The row's layout keeps its height whether the text is short or long; on the 360 px popup the datetime and the direction label fit on one line, or the datetime takes its own line. No layout shift when history arrives. - `README.md`: the sentence "A history row shows the age, with the datetime as its tooltip" states the new row. ## Definition of done - A test, failing on current `next`, that a rendered history row contains both the ISO datetime and the relative age. - A screenshot check in the headless browser harness (`make test-e2e`) of Home with history at 360 px wide, described in the PR body, showing no overflow. - `make check` green. Gate runs: `flock -w 3600 /srv/code/tmp/.autistmask-gate.lock script/cibuild`. Model: opus-5-5
clawbot self-assigned this 2026-11-10 20:54:19 +01:00
clawbot added this to the 1.0.0 milestone 2026-11-10 20:54:25 +01:00
Author
Collaborator

Built in #773. History rows now show the ISO datetime and then the age, on a line of their own. The direction label moved to its own right-aligned line below the time. The e2e check runs at the popup's real width of 396 px, not 360 px.

Model: opus-5-5

Built in https://git.eeqj.de/sneak/AutistMask/pulls/773. History rows now show the ISO datetime and then the age, on a line of their own. The direction label moved to its own right-aligned line below the time. The e2e check runs at the popup's real width of 396 px, not 360 px. Model: opus-5-5
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#760