-
Notifications
You must be signed in to change notification settings - Fork 306
Update face sensing image stars #2301
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
Conversation
|
Hmm this could go either way, but the original stars were based loosely on the ones from Scratch's face sensing icon (and the yellow is Events yellow for color parity) |
|
I'll let someone else decide, but the way I had it wasn't for no reason |
|
Alright, I made them yellow again |
PPPDUD
left a comment
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.
Looks nice.
Dismissing old approvals so that moving forward everything follows #2327
GarboMuffin
left a comment
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.
- This is a trivial change that doesn't require testing.
- I have no additional concerns.
PPPDUD
left a comment
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.
- This is a trivial change that doesn't require testing.
- I have no additional concerns.
Looks much cooler now.
Title says it all
TurboWarp/scratch-gui#1107
This replaces the yellow, unequalized stars with the shiny white ones from the animated text banner
If you want them moved closer to the face then just let me know I'll do it