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

Added an option to change player speed while playing an emote #97

Merged
merged 3 commits into from
Jun 14, 2021

Conversation

Apehum
Copy link
Contributor

@Apehum Apehum commented Jun 14, 2021

Added an option to change player speed while playing an emote. Making the speed slower prevents players from unrealistically running while using emotes, and it's a nice feature to have.

Also fixed this exception: https://paste.gg/p/anonymous/35f0eb907bfd477aadce8ee2370ad694

@KosmX
Copy link
Owner

KosmX commented Jun 14, 2021

In the future I'll reimplement the Bukkit plugin, also this should be in the server-side mod too.
Now I can merge it and I'll try to keep this feature in the future

if(this.config.getBoolean("slownessWhilePlaying.enabled")) {
if(data.emoteData != null) {
if(!data.emoteData.isInfinite) {
Bukkit.getScheduler().runTaskLater(this, () -> {
Copy link
Owner

Choose a reason for hiding this comment

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

This scheduled task may cause issues, when a player starts an emote, then stops earlier, then starts another.
I'll reimplement it, but it's mostly fine for now.

@KosmX
Copy link
Owner

KosmX commented Jun 14, 2021

Checks are green, merging

@KosmX KosmX merged commit 5ecffad into KosmX:development Jun 14, 2021
@KosmX
Copy link
Owner

KosmX commented Jun 14, 2021

I'll try to do a fix release tomorrow, after that I'll start reimplementing the whole server-side logic

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.

None yet

2 participants