search

theripper93 / Smart-Target

A module to target with Alt+Click and show portraits instead of colored pips to indicate targets
GNU General Public License v3.0
15 stars 10 forks source link

Target image X offset doesn't move the icon, just the image inside the icon #31

Closed susscribe closed 2 years ago

susscribe commented 2 years ago

as stated, 1 example, on the left side a player account logged client while on the right there's the GM account logged client: image

when a change is made: image

it may be my wild imagination but i seem to remember when the module moved the targeting icon altogether instead of the image inside the icon. moving the image without the icon is pointless as it will still overlap with other icons displayed on the token. moving the icon would prevent unreadable overlap.

could you please address this? thank you for you time!

theripper93 commented 2 years ago

that option is there to allign portraits that are now always centered, it's doing what it's supposed to, if you have issues with the target covering the effects, simply make it smaller.

susscribe commented 2 years ago

if i make it smaller enough to not cover it, then i cannot tell which token is targeting anymore, too small for targeting token icon to have a point for being there. is there any other solution where i can keep smart target's distinctive function intact?

theripper93 commented 2 years ago

i'm afraid not - if you have the technical know how you are free to add in the code and send a PR to implement this new feature. if it's clean enough i'll add it in

susscribe commented 2 years ago

fair enough, it is beyond my abilities however. thank you for your time!