=== Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [11:06] gmb, hey, I've got one question: I am looking at the mockups from user testing, and I don't see the case where you *are* marked as the bug supervisor — should that be treated as if you have a structural subscription then? [11:24] hi gmb [11:24] the fix you made worked great. [11:25] bac: Cool, glad to hear it. [11:25] there is one oddity i haven't yet investigated. when the overlay is closed, the accordion and some checkboxes remain behind [11:25] Hmm. [11:26] That's a bit weird. I didn't see that one but my testing was probably a bit perfunctory. [11:26] bac: I'm just going to grab a cup of tea, but I'll take another look in a few minutes. [11:27] yeah i only saw it yesterday but not sure if it had been there all along. happens in webkit and FF [11:27] ok thanks [11:27] it'll be a little bit before im really here [11:56] gmb, do we already have some standard code that returns all the subscriptions for a (person, bug) pair? :) if not, do you know what code should I reuse to get at least some bits of it? [11:57] danilos: I guess you're after structural subscriptions there, aren't you? [11:58] gmb, nope, quite the contrary [11:58] gmb, I need things like "direct subscription", "through duplicate", "you are bug supervisor", "through team membership",... [11:59] danilos: Ah. Take a look at lp.bugs.interfaces.bug. There's a bunch of methods there that will be useful, but nothing that does everything you need all in one method. [12:00] danilos: Thinks like isSubscribed(user), isSubscribedToDupes(user), getSubscribersForPerson(user).... etc. [12:00] gmb, right, I just want to re-use as much as possible, thanks [12:10] gmb, btw, if you've got a minute to glance over https://pastebin.canonical.com/44215/, I wonder if I am covering all the bases :) [12:15] I thought of more supervisor actions ("leave the team" and "I don't want to be supervisor" if you are not product owner) [12:41] * danilos -> food [13:06] hey benji, looks like Huw came through for the CSS. Any word on the triangle dealie? [13:07] nope, none yet; once I try out his CSS I'll reply with a "thanks that worked great... how about the other thing"-type message [13:08] heh, ok benji, sounds good :-) [13:25] benji, hi, are you working on implementing the structural subscriptions part of mock-ups on https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2? [13:25] bac: I haven't been able to reproduce the problem you mentioned. Can you send me a screenshot [13:25] ? [13:26] gmb, sure [13:27] danilos: I think you've covered everything with that list. [13:27] danilos: I'm doing the list of user/team subscriptions for a bug (but I don't think it's mocked up on the slides you reference, it's at https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/SubsListRound1/Slide1 instead) [13:28] gmb, cool, thanks [13:28] benji, right, thanks [13:29] gmb: good : http://people.canonical.com/~bac/overlay-opened.png [13:29] gmb: bad http://people.canonical.com/~bac/overlay-closed-messy.png [13:29] OIC. Hang on... [13:29] to go from good to bad i hit the 'cancel' button [13:30] oopsie [13:30] mumble nowish, bac benji danilos gmb [13:39] https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/DirectSubscriptionsOnBug [14:10] bac: So, http://pastebin.ubuntu.com/574977/ sort of solves the problem... I think it has to do with the way we deal with expanding nodes, but I'm not sure *why* [14:10] (That's not a solution, really, but it does make the controls vanish) [14:10] interesting. and it'd need to be done for submit too, i guess [14:11] The "kill them all" apprach to DOM manipulation :-) [14:12] bac: Yeah (it'd be nice if FormOverlay had a 'hide' event that we could tie onto, but anyway...) [14:12] well, perhaps the "SHUT UP ALREADY!" approach to DOM manipulation... [14:22] gary_poster, gmb, anyone else who cares: I'll start using the IBug:+subscribe page for the work I am doing (description and direct subscription actions); if you feel I should leave that alone and provide a different page (eg. because we want to continue have +subscribe gracefully degrade), just let me know :) [14:23] danilos, we have a page for this already that gmb started and that benji is working on... [14:23] gary_poster, do we? [14:23] yeah [14:23] it has nothing on it ATM :-P [14:23] danilos: NOOOOOooooo. That page is used to render the overlay for direct subscriptions. Use Bug:+subscriptions [14:23] (Which is what gary_poster is talking about) [14:23] :-) [14:23] gmb, gary_poster: but I can't find it :) [14:24] gmb, gary_poster: ok, that's the kind of info I was looking for [14:24] cool [14:25] gary_poster, that actually confuses me a bit; benji told me he's working on https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/SubsListRound1/Slide1, so just to clarify, https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2 is part of that as well? [14:25] * gary_poster looks [14:26] benji, you are working on the bottom part of https://dev.launchpad.net/LEP/BetterBugSubscriptionsAndNotifications/Testing/EditingRound2/Slide2 I thought? [14:26] ("Other subscriptions that may send you email about this bug") [14:26] danilos: I don't see anything on those slides that (as far as I'm aware) I'm working on (i.e., I'm not doing the "If you don't want to receive..." part) [14:27] yeah you are :-) [14:28] the one you are looking at is a part of the full one; you are only implementing the non-direct subscriptions, though [14:28] which is pretty much the same [14:28] across the two mockups [14:28] danilos is working on the explanation and direct subscription part [14:29] does that ring a bell, benji, or at least are you satisfied that this jibes with what you were already doing? [14:30] gary_poster: exactly, I'm doing the non-direct bits [14:30] Great. Danilos, you cool too? [14:30] gary_poster, yep :) [14:30] yay :-) [14:30] gmb, halloo. call now, or ping me when you are ready? [14:30] thanks all [14:31] Oh, bother, forgot about that. [14:31] * gmb gets his mumble on. [14:31] :-) === Ursinha is now known as Ursinha-lunch [14:47] bac: D'you have a second to mumble? [14:47] sure [14:53] gary_poster: an RM pre-nag about QAing Bug 164196 [14:53] <_mup_> Bug #164196: Quickly-undone actions shouldn't send mail notifications < https://launchpad.net/bugs/164196 > [14:54] thanks benji. Yeah, QA is blocked on the bug I'm working on :-/ [14:54] (was blocked on staging being down for about a week brfore that) [14:57] bac: http://pastebin.ubuntu.com/575007/ [15:30] hey danilos. ping when you are ready for call? [15:44] https://pastebin.canonical.com/44238/ [15:44] gary_poster, https://pastebin.canonical.com/44239/ with syntax highlighting :) [17:14] gary_poster: I've just submitted AWS expenses; rather higher than I'd hoped as there was an EC2 instance that got stuck. [17:14] gmb :-/ ack, approving. [17:15] gary_poster: Thanks. [17:16] approved. np [17:16] Ta [17:51] bac: I've added a rudimentary test suite (i.e. all the setup, doesn't do anything helpful yet) to the ~yellow branch. I'll check back later, so if you've any questions, ping me. [17:51] * gmb -> dinner [17:52] ok [18:50] gmb: another annoyance -- if you simply click away from the overlay somewhere else on the page, the overlay disappears, leaving the cruft in place. is it supposed to make the overlay disappear? seems wrong. [18:51] bac: Yes, it's supposed to hide the overlay. I wonder if the overlay does fire a 'hide' event that we can tie our cleanup to. It would seem odd not to. [19:20] bac, I still haven't had lunch and am feeling a bit lightheaded. I should get some food. I might be 10 min late for our call. Is that alright? [19:21] gary_poster: that's fine. would you like to just say 3:00? [19:21] sure, thank you bac [19:21] great. i'm going to take a walk now, then. [19:21] cool [19:21] see you in a bit [19:21] bye