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