Skip to content

feat(ui): lead the run header with the score - #38

Merged
nekomeowww merged 1 commit into
mainfrom
nekomeowww/feat/run-header-score
Oct 21, 2026
Merged

nekomeowww merged 1 commit into
mainfrom
nekomeowww/feat/run-header-score

Conversation

@nekomeowww

@nekomeowww nekomeowww commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The run page header now leads with the score, not the run's name:

  • The headline score uses the lg size, and the title (start time · run hash) drops to body size. PageHeader gets a titleSize: 'sm' option, so other pages keep their titles as they are.
  • On phones the case bar sits under the score at the full width.
  • "Compare attempts" moves from a button in the header into the "⋯" menu, below "Copy ID". IdMenu gets a default slot for a page's secondary actions, which PageHeader passes through from a new menu slot. The slot is passed only when the page fills it, so other menus get no empty group.
  • The subtitle no longer repeats the attempt's time. It matches the run start in the title, and the breadcrumb names the attempt.

How tested

  • With agent-browser on airi's tts-speech-formula reports, at 390 and 1440px: the header layout, and the "⋯" menu with Copy ID plus Compare attempts.
  • pnpm -F @vieval/ui exec vitest run: 227 passed.
  • pnpm -F @vieval/ui typecheck and eslint pass.

🤖 Generated with Claude Code

Base automatically changed from nekomeowww/fix/mobile-layout to main October 20, 2026 23:58
@nekomeowww
nekomeowww force-pushed the nekomeowww/feat/run-header-score branch from 6b348fb to 6850c60 Compare October 20, 2026 23:58
@github-actions

Copy link
Copy Markdown

Approval required for deploying to Cloudflare Workers (Preview) for docs.

Name Link
Waiting for approval For maintainers, approve here

@nekomeowww
nekomeowww merged commit e48fa38 into main Oct 21, 2026
5 checks passed
@nekomeowww
nekomeowww deleted the nekomeowww/feat/run-header-score branch October 21, 2026 00:01
nekomeowww added a commit that referenced this pull request Oct 21, 2026
Bring in #37 (phone layout and narrow cards), #38 (run header leads with the
score), and v0.10.2. Conflicts kept main's layout and behavior and applied the
translations on top; the strings #37 and #38 added (the ETA info button, Run
details) are translated in en and zh-Hans.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
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