feat: Add BrushIcon to Icon Collection#1502
Conversation
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
There was a problem hiding this comment.
Code Review
This pull request introduces a new BrushIcon component and its corresponding index file. Review feedback indicates that the SVG path provided actually represents a pencil icon rather than a brush, and a correction has been suggested. Additionally, the icon should be exported from the main icons index file to ensure it is accessible throughout the application.
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
|
@rishiraj38 thank you for the suggested changes, I have committed the update |
|
Thank you, @icoderarely. Next time, please add screenshots reflecting the changes to the PR description. Doing so makes the review process much easier. Here is an example of how you can add screenshots. I tested the icon locally inside the Layer5 website, and this is how it looks:
I added the icon with the following code inside layer5 index.js file.
|
|
LGTM, thank you @icoderarely |
|
@Katotodan I was not sure where to test earlier, but I tested the last one and it was a BrushIcon. Thank you for the tip. I usually use PoC videos or screenshots, which I missed it this time. |

Notes for Reviewers
This PR fixes #1489
Signed commits