Skip to content

fix(ui): name the machine in the draft heading, steady panel tabs, even composer glow - #597

Merged
klNuno merged 11 commits into
mainfrom
boite/mobile-ui-notifications-rework
Oct 19, 2026
Merged

klNuno merged 11 commits into
mainfrom
boite/mobile-ui-notifications-rework

Conversation

@klNuno

@klNuno klNuno commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Note

Clanker Claude Opus 5.5 is responding on behalf of meetsu

The new-thread heading says which machine the thread starts on, the focused composer glows evenly, the side panel's tabs stop shaking at its narrowest width, and the font list keeps faces chosen for reading.

new thread before and after: the heading gains "on This PC", the worktree chip moves before the permissions menu

what changed

  • The draft heading ends with "on [machine]"; the phone adds it after its project chip (ChatView.svelte).
  • The worktree chip comes before the permissions menu (ComposerBar.svelte).
  • The focus halo spreads outward: its corners read 19 of luminance against 30 on the sides (app.css).
  • Panel strip controls no longer shrink, which made the chevrons flip every frame at 300 px (RightPanel.svelte).
  • Thread row buttons sit on the title line (ThreadCard.svelte).
  • Text fonts: Inter, Atkinson Hyperlegible Next, Source Sans 3, IBM Plex Sans, system; Geist, Figtree and DM Sans leave (236 KB), an old choice reads as Inter.

to question

The font cut follows studies: readers are about 35 % faster in their own best face (Wallace et al. 2022), open humanist shapes win at a glance (Reimer 2012, Dobres 2016), distinct letterforms help (Beier and Larson 2010). The kept faces tell I, l and 1 apart; Inter stays the default.

verified

  • bun run check: 0 errors; vitest on 22 related UI files: 233 pass.
  • New panel.test.ts step: fails on main at 300 px, passes here.
  • Nine related e2e files on Linux: 50 pass, two load timeouts passed on rerun.
  • Sizes against main 7593ddca, both built here (bun scripts/ci/budgets.ts measure): see Bundle size.
  • not verified: the Windows app, a touch screen.

Bundle size

uiEntryChunk +925 bytes (0.19 %): the draft heading names its machine (MachineIcon and the machine lookup in ChatView.svelte), and the font label gains its hint. uiDist -221,194 bytes (-4.54 %): the Geist, Figtree and DM Sans files leave. The core does not change.

captures

Composer corners, before then after (brightened):

composer corners

Panel strip at 300 px, before then after:

tab strip

Row buttons:

row buttons

Phone draft, before then after:

phone draft

Font picker, desktop and phone:

fonts desktop

fonts phone

The seven former faces:

faces

…en composer glow

The draft heading ends with the machine its project is on. The worktree chip comes before the permissions menu. The composer's focus halo spreads outward so its corners light like its sides. Right panel controls no longer shrink, so the tab chevrons stop flipping at the narrowest width. Thread row buttons sit on the title's line. The text font list keeps Inter, Atkinson Hyperlegible Next, Source Sans 3, IBM Plex Sans and the system face.
@github-actions github-actions Bot added documentation Improvements or additions to documentation ui packages/ui ci Workflows and CI scripts labels Oct 18, 2026
@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 49 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: beboite/boite/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1af4263e-9c04-4395-8106-6944cbcf6a78
📥 Commits

Reviewing files that changed from the base of the PR and between 65e9e53 and ac88b09.

⛔ Files ignored due to path filters: 7
  • packages/ui/public/fonts/DMSans-latin-ext.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/DMSans-latin.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/Figtree-latin-ext.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/Figtree-latin.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/Geist-latin-ext.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/Geist-latin.woff2 is excluded by !**/*.woff2
  • packages/ui/public/fonts/Geist-rest.woff2 is excluded by !**/*.woff2
📒 Files selected for processing: 18
  • docs/performance.md
  • packages/ui/index.html
  • packages/ui/public/fonts/DMSans-OFL.txt
  • packages/ui/public/fonts/Figtree-OFL.txt
  • packages/ui/src/app.css
  • packages/ui/src/components/AppearancePage.svelte
  • packages/ui/src/components/ChatView.svelte
  • packages/ui/src/components/ComposerBar.svelte
  • packages/ui/src/components/RightPanel.svelte
  • packages/ui/src/components/ThreadCard.svelte
  • packages/ui/src/lib/fonts.ts
  • packages/ui/src/lib/strings/en/settings.ts
  • packages/ui/src/lib/strings/en/thread.ts
  • packages/ui/src/lib/strings/fr/settings.ts
  • packages/ui/src/lib/strings/fr/thread.ts
  • tests/e2e/orchestration-recovery.test.ts
  • tests/e2e/panel.test.ts
  • tests/e2e/text-alignment.test.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…ications-rework

# Conflicts:
#	docs/ci.md
#	scripts/ci/budgets.json
…ications-rework

# Conflicts:
#	docs/ci.md
#	scripts/ci/budgets.json
…ications-rework

# Conflicts:
#	docs/ci.md
#	scripts/ci/budgets.json
…ications-rework

# Conflicts:
#	docs/ci.md
#	scripts/ci/budgets.json
@klNuno

klNuno commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

Note

Clanker Claude Opus 5.5 is responding on behalf of meetsu

#614 split strings.en.ts and strings.fr.ts into one file per part. When git merge origin/main stops on them, run bun scripts/migrate/split-catalogues.ts: it replays this branch's catalogue edits onto main's parts and stages them. A part both sides changed on the same lines keeps conflict markers.

…ications-rework

# Conflicts:
#	packages/ui/src/lib/strings.en.ts
#	packages/ui/src/lib/strings.fr.ts
#	scripts/ci/budgets.json
…ications-rework

# Conflicts:
#	docs/ci.md
#	scripts/ci/budgets.json
…ications-rework

# Conflicts:
#	packages/ui/src/components/ChatView.svelte
@klNuno
klNuno merged commit 65b57cf into main Oct 19, 2026
30 checks passed
@klNuno
klNuno deleted the boite/mobile-ui-notifications-rework branch October 19, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Workflows and CI scripts documentation Improvements or additions to documentation ui packages/ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant