-
-
Notifications
You must be signed in to change notification settings - Fork 42
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
base: master
Are you sure you want to change the base?
WoW Classic Support #279
Conversation
Updated toc version and removed code relying on GetFriendshipReputation as it is not available in Classic
Also added short bar functionality
Fixed reputation panel error
This reverts commit 33c489d.
Hi ! Thank you very much. I'll check that as a hunter point of vue. |
Hi!How to Download this? I want to test. |
…ng a class that Classic doesn't know about
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. |
@Leiludallas
@lithammer @Grimsbain - Any chance a |
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. |
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 😄 |
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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()
The first file could export something like |
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. |
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
Working:
TODO:
These are a little above my basic understanding of Lua and the WoW API