Skip to content
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

Missing actionbar values #48

Open
merijn opened this issue Nov 12, 2022 · 2 comments
Open

Missing actionbar values #48

merijn opened this issue Nov 12, 2022 · 2 comments

Comments

@merijn
Copy link

merijn commented Nov 12, 2022

A number of my action bar buttons don't seem to be getting a predicted "avg value" showing (see screenshots). Some of these are macros (but some other macros show fine) and some are HOTs/DOTs (but not all of them), so I'm a bit lost why this happens. Missing in the screenshot are: Damage/heal values for Shadow Word: Death, Shadow Word: Pain, Devouring Plague, Renew, Flash Heal, Holy Fire, and Desperate Heal.

Note also that the values show on actionbars don't seem to update if/when the spells show do. Action button 2-4 show the heal values of buttons in that position in my "friendly" action bar (hidden because I have a hostile target), rather than updating and showing the damage values for the active action bar's spells.

image

@merijn
Copy link
Author

merijn commented Nov 27, 2022

Ah! I think I have identified the issue. The detection for Dominos is broken when using the latest Dominos, since it was overhauled for retail in Dragonflight. Specifically, these lines: https://github.com/coolmodi/SpellCalc/blob/wotlk/UI/actionbar/buttonText.lua#L197-L199 never run with recent Dominos loaded, since button 1 is just ActionButton1.

By changing it to check for DominosFrame1, instead of DominosActionButton1 the detect is "fixed", but the creation of font strings for some of dominos buttons is still broken and I haven't quite figured out why yet.

@merijn
Copy link
Author

merijn commented Nov 27, 2022

As far as I can tell fontstrings are properly created for all the action buttons, however only the fontstrings for the stock bars are actually updated to show info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant