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

[Feature Request] Password Tag and More #58

Open
rabrunos opened this issue Jul 6, 2023 · 8 comments
Open

[Feature Request] Password Tag and More #58

rabrunos opened this issue Jul 6, 2023 · 8 comments
Assignees
Labels
enhancement New feature or request

Comments

@rabrunos
Copy link

rabrunos commented Jul 6, 2023

Password TAG

The portal now has two TAG fields:

  • NAMETAG: The default tag for portals in the game.
  • PASSTAG: A tag that works like a password.

If the destination portal has a PASSTAG, then the origin portal must also have the same PASSTAG to be able to teleport to it.

portal

If the Destination Key field has some text and the destination portal has no password, this does not interfere to teleport!
This allows to use the mod on a PvP focused server, where only those who know the password can teleport in my portal!

Remove Distance

Add an option in the configuration to remove the display of the distance between the portals so that it is not possible to know how many meters the user is from the other portal. (And that the configuration is enforced by the server configuration)

Show List on Enter

An option in the client-only configuration that when enabled makes it so that when you try to go through the portal, the list appears and then you press a button to teleport. That way I don't have to aim at the portal and press E every time I have to change the destination and I also don't accidentally teleport before selecting the destination.

@rabrunos rabrunos added the enhancement New feature or request label Jul 6, 2023
@rabrunos rabrunos changed the title [Feature Request] Password Tag [Feature Request] Password Tag + Remove Distance Jul 6, 2023
@rabrunos rabrunos changed the title [Feature Request] Password Tag + Remove Distance [Feature Request] Password Tag and More Jul 6, 2023
@SokBojowy
Copy link

SokBojowy commented Jul 11, 2023

Big YES! for Password Tag. I created my account just to support this post. Great and very needed idea.

@SpikeHimself
Copy link
Owner

Hi, thank you for your support, and sorry for the extremely late response. Life's been very busy.

Password TAG

I have to give this a bit more time in my head. I'm unsure if adding a password is feasible, as you'd need a field to set a password as well as one where you enter it when selecting a new destination, and it will quickly get clunky. Also this would have to be an optional component to the UI, as not everyone will want this, so I have to redo some of the UI code for that too.

Remove Distance

The next XPortal update will include a config option HidePortalDistance which allows you to remove the distance indicator from the list of portals.

Show List on Enter

By 'Enter' do you mean entering the portal (i.e. walking into it)? I think that will quickly get annoying as most of the time you'd just want to travel. Maybe I misunderstood.

@rabrunos
Copy link
Author

rabrunos commented Sep 11, 2023

Yes, my suggestion is to show the list when you enter the portal. I understand that it can be annoying when you just want to travel! That's why I suggested it as an "option", so you can activate or deactivate it in the options, that way those who want the option active (like me) could just activate it! (It could be disabled by default!)

I completely understand about the password, as it would be a new system and you would have to change the UI. But it was just an idea and it would be interesting to have it in the mod, since there is no portal mod with this, it would be a totally unique mod. I hope you give this idea some thought :)

Sorry about the English, I try to say it in the best possible way using DeepL, as English is not my language! If you don't understand anything, let me know and I'll try to explain it better!

@SpikeHimself
Copy link
Owner

Yes, my suggestion is to show the list when you enter the portal. I understand that it can be annoying when you just want to travel! That's why I suggested it as an "option", so you can activate or deactivate it in the options, that way those who want the option active (like me) could just activate it!

Does this include portals that already have a destination configured? Or only for portals that don't have a connection yet?

If it is only for portals that aren't connected yet, then it is the same request as #63, which I think might be a nice addition.

If it's for all portals (connected or otherwise), I fear that, even though you say you do want this, you will inevitably end up annoyed by it, and so it's a lot of effort for nothing.

Additionally, with the request in #42 going live soon, you'll be able to immediately connect portals upon building them, which might clash with what you're asking.

@rabrunos
Copy link
Author

I understand about the Default Portal feature.
The 2 really can't work at the same time, unless "Default" was different from just checking a checkbox.
If "Default" when active forces all portals to travel to it, then "Show on Enter" is not compatible.

There are only 2 ways for them to be compatible:
1 - There is another Default check, something like "Go To Default" active by default (not feasible).
2 - There are both options in the configuration to be activated, only one of which can be activated at the same time.

I can only see these solutions!

@SpikeHimself
Copy link
Owner

If "Default" when active forces all portals to travel to it

No, that is not how that feature will work. The Default Portal feature does not make you travel anywhere. It only initiates a connection with a portal, immediately upon building a new portal (so that you don't have to configure the portal first).

You still need to walk into the portal in order to travel.

Does that change how you envision your idea?

@rabrunos
Copy link
Author

rabrunos commented Sep 11, 2023

So if "Default" only serves to define the default connection between portals when the portal is built, then I don't see any compatibility problem. Because the player will only activate Show on Enter if he wants the benefits of this function.

@Jester2020
Copy link

something similar to this would also solve my issue of simply wanting to have some portal be private and not show in the list.

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

No branches or pull requests

4 participants