Merge upstream #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Atmos wrench act & small typo fix (#80245) ## About The Pull Request Because `deconstruct()` returns the component that was unwrenched which is not a valid attack result. We have to explicitly state what our result is. The parent proc wrench act does nothing so no point calling it Also fixes small typo in attack define comment * Atmos wrench act & small typo fix --------- Co-authored-by: SyncIt21 <[email protected]>
…RE] (#25555) * Updates the Gold palette used on the HoS Shako hat (#80238) * Updates the Gold palette used on the HoS Shako hat --------- Co-authored-by: OrionTheFox <[email protected]>
* Update cable coil examine text (#80232) ## About The Pull Request Change examine description to properly indicate how to actually change cable layering (via using cable coil in hand). Also hints that there are other applications for this item. * Update cable coil examine text --------- Co-authored-by: Vorpal Void <[email protected]>
* Fixes character screen [NO GBP] (#80259) ## About The Pull Request TrackOutsideClicks would register extra clicks and immediately close <details> <summary>proof</summary>  </details> ## Why It's Good For The Game Bug fix Fixes #80239 ## Changelog :cl: fix: You should be able to edit your character using the feature buttons again. /:cl: * Fixes character screen [NO GBP] --------- Co-authored-by: Jeremiah <[email protected]>
…B IGNORE] (#25567) * Prevents something fucky with elevation and glass tables (#80187) ## About The Pull Request Objects can be destroyed during a movement loop before the abstract entered signal can be sent, so we need to make sure only mobs that have been elevated are dropped down. ## Why It's Good For The Game Fixes #80169 ## Changelog :cl: fix: climbing or being shoved into a glass table won't cause elevation issues. /:cl: --------- Co-authored-by: san7890 <the@ san7890.com> * Prevents something fucky with elevation and glass tables --------- Co-authored-by: Ghom <[email protected]> Co-authored-by: san7890 <the@ san7890.com>
* [no gbp] some few bot fixes (#80208) ## About The Pull Request i forgto to set this in my last prs closes #80204 also closes #80252 ## Why It's Good For The Game ais can summon bots ## Changelog :cl: fix: AIs can summon bots again fix: cleanbots can clean floors directly underneath them and prioritize floors nearest to them fix: medbots drop empty medkits now /:cl: * [no gbp] some few bot fixes --------- Co-authored-by: Ben10Omintrix <[email protected]>
* MOD Modules and Isolation Cell Timers * genpop instructions * fixes Blueshifts forgotten exosuit fab
Readd fan_out_items to armory spawner [SQUASH] Fix fan_out_items
* Adds Short tail override For reasons unknown to me, SR has an override on the lizard tails to turn their TG steel-blue grayscale palette into a true grayscale. This accounts for the override file and matches the SR lizard grayscale palette. * Added two-tone tail. Still fixing bugs with it. Added two-tone tail. m_tail_short2tone_BEHIND_secondary is not playing nice though. * Wagwagwagwagwagwagwag * Makes front_secondary line up with front_primary for wagging. Why were these disjointed by one pixel??? Oops! * Update skyrat_config.txt * Update skyrat_config.txt Removes moan; untested * Update skyrat_config.txt --------- Co-authored-by: Bloop <[email protected]>
…#25568) * Fixes hardcoded dmi file change * Update reinf_walls.dm * Update walls.dm
…#25571)
* Updates ocky icky phobia to more modern lingo (#80247)
## About The Pull Request
As requested by the administrative team; ocky icky phobia has been
updated with some all new lingo
Please give more ideas in the comments if you feel like something is
missing!
Full list of words added:
"amogus",
"chat",
"c*der",
"coder",
"coders",
"erp",
"joever",
"kys",
"lmao",
"lol",
"mapper",
"muderbone",
"murderboning",
"powergame",
"rp",
"sprite",
"spriter",
"sus",
"tide",
"ur",
"5head"
## Why It's Good For The Game
The ocky icky phobia is currently out of date with the hip and cool
lingo the kids are using nowadays, this updates it for any admin feeling
like granting a smite to an intrepid twitch user who is attempting to
emote twitch emotes ic.
This improves the game because it allows for the currently very un-used
ocky icky phobia smite to be more relevant in our current day and age.
## Changelog
:cl:
admin: Updates the admin smite "Ocky icky phobia" with some new words
/:cl:
* Updates ocky icky phobia to more modern lingo
---------
Co-authored-by: Rex9001 <[email protected]>
* Adds new black market pin (#80230) ## About The Pull Request  Adds a fetching red pin which you can wear to visibly align yourself with the enemies of Nanotrasen, purchaseable from the Black Market Uplink. This pin's contained RFID chip will automatically cause you to appear on Sec HUDs as a wanted criminal and will aggro securitrons, to prove your dedication to your ideals. If your convictions are a little less firm, you can also silently pin this onto _other_ people's clothing if they stand still next to you for five seconds... though they might notice that they're suddenly wearing a red badge. I didn't want this to be a subtype of medal so I made the "you can put this accessory onto someone else" behaviour into a component to sidestep object inheritance. This has been applied to the detective's spy camera, because it makes it much easier to turn someone into a mole. This also adds a trait which makes security hate you which I guess someone could use in a novelty bar drink at some point or something. ## Why It's Good For The Game It's funny I think we need more neat things in the black market to make it sometimes worth using ## Changelog :cl: add: Added subversive pins to the black market uplink which make security hate you add: The detective's spy cam can now be conveniently pinned onto people in the same manner as medals /:cl: * Adds new black market pin --------- Co-authored-by: Jacquerel <[email protected]>
* Goose are now birds! (#80206) ## About The Pull Request adds goose (including birdboat) to bird phobia ## Why It's Good For The Game realictic phobea ## Changelog :cl: fix: Bird Phobia now triggers on geese /:cl: --------- Co-authored-by: Aki Ito <11748095+ExcessiveUseOfCobblestone@ users.noreply.github.com> Co-authored-by: MrMelbert <51863163+MrMelbert@ users.noreply.github.com> * Goose are now birds! --------- Co-authored-by: Buyrcsp2 <[email protected]> Co-authored-by: Aki Ito <11748095+ExcessiveUseOfCobblestone@ users.noreply.github.com> Co-authored-by: MrMelbert <51863163+MrMelbert@ users.noreply.github.com>
* Fixes loadout menu * Refactor this a bit * Whoops * Update modular_skyrat/modules/loadouts/loadout_ui/loadout_manager.dm --------- Co-authored-by: GoldenAlpharex <[email protected]>
add towel inhand sprites
…725) Early mirror to fixing images not allowing clicks
* Fixes runtime from cyber implants (#80429) ## About The Pull Request Adds an unregister signal that was missing ## Why It's Good For The Game Runtimes from changing implant bad. ## Changelog :cl: fix: Fixes runtime from augments not unregistering a signal. /:cl: * Fixes runtime from cyber implants --------- Co-authored-by: Diamond_ <[email protected]>
* Makes borg items slightly animated (#80434) * Makes borg items slightly animated --------- Co-authored-by: MelokGleb <[email protected]>
…MDB IGNORE] (#25709) * Fixes nukeop and elite nukeop TGC cards not having sprites (#80417) ## About The Pull Request Closes #80406 This was accidentally caused by #72517 and the holograms still trying to read from the file ## Why It's Good For The Game Holograms are meant to show the card ## Changelog :cl: fix: Made Syndicate Stormtrooper and Nuclear Operative TGC card holograms visible again /:cl: * Fixes nukeop and elite nukeop TGC cards not having sprites --------- Co-authored-by: StaringGasMask <[email protected]>
* qm uses cargo tech icons (#80411) ## About The Pull Request qm now has the "x-in-training" intern title instead of "intern quartermaster", similar to the other heads of staff qm's hud and id card icons are now the cargo tech's icon (but blue), following the other heads of staff which have blue versions of their dept (blue wrench, blue cross, blue... sec badge?? blue exclamation) ## Why It's Good For The Game makes the qm more visually consistent with the other heads, makes it clearer that the qm is the head of the cargo department the current icon is kinda hard to decipher too? you can tell the cargo tech's is a crate, but the qm's is just like a weird head dude (which the hop already has, making those ids hard to distinguish since theyre both blue)? or a stamp? ## Changelog :cl: fix: qm's intern id is now "quartermaster-in-training" image: qm's id and hud icons now use the cargo tech icons (but blue) to be consistent with the other heads of staff /:cl: * qm uses cargo tech icons * Removes the icon from modular as well * Revert "Removes the icon from modular as well" This reverts commit ac6a674887ca74a1231a8433373d55aa8df912e8. * Updates the modular icons with the new version --------- Co-authored-by: Fikou <[email protected]> Co-authored-by: Giz <[email protected]>
… (#25721) get biblical Co-authored-by: Hatterhat <[email protected]>
…#25703) * Resprites SpacePol Uniforms (And accessories) (#80317) * Resprites SpacePol Uniforms (And accessories) * Digi refit by Orion --------- Co-authored-by: OrionTheFox <[email protected]> Co-authored-by: Giz <[email protected]>
commit d537e88ba23629870d4c1425f6c74193419e0531 Author: StrangeWeirdKitten <[email protected]> Date: Fri Dec 22 22:08:18 2023 -0700 Update sol_defense.dm commit a6f98b9260baaedbf83ed1f1d2c0606bdaea5d0c Merge: 9aceef22008 ea5eef9e700 Author: The Sharkening <[email protected]> Date: Fri Dec 22 21:52:56 2023 -0700 Merge branch 'master' into upstream-12/19/2023 commit 9aceef22008bcb20d4adb188c535fe8df1a105e1 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 23:30:07 2023 -0700 ^ This is a combination of 14 commits. ^ This is the 1st commit message: Upstream merge edits removes nulls JSX memes Update FishingBubbers.tsx repathing on maps Update tgstation.dme prettier + GOBLIN gas canister color Update secmed_closet.dmi I fixed shadekin, your welcome I fixed this too waddle failed my CI :( Reduces the frequency of miasma notifications/coughing. (#872) Automatic changelog for PR #872 [ci skip] Rebalances Sol-Fed Guns (#831) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> This rebalances Sol-Fed weapons to be more distinct from each other, as well as more balanced in terms of damage and wounding. Update: Now has basic Sol-Fed sidearms behind weapons permit. This also makes each of the Sol-Fed weapons distinct from each other in purpose, rather than all of them being all-around. All weapons have been tested by hand!    <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> This prevents antagonists from being shut down in 5 seconds (hyperbole) like before, and allows for longer engagements and more opportunities to escape. Which is always better. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender, The Sharkening add: Department ordered SolFed guns add: Sol-Fed Firearms now require a weapons permit to order fix: Adjusted Sol weapon balance to be more fair /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> --------- Co-authored-by: The Sharkening <[email protected]> Automatic changelog for PR #831 [ci skip] This thing is still busted More shadekin memes Update human.dm Update human.dm I fixed the changelog more changeling oh I inch closer to removing shadekin please.. Ok Kilo Stuff I wasted 4 fucking hours over this one line sure god dammit Reverts icon cutting (for now) [MIRROR] Fixes input boxes [No gbp] [MDB IGNORE] (#25761) * Fixes input boxes [No gbp] (#80490) One of the quirks of react is that we're no longer using onChange the same as Inferno - React's version is a synthetic event. I made the mistake of thinking it would be okay. Many interfaces are using onChange events to send messages to byond (very laggy), others are using it to close the input (closes each keypress). So this was the alternative- I hope to replicate the behavior via onBlur && onEnter. I went through to undo most of the onInput -> onChange replacements of #80340 where it made sense. Other inputs which should safely use onChange (DEFINITELY to send messages) remain as such. Example of an input which used onChange now working with this PR:  Bug fixes Fixes #80486 :cl: fix: Name input in character setup should work properly now. fix: Many inputs should feel more responsive. /:cl: * Fixes input boxes [No gbp] --------- Co-authored-by: Jeremiah <[email protected]> [MIRROR] Fixes dropdowns not rendering the selection's displayText post-selection [MDB IGNORE] (#25755) * Fixes dropdowns not rendering the selection's displayText post-selection (#80464) What it says on the tin. Dropdowns were displaying the `selection` instead of `displayText` after clicking an option. Note: this bug was only _noticeably_ affecting dropdowns whose `selection` differed from `displayText`. In the below example, the dropdown uses numeric indices for `selection` and a string for `displayText`. In those cases the `displayText` should take precedence, not the other way around. <details><summary>From this</summary>  </details> <details><summary>To this</summary>  </details> Fixes a minor bug :cl: fix: fixes some dropdowns not displaying the right text after selecting something /:cl: * Fixes dropdowns not rendering the selection's displayText post-selection --------- Co-authored-by: Bloop <[email protected]> [MIRROR] Fixes some missed tgui files that needed prettier run on them [MDB IGNORE] (#25756) * Fixes some missed tgui files that needed prettier run on them (#80466) that's it. Nothing player facing * Fixes some missed tgui files that needed prettier run on them --------- Co-authored-by: Bloop <[email protected]> Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. I wasted 4 fucking hours over this one line sure god dammit Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee ^ This is the commit message #2: removes nulls ^ This is the commit message #3: JSX memes ^ This is the commit message #4: Update FishingBubbers.tsx ^ This is the commit message #5: repathing on maps ^ This is the commit message #6: Update tgstation.dme ^ This is the commit message #7: prettier + GOBLIN gas canister color ^ This is the commit message #8: Update secmed_closet.dmi ^ This is the commit message #9: I fixed shadekin, your welcome ^ This is the commit message #10: I fixed this too ^ This is the commit message #11: waddle failed my CI :( ^ This is the commit message #12: Reduces the frequency of miasma notifications/coughing. (#872) ^ This is the commit message #13: Automatic changelog for PR #872 [ci skip] ^ This is the commit message #14: Rebalances Sol-Fed Guns (#831) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> This rebalances Sol-Fed weapons to be more distinct from each other, as well as more balanced in terms of damage and wounding. Update: Now has basic Sol-Fed sidearms behind weapons permit. This also makes each of the Sol-Fed weapons distinct from each other in purpose, rather than all of them being all-around. All weapons have been tested by hand!    <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> This prevents antagonists from being shut down in 5 seconds (hyperbole) like before, and allows for longer engagements and more opportunities to escape. Which is always better. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender, The Sharkening add: Department ordered SolFed guns add: Sol-Fed Firearms now require a weapons permit to order fix: Adjusted Sol weapon balance to be more fair /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> --------- Co-authored-by: The Sharkening <[email protected]> Automatic changelog for PR #831 [ci skip] This thing is still busted More shadekin memes Update human.dm Update human.dm I fixed the changelog more changeling oh I inch closer to removing shadekin please.. Ok Kilo Stuff I wasted 4 fucking hours over this one line sure god dammit Reverts icon cutting (for now) [MIRROR] Fixes input boxes [No gbp] [MDB IGNORE] (#25761) * Fixes input boxes [No gbp] (#80490) One of the quirks of react is that we're no longer using onChange the same as Inferno - React's version is a synthetic event. I made the mistake of thinking it would be okay. Many interfaces are using onChange events to send messages to byond (very laggy), others are using it to close the input (closes each keypress). So this was the alternative- I hope to replicate the behavior via onBlur && onEnter. I went through to undo most of the onInput -> onChange replacements of #80340 where it made sense. Other inputs which should safely use onChange (DEFINITELY to send messages) remain as such. Example of an input which used onChange now working with this PR:  Bug fixes Fixes #80486 :cl: fix: Name input in character setup should work properly now. fix: Many inputs should feel more responsive. /:cl: * Fixes input boxes [No gbp] --------- Co-authored-by: Jeremiah <[email protected]> [MIRROR] Fixes dropdowns not rendering the selection's displayText post-selection [MDB IGNORE] (#25755) * Fixes dropdowns not rendering the selection's displayText post-selection (#80464) What it says on the tin. Dropdowns were displaying the `selection` instead of `displayText` after clicking an option. Note: this bug was only _noticeably_ affecting dropdowns whose `selection` differed from `displayText`. In the below example, the dropdown uses numeric indices for `selection` and a string for `displayText`. In those cases the `displayText` should take precedence, not the other way around. <details><summary>From this</summary>  </details> <details><summary>To this</summary>  </details> Fixes a minor bug :cl: fix: fixes some dropdowns not displaying the right text after selecting something /:cl: * Fixes dropdowns not rendering the selection's displayText post-selection --------- Co-authored-by: Bloop <[email protected]> [MIRROR] Fixes some missed tgui files that needed prettier run on them [MDB IGNORE] (#25756) * Fixes some missed tgui files that needed prettier run on them (#80466) that's it. Nothing player facing * Fixes some missed tgui files that needed prettier run on them --------- Co-authored-by: Bloop <[email protected]> Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. I wasted 4 fucking hours over this one line sure god dammit Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee Revert "Fixes false walls icons using the wrong icon files (#80175)" This reverts commit 4dce402e72d97b462f647422ee4fd31a17c9c1c2. Revert "Reverts icon cutting (for now)" This reverts commit f07448f4fca2d1d47a7a0c49e82289fcbca0f758. lets gooo Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> Automatic changelog for PR #878 [ci skip] Automatic changelog for PR #877 [ci skip] wee commit ea5eef9e7001d6cae4a5ac2c09e88aeee98267ea Author: Bubberbot <[email protected]> Date: Fri Dec 22 05:51:27 2023 +0100 Automatic changelog for PR #877 [ci skip] commit 1f2f7ecad14ab324c7ee7de7ee8459c533d56ea9 Author: Bubberbot <[email protected]> Date: Fri Dec 22 05:51:07 2023 +0100 Automatic changelog for PR #878 [ci skip] commit ea7f9a92c3b06c3bc8ce31ff0cc0e79703cb7fcd Author: ReturnToZender <[email protected]> Date: Thu Dec 21 22:43:24 2023 -0600 Oops! Broke Job Estimation! (#877) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request I accidentally removed an ! in the job estimation menu, meaning only people who switched it on actually showed on the menu. Of course, people didn't know this, and didn't turn off their preference. This fixes that.  <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> ## Why It's Good For The Game Job estimation my beloved <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> ## Changelog <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: job estimation actually works again /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> commit 756f2af7c7d45247ded402a79853b62edf461b02 Author: ReturnToZender <[email protected]> Date: Thu Dec 21 22:43:03 2023 -0600 Fixes Obscurity examine prefs runtiming on punpun (#878) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request The PR that added this forgot to include a ?, so mobs without a client return null, and runtime, sometimes hundreds of time in a shift. This PR fixes that. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> ## Why It's Good For The Game Less runtimes in the runtime menu. God there's so many. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> ## Changelog <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender fix: Runtime on obscurity examine prefs when the mob in question lacks a client /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> commit 4189c63872dd00d19b38d94c018917826430cfc2 Author: Bubberbot <[email protected]> Date: Thu Dec 21 03:51:11 2023 +0100 Automatic changelog for PR #831 [ci skip] commit 739b2865783e57ea3e76fc1b9376f4cd8d361510 Author: ReturnToZender <[email protected]> Date: Wed Dec 20 20:50:02 2023 -0600 Rebalances Sol-Fed Guns (#831) <!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request This rebalances Sol-Fed weapons to be more distinct from each other, as well as more balanced in terms of damage and wounding. Update: Now has basic Sol-Fed sidearms behind weapons permit. This also makes each of the Sol-Fed weapons distinct from each other in purpose, rather than all of them being all-around. All weapons have been tested by hand!    <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <!-- Please make sure to actually test your PRs. If you have not tested your PR mention it. --> ## Why It's Good For The Game This prevents antagonists from being shut down in 5 seconds (hyperbole) like before, and allows for longer engagements and more opportunities to escape. Which is always better. <!-- Argue for the merits of your changes and how they benefit the game, especially if they are controversial and/or far reaching. If you can't actually explain WHY what you are doing will improve the game, then it probably isn't good for the game in the first place. --> ## Changelog <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: ReturnToZender, The Sharkening add: Department ordered SolFed guns add: Sol-Fed Firearms now require a weapons permit to order fix: Adjusted Sol weapon balance to be more fair /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> --------- Co-authored-by: The Sharkening <[email protected]> commit 250f472b8a0be98d06a4efad5bdf333bec64848d Author: Bubberbot <[email protected]> Date: Wed Dec 20 18:11:22 2023 +0100 Automatic changelog for PR #872 [ci skip] commit 8d86f7e276e5e23f207818bf3b7e5f8600dc5131 Author: BurgerLUA <[email protected]> Date: Wed Dec 20 09:10:54 2023 -0800 Reduces the frequency of miasma notifications/coughing. (#872) commit 4d98032937bc448bc29a0d4ce69b4138f89efaa2 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 22:51:56 2023 -0700 Revert "jsx renames (hopefully shit works)" This reverts commit 4d766404667227ea6efd00be7b7773345009bd55. commit 4d766404667227ea6efd00be7b7773345009bd55 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 21:14:39 2023 -0700 jsx renames (hopefully shit works) commit 308bd0af73fc1e1609cd73fc08fb74d47529ec01 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 20:38:12 2023 -0700 i hate tgui commit deaf6b7a285bfbe7c687028c56bf22d8c38f651f Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 19:15:30 2023 -0700 Revert "Merge branch 'upstream-12/19/2023' of https://github.com/StrangeWeirdKitten/Bubberstation into upstream-12/19/2023" This reverts commit 49d8e940ebc5df43b5b6bf0770185fcbaa682b87, reversing changes made to 4bc68a4cb7d02eb74d03a430de934bddb229bcfb. commit 49d8e940ebc5df43b5b6bf0770185fcbaa682b87 Merge: 4bc68a4cb7d 515d34fa683 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 19:09:48 2023 -0700 Merge branch 'upstream-12/19/2023' of https://github.com/StrangeWeirdKitten/Bubberstation into upstream-12/19/2023 commit 4bc68a4cb7d02eb74d03a430de934bddb229bcfb Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 19:09:37 2023 -0700 Revert "tgui memes" This reverts commit e0e00728d2f21bbf61a5bf7c5d9e32265fa6d505. commit e0e00728d2f21bbf61a5bf7c5d9e32265fa6d505 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 18:34:12 2023 -0700 tgui memes Update FishingBubbers.tsx Revert "tgui memes" This reverts commit 26907a086b59455a9ac347da4c268cf9c7f529f8. memes tgui fuck shit commit 515d34fa68333dc3d86f69f56c6897b2c70378b4 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 18:51:22 2023 -0700 memes tgui fuck shit commit 44f8f3cea56ad024f85c986837f64661200d80a3 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 18:42:22 2023 -0700 Revert "tgui memes" This reverts commit 26907a086b59455a9ac347da4c268cf9c7f529f8. commit 26907a086b59455a9ac347da4c268cf9c7f529f8 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 18:34:12 2023 -0700 tgui memes Update FishingBubbers.tsx commit a32a682f40d9aa5e01dd277c125a2a7c8ef63612 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 18:10:13 2023 -0700 security medic was murdered commit 27c369d73da847091b81aaf8b7dbef723cebcacd Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 17:29:54 2023 -0700 Removes more synth stuff + rewords defines for borgs commit 154c07704e740a122a13e2b89c8d9bc202b6bff9 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 17:12:44 2023 -0700 Canister + loot removal + synth recharging code removal commit b17764f46c58ceb9818834d426d595e435a19d08 Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 17:03:11 2023 -0700 map fixes for lockers commit d9c129425dc4df90ba938f172474a0b8b1241ecb Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 16:56:44 2023 -0700 Some fixes commit 28a4a6f6e16c2dfe275d5cfdfc21f774d699f35d Author: StrangeWeirdKitten <[email protected]> Date: Tue Dec 19 09:25:32 2023 -0700 missed comma
ReturnToZender
merged commit 9112d58
into
ReturnToZender:nsfw-flavor-text
3 of 4 checks passed
ReturnToZender
pushed a commit
that referenced
this pull request
Jan 7, 2024
…ORE] (#25530) * Organ movement refactor *Un-nullspaces your organs* * Fix conflicts I checked the conflicts on the two weird conflicts and no previous TG pr touches them i assume its just github being github because those shoulden't be conflicts *shrug * Fix #1 uhh...this is going to be a long one * Fix #2 Modular Movement Flags * Fix #3 It builds now * Fix #4 Oh god it builds now, I missed some things * Fix #5 No more Runtimesplosion Now time for Synths * Update nightmare_organs.dm * on_mob_insert * Skyrat-SS13/Skyrat-tg#25664 * Skyrat-SS13/Skyrat-tg#25685 * Skyrat-SS13/Skyrat-tg#25582 * Skyrat-SS13/Skyrat-tg#25686 * bro the fucking brain does not go into the chest. * seriously? undocumented code causing shit. if it breaks ghouls, so be it. --------- Co-authored-by: Time-Green <[email protected]> Co-authored-by: SomeRandomOwl <[email protected]> Co-authored-by: SomeRandomOwl <[email protected]> Co-authored-by: Gandalf <[email protected]>
Swiftfeather
pushed a commit
that referenced
this pull request
Apr 29, 2024
…e (#27489) * More power stuff * Update stomach_ethereal.dm
ReturnToZender
pushed a commit
that referenced
this pull request
Jul 28, 2024
…avaland syndicate base. Attempt #2 (#28934) * Adds holo-fan and two thermomachine and emitters boards to lavaland syndicate base. Attempt #2 (#84934) ## About The Pull Request This is my second PR after I goofed up in the first by merging from master. Adds a holofan and two thermomachine boards to the engineering crate in the lavaland syndie base, and adds two emitter boards to its SM room. ## Why It's Good For The Game The lavaland syndie base has no holofan and is limited to two thermo machines (four thermomachines and only one emitter if the SM shard room spawns). This limits atmospheric gameplay and restricts the freedom in creating set-ups within the base even though it starts with a surplus of T4 parts and four gas miners. These additions should improve the quality of base as an atmospheric ghost role and make it less frustrating when making burns, coolers, dealing with gas leaks Etc. as well as improve SM shard gameplay when creating high power set ups or tesla. ## Changelog :cl: qol: Holofan and two thermomachine and emitter boards in lavaland syndicate base. /:cl: * Adds holo-fan and two thermomachine and emitters boards to lavaland syndicate base. Attempt #2 --------- Co-authored-by: TheNobodyofimportance <[email protected]>
ReturnToZender
pushed a commit
that referenced
this pull request
Oct 24, 2024
# This is the 1st commit message: goofsec hud # This is the commit message #2: the sensible tgui errors # This is the commit message #3: Gets it to compile # This is the commit message #4: Fixes all extra runtimes # This is the commit message #5: TGUI Fixes 1 # This is the commit message #6: Removes open dream check # This is the commit message Bubberstation#7: Ticked file enforcement # This is the commit message Bubberstation#8: A very harsh way to fix potential missing icons # This is the commit message Bubberstation#9: Wipes all the errorneously included files # This is the commit message Bubberstation#10: fixes missing_icons fully # This is the commit message Bubberstation#11: some extra ci fixes before I forget # This is the commit message Bubberstation#12: WHAT THE FUCK # This is the commit message Bubberstation#13: really? # This is the commit message Bubberstation#14: bye waffles # This is the commit message Bubberstation#15: hooray
Swiftfeather
pushed a commit
that referenced
this pull request
Nov 25, 2024
…embeds on reagent spears (Bubberstation#2536) ## About The Pull Request This fixes two bugs. Quoting the player reported bug: Exploit #1 - Twitch Next Move modificier This one needs a bit of VV to test mostly to speed up the conditions, but its doable by a miner with a bit of skill. You need the berserker armor of the chosen one, or alternatively, the Dexterous mutation and a way to remove it (I think you can do so with mutadone, but never tried, regardless!) and Twitch. The short of it is that the next move variable is always edited with multiplicatives, but Twitch was coded with an aditive, so alternating one and another and then removing one effect and letting the other run out would net you with a lower than 1 next move variable. Exploit #2 - God Slaying Spear This one is easy to test, basically you all got a spear with better embeed stats than ninja stars, which embeeds most of the time thanks to its high embeed chance, plus its armor penetration, with no chance for it to fall out naturally so it has to be manually removed. Since it has a Bulky weight class, its damage multiplier is 4 plus the pain multiplier being 6, doing 24 damage per tick and on extraction, being able to kill a person that gets it in 10 seconds, even if armored. We had a whole clique abusing this as icecats on nova, and even combining it with morphine. https://github.com/Bubberstation/Bubberstation/blob/master/modular_skyrat/modules/reagent_forging/code/forge_weapons.dm#L136 ## Why It's Good For The Game Fixes some seriously game breaking bugs no one's abused yet, discovered on Nova, ## Proof Of Testing <details> <summary>Screenshots/Videos</summary> </details> ## Changelog :cl: OrbisAnima (code for 1), ReturnToZender (code for 2) fix: Fixes an exploit with Twitch, and the reagent forged spear's embedding. /:cl: --------- Co-authored-by: Arturlang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About The Pull Request
Why It's Good For The Game
Changelog
🆑
add: Added new mechanics or gameplay changes
add: Added more things
del: Removed old things
qol: made something easier to use
balance: rebalanced something
fix: fixed a few things
sound: added/modified/removed audio or sound effects
image: added/modified/removed some icons or images
spellcheck: fixed a few typos
code: changed some code
refactor: refactored some code
config: changed some config setting
admin: messed with admin stuff
server: something server ops should know
/:cl: