Skip to content

Conversation

@Katotodan
Copy link
Member

Notes for Reviewers

This PR fixes #1262

Image of the icon inside layer5 website
Screenshot 2026-02-06 at 11 22 32

Code
Screenshot 2026-02-06 at 11 22 50

Signed commits

  • Yes, I signed my commits.

Katotodan and others added 2 commits February 6, 2026 11:24
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Rajesh-Nagarajan-11
Copy link
Member

You missed one another index.ts where we export this icon

@Katotodan
Copy link
Member Author

Hello Mr. @Rajesh-Nagarajan-11,
In this case, there was already an icons/Cloud subdirectory, which isn’t always present for new icons. Inside that folder, there were two files: CloudSavedIcon.tsx and index.ts.
I added CloudItem.tsx to the icons/Cloud subdirectory because it belongs to the same icon family as CloudSavedIcon.tsx, and made both to be exported through the same index.ts file.
Additionally, everything inside the icons/Cloud subdirectory was already being exported from icons/index.ts, so the new icon did not need to be exported again there.

I double check if that icon where working inside layer5 and yes, it was working and accessible.
Please let me know if I misunderstood your point or missed anything.
Thank you!

@Katotodan
Copy link
Member Author

Katotodan commented Feb 7, 2026

I have tried to test those two Cloud icons if they are still both working locally and yes, they are all working.
Screenshot 2026-02-07 at 11 51 19
Screenshot 2026-02-07 at 11 51 05

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit e3b4cf4 into layer5io:master Feb 8, 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 CloudIcon to Icon Collection

2 participants