Skip to content

Conversation

AAJELLAL
Copy link
Contributor

No description provided.

AAJELLAL and others added 6 commits May 15, 2025 17:29
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
AAJELLAL added 3 commits May 21, 2025 18:06
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
Signed-off-by: AAJELLAL <ali.ajellalod@gmail.com>
Copy link

const theme = useTheme();

const defaultStyle: React.CSSProperties = {
width: 15,
Copy link
Contributor

Choose a reason for hiding this comment

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

why 15 ? why not 24 has other icons ????

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the default value I chose based on the Figma, you can always override it.
For the other components, the value was set based on Figma with no option to override it.

@AAJELLAL AAJELLAL requested a review from sBouzols May 23, 2025 09:19
@AAJELLAL AAJELLAL merged commit 147a517 into main May 23, 2025
5 checks passed
@AAJELLAL AAJELLAL deleted the root-network-icon branch May 23, 2025 12:53
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.

3 participants