Skip to content

fix(windows): polish the new look's details sheet, lines, activity and support - #1422

Merged
raydocs merged 2 commits into
mainfrom
claude/windows-ui-polish-20261006
Oct 6, 2026
Merged

raydocs merged 2 commits into
mainfrom
claude/windows-ui-polish-20261006

Conversation

@raydocs

@raydocs raydocs commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Owner: docs/SHIP_PLAN.md 0.0.75 (owner 2026-10-06: "uiux 细节打磨"). Presentation only: no handler, protection, routing or native change.

What changes

  • Home details sheet (IHOME-12): the reused cards start on one 14px edge, read in the page font (the line button fell back to the platform font, figures were monospace), the two small tiles share a height, and the active line's latency reads 83 ms like the home chip and the lines page.
  • Activity: an app with more than twenty connections now offers "Show all N"; every row keeps its close button on the same generation-checked path (feat(windows-ui): PR8 sea activity and account (0.0.75 preview) #1411 F3). The close-all row is not rendered while Tono is not connected (it was a permanently disabled button).
  • Lines: a line with no separate city word is named once, not twice. The recommended card's help glyph is a 20px ring button centred on the tag.
  • Disclosures (support, tray, lines technical details, home steps): one chevron that turns when open, instead of the platform triangle; no transition under prefers-reduced-motion.
  • Support: the three action buttons share one height (40px).

Verification

  • One regression per behaviour, each run red on the old source (three source files reverted: 4 failed | 69 passed): Unable to find an element with the text: 83 ms; Unable to find … Show all 23; expected <button …> to be null; Found multiple elements with the text: Singapore · Harbor.
  • MacBook: vitest run → Test Files 56 passed (56) / Tests 396 passed (396); typecheck → unchecked indexed access errors 79 (baseline 79).
  • Shell preview at 920×600 (zh; details sheet also en): support buttons measured 40,40,40; help glyph and tag centres 230.5 / 230.5.

Limitations

🤖 Generated with Claude Code

https://claude.ai/code/session_01Cq68RQJmV6Xiao13p3SEat

raydocs and others added 2 commits October 6, 2026 13:28
…d support

Presentation only. The details sheet's reused cards share one edge, the page
font and tabular figures, and write the active line's latency in ms (IHOME-12).
Activity reaches the connections beyond twenty per app and hides the close-all
row while not connected. A line without a city word is named once. Disclosures
share one chevron, the support actions one height, the lines help glyph a ring.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Cq68RQJmV6Xiao13p3SEat
…row beyond twenty in the test

Review round for #1422 (jev-route 5f90855c): the close regression now closes a
connection that only "Show all" reveals, a collapsed app drops back to its
first twenty rows, and the record describes the old latency label correctly.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Cq68RQJmV6Xiao13p3SEat
@raydocs

raydocs commented Oct 6, 2026

Copy link
Copy Markdown
Owner Author

Review record (jev-route)

  • Decision 4480cc8a, run 5f90855c, range origin/main...c7a1a0943: single slot, Opus 5.5 (jev-reviewer), verified by Codex gpt-6.1-sol. PASSED, 0 blocking.
    • opus:F1 (minor, record): the changelog misdescribed the old latency label. Corrected.
    • opus:F2 (minor, test): the close regression closed a row that was already among the first twenty. It now asserts the 23rd row is absent, clicks "Show all 23", and closes that row (closeConnection('group-22', 7)).
    • opus:F3 (suggestion): every row of an expanded app stayed mounted after collapse. A collapsed app now returns to its first twenty; one regression, red without the handler (to have a length of 20 but got 23).
  • Fix round f10ff6942. Local: vitest run 56 files / 397 tests; typecheck 79 (baseline 79).
  • ci-gate on the exact head f10ff69421bd9d0c1685531ad6c43b278c4b6448: green, https://github.com/raydocs/tono/actions/runs/37521619167
  • Review threads: 0 unresolved; no CHANGES_REQUESTED.
  • Not verified: Windows/WebView2 hardware. UI PR, merged under the owner's 2026-10-06 instruction to merge the new-look work into main.

@raydocs
raydocs marked this pull request as ready for review October 6, 2026 20:05
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@raydocs
raydocs enabled auto-merge October 6, 2026 20:06
@raydocs
raydocs merged commit bca5fa9 into main Oct 6, 2026
13 checks passed
@raydocs
raydocs deleted the claude/windows-ui-polish-20261006 branch October 6, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant