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: add light themed icons for digest #3388

Merged
merged 5 commits into from
May 11, 2023

Conversation

davidsoderberg
Copy link
Contributor

Screenshot 2023-05-09 at 15 16 30 Screenshot 2023-05-10 at 15 23 51 Screenshot 2023-05-11 at 06 33 02 Screenshot 2023-05-11 at 06 33 09

Copy link
Contributor

@ainouzgali ainouzgali left a comment

Choose a reason for hiding this comment

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

@davidsoderberg Could we maybe move all the icons to design-system/icons? even if we create a digest folder in icons I think it will be nicer - so we can easily reuse and to have all icons in the same place

@ainouzgali
Copy link
Contributor

Its not possible to reuse the dark one with changing the stroke/fill etc?

@davidsoderberg
Copy link
Contributor Author

@davidsoderberg Could we maybe move all the icons to design-system/icons? even if we create a digest folder in icons I think it will be nicer - so we can easily reuse and to have all icons in the same place

Not sure these will be reused as other because they are very specific for this location that is why I did not put them i design-system. I agree that we could have an icons folder in digest thu 😄


if (colorScheme === 'light') {
return (
<svg xmlns="http://www.w3.org/2000/svg" width="280" height="57" viewBox="0 0 280 57" fill="none">
Copy link
Contributor

Choose a reason for hiding this comment

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

please run all the svg's through this tool: https://jakearchibald.github.io/svgomg/, to make them smaller in size

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will it not ruin the quality?

Copy link
Contributor

Choose a reason for hiding this comment

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

nope, because it's svg's vector graphics :) it just removes unnecessary staff and improved the paths

@davidsoderberg davidsoderberg merged commit fded1bd into timed-digest May 11, 2023
21 of 22 checks passed
@davidsoderberg davidsoderberg deleted the update-digest-icons-light-theme branch May 11, 2023 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants