Skip to content

Flatten assistant message actions into an inline icon row - #69505

Merged
OutThisLife merged 2 commits into
mainfrom
bb/inline-msg-actions
Jul 22, 2026
Merged

Flatten assistant message actions into an inline icon row#69505
OutThisLife merged 2 commits into
mainfrom
bb/inline-msg-actions

Conversation

@OutThisLife

@OutThisLife OutThisLife commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Replaces the assistant message kebab overflow with a single inline, hover-revealed icon row — no dropdown.

Order: age → branch → copy → read aloud → refresh

  • All row icons come from one library (Tabler) at size-3.5, so stroke weight and sizing stay consistent (refresh was previously a Codicon font glyph).
  • Branch uses the fork glyph (IconGitFork); read aloud uses a sine wave (IconWaveSine) instead of the sparse speaker.
  • Age renders compact "2h ago" via a new shared formatAgo() in lib/time.ts (built on the existing coarseElapsed), with the absolute timestamp on hover. Adds the ageDays locale string across en/ja/zh/zh-hant.

Notes

  • Preserves the isInterim footer guard (branch was rebased onto latest main).
  • formatAgo() covered by a unit test; desktop tsc --noEmit clean.

Test plan

  • Hover a completed assistant message → row reads age → branch → copy → read aloud → refresh, no kebab
  • Branch, copy, read aloud, refresh all work
  • Age shows "2h ago" / "3m ago" / "now"; hovering shows the absolute time
  • Interim (sealed mid-turn) messages still show no footer

Drop the kebab overflow so age, branch, copy, read aloud, and refresh are always one hover away.
…d age util

- Use the fork glyph for branch and a sine wave for read aloud (all one lib now)
- Extract compact "2h ago" into formatAgo() in lib/time.ts (+ ageDays locale string)
- Cover formatAgo with a unit test
@OutThisLife
OutThisLife force-pushed the bb/inline-msg-actions branch from 1fa3186 to ca244e2 Compare July 22, 2026 17:19
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

running on ca244e2

looks good to me!

@OutThisLife
OutThisLife enabled auto-merge July 22, 2026 17:20
@OutThisLife
OutThisLife merged commit 59614ef into main Jul 22, 2026
27 checks passed
@OutThisLife
OutThisLife deleted the bb/inline-msg-actions branch July 22, 2026 17:24
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/desktop Electron desktop app (apps/desktop/*) labels Jul 22, 2026
randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
…-actions

Flatten assistant message actions into an inline icon row
33hodl pushed a commit to 33hodl/hermes-agent that referenced this pull request Aug 12, 2026
…-actions

Flatten assistant message actions into an inline icon row
prmartinow pushed a commit to prmartinow/hermes-agent that referenced this pull request Aug 26, 2026
…-actions

Flatten assistant message actions into an inline icon row
melon-xf added a commit to melon-xf/hermes-agent that referenced this pull request Sep 3, 2026
…-actions

Flatten assistant message actions into an inline icon row
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants