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

Replaced default sticker set with twitter emoji #137

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

Conversation

fpohtmeh-github
Copy link
Contributor

Twitter emoji set is a very popular sticker set. Looks much better than current one

@DamirPorobic
Copy link
Member

Thanks for taking your time and providing this PR. I don't know if you knew this but you can upload your own stickers and hide the default ones, which I personally actually like, that's why I added them in the first place. Adding twitter emojis kind of creates an affiliation with a big software corporation. And last, what's the licence on the twitter emojis?

@DamirPorobic DamirPorobic self-requested a review July 27, 2020 17:47
@fpohtmeh-github
Copy link
Contributor Author

Twitter emoji graphics have license CC BY 4.0. I'll add license into stickers folder if we leave them.
Default sticker set if the first impression of anyone who installs KSnip. Only 5% (even less) will play with custom sticker sets. That's why important to have well-looking defaults in application. Twitter set is popular, attractive and well-known.

I don't want to insult developers of current sticker set, but it looks too simple for serious software. Such graphics style was popular 20 years ago. Old school is not for everyone, but KSnip is :)

@fpohtmeh-github
Copy link
Contributor Author

I can look for another emoji set, but strongly recommend don't leave current one because it's not for masses.

@DamirPorobic
Copy link
Member

I can look for another emoji set, but strongly recommend don't leave current one because it's not for masses.

Thanks. We can change it but I'm kind of not warm with this one. Just link me any other example you find before you go through adding them again.

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.

None yet

2 participants