squaremap

squaremap

1.9k Downloads

Add a function to hide or show all players at the same times

HydrolienF opened this issue ยท 4 comments

commented

It would be nice to have a /squaremap show all and /squaremap hide all.

commented

The hidden status is stored in the player data, so we can't do anything about offline players. This means hiding and showing all is not a reversible action if the players online changes in between. This would lead to unexpected behavior and so isn't something I want to include. On Fabric & NeoForge you should be able to use /execute to run the command as each online player. On Paper you would need to use the API.

commented

On second thought, I guess this could be a command that just toggles the player tracker at runtime, which could actually be useful

commented

The solution that I will prefer is a config option that have 3 values: forced_show, forced_hide, player_setting. (Could be editable through command tho).

  • player_setting is the current behavior where it use visibility stored on player data.
  • forced_visible will make player visible no mater if they have run /map hide
  • forced_show will make player visible no mater if they have run /map show

If you agree with that new feature, I can try to do it and send a pull request.

commented

@jpenilla if I add this feature to my forks, would you be interested on a pull request ?