Conversation
Replace the dark charcoal (#212121) primary and purple (#C177FC) accent with the landing page's blue (#2563eb) for consistent branding. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This pull request updates the accent color from purple (#C177FC) to blue (#2563eb) across the application's theme configuration. However, the PR title "update primary color to blue" is misleading, as the actual changes modify the accentedPalette configuration, not the primaryPalette. Additionally, there are critical inconsistencies in how the color changes are applied between different files.
Changes:
- Updated
accentedPalettein main.ts from purple (#C177FC) to blue (#2563eb) - Modified three color fallback configurations in connections.service.ts, incorrectly changing both
primaryPaletteandaccentedPaletteto blue
Reviewed changes
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| frontend/src/main.ts | Updates the default accentedPalette color configuration from purple to blue |
| frontend/src/app/services/connections.service.ts | Updates fallback color configurations when no connection properties exist, but incorrectly changes primaryPalette values |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| palettes: { | ||
| primaryPalette: '#2563eb', | ||
| primaryPalette: '#212121', | ||
| accentedPalette: '#2563eb', |
There was a problem hiding this comment.
The PR title states "update primary color to blue", but this change updates the accentedPalette from '#C177FC' (purple) to '#2563eb' (blue), not the primaryPalette which remains '#212121' (dark gray/black). This creates a discrepancy between the PR description and the actual implementation. Consider updating the PR title to "Update accent color to blue" or similar to accurately reflect the change.
…s service The runtime updateColors calls were overriding primaryPalette with blue (#2563eb) instead of the intended charcoal (#212121). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 participant
No description provided.