-
Notifications
You must be signed in to change notification settings - Fork 2.8k
ux: improve Skills and Slash Commands settings UI with multi-mode support #11157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
All previously flagged issues have been resolved. No new issues found in this review.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
All previously flagged issues have been resolved. No new issues found in this review.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
Motivation
The Skills and Slash Commands settings panels had inconsistent UIs and skills could only be associated with a single mode. This PR improves the UX consistency and adds flexibility for skill mode targeting. Closes EXT-725
Key Changes
Multi-mode Support for Skills
modeSlugsarray in frontmatter (replaces singlemodefield)SkillsManager.updateSkillModes()method to update mode associations without recreating skillsUnified UI for Skills and Slash Commands
SlashCommandsSettingsto matchSkillsSettingsvisual patternCreateSlashCommandDialogcomponent with consistent dialog stylingUI Components
Checkbox,Input, andTextareafrom@/components/uifor use across settings panelsi18n
Technical Notes
modefield is deprecated but still supportedmodeSlugstakes priority overmodein frontmatter parsingskills/directory (mode info in frontmatter)