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

Inconsistent pupil colors #30

Open
winterClover opened this issue Apr 3, 2023 · 4 comments
Open

Inconsistent pupil colors #30

winterClover opened this issue Apr 3, 2023 · 4 comments

Comments

@winterClover
Copy link

Unlike the rest of the emojis, emojis with wide eyes has black pupils instead of brown.

The rolling eyes emoji and the old versions of pleading/holding back tears emoji are an exception in that they have brown pupils.

@barkloaf
Copy link

barkloaf commented Apr 5, 2023

the difference with the pleading and holding back tears emojis here is that they have much larger pupils and also have irises. changing those large pupils to black (as was done in #6) makes them look very out-of-place, and my guess is that some of that has to do with that size difference. having two large black dots in the middle of the emoji like that just looks very weird. as stated in my comment on #27, I think that there's nothing wrong with the original pleading emoji and that it shouldn't've been changed. I think that the primary issue with the holding back tears emoji is not the pupils and irises, but the shape of the sclera (the white part of the eye) and the location of the iris and pupil within it.

ultimately, I think that the pleading face and holding back tears emojis should be left out of this discussion about pupil color because they're not very comparable to the other 5 emoji. I think making the rest of the 5 consistent (either as black or as brown) is a worthwhile question, though

@winterClover
Copy link
Author

I personally think they should be brown as to be more inline with the rest of the emojis

@winterClover
Copy link
Author

On the new pleading/holding back tears emojis, they now have large black pupils. which makes them now part of the other 4 emojis to have black eyes.

image

@winterClover
Copy link
Author

I don't have much of an opinion on the changes, but I do take issue with adding inconsistency where there was once wasn't.

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

No branches or pull requests

2 participants