=== ssweeny` is now known as ssweeny [07:07] * tsdgeos waves the channel === kgunn is now known as Guest60061 [08:03] hay can i change the position of unity launcher ,,>>?? === vrruiz_ is now known as rvr === davmor2_ is now known as davmor2 [09:53] Cimi: https://code.launchpad.net/~cimi/unity8/infographics-august-merge/+merge/231344/comments/573362 [09:54] tsdgeos, I don't bother about this branch unless someone tells me "fix this, we want to merge infographics today" [09:54] ok [09:54] maybe put it as WIP then? [09:54] tsdgeos, I lost the count on how many times I fixed conflicts and noboduy reviewed [09:55] tsdgeos, I did infographics two months ago [09:55] i don't even know what it is about [09:55] but i guess we don't want it now in rtm [09:55] or do we? [09:55] tsdgeos, replace infographics with svg [09:56] i guess i'll say it put it as WIP and bug saviq when he comes back? it's not feature critical, is it? [09:57] pete-woods, do you know anything about schedule for infographics as svg? [09:58] Cimi: I haven't been told anything. I think it's up to us two to push it forward [09:59] Cimi: if we find a "quiet" week for u8, then maybe we should just push it through [10:00] pete-woods, on your side, do you think it has feature parity with the current implementation? [10:00] pete-woods, unity side, is just a frontend, we don't control things [10:00] tsdgeos: I did the translation support for the YouTube scope while you were away. wasn't sure if it was blocking you from doing the spanish translation [10:00] Cimi: I think maybe it misses the "no data" infographic [10:01] Cimi: but that's difficult to do from my end now, because of translations [10:01] pete-woods, so pleas, add this === davidcalle is now known as davidcalle_afk [10:51] pete-woods: cool tx === marcustomlinson is now known as marcustomlinson| === marcustomlinson| is now known as marcustomlinson [11:08] Holas [11:18] dednick: what's the status of https://code.launchpad.net/~nick-dedekind/unity8/indicator-polishing/+merge/228700 ? [11:18] do we want it in? need reviewing? [11:19] tsdgeos: Cimi is reviewing [11:19] dednick: good stuff === MacSlow is now known as MacSlow|lunch [11:34] hm is there a simple way to trigger a notifaction [11:34] apart of calling the phone [11:34] and adding a callendar entry with reminter :( [11:34] reminder even [11:34] anpok: lp:unity-notifications [11:34] anpok: examples directory [11:35] oh thx having a loo [11:35] k [11:54] dandrader: good morning [11:54] dandrader: hey, what's the ETA on the lost input fix? [11:54] mzanetti, hi [11:54] landing team wants to know [11:54] seems its blocking promotions [11:54] mzanetti, you mean that second patch? [11:54] dandrader: I mean a fix for the lost input :D [11:55] yes, the second patch. [11:55] people think of this as one issue though [11:55] so for the landing team the issue is not fixed. [11:56] mzanetti, https://code.launchpad.net/~dandrader/qtmir/frozenApps-lp1295623 needs a reviewer [11:56] mzanetti, wanna do it? :) [11:59] dandrader: ah, great. will look at it, yes [11:59] wow... this is a whole lot of code [12:00] mzanetti, 90% of it is refactoring to make it possible to unit test MirSurfaceItem [12:00] mzanetti, 5% is the test itself and 5% is the actual fix [12:01] yeah.. see that [12:01] mzanetti, the commit message explains that mess [12:03] dandrader: shouldn't SessionInterface be in unity-api too (not asking you to do that now - just for my undertstanding) [12:03] ? [12:03] mzanetti, yes, I even had a TODO comment in its header stating that [12:04] mzanetti, but greyback_ wants to nuke it in the nor so far future [12:04] s/nor/not [12:04] * dandrader looks for the doc [12:04] ah ok [12:04] no need to dig docs now [12:04] I just wanted to know if my understanding is ok [12:05] mzanetti, ah you opened it already. it's item #2 [12:05] * mzanetti totally forgot about that tab being open :D [12:06] mzanetti, and that nice diagram https://docs.google.com/a/canonical.com/drawings/d/1KFn03gFW1SXYOQn1l9hPCwFx_a8EAJA2AMc1hygqeEo/edit [12:06] ack. makes sens to me [12:12] dandrader: hmm... you sure you're not breaking multitouch with this? (I haven't tried it yet - but just reading through it you don't seem to care about touch ID in validateAndDeliverTouchEvent) [12:15] mzanetti, when a QEvent::TouchBegin comes, it means a whole new touch sequence is starting. Which implies that previous to that there should be no active touch points [12:15] ah ok [12:16] dandrader: what event is there when there's just an additional finger put down? [12:16] QEvent::TouchUpdate [12:16] ah [12:16] thanks [12:17] a touch sequence starts when the first touch point is pressed and ends once the last remaining touch point is released [12:17] ok... I would have expected to get a TouchBegin for each finger === davidcalle_afk is now known as davidcalle [12:18] mzanetti, that you get from QTouchPoint::touchPoints()::state() or QTouchPoint::touchPointStates() [12:18] got it [12:19] dandrader: yeah... your branch looks good. compiling it on the device to test === mpt_ is now known as mpt === MacSlow|lunch is now known as MacSlow [12:42] Is anyone looking at https://bugs.launchpad.net/ubuntu/+source/compiz/+bug/1305438? It's been broken one way or another since ~ 12.10 [12:42] Launchpad bug 1305438 in compiz (Ubuntu) "hot corners stop working time to time" [Medium,Confirmed] [12:54] tsdgeos, ping [12:56] Cimi: tsdgeos ...i thot i did get access for the team, lemme check if i need to do one more approval step of something [12:56] pstolowski: hi [12:58] tsdgeos, hey. i'm looking at an enhancement requested by thostr (and he discussed it with saviq) to delay invalidating of results model till first result of new search arrives; instead extsting results should be "disabled" (blurred, dimmed, whatever). do you know about that idea? [12:58] pstolowski: nope [12:58] tsdgeos, (and they shouldn't be clickable when new search is in progress) [12:59] tsdgeos, ok.. so i've first part more less ready, but wanted to check with you about making current results dimmed [13:00] greyback_: can you review ? https://code.launchpad.net/~dandrader/qtmir/frozenApps-lp1295623/+merge/234393 [13:00] tsdgeos, would it make sense for you if it's a single property of Scope? [13:00] kgunn, greyback_, mzanetti is on it [13:00] kgunn: I believe mzanetti was looking at it [13:00] ta [13:00] tsdgeos, (all results are either disabled or not for given scope) [13:00] mzanetti, you should probably claim it [13:01] pstolowski: i guess that's fine yes [13:03] tsdgeos, how busy are you currently? do you have time to hack something soonish (and I'd test/play with it and my plugin branch)? [13:03] pstolowski: just came from a week out, settling in, not terribly busy [13:04] i guess i could take it [13:05] tsdgeos, awesome; how about a bool property of Scope named 'resultsEnabled'? [13:06] hmmm [13:06] actually do we need a new property? [13:06] don't we have the "searching" thing already? can use that? [13:07] pstolowski: ↑ [13:07] tsdgeos, we have searchInProgress getter, but not exposed via property [13:07] tsdgeos, the only qproperty I can see is for 'favorite' flag [13:07] hmmm [13:07] and what does trigger the orange busy bar? [13:07] * tsdgeos checks [13:08] tsdgeos, but yeah, resultsEnabled would = !searchInProgress afaict [13:08] scope.searchInProgress [13:09] seems exposed to me [13:09] at least we use it :D [13:09] tsdgeos, ahhh, bloody scope interface defined somewhere else :/ [13:11] tsdgeos, yeah, you're right, we do handle it. in that case no need for new property (unless we find it's limiting us for some reason; but should be ok) [13:11] oki [13:11] pstolowski: so what should i do then? [13:11] make the thing blurred + disabled while searching? [13:12] tsdgeos, probably making them dimmed will be better. but i'm wondering if tying it to search-in-progress is a right thing, cause this will make them not-clickable until search finished [13:12] * finishes [13:13] well you said [13:13] resultsEnabled would = !searchInProgress afaict [13:13] :D [13:13] tsdgeos, yeah... but now as I think of it more.. what we want is to disable *old* results, but when new results start arriving, they should be clickable / enabled [13:14] tsdgeos, so, I think we need a new property after all :/ [13:15] ok [13:17] tsdgeos, i'll be back in 1h [13:28] dednick_: just checking, i know there's no prereq marked, but i can land lp:~nick-dedekind/ubuntu-settings-components/indicator-polishing w/o landing [13:29] lp:~nick-dedekind/unity8/indicator-polishing [13:29] ? [13:29] kgunn, yes === boiko_ is now known as boiko [13:30] kgunn, but he needs an approve first :) [13:33] Cimi: yes sir... [13:33] Cimi: was just noticing it seemed ready [13:33] Cimi: is there someone who we usually get to top approve on u-s-s ? [13:33] kgunn, it is quite ready yes, but it is affected from some changes to the sdk, I would like to wait tomorrow/wednesday [13:34] kgunn, this is settings components, not system settings [13:34] Cimi: cool, i'll keep it on the "still cookin'" pile [13:34] Cimi: you're right, sorry, mis-spoke [13:34] kgunn, basically, wait sdk to revert few things, then if nothing breaks approbe === _morphis is now known as morphis [13:35] thanks Cimi, you know which mp's we're eyeballing from sdk to land first ? [13:35] kgunn, otherwise we might just want to fine-tune some small details [13:36] dednick_: and i thot lp:~nick-dedekind/unity8/indicator-polishing was landing last week ?...what happened ? [13:36] kgunn, there are many of zsombi https://code.launchpad.net/ubuntu-ui-toolkit [13:36] Cimi: ok [13:36] kgunn, regarding listitems and divider [13:36] kgunn, it happened me not approving [13:37] Cimi: were you just being cruel ? [13:37] :) [13:37] kgunn, last week there was a bug around slow initialization of the indicators [13:37] kgunn, you were receiving a message, you pulled down the indicators... blank page for 1-2 seconds [13:37] eegads [13:37] kgunn, nick fixed friday that and is fine [13:38] Cimi: ok...well, there will also be another train [13:38] to land on [13:38] kgunn, however I dislike some UX changes daniela did (but she left) [13:38] kgunn, design also has a redesign for indicators [13:38] but she's back today right [13:38] Cimi: yeah...but that branch fixes a poop-ton of bugs [13:39] kgunn, so we might want to land the settings components part, and incorporate those design changes (that will fix my concern) [13:39] kgunn, I know, but we have policy of no regressions [13:39] Cimi: kk [13:39] re [13:39] kgunn: yes, on it [13:39] Cimi: what regression if the slow-init was addressed ? [13:41] Cimi: there's no regression in the the u8 branch [13:41] Cimi: it's a different design [13:43] dednick_, I think is a regression [13:43] dednick_, not your fault [13:43] Cimi: isn't danelia back today? [13:43] Cimi: i think she's left? [13:43] dednick_: you mean forever ? [13:43] i think so [13:44] kgunn: yeah [13:44] kgunn, I can approve nick branch, is a good improvement [13:44] kgunn, but I am sure we will receive complain for the new indicator design === _salem is now known as salem_ [13:45] the header bar loses the chevron for example, you don't know how to switch indicators [13:45] once you pulled down [13:45] Cimi: i see, at that point...that monkey is on design [13:46] it's going to change again in short order [13:46] kgunn, at the same time, vesa and ben were working on a redesign... and now we have that [13:46] greyback_, Here is QOpenGLDebugLogger::startLogging crashing on me again... http://paste.ubuntu.com/8350547/ [13:46] greyback_, commented it out [13:46] Cimi: "working on redesign" doesn't sound "done" [13:46] dandrader: weird [13:46] greyback_, that was on krillin [13:47] kgunn, they user tested this morning [13:47] kgunn, and trust me, is miles better [13:47] dandrader: looks like a null pointer.. [13:48] Cimi: sorry, user tested nick's branch that's up? [13:49] or ....new design we have yet to implement ? [13:51] kgunn, users tested the prototype [13:51] of the new indicatords [13:51] not nick branch [13:52] got it [13:52] kgunn, I don't think any designer ever saw nick branch working [13:52] kgunn, because daniela left [13:52] kgunn, and other designers were working on the redesign... [13:54] Cimi: they approved my changes. it's been there for ages. [13:54] dednick_, who approved? [13:57] Cimi: ben, daniela, esti . can't remember who all [13:57] Cimi, kgunn: https://docs.google.com/a/canonical.com/document/d/1KC40suUs13gajIlE1Ms6PJYoIPx_AjlF4ae6bvtNjNc/edit [13:57] dednick_, esti is not UX, daniela left... [13:57] removing the chevron was a direct request === lborda is now known as lborda_ [13:59] anyway, regardless we have UX approvals (only daniela), we have to change the design anyway, why not merging both? === dandrader is now known as dandrader|afk [14:00] instead of getting comments publicly, google plus/whatever "hey indicator redesign!" etc etc... then change it all over again in a couple of days === lborda_ is now known as lborda__ === dandrader|afk is now known as dandrader [14:42] Cimi: indeed much better with this option. thanks [14:42] yw [14:42] Cimi: now do you know a cure for Telepathy hiding all the time? [14:43] not using it :) [14:43] probably similar solution [14:43] tsdgeos, hey, so ok for new attribute 'resultsEnabled'? do you know if it needs to be added to ScopesInterface (I don't know what was the history of extracting this interface into separate project) [14:43] doesn't have such an option I think [14:43] Cimi: don't you use any IM? [14:43] pstolowski: everything needs to be in the interface [14:43] pstolowski: so mocks for example can use the interface [14:43] unity8 mocks [14:44] tsdgeos, ah, ok [14:44] mzanetti, I am antisocial, I don't chat apart IRC with colleagues :D [14:50] mhm... [15:11] * greyback_ needs reboot [15:32] dandrader: do you know what happened to the touch cancellation for the edges? [15:32] list time I heard from it, it was "soon after qtcomp landed" [15:33] mzanetti, right [15:33] mzanetti, waiting for it bubble up the prioritized queue of TODO work [15:34] kgunn, ^ [15:34] dandrader: ah... I think this should be critical... people start working around it: https://bugs.launchpad.net/ubuntu-ux/+bug/1368702 [15:34] Launchpad bug 1368702 in ubuntu-ui-toolkit (Ubuntu) "[SDK] HEADER ISSUE: Back arrow needs to be moved further to the right and also have a larger invisible touch target" [Undecided,New] [15:34] kgunn: ^ [15:34] I think the only reason why this isn't critical for us, is that we don't have a bug number for it [15:35] +1 === gatox is now known as gatox_lunch [15:53] marcustomlinson: please fix the tests in your branch [15:58] tsdgeos: k [16:34] Cimi: looks like i you can't put a colorized image into an UbuntuShape. [16:34] Cimi: so can't do anything about it :/ [16:34] dednick, you can use shaderEffectSource [16:35] is fixable, just, dammit! [16:35] maybe there is a better fix [16:35] like fixing the icons of the notifications [16:39] Cimi: hm? SharedEffectSource renders a texture. [16:39] *into a === dandrader is now known as dandrader|lunch [16:39] dednick, you can wrap a texture (an icon) inside a shape [16:40] dednick, we use this in the shell already somewhere === gatox_lunch is now known as gatox [17:03] * Cimi away, catch you a bit later [17:19] Cimi: that change is in usc now. === dandrader|lunch is now known as dandrader [19:10] greyback_, update the splashscreen unity8 mp [19:10] updated === dandrader is now known as dandrader|afk === salem_ is now known as _salem === dandrader|afk is now known as dandrader