search

jezzi23 / stat_weights_classic

Stat Weights Classic - Addon for World of Warcraft Classic
5 stars 3 forks source link

Default target level won't update #14

Closed ITJammish closed 4 years ago

ITJammish commented 4 years ago

Hello. Firstly amazing addon, thank you!

Briefly: The default target level of my alt mage is stuck on 59 despite my character being level 60. Any editing of the field is not saved and the values for spell hit % are not updated on the spell tooltip overlay (spells continue to be seen as having a 99% chance to hit). Creating a new profile does not help, or changing the value and logging out etc. I don't think it's ever accepting a change from '59'.

My main is a 60 Druid and has the default target value of 63 as I would expect, and the spell hit values seem accurate. When I installed the addon, the Druid was already lvl 60 while the mage was still somewhere mid 50's. I'm wondering if the character levels are snap-shotted when the addon is first used but not updated? If my mage had been lvl 56 when i'd installed the addon, this would explain why the default target is locked to 59.

I think a reinstall and re-setup would resolve the issue, but in case you want me to try anything I'll keep things as they are for a while.

Thanks again!

jezzi23 commented 4 years ago

Hey thanks!

I checked this issue out and it turns out that you have to press enter or escape after having edited the field for it to lose focus and correctly update. Which is admittedly pretty bad UI functionality :/

ITJammish commented 4 years ago

Ah ha! Thank you, that does work. I'll close the issue, but maybe consider it a minor improvement for the future.

Definitely one of the best add-ons I've used!