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

Allow tokens to carry MBL, like they can with VBL. #2730

Closed
kwvanderlinde opened this issue Jun 11, 2021 · 1 comment
Closed

Allow tokens to carry MBL, like they can with VBL. #2730

kwvanderlinde opened this issue Jun 11, 2021 · 1 comment
Assignees
Labels
documentation needed Missing, out-of-date or bad documentation feature Adding functionality that adds value tested This issue has been QA tested by someone other than the developer.

Comments

@kwvanderlinde
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Tokens are able to block vision, and they are also able to block pathfinding, but only if the AI is set to pathfind around VBL. This means a token cannot block movement without requiring everything else to pathfind around VBL as well.

Describe the solution you'd like
I would like tokens to be able to store MBL, in the same way they store VBL. The "Edit Token" dialog would be extended to allow adding MBL, either separately from VBL or in concert (much like MBL/VBL can be drawn on the map separately or together).

The following macro functions would also be added so that macros can manipulate token MBL analogously to how they can manipulate token VBL:

  • getTokenMBL(id)
  • setTokenMBL(vbl, id)
  • transferMBL(value), transferMBL(value, delete), transferMBL(value, id), transferMBL(value, delete, id)

Describe alternatives you've considered
Tokens can already block movement by setting their terrain modifier to "Block". This is quite a blunt option, though, as the token then blocks entire cells. This makes it unsuitable for representing a glass door on a grid line, for example. It also doesn't allow freely placing irregularly shaped terrain, since the token always blocks its entire footprint. Also, Free Size / Native Size tokens cannot block movement this way.

Regarding the macro functions, another option would be to define generic topology functions that can work equally well with VBL and MBL. Given that we already have VBL-specific functions and some MBL-specific functions, it makes the most sense to at least expand the MBL functions to support token MBL. Generic topology functions would then be redundant, and could even be built as UDFs if desired.

Additional context
This is somewhat of a follow up to #2234 that added functions for drawing MBL on the map. This FR seeks to expand on that and bring parity between MBL and VBL.

Not all VBL functionality requires MBL equivalents. For example, the "Is Visible over FoW" option and the related setAlwaysVisible() macro function are more about a token's relationship with Fog of War than they are about the VBL per se, and there isn't an analogous use for MBL.

@Phergus
Copy link
Contributor

Phergus commented Jul 11, 2022

Tested various iterations of get, set and transferMBL. Worked as expected.

@Phergus Phergus closed this as completed Jul 11, 2022
@Phergus Phergus added the tested This issue has been QA tested by someone other than the developer. label Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation needed Missing, out-of-date or bad documentation feature Adding functionality that adds value tested This issue has been QA tested by someone other than the developer.
Projects
None yet
Development

No branches or pull requests

2 participants