Slothpala / RaidFrameSettings

GNU General Public License v2.0
4 stars 4 forks source link

Stacks of some debuffs are not showing in Mythic+ #56

Open kpontussvensson opened 1 month ago

kpontussvensson commented 1 month ago

Describe the bug There are certain debuffs where the stacks just won't show for me in a key, it happens every run in a key but I can't replicate it in a normal/heroic/m0. I get no lua errors or anything that would indicate something is wrong otherwise. I've tested it with only RaidFrameSettings enabled and I still get the same results.

The places where it's happening every time in a key for me that I know is:

To Reproduce Steps to reproduce the behavior:

  1. Start a key of RLP (only reproducible in m+)
  2. See if you see the amount of stacks of the debuff on the tank on first boss when the whelp comes.

Expected behavior The stacks should show.

Have you disabled all other add-ons? Yes

Game Version 10.2.7

kpontussvensson commented 1 month ago

https://github.com/Slothpala/RaidFrameSettings/assets/34517108/5f8553f1-4b18-4c60-8cdf-1d34abc3c7c5

Short video above because of the GitHub size limit. Longer video below: https://streamable.com/rnvnpl

Only RaidFrameSettings and SharedMedia enabled on that test.

And here's an image of me going into Normal to test if I get stacks there: image

excorp commented 1 month ago

For normal/heroic/m0 in the Ruby Life Pool, the Primal Chill debuff probably has a duration, but for key it has no duration.

For debuffs with no duration, no cooldown is shown. The stack is under the cooldown, so if the cooldown is not shown, the stack is not shown either. I've made a pull request for this before.

https://github.com/Slothpala/RaidFrameSettings/pull/37

Now I'm using a different workaround.

in Buffs.lua

stackText:SetParent(cooldown)

->

local textFrame = CreateFrame("Frame", nil, buffFrame)
buffFrame.textFrame = textFrame
textFrame:SetAllPoints(buffFrame)
textFrame:SetFrameLevel(buffFrame.cooldown:GetFrameLevel() + 1)
stackText:SetParent(textFrame)

However, creating an extra frame in this way seems to be a speed penalty.

kpontussvensson commented 1 month ago

Ah, it makes sense. I'll grab the workaround for now and hope that it makes it into the main version as well. Thank you @excorp.

Do you have a PR open with the new workaround you're using?

excorp commented 1 month ago

The code in #37 should do the trick. Creating new frames also works, but I'm not sure which is better, as creating a lot of frames seems to cause momentary lag on named names like Volcoross. So I didn't create a pull request that generates a frame.

kpontussvensson commented 1 month ago

Okay. I just saw that #37 was closed.

Slothpala commented 1 month ago

Hey, I am aware of the problem! I am currently working on a complete rewrite of the addon. The new version will move away from reusing Blizzard's aura indicators and I will create my own "class/template" (mix of xml and lua) of aura indicators. I won't be spending much time on the old code anymore.

Slothpala commented 1 month ago

I will look into #37 and apply a temp fix

kpontussvensson commented 1 month ago

Sounds good! 👍

Slothpala commented 1 day ago

Just wanted to let you know that I've uploaded the 3.0.0 beta to Curseforge. If you're interested, you can test it now. Just a heads-up, though, that you'll lose your settings. That's why I'm waiting with the release until after the patch.

kpontussvensson commented 16 hours ago

Cool, I'll try it out over the next couple of days and see if I notice any problems.

I've another issue at times where 1 party member doesn't get the debuff color working but I haven't isolated that with only RaidFrameSettings enabled since it's also very random and hard to replicate so I haven't created an issue for it. It's been like that since as long as I remember so it's not something in the latest releases or so, I think I commented about it on CurseForge a couple of months ago, have you had that happen to you at all?

Slothpala commented 14 hours ago

The Aura Highlight module has been replaced by Debuff Highlight and Buff Highlight and the entire backend of the addon has been changed. The new HealthColor.lua file now always controls the color management of the addon, which fixes some issues in general. I will close all old issues and PRs soon, as it's basically a new addon with some reused aspects.