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]: Modify the Expose options in the token context menu for clarity. #5030

Open
kwvanderlinde opened this issue Nov 4, 2024 · 0 comments
Labels
feature Adding functionality that adds value

Comments

@kwvanderlinde
Copy link
Collaborator

Describe the Problem

When right-clicking a token and choosing one of the options in the Expose menu, it's not always clear what is going to happen.

  • Expose > Last path is good - it exposes along the entire path, or only at way points if "FoW: Expose only at way points" is enabled.
  • Expose > Visible area is confusing - it does what it says if "FoW: Expose only at way points" is disabled. Otherwise, it exposes at all waypoints in the token's last path.
  • Expose > Only currently visible is merely okay - it does what it says, but is easily confused with Expose > Visible area.

The Solution you'd like

Change the available options in the Expose menu, based on whether "FoW: Expose only at way points" is enabled:

Option Enabled Disable
Last path Exposes at the way points of the last path Exposes at every point along the last path
Last path, waypoints only Exposes at every point along the last path Exposes at every point along the last path
Visible area Exposes at the current point Exposes at the current point

Alternatives that you've considered.

Also wondering if Expose > Last path should be disabled if "FoW: Expose only at way points" is enabled, but I fear that will lead to too much surprise.

As a complete alternative, we could leave the current functionality in place but rename the various options to make it clearer. The problem I see with this is that the "FoW: Expose only at way points" suggests a more restrictive mode, but the Expose > Visible area currently becomes less restrictive. And rewording that options to communicate as much seems like a tall task.

Additional Context

No response

@kwvanderlinde kwvanderlinde added the feature Adding functionality that adds value label Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding functionality that adds value
Projects
None yet
Development

No branches or pull requests

1 participant