fix: improve apple intelligence ui and add reusable alert component#517
Merged
cjpais merged 1 commit intocjpais:mainfrom Jan 10, 2026
Merged
Conversation
- hide model dropdown when apple intelligence is selected - move error state into provider section (only shows when unavailable) - remove redundant apple intelligence infobox - add reusable Alert component with variants (error, warning, info, success) - Alert supports contained prop for use inside settings groups
Owner
|
Thank you! |
Contributor
Author
|
@cjpais are you looking for something specific? I wasn't entirely sure what you meant by improving the UI, so i went with my gut and made fixes to things-i-wanted-to-change |
Owner
|
I'll probably just need to take a look at it myself, this fixes most of my pain points I think I just wanted to get the primary fix out since I got some emails I think that PR will solve |
h0lybyte
added a commit
to KBVE/Handy
that referenced
this pull request
Jan 12, 2026
…, and UI improvements Merged upstream features: - feat: add Czech translation (cjpais#568) - fix: keybinding changes failing silently due to incorrect key ordering (cjpais#524) - fix: improve apple intelligence ui and add reusable alert component (cjpais#517) - Fix race condition when toggling transcription via SIGUSR2 (cjpais#560) - feat(linux): add ydotool support for text input (cjpais#557) - feat: add Moonshine Base speech recognition model (cjpais#556) - Prevent highlight and selection cursor hover on UI text items (cjpais#541) - feat(ui): add reusable Tooltip component and integrate with settings (cjpais#538) - PR guidelines for AI - dont allow package-lock with a bun project for now Conflicts resolved: - src-tauri/src/shortcut.rs: Kept KBVE filler detection commands, adopted upstream's improved doc comment - src/bindings.ts: Merged KBVE types (Discord, Memory, Filler) with upstream's Moonshine engine type
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Before Submitting This PR
Please confirm you have done the following:
Human Written Description
containedto handle border radius, added icon(s), and an alert UI component to standardize approachRelated Issues/Discussions
Community Feedback
requested by @cjpais in #473
Testing
Screenshots/Videos (if applicable)