oUF

97.2k Downloads

Portraits not updating properly?

Kkthnx opened this issue · 5 comments

commented

Describe the bug

When I shift into Storm, Earth, and Fire as a Monk. My portraits should change as my model does. Yet sometimes they do not update when I shift into this new model and sometimes they do. If I re-target myself, it will update it.

Do note I do not recall any changes I have made to KkthnxUI to affect this. I remember a bug report I made maybe around a year ago about the same issue which was solved though I did not see this issue again until Prepatch.

How to reproduce

I guess you could use a toy to shift into a different model or use the said spell above on a monk in Windwalker spec.

What did you expect?

Portraits should update based on your model change and not stick to a previous change model after you left that model or changed into a different model.

Error report

Screenshots

unknown
unknown

commented

What surprises me is that for some reason it doesn't count as a model change, so UNIT_MODEL_CHANGED doesn't fire 🤔 Oh well...

commented

Sup!

I'm not exactly sure what to do about portraits, they've been a pita for quite a while. Thing is, no portrait-related events fire when you cast that spell, so it doesn't update. The same happens in the default UI, btw.

image

The only solution I see so far is to try to use UNIT_AURA, but it's a VERY spammy event, and frequent updates of 3D portraits can cause stuttering (irregular fps drops) which isn't ideal, and the portrait itself will be constantly resetting, so it won't animate all that well either. So it's not a solution at all.

commented

That is rough. Honestly, it is so minor that I was debating on reporting it. Though UNIT_AURA honestly is not worth it. I mean if people really care they can do this themselves. Thank you kindly though for the reply and looking into it nonetheless <3

commented

I'll be closing it then. And have a nice day.

commented

Not to necro a closed thread, but we did some more testing, and it turns out a ton of model changes from toys are left out of portrait updates too. So "fixing" this would require a huge list of auraIDs to track, not to mention constantly monitoring UNIT_AURA for this.

So I say screw it! The portrait reflects shapeshifts and vehicles. That's enough.