Skip to content

feat(tui): trigger skill popup on '/' in edit mode instead of command…#57

Merged
iohub merged 1 commit into
mainfrom
feat-openai-go
May 8, 2026
Merged

feat(tui): trigger skill popup on '/' in edit mode instead of command…#57
iohub merged 1 commit into
mainfrom
feat-openai-go

Conversation

@iohub
Copy link
Copy Markdown
Owner

@iohub iohub commented May 8, 2026

… mode

  • Add showSkillPopup/skillPopupIdx fields to model
  • Remove :skill-name matching logic from processCommand()
  • Intercept '/' key in edit mode to show skill selection popup
  • Add skill popup navigation (j/k/up/down/enter/esc)
  • Render skill popup as centered overlay with lipgloss.Place + RoundedBorder
  • Update i18n tips: remove ::cmd from command mode, update help/edit tips
  • Update help dialog skills section to reflect new '/' trigger
  • Fix popup auto-scrolling by adding global popup guard in Update() to block non-KeyMsg messages (tickMsg, taskEventMsg, WindowSizeMsg) when any overlay is visible

… mode

- Add showSkillPopup/skillPopupIdx fields to model
- Remove :skill-name matching logic from processCommand()
- Intercept '/' key in edit mode to show skill selection popup
- Add skill popup navigation (j/k/up/down/enter/esc)
- Render skill popup as centered overlay with lipgloss.Place + RoundedBorder
- Update i18n tips: remove ::cmd from command mode, update help/edit tips
- Update help dialog skills section to reflect new '/' trigger
- Fix popup auto-scrolling by adding global popup guard in Update()
  to block non-KeyMsg messages (tickMsg, taskEventMsg, WindowSizeMsg)
  when any overlay is visible
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @iohub, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@iohub iohub merged commit 62a7bd7 into main May 8, 2026
1 check 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.

1 participant