Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/144 oss icons #176

Merged
merged 13 commits into from
Nov 24, 2023
Merged

Feat/144 oss icons #176

merged 13 commits into from
Nov 24, 2023

Conversation

Ademsk1
Copy link
Contributor

@Ademsk1 Ademsk1 commented Nov 23, 2023

Fixes #144

Also I added an mdx file to list out all the icons, renamed every single icon to be more clear, moved over the index file into components so our src index file can be more concise

@Ademsk1 Ademsk1 marked this pull request as draft November 23, 2023 17:55
Copy link

@review-bot-for-github review-bot-for-github bot left a comment

Choose a reason for hiding this comment

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

Looks good! No comments.

@Ademsk1 Ademsk1 marked this pull request as ready for review November 24, 2023 09:39
Copy link

@review-bot-for-github review-bot-for-github bot left a comment

Choose a reason for hiding this comment

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

Looks good! No comments.

@Ademsk1 Ademsk1 requested a review from swdilip November 24, 2023 09:39
@swdilip
Copy link
Contributor

swdilip commented Nov 24, 2023

The icons look great! Some of them are a bit thicker or thinner than the others. Such as UserGroupIcon vs CircleOutlineIcon. Is this how it's meant to be?

Also in dark mode the Icon's themselves don't change colour and are instead embedded inside a 'white' box which is not visible in light mode. I am also not sure how the icons are meant to be implemented for dark mode 😓.

Light Mode Dark Mode
image image

@Ademsk1
Copy link
Contributor Author

Ademsk1 commented Nov 24, 2023

@swdilip Some icons are meant to be bigger than others, hence the difference in thickness.

For dark mode, we allow the user to pass in props to the icons, so that they may alter the colors directly :D

Copy link
Contributor

@swdilip swdilip left a comment

Choose a reason for hiding this comment

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

Looks good to me! LGTM!

@Ademsk1 Ademsk1 merged commit a7d9deb into main Nov 24, 2023
3 checks passed
@Ademsk1 Ademsk1 deleted the feat/144_oss_icons branch November 24, 2023 17:23
@github-actions github-actions bot mentioned this pull request Nov 28, 2023
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.

[Component] Oss Icons
2 participants