feat: Add BrushIcon to Icon Collection#1502
feat: Add BrushIcon to Icon Collection#1502icoderarely wants to merge 3 commits intolayer5io:masterfrom
Conversation
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
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.
| xmlns="http://www.w3.org/2000/svg" | ||
| {...props} | ||
| > | ||
| <path d="M20.71 7.04c.39-.39.39-1.02 0-1.41l-2.34-2.34c-.39-.39-1.02-.39-1.41 0l-1.83 1.83 3.75 3.75 1.83-1.83zM3 17.25V21h3.75L17.81 9.94l-3.75-3.75L3 17.25z" /> |
There was a problem hiding this comment.
The SVG path provided corresponds to the 'Edit' (pencil) icon, which is already present in the icon collection. If the intention is to add a 'Brush' icon, please use the correct path from the Material Design set.
| <path d="M20.71 7.04c.39-.39.39-1.02 0-1.41l-2.34-2.34c-.39-.39-1.02-.39-1.41 0l-1.83 1.83 3.75 3.75 1.83-1.83zM3 17.25V21h3.75L17.81 9.94l-3.75-3.75L3 17.25z" /> | |
| <path d="M7 14c-1.1 0-2 .9-2 2 0 1.1.9 2 2 2s2-.9 2-2c0-1.1-.9-2-2-2zm3.71-3.12c-.39-.39-1.02-.39-1.41 0L3.27 16.9c-.39.39-.39 1.02 0 1.41.39.39 1.02.39 1.41 0l6.03-6.03c.38-.39.38-1.03 0-1.4zM21 3h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V4c0-.55-.45-1-1-1zm-8 4h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V8c0-.55-.45-1-1-1z" /> |
| @@ -0,0 +1 @@ | |||
| export { default as BrushIcon } from './BrushIcon'; | |||
There was a problem hiding this comment.
The BrushIcon is not exported from the main src/icons/index.ts file. Please add export * from './BrushIcon'; to the root index file to ensure the new icon is accessible to other components.
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
Signed-off-by: Navneet Anand <itworksnavneet@gmail.com>
| xmlns="http://www.w3.org/2000/svg" | ||
| {...props} | ||
| > | ||
| <path d="M7 14c-1.1 0-2 .9-2 2 0 1.1.9 2 2 2s2-.9 2-2c0-1.1-.9-2-2-2m13.71-6.04c.39-.39.39-1.02 0-1.41l-2.34-2.34c-.39-.39-1.02-.39-1.41 0l-1.83 1.83 3.75 3.75zM12.12 8.46 3 17.59V21h3.41l9.12-9.12-3.41-3.42zM21 3h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V4c0-.55-.45-1-1-1m-8 4h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V8c0-.55-.45-1-1-1z" /> |
There was a problem hiding this comment.
Have you tested this with the UI? I think this isn’t the intended icon.
| <path d="M7 14c-1.1 0-2 .9-2 2 0 1.1.9 2 2 2s2-.9 2-2c0-1.1-.9-2-2-2m13.71-6.04c.39-.39.39-1.02 0-1.41l-2.34-2.34c-.39-.39-1.02-.39-1.41 0l-1.83 1.83 3.75 3.75zM12.12 8.46 3 17.59V21h3.41l9.12-9.12-3.41-3.42zM21 3h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V4c0-.55-.45-1-1-1m-8 4h-3c-.55 0-1 .45-1 1v3c0 .55.45 1 1 1h3c.55 0 1-.45 1-1V8c0-.55-.45-1-1-1z" /> | |
| <path d="M7 14c-1.66 0-3 1.34-3 3 0 1.31-1.16 2-2 2 .92 1.22 2.49 2 4 2 2.21 0 4-1.79 4-4 0-1.66-1.34-3-3-3zm13.71-9.37l-1.34-1.34c-.39-.39-1.02-.39-1.41 0L9 12.25 11.75 15l8.96-8.96c.39-.39.39-1.02 0-1.41z" /> |
2 participants
Notes for Reviewers
This PR fixes #1489
Signed commits