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

Added New Cocktails and new fill level sprites to existing drinks. #33570

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

RedBookcase
Copy link
Contributor

@RedBookcase RedBookcase commented Nov 26, 2024

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

image

Requirements

Breaking changes

Changelog

🆑 RedBookcase

  • add: Added about a dozen new cocktails for the Bartender to mix up, and polished some of the sprites for older drinks.

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. size/L Denotes a PR that changes 1000-4999 lines. labels Nov 26, 2024
Copy link
Contributor

github-actions bot commented Nov 26, 2024

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

Resources/Textures/Objects/Consumable/Drinks/arnoldpalmer.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/bluehawaiian.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/coconutrum.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/cosmopolitan.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/curacaoglass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/emeraldglass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/fourteen_loko_glass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/milkshake.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/painkiller.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
fill-6 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/pinacolada.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
fill-6 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/red_meadglass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/rewriter.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/royrogers.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/shirleytemple.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/sol_dry_glass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/vodkatonicglass.rsi

State Old New Status
icon Modified
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/caipirinha.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/daiquiri.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/deathintheafternoon.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/empress75.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/espressomartini.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
fill-6 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/mayojito.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/mimeosa.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/mimosa.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/moscowmule.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/sakeglass.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/thesunalsorises.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Resources/Textures/Objects/Consumable/Drinks/whiskeysour.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
fill-5 Added
icon Added
icon_empty Added

Edit: diff updated after cbeac4d

Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link
Contributor Author

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.

@DieselMohawk
Copy link
Contributor

I wish there was an easier way to do that.

@ScarKy0 ScarKy0 added P3: Standard Priority: Default priority for repository items. T: New Feature Type: New feature or content, or extending existing content D2: Medium Difficulty: A good amount of codebase knowledge required. S: Needs Review Status: Requires additional reviews before being fully accepted A: Service Area: Service department, including cooking, botany, etc T: Visual Change Type: Deals with changes to art, sprites or other visuals in the game. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Nov 26, 2024
@Everturning
Copy link

add a beret to the mimeosa glass kinda like a lime or umbrella :trollface:

Copy link
Contributor

github-actions bot commented Dec 9, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Dec 9, 2024
@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Dec 10, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Dec 16, 2024
@github-actions github-actions bot added S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted and removed S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted labels Dec 17, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added size/M Denotes a PR that changes 100-999 lines. and removed size/L Denotes a PR that changes 1000-4999 lines. labels Dec 19, 2024
@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Service Area: Service department, including cooking, botany, etc Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. D2: Medium Difficulty: A good amount of codebase knowledge required. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted size/M Denotes a PR that changes 100-999 lines. T: New Feature Type: New feature or content, or extending existing content T: Visual Change Type: Deals with changes to art, sprites or other visuals in the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants