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

Teleport Command Rework #3

Closed
sim2kid opened this issue Jan 3, 2022 · 1 comment
Closed

Teleport Command Rework #3

sim2kid opened this issue Jan 3, 2022 · 1 comment
Labels
enhancement Modification to current Feature/System implemented The issue has been implemented into the project

Comments

@sim2kid
Copy link
Owner

sim2kid commented Jan 3, 2022

Provide different options to accessing the teleportation commands.

Currently

Currently, the tp command is split into two different sections with two different names yet are also nested under the /waystones command.
/waystone tp <waystone>
/waystone teleport <username> <waystone>
This is a bit clumsy and discourages players from teleporting to non-public waystones not owned by them.

Potential Solutions

There are many things we can do. First, we want to unnest the TP commands from inside of the /waystone command.
This would mean we give them their own command.
In theory, we can replace the Minecraft tp and the Essentials tp and province our own root /tp <waystone> command.

We can also use /warp, /way, and /teleport for this command as well.

Another thing we can do is smoosh the two current TP commands together. Potentially something like /tp <waystone> <username> where same-named waystones are smooshed together then discriminated in the username section. We will have to tell the player that there are multiple waystones in that case.

In addition to the two above solutions, we should also include filtering of the Tab responses as we are typing. This would allow people to see the waystone they're thinking of come up to the top while typing the command.

We could also solve this the way Minecraft did and have selections look like the following. <creator>:<waystone> => sim2kid:Home. Then when a player types in a waystone name, the tab results for multiple waystones will pop up, and then the player can discriminate between them. The problem here is the default cases. Who would get picked initially? The author is always first, then likely in alphabetical order.

@sim2kid sim2kid added the enhancement Modification to current Feature/System label Jan 3, 2022
@sim2kid sim2kid added this to the Road to True Plugin milestone Jan 3, 2022
@sim2kid
Copy link
Owner Author

sim2kid commented Jan 6, 2022

Solved in Release 1.1.3

@sim2kid sim2kid closed this as completed Jan 6, 2022
@sim2kid sim2kid added the implemented The issue has been implemented into the project label Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Modification to current Feature/System implemented The issue has been implemented into the project
Projects
None yet
Development

No branches or pull requests

1 participant