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

so much blobbier. #148

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

Conversation

Secret-chest
Copy link

@Secret-chest Secret-chest commented Sep 5, 2023

Resolves #65

This is a very large PR.

  • Replaces most smilies available in KitKat with their KitKat (4.4.4) versions, sometimes tweaked.
  • Newer smiles are replaced by edits of the KitKat smilies.

Previews (not full)

emoji_u1f60b
emoji_u1f62d
emoji_u1f62e
emoji_u1f62c
emoji_u1f92e
emoji_u1f618
emoji_u1f605
emoji_u1f607
emoji_u1f637
emoji_u1f911
emoji_u1f924
emoji_u1f925
emoji_u1f929
emoji_u1f970
emoji_u1f973
emoji_u1f976
face with peeking eye
face holding back tears
melting face
saluting face
smiling face with tear

Make all the faces based on the Marshmallow versions
This was referenced Sep 5, 2023
@Secret-chest
Copy link
Author

@C1710

@Bronnel721
Copy link

Bronnel721 commented Sep 5, 2023

If you really wanted to, I could also work with this PR. EDIT: Nevermind, I don't need to.

@Secret-chest
Copy link
Author

If you really wanted to, I could also work with this PR.

I don't really need you. Only do it if you want.

@Bronnel721
Copy link

So I'm gonna leave some comments, like what you did. Does that sound acceptable?

@Secret-chest
Copy link
Author

Sure, comment all you want.

Choose a reason for hiding this comment

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

The disguised face shouldn't be changed, the fact that the body has a squished shape looks weird.

Choose a reason for hiding this comment

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

Thanks for removing the bitmap.

Choose a reason for hiding this comment

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

The mouth looks too wide; use the same mouth that's in emoji_u1f604.svg.

Copy link
Author

Choose a reason for hiding this comment

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

The mouth looks too wide; use the same mouth that's in emoji_u1f604.svg.

I'll try, but I need fitting teeth.

Choose a reason for hiding this comment

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

Try to make it if you don't want me to do it.

Copy link
Author

Choose a reason for hiding this comment

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

Fork the fork and send a PR if you want.

Choose a reason for hiding this comment

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

I can't make one, I already have a fork to the original. Also, you can't create a fork of a fork if you've forked the original repository.

Copy link
Author

Choose a reason for hiding this comment

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

You can't? Interesting.

Copy link
Author

Choose a reason for hiding this comment

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

You're now a collaborator to my fork, push to so-much-blobbier

Choose a reason for hiding this comment

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

Facial expression isn't the same -- it should look something like this:
emoji_u1f60b

Copy link
Author

@Secret-chest Secret-chest Sep 5, 2023

Choose a reason for hiding this comment

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

I'm fixing it soon.

Choose a reason for hiding this comment

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

Keep this one as it is currently.

Copy link
Author

Choose a reason for hiding this comment

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

Why?

Choose a reason for hiding this comment

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

It just looks weird like that.

Choose a reason for hiding this comment

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

You did a better job with perspective, but it isn't that good with the eyes.

Copy link
Author

Choose a reason for hiding this comment

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

I'm really not good.

Choose a reason for hiding this comment

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

I'm not that good at it either, but I can try to make yours look better.

Copy link
Author

Choose a reason for hiding this comment

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

Then try it.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

Okay:
emoji_u1f612

Copy link
Author

Choose a reason for hiding this comment

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

Fork my fork and PR it.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

Perspective is MUCH more worse here, also, add the sweat back.

Copy link
Author

Choose a reason for hiding this comment

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

OK, I'm fixing it.

Choose a reason for hiding this comment

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

The mouth is too downwards, move it up a little

Choose a reason for hiding this comment

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

The color and the fact that there's NO gradient also looks weird, and again, the perspective.

Copy link
Author

Choose a reason for hiding this comment

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

The gradient should be visible in Inkscape.

Choose a reason for hiding this comment

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

Browsers are so weird, but make the face smaller.

Choose a reason for hiding this comment

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

The gradient looks so weird.

Copy link
Author

Choose a reason for hiding this comment

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

Again, view it in Inkscape.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

You're probably right, browsers show SVGs strangely sometimes.
EDIT: Yeah, it looks better.

Choose a reason for hiding this comment

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

Change where the nosedrop is to where it was originally if you're making it face to the left.

Choose a reason for hiding this comment

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

Again, the face doesn't have any perspective.

Copy link
Author

Choose a reason for hiding this comment

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

It's not mandatory. This one faces front.

Choose a reason for hiding this comment

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

Then make the face smaller.

Choose a reason for hiding this comment

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

Remove the extra teardrops you've added.

Copy link
Author

Choose a reason for hiding this comment

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

I tried to align it with other platforms more.

Choose a reason for hiding this comment

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

Well I thought they used streams.

Copy link
Author

Choose a reason for hiding this comment

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

I want to do something in-between.

Choose a reason for hiding this comment

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

Alright.

Choose a reason for hiding this comment

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

The X-shaped eyes are really wobbly, use this version:
image

Copy link
Author

Choose a reason for hiding this comment

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

That's only one X.

Choose a reason for hiding this comment

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

Make the eyes bigger.

Choose a reason for hiding this comment

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

Here's where perspective gets out of hand.

Copy link
Author

Choose a reason for hiding this comment

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

OK, fixing it.

Choose a reason for hiding this comment

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

This doesn't look THAT bad, but revert.

Choose a reason for hiding this comment

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

Again, revert.

Copy link
Author

Choose a reason for hiding this comment

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

It's a big brain, literally.

Choose a reason for hiding this comment

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

I guess that works?

Choose a reason for hiding this comment

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

You know what? That was clever.

Choose a reason for hiding this comment

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

Again, revert. I just don't really like how puffy it looks.

Choose a reason for hiding this comment

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

Again, revert. It looks like you're trying to make it look like the character's perspective is more on top.

Choose a reason for hiding this comment

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

The eyes are too black, it should look more like this:
emoji_u1f924 (blobified)

Copy link
Author

Choose a reason for hiding this comment

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

Sorry!

Choose a reason for hiding this comment

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

That's alright.

Choose a reason for hiding this comment

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

Not too bad, but the nose could be rotated to be comprehensive with the blob's shape.

Choose a reason for hiding this comment

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

No, just no. Revert.

Choose a reason for hiding this comment

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

I don't like how big the face is.

Choose a reason for hiding this comment

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

This one also looks weird.

Choose a reason for hiding this comment

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

Try to use the eyes of emoji-u1f604.svg for this one too.

Choose a reason for hiding this comment

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

The mouth is too far from the eyes, with should've already been bigger to start with.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

Again, revert, because it has the same issues as 😬.

Choose a reason for hiding this comment

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

Because of the shape.

Copy link
Author

Choose a reason for hiding this comment

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

What's so wrong?

Choose a reason for hiding this comment

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

Same with these ones, a lot of them that have this shape need to get reverted.

Copy link
Author

Choose a reason for hiding this comment

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

In this case, I picked the shape because it's filled with breath.

Choose a reason for hiding this comment

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

That's fair, but make the face smaller.

Choose a reason for hiding this comment

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

Again, the perspective of the eyes looks off.

Choose a reason for hiding this comment

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

The hand needs to be rotated the other way.

Choose a reason for hiding this comment

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

Looks like the old one's staring at yours side by side. the mouth is so downwards.

Choose a reason for hiding this comment

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

Use the eyes from this version of the dizzy face and don't make the mouth THAT far.
image

Choose a reason for hiding this comment

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

This one shouldn't be changed.

Copy link
Author

Choose a reason for hiding this comment

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

Be constructive.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

The meltiness looks weird.

Choose a reason for hiding this comment

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

Make the mouth go more far up.

Copy link

@Bronnel721 Bronnel721 Sep 5, 2023

Choose a reason for hiding this comment

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

Don't change it, and if you do, make the face smaller and more upwards, just like some other emojis with this shape.

Choose a reason for hiding this comment

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

The mouth looks WAY too rotated; should look something like this:
small_zany

Copy link
Author

Choose a reason for hiding this comment

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

OK

Choose a reason for hiding this comment

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

Remove the mouth behind.

Choose a reason for hiding this comment

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

This one? Shouldn't be changed.

@Bronnel721
Copy link

I'm sorry if I was being too rude.

@Secret-chest
Copy link
Author

I'm sorry if I was being too rude.

No problem, just give a reason instead of saying revert

@Bronnel721
Copy link

Okay then I'll edit mine.

@Bronnel721
Copy link

Actually, I'll just state it right here, for the ones that need to be reverted, I kind of change my mind -- make the faces smaller.

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