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

Delete unused icons & stroke mode #9745

Merged
merged 2 commits into from
Jan 7, 2025
Merged

Delete unused icons & stroke mode #9745

merged 2 commits into from
Jan 7, 2025

Conversation

pinotalexandre
Copy link
Collaborator

Description

Cleaning Sparkle - Deleting unused icons and stroke mode.
An exception has been made for "Star" that is used as a toggle button for favorite, so I moved it to the solid folder but with a different name.

Deleted theses :
image

Risk

Break UI

Deploy Plan

Merge PR
Deploy Sparkle

@pinotalexandre pinotalexandre changed the title Deleting unused icons & stroke mode Delete unused icons & stroke mode Jan 3, 2025
@JulesBelveze
Copy link
Contributor

@pinotalexandre FYI: No biggie but we usually split changes in front and sparkle into 2 different PRs, otherwise if your sparkle aren't retro compatible it will block front deployments. Happy to further explain IRL if needed 😃

@pinotalexandre
Copy link
Collaborator Author

Don't see any problem in front-edge, ok for you if I bump a new sparkle version and then deploy sparkle? @JulesBelveze

@pinotalexandre
Copy link
Collaborator Author

@JulesBelveze I deleted front modifications, so I guess the plan is to merge PR -> Deploy Sparkle, bump sparkle version in front in another PR?

@JulesBelveze
Copy link
Contributor

@pinotalexandre Yessir! You'll need to bump sparkle version again though (someone took 0.2.355 away from you already 👿

@pinotalexandre pinotalexandre force-pushed the clean-icon branch 2 times, most recently from 4673bbc to 7499b62 Compare January 6, 2025 16:30
Copy link
Contributor

@JulesBelveze JulesBelveze 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 sir 🔥

@pinotalexandre pinotalexandre merged commit 85e375e into main Jan 7, 2025
3 checks passed
@pinotalexandre pinotalexandre deleted the clean-icon branch January 7, 2025 09:02
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.

2 participants