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

Disabling the explorer feature with the API, user can still trigger the explorer using the mouse wheel #3926

Open
ValdoTR opened this issue May 7, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@ValdoTR
Copy link
Contributor

ValdoTR commented May 7, 2024

I needed to disable the explorer because it was not making sense for my map.
So I used WA.controls.disableMapEditor();. The map editor button is not disabled but I can always trigger the explorer if I un-zoom until triggering it.

Expected behavior

  • after using WA.controls.disableMapEditor(); the player should not be able to trigger the explorer with the mouse wheel either
  • I don't get why we have to call WA.controls.disableMapEditor(); to disable the map explorer. I know they are in the same menu but they are completely different features.
  • we should have a WA.controls.disableMapExplorer(); that remove the feature and disables the button in the map editor menu
  • I also think we should not just disable the feature button but remove it entirely (often when we need to "disable" it, we also want to remove it from the UI because it must not be used for a specific reason)
@ValdoTR ValdoTR added enhancement New feature or request front Front end tasks review UX The UX is not clear and must be review labels May 7, 2024
@gparant gparant removed their assignment May 8, 2024
@gparant gparant removed front Front end tasks review UX The UX is not clear and must be review labels May 8, 2024
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

2 participants