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

Adds the ability to jam cigarettes into the filter slot of a gas mask #76881

Merged
merged 31 commits into from Aug 9, 2023

Conversation

Momo8289
Copy link
Contributor

@Momo8289 Momo8289 commented Jul 16, 2023

About The Pull Request

This PR makes it so that when a gas mask of any type (normal, explorer, mime, etc) has isn't holding any filters, and isn't covering the mouth, you can jam a cigarette in there and smoke it like normal. Cigs placed in a mask can be lit by using the lighting implement on the mask as you would a normal cig. The gas mask can't be adjusted while holding a cig (if the gas mask can be adjusted at all). Whether or not internals can be used depends on the mask and its adjusted state.
image
A mime enjoying their favourite robusto cigars

Why It's Good For The Game

First off, I just find the idea of replacing an air filter in a gas mask, something meant to protect you from harmful gasses and fumes, with a cancer-inducing burning stick of chemicals, to be really funny.

Secondly, and the reason that was the initial inspiration for this PR, is it allows mimes and clowns to smoke without removing their masks. This opens the door for mimes and clowns to use smoking as part of their character gimmick, without losing a pretty big part of their identity.

Changelog

🆑
add: Cigarettes can now be placed in and smoked through gas masks, so long as the mask has no filters installed and is not covering the mouth.
/:cl:

@tgstation-server tgstation-server added the Feature Exposes new bugs in interesting ways label Jul 16, 2023
@EricZilla
Copy link
Contributor

Finally mimes can smoke

@Helg2
Copy link
Contributor

Helg2 commented Jul 16, 2023

Maybe also add possibility to put bubblegum in masks?

@Momo8289
Copy link
Contributor Author

Maybe also add possibility to put bubblegum in masks?

Implementing that in the same way doesn't really make a lot of sense. Shoving bubblegum into a gas mask with the intention to chew it wouldn't really work at all. The animation also wouldn't make much sense over a gas mask.

@Helg2
Copy link
Contributor

Helg2 commented Jul 16, 2023

Shoving bubblegum into a gas mask with the intention to chew it wouldn't really work at all.

But you can eat through the mask.

@dwasint
Copy link

dwasint commented Jul 17, 2023

unfathomably based

@nukularpower
Copy link

I can finally be a clown with a stogie in his mouth?? this is the PR of the year imo

@san7890
Copy link
Member

san7890 commented Jul 17, 2023

i've been waiting on this feature for a while now, thanks

Copy link
Member

@san7890 san7890 left a comment

Choose a reason for hiding this comment

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

multiple nits. Some of it was a bit confusing for some reason, but after these reviews are addressed hopefully then I can figure out why some things look strange.

code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/modules/clothing/masks/gasmask.dm Outdated Show resolved Hide resolved
code/game/objects/items/cigs_lighters.dm Outdated Show resolved Hide resolved
code/game/objects/items/cigs_lighters.dm Outdated Show resolved Hide resolved
@san7890 san7890 marked this pull request as draft July 17, 2023 22:22
@san7890
Copy link
Member

san7890 commented Jul 17, 2023

mark ready for review when this has all been addressed

@Momo8289 Momo8289 marked this pull request as ready for review July 18, 2023 00:47
@Momo8289
Copy link
Contributor Author

I've been told the failed check isn't my fault 👍

@san7890
Copy link
Member

san7890 commented Jul 18, 2023

sorry, i didn't accurately explain the balloon alert stuff. they must be complaint with the requirements in this document: https://hackmd.io/@tgstation/SJIle5wYd#balloon_alertballoon_alert_to_viewers

this is to mean short, no capitalization nor punctuation, and informative. read the document and follow that for all of the things you've added.

@san7890 san7890 marked this pull request as draft July 18, 2023 01:32
@san7890
Copy link
Member

san7890 commented Jul 24, 2023

image

undraft when CI is passing

@san7890 san7890 marked this pull request as draft July 24, 2023 04:38
@Momo8289 Momo8289 marked this pull request as ready for review July 24, 2023 16:11
@Momo8289 Momo8289 requested a review from MrMelbert July 24, 2023 18:29
@weesnaw-ops
Copy link

I’ve been trying to do my chainsmoking clown gimmick for a while thank fuck

@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 2023

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale Even the uncaring universe rejects you, why even go on label Aug 6, 2023
@Momo8289
Copy link
Contributor Author

Momo8289 commented Aug 6, 2023

@MrMelbert Changes have been done for a while and it's going stale now. Could I get a merge?

