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

WoW Classic Support #279

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

WoW Classic Support #279

wants to merge 25 commits into from

Conversation

chowarth
Copy link

@chowarth chowarth commented Sep 1, 2019

I've gone through and tried to get as much working as I possibly can. (this may will be a bit hacky as I have zero experience with Lua or the WoW API), most things were checks to see if an object exists before trying to use it.

It is by no means perfect I've only gone through everything while using a Warrior so there could well be some class specific behaviour that will cause errors.

Hopefully this will help #278

WoWScrnShot_090119_145434

Working:

  • !BeautyCase
  • !Colorz
  • Evl's Raid Status
  • nBuff
  • nChat
  • nCore
  • nMainbar (with added 'short' functionality)
  • nMinimap
  • nTooltip

TODO:

These are a little above my basic understanding of Lua and the WoW API

  • nPlates
  • nPower
  • oUF
  • oUF_Neav
  • oUF_NeavRaid

@WillScarlettOhara
Copy link

Hi ! Thank you very much. I'll check that as a hunter point of vue.

@AFK-afk
Copy link

AFK-afk commented Sep 9, 2019

Hi!How to Download this? I want to test.

@chowarth
Copy link
Author

chowarth commented Sep 9, 2019

@AFK-afk, the easiest way would be to download the branch from the repo I forked: here

@WillScarlettOhara
Copy link

Hi,

When I click on guild icon on top of the minimap, it opens the Group panel instead of guild panel.

Is there a way to apply action bar button border to raid/group unit frame within Vuhdo for example ? Thank you for you work.

@chowarth
Copy link
Author

@Leiludallas

  1. Had a quick look at the minimap button. It is calling ToggleGuildFrame() but that seems to default to the 'Groups' panel in Classic. I'll see if there's anything I can do to get the actual Guild panel showing instead.
  2. I don't use Vuhdo myself but I can have a look. I was thinking of going through and adding configuration settings for the skins as well, so if you don't want something skinned you can easily disable it.

@lithammer @Grimsbain - Any chance a Classic branch could be created and then this could be maintained alongside?

@lithammer
Copy link
Owner

Sorry for being unresponsive. I actually already have a local 'classic' branch, but I haven't had time to play as much as I'd hoped. Long queue times on Gehennas-EU, I'm still lvl 19 😞

Anyway, I'm currently trying to restore some of the lost functionality like threat, aura durations, real health values, and heal prediction (this is a tougher though). But I've only tested on warlock so far, so I have no idea what the state of things like combo points and totems are.

@chowarth
Copy link
Author

I can totally understand with the queues on Gehennas-EU, ended up rolling onto another server rather than try and get on there with old server friends.

If there's anything I can do to help, or if you want to use the changes I've made to everything outside of the unit frames, feel free to shout. Although I'm still fresh to Lua, and writing addons in general, I'm enjoying just digging around at the moment 😄

@WillScarlettOhara
Copy link

@Leiludallas

  1. Had a quick look at the minimap button. It is calling ToggleGuildFrame() but that seems to default to the 'Groups' panel in Classic. I'll see if there's anything I can do to get the actual Guild panel showing instead.
  2. I don't use Vuhdo myself but I can have a look. I was thinking of going through and adding configuration settings for the skins as well, so if you don't want something skinned you can easily disable it.

@lithammer @Grimsbain - Any chance a Classic branch could be created and then this could be maintained alongside?

Thanks for the last fix.

I talk about Vuhdo but if you can change the default wow group and raid frame, it's great too.

return
else
-- Classic TODO: Doesn't have vehilces, can this be updated to still work with Retail?
-- if UnitHasVehicleUI("player") then
Copy link

@vendethiel vendethiel Feb 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if _G['UnitHasVehicleUI'] and UnitHasVehicleUI("player") then? (or isRetail and ... for better perf)


Could be due to my limited WoW and Lua knowledge :s
]]--
for k = 3, 5 do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potentially:

for k = 3, 8 do
  local editBox = select(k, _G[self.."EditBox"]:GetRegions())
  if editBox.SetTexture then
    editBox:SetTexture(nil)
  end
end

@@ -5,7 +5,12 @@ function nCore:Dressroom()

DressUpFrameCancelButton:SetText("Naked")
DressUpFrameCancelButton:SetScript("OnClick", function()
DressUpModel:Undress()
-- Classic: Use DressUpModelFrame instead
if DressUpModelFrame then

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also (DressUpModelFrame or DressUpModel):Undress()

@vendethiel
Copy link

The first file could export something like IS_CLASSIC = select(4, GetBuildInfo()) == 11302

@lithammer
Copy link
Owner

lithammer commented Feb 17, 2020

There's also a classic branch, but I haven't worked on it for some time. So it might be broken in raids (I'm not even lvl 60) or that some Blizzard patch introduced changes that broke it.

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

Successfully merging this pull request may close these issues.

5 participants