Skip to content

Tool launcher ux#113

Merged
dantheuber merged 2 commits into
mainfrom
tool-launcher-ux
May 10, 2026
Merged

Tool launcher ux#113
dantheuber merged 2 commits into
mainfrom
tool-launcher-ux

Conversation

@dantheuber
Copy link
Copy Markdown
Owner

Summary

  • Tool Launcher: Smarter default selection and clearer template conflict feedback in QuickClipsScanner.

Added

  • "Select all / deselect all" checkbox with indeterminate state in the Found Patterns header.
  • quickClipsSelection.ts helper module with computeInitialSelection and computeAmbiguousGroups, plus unit tests in QuickClipsScanner.test.tsx.
  • Inline warning (with triangle-exclamation icon) on templates whose tokens map to multiple selected values, naming the conflicting groups.
  • Auto-expand of the Tools accordion section when tools become available.

Changed

  • Initial capture selection: when >5 patterns are found, none are pre-selected; otherwise only singleton groups are pre-selected so the user explicitly picks among duplicates.
  • Templates with ambiguous token bindings are now disabled (with tooltip + inline message) instead of silently using an arbitrary value.
  • Disabled template button cursor switched from wait to not-allowed.
  • Section title row restructured to host the select-all control alongside the heading.

Fixed

  • Ambiguity in template generation when multiple values were captured for the same named group.

Removed

  • Local CaptureItem interface in QuickClipsScanner.tsx (now imported from the shared selection helper).

@github-actions
Copy link
Copy Markdown

📊 Coverage Report

Metric Coverage
Statements 100%
Branches 100%
Functions 100%
Lines 100%

@github-actions
Copy link
Copy Markdown

🧪 E2E Test Results

18 passed, 0 failed, 0 skipped

Test Suite Result
App Launch
Clipboard
Context Menu
Image Clipboard
Quick Clips
Settings
Theme
Settings — Search Terms CRUD
Settings — Quick Tools CRUD
Settings — Templates CRUD
Tools Launcher — Pattern Scanning
Tools Launcher — Clip Templates

@dantheuber dantheuber merged commit 274e931 into main May 10, 2026
3 checks passed
@dantheuber dantheuber deleted the tool-launcher-ux branch May 10, 2026 21:54
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