| 00:22.08 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 00:24.22 | *** join/#wowuidev Seerah (~umsin@75-36-111-192.lightspeed.mmphtn.sbcglobal.net) |
| 00:24.23 | *** mode/#wowuidev [+o Seerah] by ChanServ |
| 00:28.16 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 00:31.39 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 00:35.01 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 00:39.12 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 04:14.18 | *** join/#wowuidev purl (ibot@96-86-209-99-static.hfc.comcastbusiness.net) |
| 04:14.18 | *** topic/#wowuidev is 9.0.5 ToC: 90005 || 1.13 ToC: 11307 || TBC Classic Beta Key Sign up: https://bit.ly/3vWt3J0 || http://pastebin.com || Discord: https://discord.gg/sVQCHr5 || Battletags: http://bit.ly/2MAEFxk || This channel is logged, via purl |
| 05:47.41 | *** join/#wowuidev Elmoe (~Elmoe@199.36.252.5) |
| 07:09.28 | *** join/#wowuidev Andols (~yaaic@2a00:801:447:55a8:fa6f:757b:5f3d:1034) |
| 07:20.42 | *** join/#wowuidev Andols2 (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 07:46.09 | *** join/#wowuidev Elmoe (~Elmoe@199.36.252.5) |
| 08:06.16 | *** join/#wowuidev Andols (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 08:14.07 | *** join/#wowuidev Andols2 (~yaaic@2a00:801:447:55a8:cb91:d0f5:b8b5:3b7a) |
| 09:24.24 | *** join/#wowuidev Mr_Rabies (~mrrabies@167.172.227.29) |
| 10:05.47 | *** join/#wowuidev Andols (~yaaic@2a00:801:447:55a8:1601:b10a:e71d:fd02) |
| 10:56.13 | *** join/#wowuidev Treeston (~stolenleg@unaffiliated/stolenlegacy) |
| 11:51.59 | *** join/#wowuidev brykrys (~brykrys@WoWUIDev/Norganna/Dev/Brykrys) |
| 12:11.47 | *** join/#wowuidev pompy (~Mike@2601:87:4102:8c80:916a:d8f9:3cde:9615) |
| 12:35.47 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 12:41.35 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 12:42.59 | *** join/#wowuidev Breen (~Breen@80.175.117.105) |
| 12:52.05 | *** join/#wowuidev eir (bot@freenode/utility-bot/eir) |
| 12:53.16 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 13:17.58 | *** join/#wowuidev yalb (xsbot@lua/bot/yalb) |
| 14:41.52 | *** join/#wowuidev pompy (~Mike@2601:87:4102:8c80:55dc:86ca:bee5:1bc6) |
| 15:46.09 | *** join/#wowuidev Treeston (~stolenleg@unaffiliated/stolenlegacy) |
| 15:51.09 | *** join/#wowuidev greedom (~greedom@2001:16b8:3216:6600:712f:8b3f:d08c:1d82) |
| 15:54.27 | *** join/#wowuidev greedom (~greedom@2001:16b8:3216:6600:712f:8b3f:d08c:1d82) |
| 16:31.50 | *** join/#wowuidev Crosshash (c64a2302@198.74.35.2) |
| 16:31.50 | *** mode/#wowuidev [+o Crosshash] by ChanServ |
| 16:36.49 | *** join/#wowuidev Andols (~yaaic@2a00:801:447:55a8:1601:b10a:e71d:fd02) |
| 17:02.37 | *** join/#wowuidev Seerah (~umsin@2600:1700:aac0:82e0:80b2:91e8:f81:a35b) |
| 17:02.37 | *** mode/#wowuidev [+o Seerah] by ChanServ |
| 17:31.32 | *** join/#wowuidev Andols2 (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 17:35.14 | *** join/#wowuidev Andols (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 18:16.04 | *** join/#wowuidev Kesavaa (~Kesava@unaffiliated/sliker) |
| 18:19.22 | cirdan | anyone here? have a question about fixing an addon for classic tbc/9.x with SetBackdrop |
| 18:19.48 | Seerah | just ask ;) |
| 18:20.19 | cirdan | i copypasta from https://www.wowinterface.com/forums/showthread.php?t=58109 and just trying to tweak it :) |
| 18:20.22 | cirdan | 1 sec |
| 18:20.41 | cirdan | does the end to an if statement need to be on the next line? |
| 18:22.34 | cirdan | I have this: https://github.com/illiath/EnhancedFlightMap/issues/11 |
| 18:23.56 | *** join/#wowuidev p3lim5 (~p3lim@unaffiliated/p3lim) |
| 18:25.04 | cirdan | https://github.com/illiath/EnhancedFlightMap/blob/master/ConfigScreen.lua |
| 18:25.27 | cirdan | I'm trying to wrap it like the wowinterface post says to so it works in both classic and tbc classic |
| 18:25.47 | Seerah | as for the end question, if you mean what's in the OP on that thread you linked, I don't know why that end is there. |
| 18:26.12 | cirdan | ok |
| 18:26.27 | Seerah | Classic and TBC Classic should have the same backdrop API as live, as they're based on 9.x |
| 18:26.55 | Seerah | that thread is for 8.x vs 9.x |
| 18:27.26 | cirdan | they dont |
| 18:27.46 | cirdan | at least, tbc classis is gonna break a shitlod of addons because of this |
| 18:28.02 | cirdan | classic was based on the 7.1 api |
| 18:28.13 | cirdan | seems tbc is based on the 9.x |
| 18:29.56 | cirdan | I'm wrapping the stuff as in https://www.wowinterface.com/forums/showpost.php?p=336519&postcount=20 but it seems odd to me that the end would be so far down |
| 18:31.39 | cirdan | https://paste.debian.net/1197633/ |
| 18:31.43 | cirdan | is this correct? |
| 18:32.42 | cirdan | to me it seems none of that would be set on 8.x api |
| 18:33.23 | *** join/#wowuidev pompy (~Mike@2601:87:4102:8c80:40b6:73c8:48c4:5d1) |
| 18:34.17 | Seerah | Ah, it's been a while since I played on Classic. I believe it is close to the same as 8.x, not 9.x. When you create your frame, do this CreateFrame("frame", "frameName", parent, BackdropTemplateMixin and BackdropTemplateMixin) That will use the mixin if it exists, and not if it doesn't |
| 18:34.32 | cirdan | hmm ok |
| 18:34.39 | cirdan | and it wont error out on older stuff |
| 18:34.43 | Seerah | your end looks like it's in the right place, but proper formatting/indents go a long way for understanding how your code runs. ;) |
| 18:34.51 | cirdan | it was proper in notepad++ |
| 18:38.27 | cirdan | you mean CreateFrame(frameType, frameName, frameParent, BackdropTemplateMixin and "BackdropTemplate"); ? |
| 18:41.03 | Seerah | yes :) |
| 18:43.45 | cirdan | ok this last one has an extra arg: local EFM_MapWindowNew = CreateFrame("FRAME", "EFM_MapWindowNew", UIParent, Parent, BackdropTemplateMixin and "BackdropTemplate"); |
| 18:44.07 | Seerah | You have both UIParent and Parent in there |
| 18:44.10 | cirdan | would it be Parent a BackdropTemplateMixin and "BackdropTemplate" |
| 18:44.17 | cirdan | the orig had both in there |
| 18:44.57 | Seerah | local EFM_MapWindowNew = CreateFrame("frame", "EFM_MapWindowNew", UIParent, BackdropTemplateMixin and "BackdropTemplate") |
| 18:45.14 | cirdan | what about the parent? |
| 18:45.15 | *** join/#wowuidev Andols (~yaaic@2a00:801:447:55a8:aaf0:a89a:739f:5adc) |
| 18:45.19 | Seerah | if the original had both, the original was wrong ;) |
| 18:45.36 | cirdan | the orig modified the flight map window/points with more info |
| 18:46.00 | Seerah | there is no such thing as Parent, it's nil. so the game ignored it, as it wasn't a valid template to use (the 4th arg) |
| 18:46.07 | cirdan | no errors no reload :) |
| 18:46.39 | cirdan | ok now that I have you here and engaged... |
| 18:46.46 | cirdan | ;-) |
| 18:47.30 | cirdan | I noticed some addons are able to open the settings window under interface directly, like with /efm config |
| 18:47.44 | cirdan | right now it just opens the interface window and you have to click the addons tab |
| 18:48.05 | AcidWeb | call it twice |
| 18:48.13 | Seerah | it's a glitch |
| 18:48.32 | cirdan | finding out some of this stuff is hard |
| 18:48.38 | cirdan | huh. in the code you mean? |
| 18:48.48 | Seerah | opentocategory |
| 18:48.58 | Seerah | or whatever it is |
| 18:49.40 | nevcairiel | yeah just run the open command twice |
| 18:49.57 | *** join/#wowuidev Andols2 (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 18:52.00 | *** join/#wowuidev Andols3 (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 18:52.58 | cirdan | weird. thanks. |
| 18:53.02 | cirdan | that was driving my crazy |
| 18:53.39 | cirdan | I just spent about 20h in the beta recording every horde FP and the times between each direct fligh |
| 18:53.42 | cirdan | and all the TBC ones |
| 18:53.54 | cirdan | my brain is mush, glad to clear up these errors |
| 18:54.11 | Seerah | used to happen randomly, then it was if your addon was lower in the list and not visible, then it started happening randomly again to any addon |
| 18:54.36 | znf | wasn't there an addon that did that back in BC days anyway? |
| 18:55.01 | cirdan | yeah |
| 18:55.02 | cirdan | this one |
| 18:55.18 | cirdan | the dev came back for classig for a bit but has been awol |
| 18:55.38 | znf | actually I remember it being a feature of DBM, I think? |
| 18:55.41 | cirdan | https://github.com/illiath/EnhancedFlightMap |
| 18:55.45 | cirdan | no |
| 18:55.51 | cirdan | EFM |
| 18:56.06 | znf | that addon doesn't ring any bell, and I can swear I had flightpath timers back in BC |
| 18:56.21 | cirdan | I never did even with dbm |
| 18:57.02 | cirdan | so this was also a big in tbc and wrath. to get to ghostlands you have to fly through the portal |
| 18:57.13 | cirdan | this would cause EFM to stop the flight timer then |
| 18:57.24 | znf | yeah, I can't recall that long ago |
| 18:57.37 | cirdan | is there any way to keep the timer going through the server xfer? |
| 18:58.31 | znf | you sure it's an issue with the current game timer? |
| 18:58.34 | cirdan | I can hand enter times and somehow block them from being updated but the timer UI is gone after the portal as well |
| 18:58.37 | znf | erm, current game client* |
| 18:58.41 | cirdan | yes |
| 18:58.43 | cirdan | I've tested it |
| 19:07.39 | cirdan | it really is a server transfer, sadly |
| 19:08.34 | cirdan | it seems the slider stuff has changed? |
| 19:08.51 | cirdan | the code has SetValueStep 10 but it doesn't |
| 19:09.37 | cirdan | oh Slider:SetObeyStepOnDrag |
| 19:10.18 | znf | Now you got me wondering if it's the same on retail (re: timer) |
| 19:10.23 | AcidWeb | https://github.com/Stanzilla/WoWUIBugs/wiki/2.5.1-Consolidated-UI-Changes |
| 19:10.44 | cirdan | https://wowpedia.fandom.com/wiki/API_Slider_SetObeyStepOnDrag that says 5.4.1 |
| 19:11.11 | cirdan | znf: what ghostlands? |
| 19:11.17 | cirdan | i think they revamped it in cata |
| 19:11.32 | znf | no, I mean, just instance transfer |
| 19:11.40 | cirdan | it was originally to keep people out of the new areas if you didn't have the xpac |
| 19:11.45 | cirdan | no tbc no glostlands |
| 19:13.59 | znf | indeed, bigwigs timers don't survive trough swapping instance |
| 19:14.43 | cirdan | gonna have to figure out how to special case those zones or something |
| 19:15.01 | cirdan | it would be really nice to re-int the countdown with remaining time |
| 19:15.06 | cirdan | so the user wouldn't really notice |
| 19:15.25 | cirdan | no idea if that's possible |
| 19:25.10 | cirdan | does anyone have a min to take a look at https://github.com/illiath/EnhancedFlightMap/blob/master/MapWindow.lua#L78 and see why it's not working? |
| 19:25.17 | cirdan | the druid flight point is always shown |
| 19:43.33 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 19:52.54 | Seerah | This is what you were thinking of, znf https://www.wowinterface.com/downloads/info11178-InFlight.html |
| 19:53.16 | *** join/#wowuidev Kalroth (~Kalroth@knas.io) |
| 19:57.42 | cirdan | yyeah that's the other one, less features, better timer ;-) |
| 20:07.33 | cirdan | hmm there's a prblem with https://github.com/illiath/EnhancedFlightMap/blob/master/message.lua#L66 |
| 20:08.11 | cirdan | /efm report party|guild|raid|4|nick works |
| 20:08.23 | cirdan | but nothing tries to whisper nobody and that throws an error |
| 20:08.46 | cirdan | whereas it should just be printed in the default chat frame |
| 20:12.17 | Treeston | cirdan: if it helps, this is what i'm currently using to open IOF |
| 20:12.20 | Treeston | https://gist.github.com/Treeston/9a9c6f4a40ce6e0bd5d66f4c77f308e5 |
| 20:12.38 | cirdan | I just called it twice like they said |
| 20:12.58 | Treeston | ah, that should also work (sorry, didn't read full scrollback) |
| 20:14.37 | cirdan | np thanks |
| 20:14.40 | Treeston | also in the lua you linked cirdan |
| 20:14.43 | Treeston | messageDest will never be nil |
| 20:14.57 | Treeston | you get it from string.lower(reportTo) |
| 20:15.02 | Treeston | if you pass nil to string.lower it will error |
| 20:15.08 | Treeston | so the return will always be a string |
| 20:15.23 | Treeston | you likely want (messageDest ~= "") rather than (messageDest ~= nil) |
| 20:15.40 | cirdan | for all of them? |
| 20:15.46 | cirdan | the nils |
| 20:15.49 | Treeston | https://github.com/illiath/EnhancedFlightMap/blob/master/message.lua#L58 on this line |
| 20:18.08 | cirdan | damn i broke something |
| 20:19.36 | cirdan | something i didnt touch |
| 20:23.04 | cirdan | this is weird |
| 20:34.32 | cirdan | hmm what the hell |
| 20:35.01 | cirdan | now when i open the flight map it's telling me attempt to concatenate global "myContinent" and i dont htink I touched that stuff |
| 20:35.05 | cirdan | gonna have to do a diff now |
| 20:53.22 | *** join/#wowuidev Andols (~yaaic@2a00:801:447:55a8:b74:9e36:b806:ad1a) |
| 20:58.47 | *** join/#wowuidev Andols2 (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 23:04.55 | *** join/#wowuidev Andols (~yaaic@217-209-12-6-no170.tbcn.telia.com) |
| 23:17.09 | cirdan | it's kinda amusing. fixing some bugs is uncovering others I never knew existed |
| 23:17.19 | cirdan | it's also annoying cause I no ~nothin about lua |
| 23:17.26 | cirdan | like it's syntax and all |
| 23:17.54 | cirdan | the bad line this time is --EFM_Shared_DebugMessage("Checking if node "..myNode.." is reachable on continent "..myContinent, myDebug); |
| 23:18.01 | cirdan | what does the .. mean |
| 23:20.22 | cirdan | it's also weird cause it's not an issue on classic |
| 23:20.25 | cirdan | but it is on tbc classic |
| 23:30.05 | nevcairiel | .. is string concat |
| 23:30.29 | *** join/#wowuidev VoraciousGhost (b8611086@184.97.16.134) |