Slothpala / RaidFrameSettings

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

Show buff timer in vanilla and bugfixes #38

Closed excorp closed 3 months ago

excorp commented 4 months ago

Blizzard has made the cooldown timer invisible on buffs before TBC. I don't know why, but I've changed it to show the timer.

function CompactUnitFrame_UpdateCooldownFrame(frame, expirationTime, duration, buff)
    if GetClassicExpansionLevel() < LE_EXPANSION_BURNING_CRUSADE and buff then
        return;
    end

    local enabled = expirationTime and expirationTime ~= 0;
    if enabled then
        local startTime = expirationTime - duration;
        CooldownFrame_Set(frame.cooldown, startTime, duration, true);
    else
        CooldownFrame_Clear(frame.cooldown);
    end
end

I also updated the version number and fixed the roster configuration. There are differences between vanilla and wrath.
The roster issue is on the wrath side, but I didn't make it a separate PR because it's too minor a fix.

Slothpala commented 3 months ago

Thanks for pointing that out! I did some major changes to the module so this pr was to outdated but i implemented the timer and did some fixes to the roster function.