=== davmor2_HOLS is now known as davmor2 === marcusto_ is now known as marcustomlinson === alan_g is now known as alan_g|lunch === dandrader is now known as dandrader|afk === marcust__ is now known as marcustomlinson === dandrader|afk is now known as dandrader === alan_g|lunch is now known as alan_g [14:22] mzanetti: I'm looking at unity8's usageMode calculation -- is there a reason it compares current pointer count against the last pointer count? (I'm thinking of the case where you connect a mouse to your phone, then connect your TV to your phone -- looks like we'd stay in Staged mode then. If we get rid of that, we could have a real simple Binding that just [14:22] checks if we are big enough and have a pointer -> Windowed, else Staged. And not even bother looking at current value of usageMode either. [14:22] mterry, yes, there is a reason :) [14:23] I figured :) [14:23] it's designed that way [14:23] mterry, mainly: if the user manually switches to something, then this makes a difference on when the automatism kicks in again [14:23] mzanetti: I thought we didn't support manually switching that anymore [14:24] huh? [14:24] mzanetti: I thought we went from that being an actual setting to that just being a cache location for where u8 stores its current mode [14:24] mterry, no... there is a switch in the system indicator [14:24] mzanetti: oh right, we do have that [14:25] (which I think should eventually be moved to the display indicator but that's a different story) [14:25] but in any case, that switch is here to stay [14:25] mterry, that said, that whole logic still has a looooooong way to go [14:26] mzanetti: yeah I think my case above is still a bug? Looking at code anyway, maybe I'm missing something. But I feel like this could be simpler. (I'm hitting what might be a race condition on startup with the snap and its gsettings that this logic is choking on) [14:27] mzanetti: I'll continue testing and see if there's a way to fix that makes it easier to read too, I feel like this is not the first time I've asked you that question above ;) [14:27] indeed [14:27] mterry, I'd need to look into it myself again [14:28] also dig out the reasons design gave me for this [14:28] but in any case, right now it behaves as specced [14:28] Saviq, can we haz this in the pre OTA-14 silo? https://code.launchpad.net/~mzanetti/unity8/spread-blur/+merge/309335 [14:29] Saviq, and this? ;) https://code.launchpad.net/~lukas-kde/unity8/superKeyPressFix/+merge/307977 [14:30] ltinkl, hey, I got a mail from design that they reviewed the spread again, now that it's landed and sent me a mail with a list of things to fix. Mind reviewing this branch: lp:~mzanetti/unity8/spread-fixes [14:31] mzanetti, sure [14:31] mzanetti, got an MP? [14:31] preparing as we speek [14:31] speak [14:32] mzanetti, cool, mind forwarding that email? [14:32] ltinkl, mp: https://code.launchpad.net/~mzanetti/unity8/spread-fixes/+merge/310187 [14:33] ltinkl, mail should be in your inbox [14:33] mzanetti, thx, does this also contain the short Alt-TAB fix? [14:33] ltinkl, note that 2 points from that mail are not fixed yet. One I can't repro, the other seems to tricky to do in a rush [14:33] ltinkl, hmm... I have submitted that in some other MP, haven't I? [14:34] mzanetti, yeah maybe... getting lost in those many MPs :) [14:34] ltinkl, that one https://code.launchpad.net/~mzanetti/unity8/delay-alt+tab/+merge/309339 [14:34] ltinkl, you happroved already [14:35] mzanetti, yup yup [14:36] hmm didn't think to have a silo with those, but maybe we should indeed [14:41] Saviq, yeah, something for generic u8 fixes (we have accumulated quite some over the time) [14:58] testing 2106, might just pile on there [15:27] nope, test fails there too [15:31] Trevinho, rebuilding 2106 again? :) [15:32] ltinkl: oh, did you do that already? [15:32] Trevinho, Saviq did [15:32] ltinkl: yeah, in fact I checked the autdit log but didn't see that [15:33] ltinkl: while I was curious why there was no complain about new commits [15:33] ltinkl: cancelled though [15:33] Trevinho, there had been, before Saviq rebuilt it [15:33] ltinkl: good, I'll wait before approving it again then === dandrader is now known as dandrader|afk [16:50] @unity this isn't in u8 code proper, but can someone review https://code.launchpad.net/~mterry/unity8-desktop-session/accountsservice/+merge/310005 ? (needed for snapped unity8 to see AS currently) === dandrader|afk is now known as dandrader === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader