-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix(icons): changed piano icon
#3974
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 ChatGPT Tags suggestions ✨
I've asked ChatGPT for some suggestions for tags.
| "karsa-mistmere", | ||
| "jguddas" | ||
| ], | ||
| "tags": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestions for the piano icon.
Try asking it your self if you want more suggestions. Open ChatGPT
Here are the suggestions:
| "tags": [ | |
| "tags": [ | |
| "melody", | |
| "scale", | |
| "art", | |
| "entertainment", | |
| "soundwave", |
Added or changed iconsIcons as codeWorks for: const PianoIcon = createLucideIcon('Piano', [
["path",{"d":"M12 14v3"}],
["path",{"d":"M16 14v3"}],
["path",{"d":"M19 21a2 2 0 0 0 2-2v-8a2 2 0 0 0-2-2h-.416a1 1 0 0 1-.87-.509L16.066 5.5A7 7 0 0 0 3 9.013V19a2 2 0 0 0 2 2z"}],
["path",{"d":"M3 14h18"}],
["path",{"d":"M8 14v3"}]
]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whatever we end up going with, this one should be consistent with keyboard-music.
It's not really possible to fit 4 vertical lines without increasing the size beyond what is allowed. I guess we could change |
What is the purpose of this pull request?
Description
Decreased size
Before Submitting