Skip to content

fix(notes): match compact library toolbar controls - #60

Open
luannn010 wants to merge 2 commits into
feat/notes-route-heading-motionfrom
fix/notes-library-toolbar
Open

luannn010 wants to merge 2 commits into
feat/notes-route-heading-motionfrom
fix/notes-library-toolbar

Conversation

@luannn010

Copy link
Copy Markdown
Collaborator

Summary

  • Fix the64.5px toolbar stretch: compact30px actions,28px genuine sort/view group and24px List/Grid choices.
  • Match inline title/count, spacing and Ghost color in Notes library-list/grid/empty headers; preserve real callbacks, guards and other header consumers. Depends on feat(notes): match admitted route heading motion #59.

Testing

-35 focused tests, TypeScript/build and independent source review passed.

  • Real encrypted browser checks passed27 theme/viewport/sidebar cases plus long tag, focus, create, List/Grid, Markdown picker cancel/import, persisted sort/reload, Trash and Clips/Tags isolation.
  • Existing complete library journey passed with unchanged assertions. Native rebuild and independent normal-host clean export follow; shared restricted-env Windows verifier remains unusable before Node startup.
  • Full-frame/native FPS/backend qualification remains open. No migration; revert to undo.

Co-authors

none

AI usage

  • Written by hand, with no AI
  • Harness and version: Codex desktop, codex-cli0.160.0; Git/gh, Node, Vitest, Playwright/Chrome, Rust/Tauri and bundled Computer Use.
  • Models: gpt-6.1-sol (UI/UX, implementation, browser checks, integration and independent source review); gpt-6.1-terra (independent clean-export verification).
  • Effort: medium.
  • Human role: requested design-gap audit/fixes, local build and focused PR delivery; merge review pending.
  • Sources: project-owned Notes artifact706a8d07 at designf610ced1da443b04699137333c1309925092fffa, frames library-list/library-grid/empty-library; existing Notes contracts. No external implementation copied; existing notices retained.

Checklist

  • Works on Windows, macOS and Linux
  • Architecture updated for scoped Notes header geometry
  • No secrets, profiles, screenshots or build outputs included

@luannn010
luannn010 added this pull request to stack #51 November 11, 2026 16:05
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