Conversation
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
1 participant
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.
Hide disabled deprecated providers from the Providers screen by default while keeping enabled deprecated providers visible. A top-right selector offers Current & enabled, All, and Deprecated; text search applies within the selected view. Disabling a deprecated provider removes it from the default view after refresh. All views retain Add Provider, and an empty Deprecated view explains that no deprecated providers exist.
Depends on #12242 for explicit provider enablement (related issues #11331 and #11212). This is a separate follow-up PR requested by James. Until #12242 merges, the main-branch diff includes its prerequisite commit; the filter-only change is commit
0742df1fd30329a7b8b00cda3646ce1b954af9d8. Merge #12242 first and rebase this PR afterward.Validation: all 916 Desktop tests passed with the repository's Node 24 runtime, including four new UI tests for default visibility, All/Deprecated plus search, disabling and refreshing, and the empty state. TypeScript, ESLint for the changed components, locale validation, formatting, and diff checks passed. Added labels to all 16 locale catalogs and updated the self-test recipe. Native Electron visual testing and the agent-driven self-test recipe were not run for this UI follow-up; the parent PR records backend validation and its live-account test limitations.
Fixes #12245.
Screenshots below render the actual ProviderGrid and app theme with sample provider states. They are component previews, not an installed Electron session.
All providers view. Screenshot review exposes existing Configure-button clipping within the fixed-width cards; this PR does not change card actions/layout.