bacmorning frankban, teknico.11:48
frankbanhi bac 11:48
* bac starts reviewing11:49
teknicobac, good morning11:49
bacteknico: better today?11:49
teknicobac, yes, much better, thanks :-)11:50
bacfrankban: first one done.  land away11:51
bacfrankban: 'file not found' reviewed.  teknico can you do a second review?12:10
bacteknico: nm, i see you're on it12:11
teknicobac, I am :-)12:11
frankbanbac, teknico: thanks12:11
frankbanbac: I think the comment you suggested makes sense12:12
bacfrankban: good12:24
bachi teknico, do you have a moment to chat about bug 1102640?13:39
_mup_Bug #1102640: Selecting a target service when adding a relation brings up the menu <juju-gui:Triaged> < https://launchpad.net/bugs/1102640 >13:39
bachi bcsaller13:47
bcsallerhey :)13:50
* teknico is back from lunch14:05
teknicobac, sure14:05
bacsorry bcsaller, i didn't see you respond.  i was going to ask if your branch fixed bug 1102640 but i see it didn't14:06
_mup_Bug #1102640: Selecting a target service when adding a relation brings up the menu <juju-gui:Triaged> < https://launchpad.net/bugs/1102640 >14:06
bacteknico: ok, regular hangout?14:06
teknicobac, yep14:07
bcsallerbac: That may be covered by the fix, I didn't see that one to specifically include it in testing ,checking now14:08
bcsallerbac: appears to be fixed as well14:09
bacbcsaller: not really14:09
bacon call.  want to join?14:09
bcsallerbac: yeah14:09
frankbanbac, bcsaller: one test fails in trunk, adding the trivial fix in my current branch, landing soon.14:13
bacfrankban: thanks14:18
bcsallerfrankban: thanks14:18
gary_posteryou guys rock, thank you14:53
bacbcsaller: i think i fully understand what is going on now...but not how to fix it.  can we chat before or after the standup?15:27
bcsallerbac: sure15:27
bacalejandraobregon, goodspud, teknico, benji: call now15:34
alejandraobregonbac: hi, we're at the sprint in a meeting, sorry we can't join..15:35
bacalejandraobregon: right, forgot15:35
alejandraobregonbac: no worries15:35
bacbcsaller: can you give me a quick idea on where to make the framework changes?15:42
bcsallerbac: it looks like we *could* but don't need to. app/assets/javascripts/d3-components.js:202 We do assign the global d3.event object15:44
bcsallerand can use that to cancel the event15:44
bcsallerso the proper fix should be available now15:45
bacbcsaller: oh, nice15:46
bacbcsaller: using d3.event to preventDefault did not prevent serviceClick from being called.  a crude work-around that does work is http://paste.ubuntu.com/1560040/16:35
bcsallerbac: I think you want stopPropagation, not preventDefault16:38
bacbcsaller: ok i'll try that16:38
bacbcsaller: neither stopPropagation nor stopEvent prevent serviceClick from being called16:43
bcsallerbac: hmm, that seems wrong because it should. I'll poke around a bit but don't let that interfere with what you're doing.16:44
bacbcsaller: did you note i was using d3.event.sourceEvent?  the actuall d3.event doesn't have those methods.16:44
bacbcsaller: i'm going to grab some lunch16:45
bcsallerThe event facade should have them as well, but I'll look into it more, thanks16:45
therveis the gui supposed to work in firefox?16:47
therveI can't make it work for some reason here16:47
thervebut it works in chromium16:47
gary_postertherve, not really, no.  We occasionally make sure it works in FF but we won't say it is supported until we have something like daily CI tests for the browser.  That's almost certainly happening in the next two months.  Meanwhile, we use chromium because it is wildly faster, particularly for the things we are doing.16:55
gary_posterMedium priority: FF seems to be broken.17:02
hazmatgary_poster, bummer17:12
bacbcsaller, benji: can you have a look at my review?21:05
benjibac: sure21:05
bcsallerbac: doing it now21:06
bacbcsaller: i attempted to come up with a test but so far have not been able to craft one21:10
bacbcsaller: the existing test doesn't simulate click events but calls the underlying methods directly.21:11
bacbcsaller: so, i'll make a card and perhaps collaborate with someone tomorrow to figure out how to get something working21:11
bcsallerbac: a card is fine. 21:11
gary_postergoodspud, try

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!