Skip to content

Fix/dark mode provider logos - #6299

Open
Aman-goel-04 wants to merge 3 commits into
Agenta-AI:mainfrom
Aman-goel-04:fix/dark-mode-provider-logos
Open

Fix/dark mode provider logos#6299
Aman-goel-04 wants to merge 3 commits into
Agenta-AI:mainfrom
Aman-goel-04:fix/dark-mode-provider-logos

Conversation

@Aman-goel-04

@Aman-goel-04 Aman-goel-04 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes provider/LLM icon colors (AlephAlpha, Anthropic, Groq, OpenAI, OpenRouter, Perplexity, Replicate, Together, XAI) so they render correctly in dark mode. Previously these icons used hardcoded fill/stroke colors that were tuned for light mode only, making them low-contrast or visually broken against a dark background.

The fix updates the SVG color values in each icon component to respect the current theme rather than assuming a light background.

Closes #6275

Testing

Verified locally

Ran the app locally via pnpm dev-oss with the full Docker Compose backend stack up, opened the "Add a provider" panel, and visually confirmed all 9 updated provider logos render with correct contrast in both light and dark mode (see screenshots below).

Added or updated tests

N/A — this is a visual/styling fix to static icon components with no logic changes, so no new automated coverage was needed.

QA follow-up

N/A — verified directly in the real "Add a provider" screen in both light and dark mode.

Demo

Before (SS from the issue #6275's description):
image

After:
Light mode:
Screenshot 2026-08-27 at 2 20 08 AM

Dark mode:
Screenshot 2026-08-27 at 1 42 44 AM

Checklist

  • Demo shows the real app running this branch (not a mock-up or recreated UI), or is marked N/A
  • Relevant tests pass locally
  • Relevant linting and formatting pass locally
  • I have signed the CLA, or I will sign it when the bot prompts me

Copilot AI lite review requested due to automatic review settings August 26, 2026 20:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Aman-goel-04 is attempting to deploy a commit to the agenta projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: d6b86225-b9e2-4dd1-b444-96d91a34ecb0

📥 Commits

Reviewing files that changed from the base of the PR and between 6d93cca and 3c3ed6a.

📒 Files selected for processing (1)
  • web/packages/agenta-ui/src/LLMIcons/assets/Anthropic.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • web/packages/agenta-ui/src/LLMIcons/assets/Anthropic.tsx

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Style
    • Updated AI provider icons to inherit the surrounding text color.
    • Improved icon consistency across different themes and interface contexts.

Walkthrough

Nine LLM provider icons now use currentColor instead of fixed black fills. Their SVG paths inherit the surrounding text color.

Changes

LLM icon color updates

Layer / File(s) Summary
Provider icon fill updates
web/packages/agenta-ui/src/LLMIcons/assets/{AlephAlpha,Anthropic,Groq,OpenAi,OpenRouter,Perplexity,Replicate,Together,XAI}.tsx
The SVG paths now use currentColor instead of black.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 3c3ed

This change updates provider logo colors for correct light- and dark-mode contrast without changing application logic, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The changes update all nine icons identified in issue #6275 to use theme-aware colors, which addresses the dark-mode legibility requirement.
Out of Scope Changes check ✅ Passed All changes are limited to provider icon color updates and related formatting. No unrelated changes are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 9…
Title check ✅ Passed The title clearly identifies the main change: fixing provider logos for dark mode.
Description check ✅ Passed The description accurately explains the nine icon updates, dark-mode objective, validation steps, and lack of automated tests for this visual change.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 9 files.

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: ef5030f8-d205-4ca4-8d81-c0db9a95e82a

📥 Commits

Reviewing files that changed from the base of the PR and between e24609c and 6d93cca.

📒 Files selected for processing (9)
  • web/packages/agenta-ui/src/LLMIcons/assets/AlephAlpha.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/Anthropic.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/Groq.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/OpenAi.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/OpenRouter.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/Perplexity.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/Replicate.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/Together.tsx
  • web/packages/agenta-ui/src/LLMIcons/assets/XAI.tsx

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Comment thread web/packages/agenta-ui/src/LLMIcons/assets/Anthropic.tsx Outdated
Copilot AI review requested due to automatic review settings August 26, 2026 21:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

(bug) Some provider logos are invisible in dark mode

2 participants