Skip to content

Add gridon Icon to sistent#1473

Merged
ritzorama merged 3 commits intolayer5io:masterfrom
indevi:gridon
May 3, 2026
Merged

Add gridon Icon to sistent#1473
ritzorama merged 3 commits intolayer5io:masterfrom
indevi:gridon

Conversation

@indevi
Copy link
Copy Markdown
Contributor

@indevi indevi commented May 3, 2026

Notes for Reviewers

This PR fixes #1471

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Shivam Tyagi <tyagishivam.dev@gmail.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the GridOnIcon component and updates the icon exports. The review feedback suggests using DEFAULT_FILL_NONE for the fill prop to allow the icon to inherit text color and adding a trailing newline to the GridOnIcon.tsx file for standard compliance.

Comment thread src/icons/GridOn/GridOnIcon.tsx Outdated
Comment thread src/icons/GridOn/GridOnIcon.tsx Outdated
indevi and others added 2 commits May 4, 2026 00:10
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: shivam tyagi <tyagishivam.dev@gmail.com>
Copy link
Copy Markdown
Contributor

@ritzorama ritzorama left a comment

Choose a reason for hiding this comment

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

Thanks, @indevi!

@ritzorama ritzorama merged commit 15f2357 into layer5io:master May 3, 2026
5 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.

[Feature] Add GridOnIcon to Sistent

2 participants