sPOiDar / fvtt-module-stream-view

FoundryVTT module that provides a minimal UI view with automated camera work, ideal for streaming or recording games.
MIT License
11 stars 11 forks source link

Compatibility Request: Levels Module #21

Closed unrealkakeman89 closed 2 years ago

unrealkakeman89 commented 2 years ago

I would like to request compatibility with the Levels Module. Currently, when using Stream View and Levels, it does follow the player during combat, however, it shows only the uppermost level and all of the tiles from the levels below. It would be ideal if it would use the Tokens vision to see only the level the token is on and only what the token can see. I, unfortunately, am not familiar enough with programming to be of assistance, I am just able to make the request.

sPOiDar commented 2 years ago

This is not something that can be fixed here - this module simply relies on the Observer permission built-in to core Foundry to share vision.

It sounds like the Levels module does not work well when other players have Observer vision, and any changes in that behaviour would have to happen inside the Levels module, so I'd suggest you place a feature request over there.

theripper93 commented 2 years ago

If you wish to make it work with levels, you would need to select the token whenever focusing on it, so that levels can calculate the appropriate levels, that's the extent of what can be done to build some sort of compat. I would assume that to control the token you would need owner permission - after that you could deselect the token to get back to the shared view. As an alternative you could call _levels._onElevationChangeUpdate(token) whenever you focus on a token and it should (tm) work.

sPOiDar commented 2 years ago

Stream View doesn't really focus a specific token, it just adjusts the camera bounds to keep tokens in view (in automatic mode, in directed mode, tokens aren't considered at all). That said, this is not just a problem for Stream View, but for any game where players want shared party vision.

IMO if Levels wanted to properly support observers or multiple owners, it would push updates to players that have vision perms for the token when the token's elevation changes.

theripper93 commented 2 years ago

Well, then I guess compat is not really possible , if I have some extra time I might do something as you suggest