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

Randomized lollipop and gumball flavors #1074

Merged
merged 14 commits into from
Apr 14, 2024

Conversation

pissdemon
Copy link
Contributor

@pissdemon pissdemon commented Apr 10, 2024

About the PR

This PR makes gumballs and lollipops have random flavors! And it will also make borgs make a noise when dispensing them, as well as allow you to put lollipops in your mouth as god intended. (And I'm also contemplating buffing the lollipop a little.)

Why / Balance

Because nobody could tell me how they taste like and that's literally unacceptable. It also allows people to have really intense schoolyard disputes over their favorite flavors.

Technical details

Adds a candyFlavor proto. You can add new candy flavors, giving them a name, a color and a set of flavors (uses the Flavor proto).

Media

image
image
image

Also, when you eat them it shows the appropriate flavors, but you can probably use your imagination for that.

Breaking changes

None.

Changelog
🆑

  • add: Gumballs and lollipops now come in 32 awesome flavors! Try them all!

TODO

  • More flavors :9
  • Borg noise (play machine_vend.ogg probably) (done, but maybe should be quieter?)
  • Lollipop in mouth slot (Laika, who I blame for this PR, said they would help with sprite stuff for this)
  • Merge lollipop stick and shine, they don't need to be separate

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files labels Apr 10, 2024
Copy link
Contributor

github-actions bot commented Apr 10, 2024

RSI Diff Bot; head commit f16967f merging into 4b80e48
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Nyanotrasen/Objects/Consumable/Food/candy.rsi

State Old New Status
gumball Modified
gumball-shine Added
lollipop-ball Added
lollipop-equipped-MASK-vulpkanin Added
lollipop-equipped-MASK Added
lollipop-stickandshine Added

Edit: diff updated after f16967f

@pissdemon pissdemon marked this pull request as ready for review April 13, 2024 03:25
Copy link
Member

@deltanedas deltanedas left a comment

Choose a reason for hiding this comment

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

new code should be in deltav namespace and minor stuff

@deltanedas deltanedas merged commit d8b97e8 into DeltaV-Station:master Apr 14, 2024
13 checks passed
@pissdemon pissdemon deleted the howitfeelstochewfivegum branch April 14, 2024 18:13
nearlyNonexistent referenced this pull request in nearlyNonexistent/Horizon-Station Apr 14, 2024
* Give lollipops and gumballs random flavors

* RandomizedCandy: Fix borg lollipop action

* RandomizedCandy: Play sound when candy is fabricated

* RandomizedCandy: Lollipop can now be put in mouth

* RandomizedCandy: Quieten fab sound

* RandomizedCandy: merge icons and reexport without pHYs chunk

should speed up downloading these textures on 9600 baud modems

* RandomizedCandy: loads more flavors and update description

* RandomizedCandy: Fix duplicate candyFlavor IDs

* RandomizedCandy: Fix attrib and preserve flavor order

* RandomizedCandy: init on MapInit instead of ComponentInit

* RandomizedCandy: Fix minor code style issues, file placement

* RandomizedCandy: put new code into DeltaV namespace

* RandomizedCandy: Removed redundant datafield tags

Signed-off-by: no <165581243+pissdemon@users.noreply.github.com>

---------

Signed-off-by: no <165581243+pissdemon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants