caewok / fvtt-token-visibility

Advanced token visibility rules for Foundry VTT
MIT License
4 stars 8 forks source link

Fix PV compatibility #29

Closed dev7355608 closed 1 year ago

dev7355608 commented 1 year ago

This fixes the issue that constraints applied to the token's LOS polygon (by Vision Limit or Vision Range (In light)) affects all detection modes instead of just basic sight with Area2d.

I don't use _getPolygonConfiguration on purpose, because PV may add radius, which we don't want. Also we can't use _getPolygon, because PV would apply the constraints to it and we don't want that.

Once this is merged, all outstanding issues related to PV can be closed, I think. Everything seems to work as expected now.

caewok commented 1 year ago

Thanks for doing this!