Skip to content

Fix toolbar dark-mode mismatch and oversized icons - #3

Merged
PermutationResearch merged 2 commits into
mainfrom
fix/toolbar-appearance-and-shell
Jul 10, 2026
Merged

Fix toolbar dark-mode mismatch and oversized icons#3
PermutationResearch merged 2 commits into
mainfrom
fix/toolbar-appearance-and-shell

Conversation

@PermutationResearch

Copy link
Copy Markdown
Owner

Summary

  • Pin the app to the light (aqua) appearance so the titlebar/toolbar no longer render dark against Loci's light-authored palette when the system is in dark mode
  • Render the command palette, Ask Loci, and settings toolbar items as bordered buttons with 13pt symbol configurations so the glyphs draw at standard compact-toolbar size instead of oversized
  • Decompose the LociShell body into workspace/chrome/palette sections, keep the zoom slider pinned top-trailing with a fixed 20pt track height, and fix a scaled-font modifier recursion that crashed at launch

Testing

  • swift build -c release clean
  • swift test — 23/23 passing

🤖 Generated with Claude Code

arnaudbellemare and others added 2 commits July 10, 2026 00:01
Split the shell ZStack into workspace, chrome, and command-palette
sections, keep the zoom slider anchored top-trailing with a fixed
20pt track height, and stop LociScaledFontModifier from reapplying
itself through lociFont, which recursed until launch crashed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Loci's palette is authored against light surfaces, so the titlebar
went dark-on-light when the system was in dark mode. Pin NSApp to
aqua until a real dark palette exists, and render toolbar items as
bordered buttons with 13pt symbol configurations so the command,
sidebar, and settings glyphs stop drawing oversized.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@PermutationResearch
PermutationResearch merged commit 0349dc0 into main Jul 10, 2026
1 check passed
@PermutationResearch
PermutationResearch deleted the fix/toolbar-appearance-and-shell branch July 10, 2026 04: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.

2 participants