| === iahmad is now known as iahmad|afk | ||
| === iahmad|afk is now known as iahmad | ||
| tsdgeos | i can't reproduce the test failure of https://code.launchpad.net/~aacid/unity8/no_hswipe_if_vswipe_test/+merge/193401 :-/ | 08:42 |
|---|---|---|
| tsdgeos | Saviq: if you have some time can you try to? | 08:43 |
| mzanetti | mhr3: hey, I bet you know why this happens: https://bugs.launchpad.net/unity8/+bug/1249255 | 09:25 |
| ubot5 | Ubuntu bug 1249255 in Unity 8 "all but 9 apps disappear for minutes" [Undecided,New] | 09:25 |
| mhr3 | bug in click scope | 09:25 |
| tsdgeos | anyone for https://code.launchpad.net/~aacid/unity8/robust_test_show_scope_on_load/+merge/194470 ? | 09:30 |
| tsdgeos | Cimi: can you to approve https://code.launchpad.net/~aacid/unity8/no_hswipe_if_vswipe_test/+merge/193401 ? there is a dancing test that fails sometimes but is not related to the changes there and i think i've fixed it in https://code.launchpad.net/~aacid/unity8/robust_test_show_scope_on_load/+merge/194470 | 09:45 |
| Cimi | tsdgeos, yup | 09:45 |
| tsdgeos | tx | 09:46 |
| nic-doffay | Cimi, got time for a small review? | 09:51 |
| nic-doffay | https://code.launchpad.net/~nicolas-doffay/unity8/infographic-z-order-fix | 09:51 |
| nic-doffay | it's one line | 09:51 |
| Cimi | nic-doffay, makes sense ::) | 09:52 |
| nic-doffay | Cimi, think jenkins was just **** with that instability | 09:52 |
| Cimi | nic-doffay, otherwise you can just put it at the bottom of the file | 09:54 |
| Cimi | nic-doffay, not sure which approach we like to follow for qml | 09:54 |
| Cimi | Saviq, tsdgeos if we want to move an element on higher z order, shall we change z order or play with the ordering of components in the file? | 09:54 |
| Cimi | I'd go to put components that should stay on top at the bottom of the file | 09:55 |
| Cimi | z index can be more tricky | 09:55 |
| nic-doffay | Cimi, the bottom of the file puts it above the mouse area. | 09:56 |
| nic-doffay | Figured it's safer doing that. | 09:56 |
| tsdgeos | i think we mostly do "ordering" based z-order | 09:56 |
| Cimi | nic-doffay, you can move the mouse area as well | 09:56 |
| Cimi | tsdgeos, indeed | 09:57 |
| nic-doffay | Cimi, I didn't want to juggle the components in case I caused a bug which I wasn't aware about. | 09:57 |
| Cimi | nic-doffay, could you pls do the say ordering the components instead changing z order? | 09:57 |
| Cimi | *same | 09:57 |
| nic-doffay | Cimi, if you can without a doubt say it hasn't broken anything afterwards then yeah. | 09:57 |
| Cimi | well you have it working with the z index, try changing order and obtain the same | 09:58 |
| nic-doffay | Cimi, I've pushed. To test go to tests/mocks/libusermetrics/UserMetrics.cpp then on line 279 just add a + 0.8 to the rand() | 10:02 |
| nic-doffay | That should make the circles big enough to see. | 10:03 |
| nic-doffay | Cimi, then run ./run -f | 10:03 |
| nic-doffay | (after building and all) | 10:03 |
| nic-doffay | pstolowski, you busy with this? https://bugs.launchpad.net/unity8/+bug/1226311 | 10:08 |
| ubot5 | Ubuntu bug 1226311 in Unity 8 "Cannot search for apps on the home scope" [High,Confirmed] | 10:08 |
| nic-doffay | or can I steal it? | 10:08 |
| Cimi | nic-doffay, I'm building unity, but I bet it works | 10:10 |
| Cimi | nic-doffay, Clock does not have a mouse area so should not be a problem with the other MouseArea | 10:10 |
| pstolowski | nic-doffay, nope, i havne't been looking at it | 10:12 |
| nic-doffay | pstolowski, mind if I take it over? | 10:14 |
| pstolowski | nic-doffay, no problem, you can take it | 10:15 |
| nic-doffay | pstolowski, what regressions were you worrying about in the one comment? | 10:15 |
| pstolowski | nic-doffay, previews, progress for app install | 10:18 |
| nic-doffay | pstolowski, how can I test for those regressions? | 10:21 |
| nic-doffay | Just to make sure it doesn't "$%"$% anything | 10:21 |
| pstolowski | nic-doffay, well, manually, on the phone | 10:22 |
| Cimi | nic-doffay, approved | 10:23 |
| nic-doffay | Cimi, cool thanks | 10:29 |
| nic-doffay | tsdgeos, got time for a review? | 10:45 |
| tsdgeos | nic-doffay: sure | 10:46 |
| nic-doffay | tsdgeos, it's what I was working on before with the search history being persistent. | 10:49 |
| nic-doffay | https://code.launchpad.net/~nicolas-doffay/unity8/search-history-persist/+merge/193935 | 10:49 |
| tsdgeos | oki | 10:49 |
| nic-doffay | Except it's kept QML side this time at Saviq's request | 10:49 |
| nic-doffay | tsdgeos, thanks dude | 10:50 |
| tsdgeos | so we don't save to disk? | 10:50 |
| Saviq | tsdgeos, not yet - we're not sure where to save it to | 10:50 |
| tsdgeos | ok | 10:50 |
| Saviq | tsdgeos, we might do zeitgeist, we might do U1Db, we might do something else entirely... | 10:50 |
| dednick | MacSlow: ping | 10:58 |
| * greyback bbiab | 11:37 | |
| === dandrader is now known as dandrader|afk | ||
| === MacSlow is now known as MacSlow|lunch | ||
| === dandrader|afk is now known as dandrader | ||
| tsdgeos | arggg | 12:16 |
| tsdgeos | the Actual (lvwph->m_headerItem->y()): -1.42109e-14 Expected (0.): 0 thing is getting annoying :S | 12:16 |
| tsdgeos | wonder if we should disable it until we get the 5.2 (i'm convinced it has to do with poor qml scenegraph loop forgetting to send the last event) | 12:17 |
| tsdgeos | and if with 5.2 still fails we debug it then on the jenkins instances | 12:18 |
| tsdgeos | Saviq: ↑↑↑↑↑ | 12:18 |
| Saviq | tsdgeos, +1 | 12:18 |
| Saviq | tsdgeos, + console.log("scope", scope) | 12:20 |
| tsdgeos | wops | 12:20 |
| tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/disableTestMaximizeVisibleAreaMoveUpAndShowHeader/+merge/194492 | 12:34 |
| Saviq | tsdgeos, happroved | 12:34 |
| === iahmad is now known as iahmad|afk | ||
| jalcine | anyone has tips for quick importings from git to bzr on a junk branch launchpad? | 13:04 |
| === alan_g is now known as alan_g|lunch | ||
| Saviq | jalcine, there's git-bzr | 13:14 |
| Saviq | jalcine, that allows you to push from a git repo to bzr directly | 13:14 |
| tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/killApplicationsFilterGrid.qml/+merge/194511 | 13:23 |
| jalcine | Saviq: thanks | 13:23 |
| Saviq | tsdgeos, ooh cleanup, will have a look later ;) | 13:24 |
| tsdgeos | i have a few more cleanups comming, basically introducing some "base items" since we now for example define "signal clicked(int index, var model, real itemY)" in various and then let the GenericScopeView use it, planning to create a item that has those and make all the other inherit from it | 13:26 |
| tsdgeos | Saviq: ↑↑↑ | 13:26 |
| Saviq | tsdgeos, yeah, we need a refactor of the tiles indeed | 13:26 |
| === alan_g|lunch is now known as alan_g | ||
| dednick | mhr3: ping | 13:42 |
| === MacSlow|lunch is now known as MacSlow | ||
| mhr3 | dednick, pong | 14:21 |
| mhr3 | dednick, did you decide to switch from the free trial? :) | 14:22 |
| mhr3 | dednick, you get faster response times as a proper subscriber | 14:22 |
| dednick | mhr3: ahha. | 14:22 |
| dednick | mhr3: https://bugs.launchpad.net/unity8/+bug/1238302 . still happening for you? cant reproduce on mine. | 14:23 |
| ubot5 | Ubuntu bug 1238302 in Unity 8 "Category reordering causing column "shifting" in overridden result model" [Medium,Triaged] | 14:23 |
| mhr3 | dednick, think it's only happening with the category reordering | 14:23 |
| mhr3 | dednick, https://bugs.launchpad.net/unity8/+bug/1240118 | 14:25 |
| ubot5 | Ubuntu bug 1240118 in Unity 8 "Scrolling in dash breaks" [High,Triaged] | 14:25 |
| mhr3 | see the first comment | 14:25 |
| mhr3 | same applies to this | 14:25 |
| dednick | ok | 14:26 |
| dednick | mhr3: right, so they're still issues though i guess... | 14:28 |
| mhr3 | dednick, yes, those two issues were why we disabled the reordering | 14:28 |
| Cimi | Saviq, I'm using the new inversemousearea with topmost | 14:29 |
| Cimi | Saviq, but it's still limited to the runningapplicationgrid | 14:29 |
| Saviq | Cimi, with no sensingArea set? | 14:30 |
| Cimi | Saviq, no | 14:30 |
| Saviq | Cimi, please talk to zsombor | 14:30 |
| Saviq | Cimi, he added a slew of tests to make sure it's working, though, AFAIK, so that'd be real weird :/ | 14:31 |
| Cimi | Saviq, http://paste.ubuntu.com/6382346/ | 14:32 |
| Cimi | Saviq, against trunk | 14:32 |
| Cimi | Saviq, notes | 14:32 |
| Cimi | dandrader, notes | 14:33 |
| Saviq | greyback, notes | 14:34 |
| Cimi | this game is fun :D | 14:34 |
| Saviq | Cimi, you won ;) | 14:35 |
| greyback | dammit | 14:37 |
| greyback | I was being interviewed | 14:37 |
| mhr3 | sil2100, hello... zmqpp? :) | 14:39 |
| kgunn | greyback: like your famous ? or getting a new job ? :) | 14:39 |
| greyback | kgunn: heh, I wish. Helping out a college student for a survey on co-workers and "digital nomads" | 14:40 |
| MacSlow | Saviq, https://code.launchpad.net/~macslow/unity8/notification-fullscreen-support https://code.launchpad.net/~macslow/unity8/fix-1200569 | 14:45 |
| MacSlow | Saviq, still need to retrigger the last one | 14:45 |
| Saviq | MacSlow, I'll retrigger both and test locally, too | 14:45 |
| MacSlow | ok | 14:46 |
| Saviq | MacSlow, actually, merge trunk in both of them? | 14:46 |
| Saviq | MacSlow, that'll trigger and be more reliable | 14:46 |
| === dandrader is now known as dandrader|lunch | ||
| MacSlow | Saviq, yeah... but I don't do that daily really... | 14:46 |
| MacSlow | Saviq, I just refresh both of them just now... | 14:47 |
| Saviq | MacSlow, sure, no need to do daily - but it makes sense when there's problems | 14:47 |
| Saviq | MacSlow, trunk has some test reliability improvements | 14:47 |
| MacSlow | Saviq, yeah... I'll do that anyway for both now | 14:47 |
| tedg | Is there a way to run Unity8 with indicators on my desktop? | 15:04 |
| tedg | Seems just /usr/bin/unity8 doesn't have them? | 15:04 |
| mterry | Saviq, I think I may have helped re-introduce tags to lp:unity8. Is there a special trick to deleting them again, or can I just use your recipe to delete, pointing at lp:unity8? | 15:04 |
| Saviq | mterry, just do the same for lp:unity8, yeah | 15:05 |
| Saviq | tedg, should work just fine | 15:06 |
| Saviq | tedg, pass -mousetouch to get gesture support | 15:06 |
| tedg | Ah, that was the trick. I just couldn't open them! Thanks Saviq | 15:07 |
| MacSlow | Saviq, ok merged said branches with unity8 trunk again and resolved all merge-conflicts... lp:~macslow/unity8/fix-1200569 (r428 now) lp:~macslow/unity8/notification-fullscreen-support (r488 now) | 15:14 |
| Saviq | MacSlow, cheers, will monitor them | 15:17 |
| om26er | mzanetti, hey! now that automerger seems to be working. Can we land this https://code.launchpad.net/~mzanetti/unity8/launcher-small-tweaks/+merge/191380 ? | 15:18 |
| Saviq | om26er, he's on holiday | 15:19 |
| Saviq | om26er, but we'll keep an eye on it | 15:20 |
| === _salem` is now known as _salem | ||
| om26er | Saviq, ok, thanks | 15:20 |
| === _salem is now known as salem_ | ||
| mhr3 | osk still not working in 15? | 15:27 |
| Saviq | mhr3, surfaceflinger? | 15:30 |
| Saviq | mhr3, works on mir, Cimi reported problems on sf, but I doubt we'll be fixing anything for sf | 15:30 |
| mhr3 | Saviq, yea | 15:30 |
| mhr3 | Saviq, :/ mir on galaxy nexus is slow, and no osk without mir... just suc... is bad | 15:31 |
| === alan_g is now known as alan_g|afk | ||
| tsdgeos | Saviq: the Verifier thing you made, did check signals signature? | 15:34 |
| Saviq | tsdgeos, no, I don't think so | 15:45 |
| tsdgeos | was trying to see if we can do it | 15:45 |
| tsdgeos | i can see if the signal is defined, but can't seem to be able to find out the params | 15:46 |
| tsdgeos | from JS i mean | 15:46 |
| Saviq | tsdgeos, I don't think we can | 15:46 |
| Saviq | tsdgeos, if you did that in C++, we might | 15:47 |
| tsdgeos | Saviq: so, let me dump my mind :D This is about the "we are using random stuff as source for the genericscopeview delegate and it just happens that they all have mostly the same properties" and i was thinking if A) I should introduce a base Item that all those should use or B) If I should introduce a checker that complains in a more understandable way if some of the properties are not there as expected. A seems more C++-y and B seems more JS-ducktaping-y | 15:51 |
| tsdgeos | what's your opinion? | 15:51 |
| tsdgeos | i.e. we need those delegates to have signal pressAndHold, signal clicked, a expandable: property, etc. | 15:51 |
| Saviq | tsdgeos, A | 15:57 |
| Saviq | tsdgeos, just have an "abstract" BaseTile or something | 15:58 |
| Saviq | tsdgeos, you can name that BaseCard as "Cards" is the new word for items in dash | 15:58 |
| tsdgeos | Saviq: this is not about the Tile but about the FilterGrid/Carousel level | 15:58 |
| fginther | Saviq, is this something you created? http://10.97.0.26:8080/job/jenkins-autocheck/ | 15:58 |
| Saviq | tsdgeos, ah, then yeah BaseRenderer | 15:58 |
| Saviq | fginther, nope | 15:58 |
| Saviq | fginther, I don't have rights to do so | 15:58 |
| fginther | Saviq, thx | 15:58 |
| Saviq | tsdgeos, only thing missing is multi-inheritance ;) | 15:59 |
| tsdgeos | Saviq: ok, then my other "problem", is that e.g. Carousel is in Components/ not in Dash/, so it's a bit weird, move it over? | 15:59 |
| Saviq | tsdgeos, where we want a BlahCarousel to inherit from BaseRenderer and a non-dash-specific Carousel | 15:59 |
| Saviq | tsdgeos, right, ↑ that ;) | 15:59 |
| tsdgeos | yes | 15:59 |
| tsdgeos | you can actually do that | 15:59 |
| tsdgeos | i've checked | 15:59 |
| Saviq | just not in QML? | 16:00 |
| tsdgeos | well | 16:00 |
| tsdgeos | not that exactly | 16:00 |
| Saviq | tsdgeos, either move it or wrap in DashCarousel | 16:00 |
| tsdgeos | yeah ignore me, was thinking crap :D | 16:00 |
| Saviq | tsdgeos, but that unfortunately means aliasing all the carousel's props, which is unfortunate | 16:00 |
| Saviq | that's why multi-inheritance would be cool | 16:00 |
| === dandrader|lunch is now known as dandrader | ||
| === alan_g|afk is now known as alan_g | ||
| mhr3 | Saviq, blasphemy! | 16:12 |
| mhr3 | i mean using multi-inheritance and good in the same sentence | 16:13 |
| Saviq | mhr3, lol | 16:33 |
| Saviq | mhr3, that a personal vendetta or should I read up? ;) | 16:34 |
| mhr3 | Saviq, interfaces are the way to go, not multi-inheritance, it's been known for 20years :) | 16:37 |
| dandrader | heh, I thought the same (multi-inheritance + good = blasphemy) :) | 16:45 |
| Saviq | mhr3, sure, in this case I meant multi-inheritance *with* just an interface (although such a thing doesn't exist in QML, nor really in C++, does it ;)) | 16:50 |
| Saviq | although /me likes mixins sometimes ;P | 16:51 |
| mhr3 | Saviq, yea, some languages pretend to be better by not having a clear separation there, but they just got it wrong | 16:52 |
| === jhodapp is now known as jhodapp|brb | ||
| MacSlow | seb128, if you look again at https://bugs.launchpad.net/ubuntu/+source/notify-osd/+bug/404658 I've attached the screencast with the two alternative solution-suggestions, which stay within notify-osd design-guidelines and still gives the needed info to the user. | 17:10 |
| ubot5 | Ubuntu bug 404658 in notify-osd (Ubuntu) "notification summary doesn't change for synchronous messages" [Low,Confirmed] | 17:10 |
| * greyback eow | 17:23 | |
| tedg | mterry, Can you look at: https://code.launchpad.net/~ted/unity-greeter/upstart-indicators/+merge/192936 | 17:24 |
| tedg | mterry, The greeter is broken in current without that. | 17:24 |
| mterry | tedg, you and upstart-in-greeter! | 17:25 |
| tedg | mterry, Hey, I even wrote this patch! ;-) | 17:25 |
| === dandrader is now known as dandrader|afk | ||
| mterry | tedg, you've tested and it works for you? | 17:26 |
| tedg | mterry, Yes, me and charles | 17:26 |
| mterry | tedg, OK. Code looks reasonable | 17:27 |
| charles | mterry: +1 for this, it's a good change | 17:27 |
| tedg | We really need a name for using trunk's now that trusty is "managed" | 17:28 |
| tedg | Thanks mterry! | 17:33 |
| mterry | tedg, thank you! | 17:33 |
| === jhodapp|brb is now known as jhodapp | ||
| === dandrader|afk is now known as dandrader | ||
| === alan_g is now known as alan_g|EOW | ||
| === dandrader is now known as dandrader|bbl | ||
| === salem_ is now known as _salem | ||
| === dandrader|bbl is now known as dandrader | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!