Skip to content

Change the sparkle #48

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

Open
wants to merge 2 commits into
base: tools
Choose a base branch
from
Open

Change the sparkle #48

wants to merge 2 commits into from

Conversation

Mark-H
Copy link
Collaborator

@Mark-H Mark-H commented Apr 17, 2025

Make the sparkle button more subtle and closer to the sidebar's color.. but with a little splash of animation to make sure it draws a bit of attention and people see it.

Schermopname.2025-04-17.om.19.21.24.mov

@Mark-H Mark-H requested a review from rthrash April 17, 2025 17:23
@rthrash
Copy link
Member

rthrash commented Apr 22, 2025

Definitely a step in the right direction. Maybe we need to mark the prev/next arrows more in line with those? Will load this into my test site and give it a whirl.

@Mark-H
Copy link
Collaborator Author

Mark-H commented Apr 23, 2025

Maybe we need to mark the prev/next arrows more in line with those?

Yeah, I thought about that. The main reason I didn't is because to me they serve a different purpose.

The sparkle is to let people know what they can auto-generate. Because that's pretty much everywhere, I felt like removing the button shape from it to make it more subtle, makes the overall manager interface that much cleaner. Sorta reminiscent of a little asterisk to indicate a field is required - except it indicates it has be enhanced and can generate the content for you.

The buttons serve actual button-pressy features where it updates the content, plus those only show when something has been generated, so I don't mind those as much. Should probably update the color to match at least, though.

Overall, I'm like a 7/10 on changing the sparkle and only a 2/10 on what the other buttons should look like.

@theboxer
Copy link
Member

@Mark-H I'd put the style tag to the css file

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.

3 participants