@github-actions github-actions bot removed the Stale Even the uncaring universe rejects you, why even go on label Aug 7, 2023
code/game/objects/items/cigs_lighters.dm Outdated Show resolved Hide resolved
@Momo8289 Momo8289 requested a review from Ghommie August 9, 2023 16:01
@Ghommie Ghommie merged commit 7e548b2 into tgstation:master Aug 9, 2023
19 checks passed
comfyorange added a commit that referenced this pull request Aug 9, 2023
github-actions bot added a commit that referenced this pull request Aug 9, 2023
NaakaKo referenced this pull request in Bird-Lounge/Skyraptor-SS13 Aug 9, 2023
… (#76881)

## About The Pull Request
This PR makes it so that when a gas mask of any type (normal, explorer,
mime, etc) has isn't holding any filters, and isn't covering the mouth,
you can jam a cigarette in there and smoke it like normal. Cigs placed
in a mask can be lit by using the lighting implement on the mask as you
would a normal cig. The gas mask can't be adjusted while holding a cig
(if the gas mask can be adjusted at all). Whether or not internals can
be used depends on the mask and its adjusted state.

![image](https://github.com/tgstation/tgstation/assets/42454181/a1d00139-eef3-479a-b8f2-b12b531f6833)
A mime enjoying their favourite robusto cigars
## Why It's Good For The Game
First off, I just find the idea of replacing an air filter in a gas
mask, something meant to protect you from harmful gasses and fumes, with
a cancer-inducing burning stick of chemicals, to be really funny.

Secondly, and the reason that was the initial inspiration for this PR,
is it allows mimes and clowns to smoke without removing their masks.
This opens the door for mimes and clowns to use smoking as part of their
character gimmick, without losing a pretty big part of their identity.
## Changelog
:cl:
add: Cigarettes can now be placed in and smoked through gas masks, so
long as the mask has no filters installed and is not covering the mouth.
/:cl:

---------

Co-authored-by: san7890 <the@san7890.com>
Jolly-66 pushed a commit to TaleStation/TaleStation that referenced this pull request Aug 9, 2023
…ter slot of a gas mask (#7219)

Original PR: tgstation/tgstation#76881
-----

## About The Pull Request
This PR makes it so that when a gas mask of any type (normal, explorer,
mime, etc) has isn't holding any filters, and isn't covering the mouth,
you can jam a cigarette in there and smoke it like normal. Cigs placed
in a mask can be lit by using the lighting implement on the mask as you
would a normal cig. The gas mask can't be adjusted while holding a cig
(if the gas mask can be adjusted at all). Whether or not internals can
be used depends on the mask and its adjusted state.

![image](https://github.com/tgstation/tgstation/assets/42454181/a1d00139-eef3-479a-b8f2-b12b531f6833)
A mime enjoying their favourite robusto cigars
## Why It's Good For The Game
First off, I just find the idea of replacing an air filter in a gas
mask, something meant to protect you from harmful gasses and fumes, with
a cancer-inducing burning stick of chemicals, to be really funny.

Secondly, and the reason that was the initial inspiration for this PR,
is it allows mimes and clowns to smoke without removing their masks.
This opens the door for mimes and clowns to use smoking as part of their
character gimmick, without losing a pretty big part of their identity.
## Changelog
:cl:
add: Cigarettes can now be placed in and smoked through gas masks, so
long as the mask has no filters installed and is not covering the mouth.
/:cl:

---------

Co-authored-by: Nick <42454181+Momo8289@users.noreply.github.com>
Co-authored-by: san7890 <the@san7890.com>
MrMelbert pushed a commit that referenced this pull request Sep 25, 2023
## About The Pull Request

#76881 allowed you to put cigarettes in a gas mask but removed the check
for, if the thing was a cigarette, making sure it was in your mask slot.

What this meant was that standing around with a cigarette in your hand
would still make you passively smoke it - although this is somewhat more
immersive to me as a former smoker, it also gets obnoxious having to set
the cigarette on a table or put it in my bag to avoid racking up the
ridiculously easy-to-accumulate nicotine addiction points you otherwise
get from even one cigarette in-game.

This just readds that check after the checks for gas masks/non-smoker
locations (which may also be broken but are beyond me right now - trying
to put a cigarette into a gas mask's filter* slot returns 'mask's mouth
is covered.')
Absolucy pushed a commit to Absolucy/Monkestation that referenced this pull request Mar 1, 2024
…tgstation#76881)

This PR makes it so that when a gas mask of any type (normal, explorer,
mime, etc) has isn't holding any filters, and isn't covering the mouth,
you can jam a cigarette in there and smoke it like normal. Cigs placed
in a mask can be lit by using the lighting implement on the mask as you
would a normal cig. The gas mask can't be adjusted while holding a cig
(if the gas mask can be adjusted at all). Whether or not internals can
be used depends on the mask and its adjusted state.

![image](https://github.com/tgstation/tgstation/assets/42454181/a1d00139-eef3-479a-b8f2-b12b531f6833)
A mime enjoying their favourite robusto cigars
First off, I just find the idea of replacing an air filter in a gas
mask, something meant to protect you from harmful gasses and fumes, with
a cancer-inducing burning stick of chemicals, to be really funny.

Secondly, and the reason that was the initial inspiration for this PR,
is it allows mimes and clowns to smoke without removing their masks.
This opens the door for mimes and clowns to use smoking as part of their
character gimmick, without losing a pretty big part of their identity.
:cl:
add: Cigarettes can now be placed in and smoked through gas masks, so
long as the mask has no filters installed and is not covering the mouth.
/:cl:

---------

Co-authored-by: san7890 <the@san7890.com>
Absolucy pushed a commit to Absolucy/Monkestation that referenced this pull request Mar 1, 2024
## About The Pull Request

tgstation#76881 allowed you to put cigarettes in a gas mask but removed the check
for, if the thing was a cigarette, making sure it was in your mask slot.

What this meant was that standing around with a cigarette in your hand
would still make you passively smoke it - although this is somewhat more
immersive to me as a former smoker, it also gets obnoxious having to set
the cigarette on a table or put it in my bag to avoid racking up the
ridiculously easy-to-accumulate nicotine addiction points you otherwise
get from even one cigarette in-game.

This just readds that check after the checks for gas masks/non-smoker
locations (which may also be broken but are beyond me right now - trying
to put a cigarette into a gas mask's filter* slot returns 'mask's mouth
is covered.')
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants