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.

Sign up for GitHub

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

Jump to bottom

Updated sprites for doughnuts, bagels, english muffin, zongzi and fries #190

Merged
merged 6 commits into from Apr 11, 2020
Merged

Updated sprites for doughnuts, bagels, english muffin, zongzi and fries #190

merged 6 commits into from Apr 11, 2020

Conversation

binarysudoku
Copy link
Contributor

About the PR

Sprite updates 4 doughnuts, bagels, english muffin, zongzi and fries

Why's this needed?

Sprite reworks fam

Changelog

Erinexx:
* New doughnuts sprites!
* New bagel sprites!
* New english muffin sprite!
* Tweaked outline on fries sprite (what the fuck was i thinking before)

@ZeWaka ZeWaka added the C-Sprites Automatically applied on any .dmi or icons folder change label Apr 11, 2020
@flrsh
Copy link
Contributor

flrsh commented Apr 11, 2020

Nice sprites, the donuts in particular look super delicious!

I'm a fan of the old zongzi though. Also, there isn't really a noticeable difference between the old and new fries, so I'm not sure if they really need changing? They already look good to me!

@Gannets
Copy link
Contributor

Gannets commented Apr 11, 2020

I could see a further improved fries sprite in future too, but otherwise this is nice. Good work!

@binarysudoku
Copy link
Contributor Author

I could see a further improved fries sprite in future too, but otherwise this is nice. Good work!

Yeah I totally agree - I'll probably look into it at some point :)

pali6
pali6 approved these changes Apr 11, 2020
View reviewed changes
Copy link
Member

@pali6 pali6 left a comment

Choose a reason for hiding this comment

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

good sprites

pali6 added a commit that referenced this pull request Apr 11, 2020
@pali6
Changelog for #190
950323e
@pali6 pali6 merged commit 8090cbb into goonstation:master Apr 11, 2020
@binarysudoku binarysudoku deleted the MoreFoodSpriteReworks branch April 11, 2020 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pali6 pali6 pali6 approved these changes

@Gannets Gannets Gannets approved these changes

Assignees
No one assigned
Labels
C-Sprites Automatically applied on any .dmi or icons folder change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@binarysudoku @flrsh @Gannets @pali6 @ZeWaka