Skip to content

[Icon] Add CloudUploadIcon to Sistent#1469

Merged
saurabhraghuvanshii merged 4 commits intolayer5io:masterfrom
rishiraj38:feat/add-cloudupload-icon
May 3, 2026
Merged

[Icon] Add CloudUploadIcon to Sistent#1469
saurabhraghuvanshii merged 4 commits intolayer5io:masterfrom
rishiraj38:feat/add-cloudupload-icon

Conversation

@rishiraj38
Copy link
Copy Markdown
Member

@rishiraj38 rishiraj38 commented May 2, 2026

Notes for Reviewers

This PR fixes #1465

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Rishi Raj <rishiraj438gt@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 CloudUploadIcon component along with its corresponding export files. The review feedback suggests improving the SVG's robustness by ensuring critical attributes like viewBox cannot be overridden via props and enhancing accessibility by conditionally applying role and aria-hidden attributes based on the presence of a title.

Comment thread src/icons/CloudUpload/CloudUploadIcon.tsx
Comment thread src/icons/CloudUpload/CloudUploadIcon.tsx Outdated
rishiraj38 and others added 2 commits May 3, 2026 00:39
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Rishi Raj <180109771+rishiraj38@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Rishi Raj <180109771+rishiraj38@users.noreply.github.com>
Signed-off-by: Rishi Raj <rishiraj438gt@gmail.com>
@saurabhraghuvanshii saurabhraghuvanshii merged commit 7618f3a 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 CloudUploadIcon to Sistent

2 participants