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

Emote Refractoring #1204

Merged
merged 66 commits into from
Oct 24, 2024
Merged

Emote Refractoring #1204

merged 66 commits into from
Oct 24, 2024

Conversation

MosleyTheMalO
Copy link

@MosleyTheMalO MosleyTheMalO commented Oct 8, 2024

About The Pull Request

Refractors some emote code and functions, and also adds the emote panel to the game.
Also undoes some unnecessary edits from the previous port

Why It's Good For The Game

A Port?

From venus. Made mostly by @Just-Sim

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

Changelog

🆑
add: adds the emote panel using tgui
code: refractors emote code
/:cl:

MosleyTheMalO and others added 30 commits September 21, 2024 14:55
This reverts commit ab2ff17.
SPLURT maintainers try not to fuck up code challenge (impossible!) If you're gonna edit someone else's code, at least understand what you're editing please!
Why is *help so big...
Also got rid of The Great SPLURT fart library.
oops silly me haha i accidentally left these sound files on my repo
might as well update while I'm at it
Reverts changes to *meow, *mrrp
Why are these global variables but you handled emote lists as E.emote_list?? you call them once and never again!!!
adjusted cooldown
This Commit does the following:
Ports the Emote Panel
Turns SPLURT's emotes into /sound/human datums to reduce redundancy in the amount of emotes.
Gives almost every emote a name for the emote panel.
Removes herobrine.
Also fixes a few errors and makes sure all files are properly tested for Pull Request
[PORT] Modern TG Emote Panel
silicons can now use the emote panel
@@ -1,3 +1,31 @@
/* SPLURT STATION Guide:
Copy link

Choose a reason for hiding this comment

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

Hm. I Don't Think This Is Right.

Just-Sim and others added 13 commits October 8, 2024 01:50
…lls?

AKA [PORT] Refactors smiting, adds smite build mode tgstation/tgstation#55327

!! NOT!! A!! RUNNING!! BUILD!!

LIGHTNING IS BROKEN
IMMERSE IS COMMENTED OUT
HALF OF THIS SHIT IS BROKEN
DO NOT BASE YOURSELF ON THIS BUILD!!
faster aheals faster tickets
this one's coded by me

Context Menu is ordered by category so switching it to Debug bumps it up a few spaces.
Please roleplay appropriately, okay?
Commit can also be called "How I Lost My Entire Playerbase"
Popups, Smites, Immersion, Rejuvenation & the Ocky Icky
@SPLURT-Station-Bot
Copy link
Collaborator

Test Merge Deployed

Details
Server Instance

SPLURT

Revision

Origin: 2633bae
Pull Request: fe54fd6
Server: 27d3ecf338a3cf5f22474bc828332d4e25180dba

Comment

ariaworld and others added 6 commits October 9, 2024 11:22
@SPLURT-Station-Bot
Copy link
Collaborator

Test Merge Updated

Details
Server Instance

SPLURT

Revision

Origin: f11e8e7
Pull Request: fe54fd6
Server: d0807e6c8abd1555879946911e29ed1ba896e03a

Comment

@SPLURT-Station-Bot
Copy link
Collaborator

Test Merge Updated

Details
Server Instance

SPLURT

Revision

Origin: 35474e4
Pull Request: 7c8cbd3
Server: c6403e767187b9010a168b65be64aaf4349cb348

Comment

@MosleyTheMalO MosleyTheMalO merged commit 76661cc into master Oct 24, 2024
6 checks passed
@MosleyTheMalO MosleyTheMalO deleted the content-stuff branch October 24, 2024 20:24
github-actions bot pushed a commit that referenced this pull request Oct 24, 2024
@SPLURT-Station-Bot
Copy link
Collaborator

Test Merge Updated

Details
Server Instance

SPLURT

Revision

Origin: 30d1b15
Pull Request: 7c8cbd3
Server: c22f1f596769b9e57b165db7d127861317ca1275

Comment

@SPLURT-Station-Bot
Copy link
Collaborator

Test Merge Removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants