Skip to content

Prevent duplicate Model buttons and keep the picker on-screen - #91

Merged
chee merged 3 commits into
mainfrom
copilot/fix-model-button-dialogue-issue
Oct 18, 2026
Merged

chee merged 3 commits into
mainfrom
copilot/fix-model-button-dialogue-issue

Conversation

Copilot AI commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Refreshing could add another 🤖 Model button to the system tray. The Model picker could also open above the visible screen.

  • Tray entries: Remove duplicate tool IDs from saved tray configuration and render each ID only once, preserving the first entry’s position.
  • Picker placement: Open above or below the button based on available viewport space, with its height constrained to fit.
  • Regression coverage: Add focused tests for tray deduplication and picker placement.

Copilot AI and others added 2 commits October 17, 2026 00:56
Co-authored-by: chee <178266+chee@users.noreply.github.com>
Co-authored-by: chee <178266+chee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix model button dialogue opening off-screen Prevent duplicate Model buttons and keep the picker on-screen Oct 17, 2026
Copilot AI requested a review from chee October 17, 2026 00:58
@chee
chee marked this pull request as ready for review October 18, 2026 21:02
@patchcrow

patchcrow commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Patchwork glance ready

https://patchwork-preview-91--patchwork-base.netlify.app

Patchwork from inkandswitch/patchwork.inkandswitch.com, packages from this PR at 43e3c85.

build-info.json

@chee
chee merged commit 755a42e into main Oct 18, 2026
7 checks passed
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.

every time i refresh i seem to have a new 🤖 model button?

3 participants