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

Rips out the old keybind system and ports a better version from /tg/ #17216

Merged
merged 16 commits into from Jan 4, 2023

Conversation

ThatLing
Copy link
Contributor

@ThatLing ThatLing commented Dec 29, 2022

Document the changes in your pull request

At the start of the last rebase, I made a shitty keybind system that allowed you to rebind actions. 4 years later, it's about time to upgrade it.

This PR completely removes the old system and ports the new version used by /tg/. It adds more actions and a better way to assign keys.

I tested each keybind to ensure it works correctly, however there might be some issues still. Since it's such a big change, I decided not to convert existing keybinds to the new system. As such, people will need to re-assign their keys.

List of ports below, full credits to the original authors

Wiki Documentation

No clue if there is documentation on the wiki, but this system should work largely similar

Changelog

馃啈
rscadd: Added a new keybind system, your old keybinds have been reset
rscadd: You can now bind a key to a specific emote
/:cl:

@ThatLing ThatLing requested a review from a team as a code owner December 29, 2022 15:08
@Yogbot-13 Yogbot-13 added the Feature This adds new content to the game label Dec 29, 2022
@RG4ORDR
Copy link
Contributor

RG4ORDR commented Dec 29, 2022

Flip bound to scroll wheel.

@ThatLing
Copy link
Contributor Author

Flip bound to scroll wheel.

There's a cooldown, and if people abuse it I will just remove the emote keybinds

@Hopekz
Copy link
Contributor

Hopekz commented Dec 29, 2022

Epic

@Hopekz
Copy link
Contributor

Hopekz commented Dec 29, 2022

Flip bound to scroll wheel.

There's a cooldown, and if people abuse it I will just remove the emote keybinds

What's wrong with flipping fast?

@ToasterBiome
Copy link
Contributor

we should not have bindable emotes

@ThatLing
Copy link
Contributor Author

Why not?

@ToasterBiome
Copy link
Contributor

Why not?

flip and spin spam is annoying

@Hopekz
Copy link
Contributor

Hopekz commented Dec 29, 2022

Why not?

flip and spin spam is annoying

Why not fix it with a different solution like " Flipping too quick gives a chance to knock you on your ass" or " spinning too fast makes your character dizzy"

Why do you think putting more user friendly features in the game like being able to bind things to a hotkey is the issue in this case?

@neeshacark
Copy link
Contributor

id rather we not add macros, ive caught and banned or had banned a surprising amount of people using external clients that allow similar effects like flipping a bunch while moving (including one literal hours ago). these clients also come with exploits like aimbot that are much harder to find and prove.

if for some reason it must be added, at minimum logging so admins can see peoples macros and a max amount/cooldown of macro emotes that doesn't prevent spamming non-macro flips so I can still use this to weed out hacking griefers would be crucial.

@JamieD1 JamieD1 merged commit 50e3477 into yogstation13:master Jan 4, 2023
Yogbot-13 added a commit that referenced this pull request Jan 4, 2023
@burn1cycle
Copy link

Game's broken #17646

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This adds new content to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants