Skip to content

Adding People icon#1275

Merged
Rajesh-Nagarajan-11 merged 7 commits intolayer5io:masterfrom
Katotodan:feat/icons/people-icon
Feb 4, 2026
Merged

Adding People icon#1275
Rajesh-Nagarajan-11 merged 7 commits intolayer5io:masterfrom
Katotodan:feat/icons/people-icon

Conversation

@Katotodan
Copy link
Copy Markdown
Member

Notes for Reviewers
Adding People icon

This PR fixes #1260

Signed commits

  • Yes, I signed my commits.

Katotodan and others added 4 commits January 30, 2026 16:45
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Rajesh-Nagarajan-11
Copy link
Copy Markdown
Member

Any images will be helpful

Copy link
Copy Markdown
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

Why test file here ? , I guess already some test files are there for icons

@Katotodan
Copy link
Copy Markdown
Member Author

@Rajesh-Nagarajan-11 I’ve been having some trouble importing (using) the local sistent package into other local projects.
I tried using both Vite and React (with Create React App), but the React app couldn’t find the elements from my local sistent package.
To verify that the SVG code is correct, I copied the SVG into the browser (using an SVG-only viewer) to check the output.
I also followed the same approach used for the previous icons.
Additionally, I added tests to ensure everything works as expected, and all tests passed.

@saurabhraghuvanshii
Copy link
Copy Markdown
Member

@Katotodan test with layer5 repo

@Katotodan
Copy link
Copy Markdown
Member Author

Thanks @saurabhraghuvanshii,
I’ve now been able to test it, and it’s working.
The only issue is that the default fill color is black. If the icon is rendered on a black background, it won’t be visible. I don’t think this is a major issue, but if needed, I can change the fill color to a brighter one.

@Katotodan
Copy link
Copy Markdown
Member Author

Any images will be helpful

I added it inside layer5 IndexPage
Screenshot 2026-01-31 at 12 38 39

Code
Screenshot 2026-01-31 at 12 39 17

@Bhumikagarggg
Copy link
Copy Markdown
Contributor

@Katotodan Thank you for your contribution! Let's discuss this during the website call tomorrow at 6:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Katotodan
Copy link
Copy Markdown
Member Author

Here is the icon without adding any style on it.
Screenshot 2026-02-02 at 12 17 24

The code inside layer5
Screenshot 2026-02-02 at 12 18 24

I added that div container just in order to properly view the image on a white container.

@Rajesh-Nagarajan-11
Copy link
Copy Markdown
Member

@Katotodan Use KEPPEL_GREEN_FILL as Default fill

Signed-off-by: DANIEL KATOTO <katotodan@gmail.com>
@Katotodan
Copy link
Copy Markdown
Member Author

Here is the image of the icon after updating its fill property
Screenshot 2026-02-03 at 19 27 44

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 merged commit 671dc2e into layer5io:master Feb 4, 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 PeopleIcon to Icon Collection

4 participants