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

Redesigned icon and support for monochromatic icon. #690

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Pindrop12344321
Copy link

I have changed the icon, and not deleted the previous one from the project, instead added new folder for the new "svg" icon.
Here are the samples


Copy link
Owner

@Julow Julow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think I'd prefer if the old icon was gone and replaced with an image version of the new icon (for compatibility). The old icon was made quickly and doesn't look very good.

@ChasmSolacer
Copy link
Contributor

The old icon shows these small chars in key corners, I don't remember if I noticed this app because of this, but I think it quickly shows off the keyboard's main feature.
I'd rather insert random symbols in some corners and maybe remove the _ key and enlarge the other keys a bit to make them visible. What do you think?

@Pindrop12344321
Copy link
Author

Pindrop12344321 commented Jul 7, 2024

You are right 👍, but in the replacement of _ what icon do you prefer

@ChasmSolacer
Copy link
Contributor

Honestly idk, if it needs replacement, maybe a key that doesn't appear on most keyboards like arrows or Fn.
Otherwise, the icon you propose looks elegant. That's just a random remark

@Pindrop12344321
Copy link
Author

Honestly idk, if it needs replacement, maybe a key that doesn't appear on most keyboards like arrows or Fn. Otherwise, the icon you propose looks elegant. That's just a random remark

** I think that was a great remark **, going to replace the underscore

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