Skip to content

Make illustration icons color responsive using accent theme colors#17107

Merged
charlesBochet merged 1 commit intomainfrom
fix/illustration-icons-accent-colors
Jan 12, 2026
Merged

Make illustration icons color responsive using accent theme colors#17107
charlesBochet merged 1 commit intomainfrom
fix/illustration-icons-accent-colors

Conversation

@Bonapara
Copy link
Copy Markdown
Member

Summary

  • Update all illustration icons to use theme.accent.accent3 for fill and theme.accent.accent8 for border
  • Replaces hardcoded IllustrationIcon.blue values with theme-responsive accent colors
  • Icons will now adapt correctly to theme changes

Test plan

  • Navigate to Settings > Data model > select any object
  • Verify the Relations "Type" column icons use the accent colors
  • Verify the Fields "Data type" column icons use the accent colors
  • Switch themes (if available) and verify icons adapt accordingly

Update all illustration icons to use theme.accent.accent3 for fill and
theme.accent.accent8 for border instead of hardcoded IllustrationIcon.blue values
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@Bonapara
Copy link
Copy Markdown
Member Author

image

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 21 files

@charlesBochet charlesBochet merged commit 88d613d into main Jan 12, 2026
67 checks passed
@charlesBochet charlesBochet deleted the fix/illustration-icons-accent-colors branch January 12, 2026 18:24
@twenty-eng-sync
Copy link
Copy Markdown

Hey @charlesBochet! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

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