Skip to content

feat: move settings document action and context meter, tighten composer - #357

Open
KinyooZ wants to merge 1 commit into
mainfrom
feat/settings-composer-layout-tweaks
Open

feat: move settings document action and context meter, tighten composer#357
KinyooZ wants to merge 1 commit into
mainfrom
feat/settings-composer-layout-tweaks

Conversation

@KinyooZ

@KinyooZ KinyooZ commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Settings dialog: the "Open configuration file" action now sits at the bottom of the left nav rail instead of the content header (new patch for dsh-client-ui-settings-general).
  • Composer: the context-usage ring moves from the trailing controls into the stats row under the card, centred under the send button; the "+" button loses its resting background; the gap between "+" and the mode selectors is 4px (updated dsh-client-ui-conversation patch).
  • Patch contract tests added for both changes.

Both packages are only available as local tarballs, so the patches were generated by diffing the pristine tarball files and verified by re-applying with patch-package.

Test plan

  • npx vitest run — 91 files / 748 tests pass
  • npm run typecheck
  • Dev app: settings dialog button at bottom-left; composer ring aligned under send button; "+" flat; 4px gap

🤖 Generated with Claude Code

- Settings dialog: render the `settings.action` slot (Open configuration
  file) at the bottom of the nav rail instead of the content header.
- Composer: move the context meter from the trailing controls into the
  dock row under the card, aligned with the primary button; drop the
  resting background on the "+" button; tighten the tools gap to 4px.
- Add patch contract tests for both upstream patches.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@KinyooZ
KinyooZ force-pushed the feat/settings-composer-layout-tweaks branch 2 times, most recently from e6dbfae to f587f82 Compare September 10, 2026 06:25
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