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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

six new drinks #16320

Merged
merged 8 commits into from Sep 2, 2021
Merged

Conversation

thatdanguy23
Copy link
Contributor

@thatdanguy23 thatdanguy23 commented Jul 7, 2021

What Does This PR Do

adds six new drinks; one non-alcoholic and five alcoholic. sprites by Red966#7385 (Discord).

Why It's Good For The Game

more recipes for bartenders to flaunt

Images of changes

image
(left to right) gimlet, sidecar, whiskey sour, mint julep, pina colada, arnold palmer

Changelog

馃啈 thatdanguy23 Red966
add: added six new drinks
/:cl:

@farie82
Copy link
Member

farie82 commented Jul 7, 2021

You can use the following as a CL to also include Red in it:

馃啈 thatdanguy23 Red966
add: added six new drinks
/馃啈

@thatdanguy23
Copy link
Contributor Author

You can use the following as a CL to also include Red in it:

馃啈 thatdanguy23 Red966
add: added six new drinks
/馃啈

ooh perfect! thank you!

@AffectedArc07 AffectedArc07 added the Feature This PR is a new addition to the game label Jul 7, 2021
@Spacemanspark
Copy link
Contributor

D r i n k

@@ -44,7 +44,8 @@ GLOBAL_LIST_INIT(drinks, list("beer2","hot_coco","orangejuice","tomatojuice","li
"vodkatonic","ginfizz","bahama_mama","singulo","sbiten","devilskiss","red_mead",
"mead","iced_beer","grog","aloe","andalusia","alliescocktail","soy_latte",
"cafe_latte","acidspit","amasec","neurotoxin","hippiesdelight","bananahonk",
"silencer","changelingsting","irishcarbomb","syndicatebomb","erikasurprise","driestmartini", "flamingmoe"))
"silencer","changelingsting","irishcarbomb","syndicatebomb","erikasurprise","driestmartini", "flamingmoe",
"arnold_palmer","gimlet","sidecar","whiskeysour","mintjulep","pinacolada"))
Copy link
Contributor

Choose a reason for hiding this comment

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

out of scope but MY GOD that entire list needs spaces after to commas.

Copy link
Contributor

@SabreML SabreML left a comment

Choose a reason for hiding this comment

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

Always good to get new drinks. 馃憖

code/modules/reagents/chemistry/reagents/alcohol.dm Outdated Show resolved Hide resolved
code/modules/reagents/chemistry/reagents/alcohol.dm Outdated Show resolved Hide resolved
code/modules/reagents/chemistry/reagents/alcohol.dm Outdated Show resolved Hide resolved
drink_desc = "Lemon and whiskey, with a cute foamy head!"
taste_description = "warm whiskey and sweetness"

/datum/reagent/consumable/ethanol/mint_julep
Copy link
Contributor

Choose a reason for hiding this comment

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

This and the Pina Colada are fine actually, just the others. 馃憤

Comment on lines 181 to 189
/datum/reagent/consumable/drink/cold/arnold_palmer
name = "Arnold Palmer"
id = "arnold_palmer"
description = "A wholesome mixture of lemonade and iced tea."
color = "#8B5427" // rgb: 139, 84, 39
drink_icon = "arnoldpalmer"
drink_name = "Arnold Palmer"
drink_desc = "A wholesome mixture of lemonade and iced tea...looks like somebody didn't stir this one very well."
taste_description = "sweet and fizzy"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
/datum/reagent/consumable/drink/cold/arnold_palmer
name = "Arnold Palmer"
id = "arnold_palmer"
description = "A wholesome mixture of lemonade and iced tea."
color = "#8B5427" // rgb: 139, 84, 39
drink_icon = "arnoldpalmer"
drink_name = "Arnold Palmer"
drink_desc = "A wholesome mixture of lemonade and iced tea...looks like somebody didn't stir this one very well."
taste_description = "sweet and fizzy"
/datum/reagent/consumable/drink/cold/arnold_palmer
name = "Arnold Palmer"
id = "arnold_palmer"
description = "A wholesome mixture of lemonade and iced tea."
color = "#8B5427" // rgb: 139, 84, 39
drink_icon = "arnoldpalmer"
drink_name = "Arnold Palmer"
drink_desc = "A wholesome mixture of lemonade and iced tea...looks like somebody didn't stir this one very well."
taste_description = "sweet and fizzy"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one ended up being counted as outdated so if it's still spaces rather than tabs feel free to re-suggest it

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, right I didn't include all the changes in both. Just a second.

/datum/reagent/consumable/ethanol/whiskey_sour
name = "Whiskey Sour"
id = "whiskeysour"
description = "A tantalizing mixture of whiskey, sugar, lemon juice...and egg whites?"
Copy link
Contributor

Choose a reason for hiding this comment

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

Also for this and the drink description for the 'Arnold Palmer', you probably want an extra space after the ellipses.

Suggested change
description = "A tantalizing mixture of whiskey, sugar, lemon juice...and egg whites?"
description = "A tantalizing mixture of whiskey, sugar, lemon juice... and egg whites?"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

also git is angry now and they failed the PR checks, f u g

code/modules/reagents/chemistry/reagents/drink_cold.dm Outdated Show resolved Hide resolved
thatdanguy23 and others added 4 commits July 28, 2021 15:27
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Copy link
Contributor

@SabreML SabreML left a comment

Choose a reason for hiding this comment

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

Same as above but git doesn't like me, this'll fix the CI checks too.

code/modules/reagents/chemistry/reagents/alcohol.dm Outdated Show resolved Hide resolved
code/modules/reagents/chemistry/reagents/drink_cold.dm Outdated Show resolved Hide resolved
thatdanguy23 and others added 2 commits July 28, 2021 15:35
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
Co-authored-by: SabreML <57483089+SabreML@users.noreply.github.com>
@yeetethdeleteth
Copy link

Yes! We need Arnold Palmers!

@variableundefined variableundefined merged commit 50ec519 into ParadiseSS13:master Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This PR is a new addition to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants