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

feat(@clayui/css): Add new icons: content-shield, scim, shield-asterisk #5804

Merged
merged 2 commits into from
Apr 16, 2024

Conversation

SelenaAungst
Copy link
Member

Copy link
Member

@pat270 pat270 left a comment

Choose a reason for hiding this comment

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

Lastly, we should use tabs instead of spaces to indent code.

* SPDX-License-Identifier: BSD-3-Clause
-->
<svg viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg">
<path fill-rule="evenodd" clip-rule="evenodd" d="M0 1.625V11.375C0 12.2713 0.728394 13 1.625 13H7V11.375H1.625V1.625H11.375V6H13V1.625C13 0.728716 12.2716 0 11.375 0H1.625C0.728394 0 0 0.728716 0 1.625ZM9.75 4.0625C9.75 4.95997 9.0224 5.6875 8.125 5.6875C7.2276 5.6875 6.5 4.95997 6.5 4.0625C6.5 3.16503 7.2276 2.4375 8.125 2.4375C9.0224 2.4375 9.75 3.16503 9.75 4.0625ZM3.25 5.6875L4.875 2.4375L6.5 5.6875H3.25ZM7.00002 9V10H3.50007C2.81257 10 2.84382 9 3.50007 9H7.00002ZM7.00002 8V7.42425C6.999 7.29597 7.03622 7.17011 7.10724 7.06172C7.12131 7.04026 7.13658 7.01966 7.15296 7H3.5C2.84375 7 2.8125 8 3.5 8H7.00002Z" fill="#6B6C7E"/>
Copy link
Member

Choose a reason for hiding this comment

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

@SelenaAungst we need to remove fill="#6B6C7E" from the paths. This prevents us from changing the color of the icon.

*
* SPDX-License-Identifier: BSD-3-Clause
-->
<svg viewBox="0 0 16 16" xmlns="http://www.w3.org/2000/svg">
Copy link
Member

Choose a reason for hiding this comment

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

Right now viewBox="0 0 16 16" won't work with the clay-icon tag in DXP. It's hardcoded to be viewBox="0 0 512 512". We need the viewBox="0 0 512 512" version until we can convert everything to 0 0 16 16.

@SelenaAungst SelenaAungst force-pushed the LPD-21829 branch 2 times, most recently from 4da249d to 9914dbe Compare April 15, 2024 18:12
@SelenaAungst
Copy link
Member Author

Hi @pat270,
I tried changing the viewBox values to 512, should this be manually updated in icons.svg as well or is there a way to update automatically? Additionally, where are there spaces that need to be changed to tabs?
Thanks for reviewing this!

@pat270
Copy link
Member

pat270 commented Apr 15, 2024

@SelenaAungst we can't change the viewBox manually here. We need Lexicon to provide us with svg's that are 512 x 512.

* SPDX-License-Identifier: BSD-3-Clause
-->
<svg viewBox="0 0 512 512" xmlns="http://www.w3.org/2000/svg">
<path fill-rule="evenodd" clip-rule="evenodd" d="M0 1.625V11.375C0 12.2713 0.728394 13 1.625 13H7V11.375H1.625V1.625H11.375V6H13V1.625C13 0.728716 12.2716 0 11.375 0H1.625C0.728394 0 0 0.728716 0 1.625ZM9.75 4.0625C9.75 4.95997 9.0224 5.6875 8.125 5.6875C7.2276 5.6875 6.5 4.95997 6.5 4.0625C6.5 3.16503 7.2276 2.4375 8.125 2.4375C9.0224 2.4375 9.75 3.16503 9.75 4.0625ZM3.25 5.6875L4.875 2.4375L6.5 5.6875H3.25ZM7.00002 9V10H3.50007C2.81257 10 2.84382 9 3.50007 9H7.00002ZM7.00002 8V7.42425C6.999 7.29597 7.03622 7.17011 7.10724 7.06172C7.12131 7.04026 7.13658 7.01966 7.15296 7H3.5C2.84375 7 2.8125 8 3.5 8H7.00002Z"/>
Copy link
Member

Choose a reason for hiding this comment

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

@SelenaAungst these indent's are using spaces. It should be tabs. Your code editor should have something that can convert to tabs.

@SelenaAungst
Copy link
Member Author

Thanks for pointing this out! I think I resolved those issues, please let me know if there is anything else I can add/change for this @pat270

@pat270
Copy link
Member

pat270 commented Apr 16, 2024

@SelenaAungst I'm getting this

shield-icons

@pat270
Copy link
Member

pat270 commented Apr 16, 2024

@marcoscv-work can we get these icons in 512 x 512? I'm still working on the 16x16. There are some inconsistencies in the icons we need to work through together.

@SelenaAungst
Copy link
Member Author

@pat270 hmm yeah I am not seeing them now either after changing to 512

@pat270
Copy link
Member

pat270 commented Apr 16, 2024

@SelenaAungst I was able to change the size export them from Lexicon's Icon Figma Library.

@SelenaAungst
Copy link
Member Author

Thanks @pat270!

@pat270 pat270 merged commit 92a194b into liferay:master Apr 16, 2024
4 checks passed
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.

None yet

2 participants