Skip to content

feat(vscode): make Show more providers prominent and collapse disabled providers - #11881

Merged
chrarnoldus merged 5 commits into
mainfrom
feat/providers-tab-show-more
Jul 2, 2026
Merged

feat(vscode): make Show more providers prominent and collapse disabled providers#11881
chrarnoldus merged 5 commits into
mainfrom
feat/providers-tab-show-more

Conversation

@chrarnoldus

Copy link
Copy Markdown
Collaborator

What

Improves the Providers settings tab layout:

  • "Show more providers" is now a prominent row appended to the bottom of the Popular providers list, with a providers icon and a chevron, instead of a low-key ghost text link below the card.
  • Disabled Providers is now a ghost Collapsible that is collapsed by default and rendered with a smaller, muted trigger, keeping the focus on active/connected providers.

Why

The full provider catalog was hard to discover behind an unobtrusive text link, while the Disabled Providers section took up prominent space despite being rarely used. This rebalances visual weight toward the actions users take most often.

Notes

  • Button label (dialog.provider.viewAll = "Show more providers") is unchanged, so existing docs referencing it remain accurate.
  • No screenshots were removed.

…roviders

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Jul 2, 2026
github-actions Bot and others added 2 commits July 2, 2026 09:20
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-vscode/webview-ui/src/stories/settings.stories.tsx - adds a ProvidersDisabledExpanded story that opens the Disabled Providers collapsible on mount, resolving the previous suggestion about missing expanded-state Storybook/visual coverage
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/settings/providers-disabled-expanded-chromium-linux.png - new CI-generated baseline for the story above, no issues
Previous Review Summary (commit 21e958b)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 21e958b)

Status: 2 Issues Found | Recommendation: Nice-to-have, safe to merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 2
Issue Details (click to expand)

SUGGESTION

File Line Issue
packages/kilo-vscode/webview-ui/src/components/settings/ProvidersTab.tsx 393 Raw <button> with inline styles bypasses kilo-ui Button (no full-width/space-between variant exists yet; similar precedent exists in PermissionEditor.tsx)
packages/kilo-vscode/webview-ui/src/components/settings/ProvidersTab.tsx 429 No Storybook/visual coverage for the expanded Disabled Providers state; only the closed default is captured in the updated baseline
Files Reviewed (3 files)
  • .changeset/providers-tab-show-more.md - no issues
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/settings/providers-configure-chromium-linux.png - no issues (baseline regenerated, matches layout change)
  • packages/kilo-vscode/webview-ui/src/components/settings/ProvidersTab.tsx - 2 issues

Fix these issues in Kilo Cloud


Reviewed by claude-sonnet-5-20260630 · Input: 36 · Output: 6.1K · Cached: 860.7K

Review guidance: REVIEW.md from base branch main

chrarnoldus and others added 2 commits July 2, 2026 09:48
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus
chrarnoldus merged commit 1d714c6 into main Jul 2, 2026
23 checks passed
@chrarnoldus
chrarnoldus deleted the feat/providers-tab-show-more branch July 2, 2026 11:29
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…ow-more

feat(vscode): make Show more providers prominent and collapse disabled providers
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.

2 participants