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

Halved the size of the spinner (back to the size it originally was) #328

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

Conversation

CeeJayDK
Copy link
Contributor

Halved the size of the spinner (back to the size it originally was)

It's now about the same size (42 x 42px) as a taskbar icon in Windows 10 (48x48px) which I feel is about the perfect size.

If I knew how I would also add a 1 or 2 second wait before it shows up, because if the compile time is THAT short, why even bother people with a message since it's near instantaneous anyways.

Maybe we should also add it to the initial message - since it is the logo after all.

@CeeJayDK
Copy link
Contributor Author

Hmm hang on .. I think I might have forgotten to fix the width too.
I'll go check

The width of the spinner I think
@CeeJayDK
Copy link
Contributor Author

OK now I'm happy with the commit.

@crosire
Copy link
Owner

crosire commented Sep 15, 2024

Not sure I agree with this, it is awfully small now and heavily aliased (even if it matches the taskbar icon size).

@CeeJayDK
Copy link
Contributor Author

I'll see if I can make a compromise we are both happy with.

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