Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DS-590 new icons for Pega sites #2411

Merged
merged 4 commits into from
Jan 5, 2022

Conversation

mikemai2awesome
Copy link
Collaborator

@mikemai2awesome mikemai2awesome commented Dec 22, 2021

Jira

https://pegadigitalit.atlassian.net/browse/DS-590

Summary

New icons added to represent the network of Pega sites.

Details

SVG files are added for the new icons.

How to test

Run the branch locally and check out the icon element docs. Make sure all new icons are rendering without any kinks.

Release notes

Visual changes

customer-service icon has been updated to match brand icon. The double chat bubbles are slightly different from before.

@github-actions github-actions bot added the type: feature List this PR in the 'Features' section of the release notes. label Dec 22, 2021
@colbytcook colbytcook temporarily deployed to feature/ds-590-icons-for-pega-sites--branch-preview December 22, 2021 20:24 Inactive
@colbytcook colbytcook temporarily deployed to feature/ds-590-icons-for-pega-sites--branch-preview December 23, 2021 12:58 Inactive
Copy link
Collaborator

@danielamorse danielamorse left a comment

Choose a reason for hiding this comment

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

@mikemai2awesome the docs say to use #151619 as the fill color. This PR uses #151618. Want to update before we merge this?

@mikemai2awesome
Copy link
Collaborator Author

@danielamorse Fixed.

@danielamorse danielamorse self-requested a review January 4, 2022 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature List this PR in the 'Features' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants