-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Icon texture does not update on talent or gear swap #749
Comments
I've seen this as well when I wear two on-use trinkets and I swap the locations of the two trinkets. |
Even worse for Havoc Demonhunters (after merging Hemario@b2fd91d)
Casting meta will not change the texture of the icons |
Here is an even simpler example:
Using this script, Ovale will show "0.0" in the icon frame, but if you toggle the checkbox, it doesn't change to "1.0" unless you reload the UI. I've turned on debugging for OvaleCompile and it does say:
|
I've figured out why this is happening, and I will have a pullup for this issue shortly. |
The "constant" optimization in RecursiveCompute() was returning the wrong results for "if" nodes when the result was a "value" node because the cached result from a previous computation wasn't being updated if the condition's computed result was different than before. Remove the "constant" optimization from RecursiveCompute() and add a new visitor computeUndefined() to visit any nodes of type "undefined" that were generated by parsing typed functions. Export type AstUndefinedNode in engine/ast so that in can be used in engine/runner in computeUndefined(). This fixes part of issue Sidoine#749.
The idiom that was used to set the texture was unnecessary when the variables were all local, but is plain wrong when using "result" fields because it would never overwrite a cached result's texture. Simplify by using a simple if-else statement to set the texture. This fixes the rest of issue Sidoine#749.
The idiom that was used to set the texture was unnecessary when the variables were all local, but is plain wrong when using "result" fields because it would never overwrite a cached result's texture. Simplify by using a simple if-else statement to set the texture. This fixes issue Sidoine#749.
The idiom that was used to set the texture was unnecessary when the variables were all local, but is plain wrong when using "result" fields because it would never overwrite a cached result's texture. Simplify by using a simple if-else statement to set the texture. This fixes issue #749.
@Hemario I think this is fixed in Ovale 9.0.34. Can you verify? |
Describe the bug
Icon texture does not update for spells when they get replaced by another spell due to talent swaps.
Icon texture of trinket does not change when you swap trinket.
To Reproduce
Custom script to test (protection paladin)
Expected behavior
The icon textures should update according to the newly selected talent or trinket.
After a /reload the item texture is correct.
The text was updated successfully, but these errors were encountered: