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

(old and new block on walk) and (single walk event) events. #716

Closed
FUZIK opened this issue Jul 10, 2017 · 3 comments
Closed

(old and new block on walk) and (single walk event) events. #716

FUZIK opened this issue Jul 10, 2017 · 3 comments
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.

Comments

@FUZIK
Copy link

FUZIK commented Jul 10, 2017

(old and new block on walk) and (single walk event) events.

examples:

on step on ores:
	broadcast "%past location%"
	broadcast "%future block%"

please add single walk event

on step:
	broadcast "top-top-top of %player%" to block

spigot api:

methods:

getFrom(); setFrom(loc);
getTo(); setTo(loc).

thanks to all and Bensku. ✌️

@TheBentoBox TheBentoBox added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Jul 16, 2017
@Snow-Pyon Snow-Pyon added the priority: low Issues that are not harmful to the experience but are related to useful changes or additions. label Jan 31, 2018
@Nicofisi
Copy link
Member

Wouldn't it hurt performance? If not, I can just add it because it's rather trivial

@bensku
Copy link
Member

bensku commented Jul 12, 2018

Skript doesn't have generic on move for performance reasons. It is likely that even excessive use of specific walk events might cause lag.

@Nicofisi
Copy link
Member

Nicofisi commented Jul 12, 2018

Can this be marked as wontfix then, or will we keep it open until someone rewrites skript in bytecode for performance lolol

On the other hand and for real though, while I understand it might lag a lot, I see how it might be considered a rather basic feature by some people, hmm. How about implementing this, with a config option to enable the whole thing?

@FranKusmiruk FranKusmiruk added the completed The issue has been fully resolved and the change will be in the next Skript update. label May 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.
Projects
None yet
Development

No branches or pull requests

7 participants