KayelGee / token-attacher

MIT License
11 stars 18 forks source link

Select and Assign a Pivot (feature request) #54

Closed MarkPearce closed 3 years ago

MarkPearce commented 3 years ago

Feature Request.

With vehicles and complicated rigs (eg bailywiki) it's common to use a controller token and link everything to that for moving large vehicles. But this leaves rotation at some strange offset outside the actual vehicle.

This is similar to the vehicles mod which lets you name a token in the scene to use as a pivot - BUT using a token actor is a poor choice as tokens within a vehicle are likely to overlap which risks accidently moving the pivot, or being unable to select a token due to zdepth layering problems. A shape or tile would be better, or even just enter an offset value in X/Y grid spaces.

Bonus points if when targeting a "controller" it can report its position as the pivot location, allowing players to utilize automation in vehicle combat (in systems that support it)

KayelGee commented 3 years ago

I've thought about that and I'm not sure this is the right way to do it. In my option the base token is the pivot, if you need another pivot you can attach the token to another.

This isn't working with v&m because a token with an attached drawing can't be inside that drawing. So the solution you're describing is just trying to find a workaround for that. I'd rather have a solution where the token can be inside of a v&m drawing instead of a more complex workaround for that.

KayelGee commented 3 years ago

The next update will have an option to enable a base token to be inside of a v&m drawing without being moved by anything related to mlt/v&m which might solve the issue you're having.

MarkPearce commented 3 years ago

Only the problem of Z-fighting with other tokens would remain....

KayelGee commented 3 years ago

I think you'll love to hear of the https://github.com/theripper93/token-z module by theripper93 :D

KayelGee commented 3 years ago

I'll close this for now as I think with the new update and token-z module this should be in a working state

MarkPearce commented 2 years ago

Loving the new update. Removing the redundant controller tokens now. Thanks!

On Mon, Sep 13, 2021 at 11:28 PM KayelGee @.***> wrote:

I'll close this for now as I think with the new update and token-z module this should be in a working state

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/KayelGee/token-attacher/issues/54#issuecomment-918591892, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVHG6W57EHNEYWEUIOG5BTUBZUGHANCNFSM5D3OVAQQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.