-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Added New Cocktails and new fill level sprites to existing drinks. #33570
base: master
Are you sure you want to change the base?
Added New Cocktails and new fill level sprites to existing drinks. #33570
Conversation
RSI Diff Bot; head commit cbeac4d merging into 0462993 Resources/Textures/Objects/Consumable/Drinks/arnoldpalmer.rsi
Resources/Textures/Objects/Consumable/Drinks/bluehawaiian.rsi
Resources/Textures/Objects/Consumable/Drinks/coconutrum.rsi
Resources/Textures/Objects/Consumable/Drinks/cosmopolitan.rsi
Resources/Textures/Objects/Consumable/Drinks/curacaoglass.rsi
Resources/Textures/Objects/Consumable/Drinks/emeraldglass.rsi
Resources/Textures/Objects/Consumable/Drinks/fourteen_loko_glass.rsi
Resources/Textures/Objects/Consumable/Drinks/milkshake.rsi
Resources/Textures/Objects/Consumable/Drinks/painkiller.rsi
Resources/Textures/Objects/Consumable/Drinks/pinacolada.rsi
Resources/Textures/Objects/Consumable/Drinks/red_meadglass.rsi
Resources/Textures/Objects/Consumable/Drinks/rewriter.rsi
Resources/Textures/Objects/Consumable/Drinks/royrogers.rsi
Resources/Textures/Objects/Consumable/Drinks/shirleytemple.rsi
Resources/Textures/Objects/Consumable/Drinks/sol_dry_glass.rsi
Resources/Textures/Objects/Consumable/Drinks/vodkatonicglass.rsi
Resources/Textures/Objects/Consumable/Drinks/caipirinha.rsi
Resources/Textures/Objects/Consumable/Drinks/daiquiri.rsi
Resources/Textures/Objects/Consumable/Drinks/deathintheafternoon.rsi
Resources/Textures/Objects/Consumable/Drinks/empress75.rsi
Resources/Textures/Objects/Consumable/Drinks/espressomartini.rsi
Resources/Textures/Objects/Consumable/Drinks/mayojito.rsi
Resources/Textures/Objects/Consumable/Drinks/mimeosa.rsi
Resources/Textures/Objects/Consumable/Drinks/mimosa.rsi
Resources/Textures/Objects/Consumable/Drinks/moscowmule.rsi
Resources/Textures/Objects/Consumable/Drinks/sakeglass.rsi
Resources/Textures/Objects/Consumable/Drinks/thesunalsorises.rsi
Resources/Textures/Objects/Consumable/Drinks/whiskeysour.rsi
|
Resources/Textures/Objects/Consumable/Drinks/red_meadglass.rsi/meta.json
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mojito recipes often times require shaking them, did you intend to remove this recipe step?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be considered something of a band-aid solution currently. The recipes added for the Daiquiri and Caipirinha were conflicting with the mojito's recipe. I'll update it to re-add the old mojito recipe and change the Daiquiri/Caipirinha in a way that fixes the conflict.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has been updated, Mojito once again requires shaking to be made and the Caipirinha recipe now requires Ice to prevent conflicts between the recipes for the two drinks.
I wish there was an easier way to do that. |
add a beret to the mimeosa glass kinda like a lime or umbrella |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
About the PR
There are just too few cocktails in the game right now, and the fact that we had absolutely none that used absinthe or champagne seemed criminal. So I made some new sprites, added some new drinks, got distracted and added sprites/fill levels for a bunch of drinks that lacked them previously, and got them all finished here.
New drinks include the Caipirinha, Daiquiri, Death In The Afternoon, Empress 75, Espresso Martini, Mayojito, Mimeosa, Mimosa, Moscow Mule, The Sun Also Rises, and the Whiskey Sour.
Old Drinks that now have new sprites or proper fill levels include the Arnold Palmer, Blue Hawaiian, Coconut Rum, Cosmopolitan, Blue Curacao, Emeraldine Melon, Fourteen Loko, Milkshake (Unused Currently), Painkiller, Pina Colada, Red Mead, Rewriter, Roy Rogers, Sake, Shirley Temple, Sol Dry, and Vodka Tonic.
Why / Balance
Balance should be largely unchanged. A few of the drinks have very minor effects (Moscow Mule contains small amounts of copper, Mayojito causes vomiting and some poison damage, Mimeosa causes mute condition) but none of them are going to be game-breakingly powerful.
Technical details
Includes lots of changes to files in Textures/Objects/Consumables/Drinks, both in adding new folders for the new drinks and modifying old drinks to add new sprites. Files in Consumable/Drink and Recipies/Reactions were tweaked for the same reason, as were drinks.yml, flavors.yml, alcohol.ftl, and flavor-profiles.ftl.
Media
Requirements
Breaking changes
Changelog
🆑 RedBookcase