/srv/irclogs.ubuntu.com/2014/09/15/#ubuntu-unity.txt

=== ssweeny` is now known as ssweeny
* tsdgeos waves the channel07:07
=== kgunn is now known as Guest60061
Jessedhammuhay can i change the position of unity launcher ,,>>??08:03
=== vrruiz_ is now known as rvr
=== davmor2_ is now known as davmor2
tsdgeosCimi: https://code.launchpad.net/~cimi/unity8/infographics-august-merge/+merge/231344/comments/57336209:53
Cimitsdgeos, I don't bother about this branch unless someone tells me "fix this, we want to merge infographics today"09:54
tsdgeosok09:54
tsdgeosmaybe put it as WIP then?09:54
Cimitsdgeos, I lost the count on how many times I fixed conflicts and noboduy reviewed09:54
Cimitsdgeos, I did infographics two months ago09:55
tsdgeosi don't even know what it is about09:55
tsdgeosbut i guess we don't want it now in rtm09:55
tsdgeosor do we?09:55
Cimitsdgeos, replace infographics with svg09:55
tsdgeosi guess i'll say it put it as WIP and bug saviq when he comes back? it's not feature critical, is it?09:56
Cimipete-woods, do you know anything about schedule for infographics as svg?09:57
pete-woodsCimi: I haven't been told anything. I think it's up to us two to push it forward09:58
pete-woodsCimi: if we find a "quiet" week for u8, then maybe we should just push it through09:59
Cimipete-woods, on your side, do you think it has feature parity with the current implementation?10:00
Cimipete-woods, unity side, is just a frontend, we don't control things10:00
pete-woodstsdgeos: 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 translation10:00
pete-woodsCimi: I think maybe it misses the "no data" infographic10:00
pete-woodsCimi: but that's difficult to do from my end now, because of translations10:01
Cimipete-woods, so pleas, add this10:01
=== davidcalle is now known as davidcalle_afk
tsdgeospete-woods: cool tx10:51
=== marcustomlinson is now known as marcustomlinson|
=== marcustomlinson| is now known as marcustomlinson
facundobatistaHolas11:08
tsdgeosdednick: what's the status of https://code.launchpad.net/~nick-dedekind/unity8/indicator-polishing/+merge/228700 ?11:18
tsdgeosdo we want it in? need reviewing?11:18
dednicktsdgeos: Cimi is reviewing11:19
tsdgeosdednick: good stuff11:19
=== MacSlow is now known as MacSlow|lunch
anpokhm is there a simple way to trigger a notifaction11:34
anpokapart of calling the phone11:34
anpokand adding a callendar entry with reminter :(11:34
anpokreminder even11:34
mzanettianpok: lp:unity-notifications11:34
mzanettianpok: examples directory11:34
anpokoh thx having a loo11:35
anpokk11:35
mzanettidandrader: good morning11:54
mzanettidandrader: hey, what's the ETA on the lost input fix?11:54
dandradermzanetti, hi11:54
mzanettilanding team wants to know11:54
mzanettiseems its blocking promotions11:54
dandradermzanetti, you mean that second patch?11:54
mzanettidandrader: I mean a fix for the lost input :D11:54
mzanettiyes, the second patch.11:55
mzanettipeople think of this as one issue though11:55
mzanettiso for the landing team the issue is not fixed.11:55
dandradermzanetti, https://code.launchpad.net/~dandrader/qtmir/frozenApps-lp1295623 needs a reviewer11:56
dandradermzanetti, wanna do it? :)11:56
mzanettidandrader: ah, great. will look at it, yes11:59
mzanettiwow... this is a whole lot of code11:59
dandradermzanetti, 90% of it is refactoring to make it possible to unit test MirSurfaceItem12:00
dandradermzanetti,  5% is the test itself and 5% is the actual fix12:00
mzanettiyeah.. see that12:01
dandradermzanetti, the commit message explains that mess12:01
mzanettidandrader: shouldn't SessionInterface be in unity-api too (not asking you to do that now - just for my undertstanding)12:03
mzanetti?12:03
dandradermzanetti, yes, I even had a TODO comment in its header stating that12:03
dandradermzanetti, but greyback_ wants to nuke it in the nor so far future12:04
dandraders/nor/not12:04
* dandrader looks for the doc12:04
mzanettiah ok12:04
mzanettino need to dig docs now12:04
mzanettiI just wanted to know if my understanding is ok12:04
dandradermzanetti, ah you opened it already. it's item #212:05
* mzanetti totally forgot about that tab being open :D12:05
dandradermzanetti, and that nice diagram https://docs.google.com/a/canonical.com/drawings/d/1KFn03gFW1SXYOQn1l9hPCwFx_a8EAJA2AMc1hygqeEo/edit12:06
mzanettiack. makes sens to me12:06
mzanettidandrader: 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:12
dandradermzanetti, 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 points12:15
mzanettiah ok12:15
mzanettidandrader: what event is there when there's just an additional finger put down?12:16
dandraderQEvent::TouchUpdate12:16
mzanettiah12:16
mzanettithanks12:16
dandradera touch sequence starts when the first touch point is pressed and ends once the last remaining touch point is released12:17
mzanettiok... I would have expected to get a TouchBegin for each finger12:17
=== davidcalle_afk is now known as davidcalle
dandradermzanetti,  that you get from QTouchPoint::touchPoints()::state() or QTouchPoint::touchPointStates()12:18
mzanettigot it12:18
mzanettidandrader: yeah... your branch looks good. compiling it on the device to test12:19
=== mpt_ is now known as mpt
=== MacSlow|lunch is now known as MacSlow
darklight_Is anyone looking at https://bugs.launchpad.net/ubuntu/+source/compiz/+bug/1305438? It's been broken one way or another since ~ 12.1012:42
ubot5Launchpad bug 1305438 in compiz (Ubuntu) "hot corners stop working time to time" [Medium,Confirmed]12:42
pstolowskitsdgeos, ping12:54
Guest60061Cimi: tsdgeos ...i thot i did get access for the team, lemme check if i need to do one more approval step of something12:56
tsdgeospstolowski: hi12:56
pstolowskitsdgeos, 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
tsdgeospstolowski: nope12:58
pstolowskitsdgeos, (and they shouldn't be clickable when new search is in progress)12:58
pstolowskitsdgeos, ok.. so i've first part more less ready, but wanted to check with you about making current results dimmed12:59
kgunngreyback_: can you review ? https://code.launchpad.net/~dandrader/qtmir/frozenApps-lp1295623/+merge/23439313:00
pstolowskitsdgeos, would it make sense for you if it's a single property of Scope?13:00
dandraderkgunn, greyback_, mzanetti is on it13:00
greyback_kgunn: I believe mzanetti was looking at it13:00
kgunnta13:00
pstolowskitsdgeos, (all results are either disabled or not for given scope)13:00
dandradermzanetti, you should probably claim it13:00
tsdgeospstolowski: i guess that's fine yes13:01
pstolowskitsdgeos, 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
tsdgeospstolowski: just came from a week out, settling in, not terribly busy13:03
tsdgeosi guess i could take it13:04
pstolowskitsdgeos, awesome; how about a bool property of Scope named 'resultsEnabled'?13:05
tsdgeoshmmm13:06
tsdgeosactually do we need a new property?13:06
tsdgeosdon't we have the "searching" thing already? can use that?13:06
tsdgeospstolowski: ↑13:07
pstolowskitsdgeos, we have searchInProgress getter, but not exposed via property13:07
pstolowskitsdgeos, the only qproperty I can see is for 'favorite' flag13:07
tsdgeoshmmm13:07
tsdgeosand what does trigger the orange busy bar?13:07
* tsdgeos checks13:07
pstolowskitsdgeos, but yeah, resultsEnabled would = !searchInProgress afaict13:08
tsdgeosscope.searchInProgress13:08
tsdgeosseems exposed to me13:09
tsdgeosat least we use it :D13:09
pstolowskitsdgeos, ahhh, bloody scope interface defined somewhere else :/13:09
pstolowskitsdgeos, 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
tsdgeosoki13:11
tsdgeospstolowski: so what should i do then?13:11
tsdgeosmake the thing blurred + disabled while searching?13:11
pstolowskitsdgeos, 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 finished13:12
pstolowski* finishes13:12
tsdgeoswell you said13:13
tsdgeos resultsEnabled would = !searchInProgress afaict13:13
tsdgeos:D13:13
pstolowskitsdgeos, 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 / enabled13:13
pstolowskitsdgeos, so, I think we need a new property after all :/13:14
tsdgeosok13:15
pstolowskitsdgeos, i'll be back in 1h13:17
kgunndednick_: just checking, i know there's no prereq marked, but i can land lp:~nick-dedekind/ubuntu-settings-components/indicator-polishing w/o landing13:28
kgunnlp:~nick-dedekind/unity8/indicator-polishing13:29
kgunn?13:29
Cimikgunn, yes13:29
=== boiko_ is now known as boiko
Cimikgunn, but he needs an approve first :)13:30
kgunnCimi: yes sir...13:33
kgunnCimi: was just noticing it seemed ready13:33
kgunnCimi: is there someone who we usually get to top approve on u-s-s ?13:33
Cimikgunn, it is quite ready yes, but it is affected from some changes to the sdk, I would like to wait tomorrow/wednesday13:33
Cimikgunn, this is settings components, not system settings13:34
kgunnCimi: cool, i'll keep it on the "still cookin'" pile13:34
kgunnCimi: you're right, sorry, mis-spoke13:34
Cimikgunn, basically, wait sdk to revert few things, then if nothing breaks approbe13:34
=== _morphis is now known as morphis
kgunnthanks Cimi, you know which mp's we're eyeballing from sdk to land first ?13:35
Cimikgunn, otherwise we might just want to fine-tune some small details13:35
kgunndednick_: and i thot lp:~nick-dedekind/unity8/indicator-polishing was landing last week ?...what happened ?13:36
Cimikgunn, there are many of zsombi https://code.launchpad.net/ubuntu-ui-toolkit13:36
kgunnCimi: ok13:36
Cimikgunn, regarding listitems and divider13:36
Cimikgunn, it happened me not approving13:36
kgunnCimi: were you just being cruel ?13:37
kgunn:)13:37
Cimikgunn, last week there was a bug around slow initialization of the indicators13:37
Cimikgunn, you were receiving a message, you pulled down the indicators... blank page for 1-2 seconds13:37
kgunneegads13:37
Cimikgunn, nick fixed friday that and is fine13:37
kgunnCimi: ok...well, there will also be another train13:38
kgunnto land on13:38
Cimikgunn, however I dislike some UX changes daniela did (but she left)13:38
Cimikgunn, design also has a redesign for indicators13:38
kgunnbut she's back today right13:38
kgunnCimi: yeah...but that branch fixes a poop-ton of bugs13:38
Cimikgunn, so we might want to land the settings components part, and incorporate those design changes (that will fix my concern)13:39
Cimikgunn, I know, but we have policy of no regressions13:39
kgunnCimi: kk13:39
mzanettire13:39
mzanettikgunn: yes, on it13:39
kgunnCimi: what regression if the slow-init was addressed ?13:39
dednick_Cimi: there's no regression in the the u8 branch13:41
dednick_Cimi: it's a different design13:41
Cimidednick_, I think is a regression13:43
Cimidednick_, not your fault13:43
kgunnCimi: isn't danelia back today?13:43
dednick_Cimi: i think she's left?13:43
kgunndednick_: you mean forever ?13:43
Cimii think so13:43
dednick_kgunn: yeah13:44
Cimikgunn, I can approve nick branch, is a good improvement13:44
Cimikgunn, but I am sure we will receive complain for the new indicator design13:44
=== _salem is now known as salem_
Cimithe header bar loses the chevron for example, you don't know how to switch indicators13:45
Cimionce you pulled down13:45
kgunnCimi: i see, at that point...that monkey is on design13:45
dednick_it's going to change again in short order13:46
Cimikgunn, at the same time, vesa and ben were working on a redesign... and now we have that13:46
dandradergreyback_, Here is QOpenGLDebugLogger::startLogging crashing on me again... http://paste.ubuntu.com/8350547/13:46
dandradergreyback_, commented it out13:46
kgunnCimi: "working on redesign" doesn't sound "done"13:46
greyback_dandrader: weird13:46
dandradergreyback_, that was on krillin13:46
Cimikgunn, they user tested this morning13:47
Cimikgunn, and trust me, is miles better13:47
greyback_dandrader: looks like a null pointer..13:47
kgunnCimi: sorry, user tested nick's branch that's up?13:48
kgunnor ....new design we have yet to implement ?13:49
Cimikgunn, users tested the prototype13:51
Cimiof the new indicatords13:51
Ciminot nick branch13:51
kgunngot it13:52
Cimikgunn, I don't think any designer ever saw nick branch working13:52
Cimikgunn, because daniela left13:52
Cimikgunn, and other designers were working on the redesign...13:52
dednick_Cimi: they approved my changes. it's been there for ages.13:54
Cimidednick_, who approved?13:54
dednick_Cimi: ben, daniela, esti . can't remember who all13:57
dednick_Cimi, kgunn: https://docs.google.com/a/canonical.com/document/d/1KC40suUs13gajIlE1Ms6PJYoIPx_AjlF4ae6bvtNjNc/edit13:57
Cimidednick_, esti is not UX, daniela left...13:57
dednick_removing the chevron was a direct request13:57
=== lborda is now known as lborda_
Cimianyway, regardless we have UX approvals (only daniela), we have to change the design anyway, why not merging both?13:59
=== dandrader is now known as dandrader|afk
Cimiinstead of getting comments publicly, google plus/whatever "hey indicator redesign!" etc etc... then change it all over again in a couple of days14:00
=== lborda_ is now known as lborda__
=== dandrader|afk is now known as dandrader
mzanettiCimi: indeed much better with this option. thanks14:42
Cimiyw14:42
mzanettiCimi: now do you know a cure for Telepathy hiding all the time?14:42
Ciminot using it :)14:43
Cimiprobably similar solution14:43
pstolowskitsdgeos, 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
mzanettidoesn't have such an option I think14:43
mzanettiCimi: don't you use any IM?14:43
tsdgeospstolowski: everything needs to be in  the interface14:43
tsdgeospstolowski: so mocks for example can use the interface14:43
tsdgeosunity8 mocks14:43
pstolowskitsdgeos, ah, ok14:44
Cimimzanetti, I am antisocial, I don't chat apart IRC with colleagues :D14:44
mzanettimhm...14:50
* greyback_ needs reboot15:11
mzanettidandrader: do you know what happened to the touch cancellation for the edges?15:32
mzanettilist time I heard from it, it was "soon after qtcomp landed"15:32
dandradermzanetti, right15:33
dandradermzanetti, waiting for it bubble up the prioritized queue of TODO work15:33
dandraderkgunn, ^15:34
mzanettidandrader: ah... I think this should be critical... people start working around it: https://bugs.launchpad.net/ubuntu-ux/+bug/136870215:34
ubot5Launchpad 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
mzanettikgunn: ^15:34
mzanettiI think the only reason why this isn't critical for us, is that we don't have a bug number for it15:34
tsdgeos+115:35
=== gatox is now known as gatox_lunch
tsdgeosmarcustomlinson: please fix the tests in your branch15:53
marcustomlinsontsdgeos: k15:58
dednickCimi: looks like i you can't put a colorized image into an UbuntuShape.16:34
dednickCimi: so can't do anything about it :/16:34
Cimidednick, you can use shaderEffectSource16:34
Cimiis fixable, just, dammit!16:35
Cimimaybe there is a better fix16:35
Cimilike fixing the icons of the notifications16:35
dednickCimi: hm? SharedEffectSource renders a texture.16:39
dednick*into a16:39
=== dandrader is now known as dandrader|lunch
Cimidednick, you can wrap a texture (an icon) inside a shape16:39
Cimidednick, we use this in the shell already somewhere16:40
=== gatox_lunch is now known as gatox
* Cimi away, catch you a bit later17:03
dednickCimi: that change is in usc now.17:19
=== dandrader|lunch is now known as dandrader
dandradergreyback_, update the splashscreen unity8 mp19:10
dandraderupdated19:10
=== dandrader is now known as dandrader|afk
=== salem_ is now known as _salem
=== dandrader|afk is now known as dandrader

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