Skip to content

v5.5.7 - #637

Merged
kapantzak merged 2 commits into
masterfrom
feat/group-email-colored-icon
Aug 10, 2026
Merged

v5.5.7#637
kapantzak merged 2 commits into
masterfrom
feat/group-email-colored-icon

Conversation

@car12o

@car12o car12o commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Adds the group_email_colored SVG asset under src/components/icon/assets/integrations/ and registers it as integrationGroupEmailColored in iconsList.js so it is available through the Icon component.

@@ -0,0 +1,4 @@
<svg width="24" height="24" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this tested n the browser in both themes? I think it has too many inline color styling.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, I don't know how I can deploy this. Just craft a simple HTML page to render in the browser, and it looks good 😄
Can you take it from here pls?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yep that's why I asked, I will test it and merge it thanks man

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks man 🙇

@kapantzak kapantzak changed the title feat(icon): add group email colored integration icon v5.5.7 Aug 10, 2026
@CLAassistant

CLAassistant commented Aug 10, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@kapantzak
kapantzak merged commit 8737924 into master Aug 10, 2026
2 of 3 checks passed
@kapantzak
kapantzak deleted the feat/group-email-colored-icon branch August 10, 2026 07:31
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.

4 participants