=== boiko_ is now known as boiko === _salem is now known as salem_ === salem_ is now known as _salem [07:34] larsu: ping [07:44] Saviq: think did not understand your comment in https://code.launchpad.net/~aacid/unity8/category-expansion/+merge/178726/comments/404942 [07:44] tsdgeos, I meant that on ListView, there's already a positionViewAtIndex() that does so with animation [07:44] tsdgeos, and I think we'll need LVWPH::positionViewAtIndex() soon enough [07:45] tsdgeos, so I was wondering if we could add it (doing as close to what LV::pVAI() does) [07:45] yeah well i can rename maximizeVisibleArea :D [07:45] tsdgeos, and then LVWPH::maximizeVisibleArea() would just be responsible for calculation what to do, and dispatch to LVWPH::pVAI() for the actuall scrolling [07:46] tsdgeos, yeah, but ::mVA does more than pVAI, doesn't it? [07:46] hello, i have some problem regarding the ubuntu search [07:46] Saviq: maybe, but not muchh more really, just positions the view so that the most of the index is visible [07:47] tsdgeos, so it does more - ::pVAI is "dumb" in the sense that it just does what you tell it to [07:47] tsdgeos, ::mVA determines what to do [07:47] hmmm [07:47] i don't see the difference [07:48] pVAI will scroll the view too, no? [07:48] tsdgeos, what I really mean is that pVAI takes a second argument [07:48] tsdgeos, mVA does not [07:48] ah [07:48] ok [07:48] yes [07:48] tsdgeos, that's the difference [07:48] tsdgeos, and we might need that [07:49] tsdgeos, being able to explicitly tell to position at beginning or end [07:49] sure, thing is that implementing pVAI nicely takes time [07:49] so i just did what we needed for now [07:49] tsdgeos, yeah, got it [07:49] i.e. i guess pVAI should work on all items [07:50] mVA just works on indexes that are already visible or cached [07:50] because we just need it for that [07:50] it's a reasonable compromise [07:50] tsdgeos, ah, got it [07:50] because for non created indexes [07:50] tsdgeos, ok, do the quickest thing for now [07:50] i have to go and calculate "where" they may be [07:50] tsdgeos, yup, got it [07:50] and then stuff starts to be harder [08:17] Cimi, can you do https://code.launchpad.net/~saviq/unity8/support-plurals/+merge/179350 please? [08:18] Saviq, y [08:19] Saviq, on my side, I have issues with testing crossfadeimage [08:20] Saviq, crossfadeImage status can be Image.ready with a not-existing image :O [08:21] Cimi, file a bug for SDK, create a test for it, fix it! :) [08:21] Cimi, should be easy [08:22] Saviq, it's our crossfadeimage componnt [08:22] Saviq, we're not using sdk [08:22] Saviq, this is opened https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1204565 [08:22] Launchpad bug 1204565 in Ubuntu UI Toolkit "CrossFadeImage needs properties to control crossfade and fade" [Undecided,New] [08:23] Cimi, ok, fix in ours, then [08:23] ok [08:39] Saviq: https://code.launchpad.net/~saviq/unity8/support-plurals/+merge/179350 [08:40] tsdgeos, yeah? [08:40] read the comment :D [08:40] tsdgeos, ;) [08:40] the make line you gave doesn't work here [08:40] tsdgeos, interesting, try "unity8.pot" [08:40] tsdgeos, maybe make behaves differently than ninja here [08:40] nope [08:40] doesn't work ether [08:40] +i [08:41] * Saviq tries [08:41] tsdgeos, make -C builddir pot_file [08:41] ah, that [08:41] yep [08:43] Cimi, tsdgeos beat you to it ;) [08:44] Saviq, I am running that command now :P [08:44] Saviq, took time branching [08:44] :) [08:44] sorry [08:44] * tsdgeos hides [08:46] works here too [10:19] tsdgeos, greyback, dednick, I will have to ask one of you guys to do https://code.launchpad.net/~unity-team/unity8/app-preview-data/+merge/179348 [10:19] as I've been involved with it too much to review it properly again [10:20] or you could wait for mzanetti on Monday ;) [10:20] wait! wait! wait! [10:20] * tsdgeos runs [10:20] ouch this one's big [10:20] tsdgeos, BIG?! [10:21] 2.5KLOC [10:21] tsdgeos, may I remind https://code.launchpad.net/~unity-team/unity8/indicators-client/+merge/172582 [10:21] tsdgeos, it was split in two before [10:21] that one is big-er [10:21] :D [10:21] tsdgeos, but it was so intertwined [10:21] tsdgeos, I decided to merge [10:21] sure, makes sense [10:21] sometimes stuff has to be big [10:21] tsdgeos, it's mostly green, so a good thing [10:22] lol [10:22] tsdgeos, ah also [10:22] * tsdgeos was about to do a bad joke [10:22] tsdgeos, DashPreview was rm'ed and added [10:22] tsdgeos, so it's not really 2.5k [10:22] tsdgeos, because bzr is stupid [10:23] meh [10:23] tsdgeos, what was the bad joke? what was the bad joke?! [10:23] * greyback theorizing about tsdgeos' joke [10:23] git is not that much smarter [10:23] lol [10:23] when you move files [10:23] you need to tell it something like [10:23] --findharder [10:23] or something [10:23] lol [10:23] so it actually realizes you moved the files [10:23] tsdgeos, you can always go back, though [10:23] tsdgeos, with bzr you're stuck [10:23] I want 'find' to have that switch [10:23] lol more [10:23] --find-copies-harder [10:24] --find-copies-harder [10:24] For performance reasons, by default, -C option finds copies only if the original file of the copy was modified in the same changeset. This flag makes the command inspect unmodified files as candidates for the source of copy. This is a very expensive [10:24] operation for large projects, so use it with caution. Giving more than one -C option has the same effect. === jibel_ is now known as jibel [10:24] tsdgeos, at least it has that option [10:24] yeah [10:24] tsdgeos, and anyway, rebase -i FTW [10:24] oh i'm not defending bzr :D [10:24] Saviq: i guess i can try to have a look at that Dash thig [10:25] I always enjoy bzr bashing Friday [10:25] the bad "joke" was: "it's mostly green, so a good thing" -> "I don't like vegetables" [10:25] now you can hate me more [10:25] rotfl [10:25] tsdgeos, let me try to repro [10:26] tsdgeos: the quotes around "joke" were well placed [10:27] you're killing me guys ;D [10:29] tsdgeos, http://pastebin.ubuntu.com/5965895/ [10:29] tsdgeos, play with the home scope a bit (search, scroll, clear the search) [10:29] Saviq: that on "plain" unity8? [10:30] tsdgeos, yes [10:30] tsdgeos, needs to be device, though, as on desktop you don't actually get empty homescope for long [10:30] ok, let me see [10:31] boooooo [10:31] i just set up the device for unity-mir [10:31] don't really want to go back [10:31] tsdgeos, you have the Nexus 10 don't ypu [10:31] you [10:32] Saviq: you speak the truth [10:32] tsdgeos, charged? ;) [10:32] yeah charged it yesterday [10:32] tsdgeos, you charged it recently, so should be fine [10:32] ok [10:32] biab [10:32] Saviq: but do you think it'll repro there? [10:42] tsdgeos, should, I think [10:51] tsdgeos: you doing that review then? [10:51] dednick: unless you are crazy about do it yourself, yeah [10:51] tsdgeos: lol. crazy about a review. funny man [10:52] i know, trying to get my comedy skills up in case i need to change careers [10:52] :D [10:52] tsdgeos: a 2.5k review esspecially :) [10:52] but better then a 9k [10:55] Saviq: if you're upset about not being able to review it yourself, you can do https://code.launchpad.net/~nick-dedekind/unity8/indicators-behaviour.tabs/+merge/175334 ;) [10:55] a measly 500 lines [10:57] dednick, yup, have that on my radar [10:57] dednick, not sure about the "upset" part, though :D === MacSlow is now known as MacSlow|lunch [10:58] Saviq: i just assumed after how you got stuck into that indicators review ;) [11:06] Saviq: lol if i do that patch on the Nexus10 i get a totally empty home scope === greyback is now known as greyback|lunch [11:07] but if i search stuff it appear [11:07] s [11:07] :S [11:07] err, do i have local stuff? [11:08] no i don't [11:10] but yeah [11:10] can reproduce the header getting crazy on the middle of the screen [11:10] not cool [11:31] Saviq: paulliu: how do i test that app preview thing? [11:31] tsdgeos, install on device, gsettings get com.canonical.Unity.Dash scopes [11:31] tsdgeos, append "click/click.scope" [11:32] to the list from gsettings [11:32] and gsettings set com.canonical.Unity.Dash scopes "['the', 'list', 'here']" [11:32] tsdgeos, you shouldn't need to restart unity8, and you should get the click scope as a new master scope [11:32] tsdgeos, either tapping or long-pressing on an app in there should show you the preview [11:33] ok [11:33] tsdgeos, not to mention `make -C builddir tryAppPreview` for the basic tests [11:33] sure [11:33] that's test, i wanted to see it in real life [11:33] tsdgeos, yup, it should even install the click package for you and allow to run it [11:33] tsdgeos, assuming everything else is working in the image already [11:36] Saviq: changing on the fly didn't seem to work [11:36] tsdgeos, weird, worked for me [11:38] tsdgeos, see with 'get' if you actually got it updated [11:38] after a rerun of the shell it worked [11:38] ok [11:38] filing bug [11:38] Saviq: is "Rate This" supposed to work? [11:38] tsdgeos, no [11:38] tsdgeos, there's no support yet in the scope and friends [11:38] ok === greyback|lunch is now known as greyback [11:38] install didn't work [11:38] but i guess that's not your fault [11:39] Saviq: i find it a bit weird that i can scroll the left size of the preview when on the tablet even if it fits fine on screen [11:40] tsdgeos, OTOH having it static might suggest it won't ever be scrollable... not sure we have a generic rule here [11:40] tsdgeos, but I know what you mean [11:40] sure [11:40] not blocking on that [11:40] just saying it's a big strange [11:40] /it's/felt [11:41] tsdgeos, did it install fine? [11:41] nope [11:41] didn't do anything [11:41] after pressing the install button [11:42] tsdgeos, ok, might be the backends didn't land yet, will verify [11:42] later [11:42] Saviq: wow, this is even weirded :D [11:42] press the install button and drag down [11:42] only the install button moves (the icon stays) and at some point the button disappers from screen [11:42] :S [11:43] tsdgeos, lol [11:43] tsdgeos, yeah, GridView should be interactive [11:43] same if you go up btw [11:43] *not [11:43] lol, and i can move the top left icon right :D [11:43] can i block on that? [11:44] tsdgeos, pushed [11:44] tsdgeos, yes [11:45] tsdgeos, "top left icon"? [11:45] the thing above the install button [11:45] tsdgeos, hmm not interactive at all here... [11:46] tsdgeos, tablet mode? [11:46] yep [11:46] tsdgeos, please write steps to repro on the MR, not seeing it here === alan_g is now known as alan_g|lunch [11:49] Saviq: ahhh [11:49] i think it makes sense now [11:49] it's in case you have more images [11:50] tsdgeos, yeah, the top part is a listview [11:50] ok [11:50] we definitely need to revisit the idea of adding a "max overshoot" to listviews [11:50] tsdgeos, same "should we scroll if nothing there" applies [11:50] this is getting silly [11:50] i can understand it should scroll a bit to show it's scrollable [11:51] but letting me move the screenshot 500px in empty spaces is confusing [11:51] lunch time [11:51] bbl [11:51] o/ === MacSlow|lunch is now known as MacSlow === dednick is now known as dednick|lunch === _salem is now known as salem_ [12:12] tsdgeos, btw, file a bug with ubuntu-ux about a generic rule for overshooting === alan_g|lunch is now known as alan_g [12:34] Saviq: i will, Oren already told me they want to limit ages ago and i even did a dirty hack when coding the hud appstack that gerry smartly told me to drop [12:34] :D [12:34] but i'll file it [12:44] Saviq: https://bugs.launchpad.net/ubuntu-ux/+bug/1210507 [12:44] Launchpad bug 1210507 in Ubuntu UX "Generic rule for overshooting scrollables/lists/etc" [Undecided,New] [12:55] Saviq: ahhhhhhhh [12:56] autogenerated files [13:18] larsu: ping === jhodapp|afk is now known as jhodapp [13:37] tsdgeos, yeah, I know [13:38] tsdgeos, will try and grab the source file and generate them build time [13:38] +1 [13:44] tsdgeos, btw, do you know if, when cross-compiling, will it be safe to use amd64 moc and qdbusxml2cpp for armhf sources? [13:45] tsdgeos, I don't expect them to be different, but then who knows... [13:45] they should [13:45] can't say for sure [13:45] but i don't see why they shouldn'y [13:45] not much a cross compile guy myself [13:45] much -> at all [13:45] :D === lborda is now known as lborda_afk === alan_g is now known as alan_g|afk [14:25] tsdgeos, did you have time to look into the proxymodel count thing? [14:28] mhr3: nope sorry [14:29] dednick, heyo! Thanks for your review of the powerd-test branch. You noted that I might not be generating the qmltypes file needed for "typeinfo plugin.qmltypes". I don't really understand the whole qmltypes thing, I just cargo culted that. Is there manual work needed to generate that? It doesn't happen automatically? [14:30] mterry: we don't need to do the qmltypes for mocks. just remove the line :) [14:31] mterry: it's just to get type info into qtcreator [14:31] dednick, fair. But for normal plugins, it does get generated automatically? [14:31] Using the macros for installing the qmldir file? [14:32] mterry: the export_qmlplugin does it for you using qmlplugindump [14:32] mterry: export_qmlplugin macro i mean [14:33] dednick, OK, makes sense [14:34] Wellark, For quick lists are you creating a new service or is it part of the Unity8 mega memory space? [14:38] guys, who has mako? [15:00] tedg: ping. can you please put a Position key in the indicator-network service file please? [15:00] NO! [15:00] Oh, okay, since you asked so nicely. [15:00] :-) [15:01] tedg: :) thanks === alan_g|afk is now known as alan_g [15:39] mterry: ping [15:42] kgunn, hello [15:43] mterry: hey....you gotta nexus4 right ? [15:43] we're in the "go green" hotseat [15:43] http://reports.qa.ubuntu.com/smokeng/saucy/image/3439/ [15:43] kgunn, yeah [15:43] mterry: can you take a look [15:44] kgunn, ok [15:44] mterry: thanks so much....all failed which is weird...i hope its just some global bug with fmwk or something.... [15:46] kgunn, do you know if there's a trick to running the tests, or do I just adb shell in and run "autopilot run unity8"? [15:50] Saviq: ^ ? [15:50] or MacSlow ^ ? [15:50] since he's had so much AP fun lately :) [15:51] mterry, that's what I do... [15:51] mterry, did you do setup described here https://wiki.ubuntu.com/Touch/Testing/Autopilot/ ? [15:52] kgunn, hm... that seems to have been too much ;) [15:53] mterry, did you do setup described here https://wiki.ubuntu.com/Touch/Testing/Autopilot/ ? [15:53] mterry, not sure if you got that... you just disconnected [15:53] MacSlow, yeah thanks, I just got it [15:54] MacSlow, will set up [15:54] is that autopilot PPA really still needed? thought it all got in saucy [15:57] mterry, well it's got the most current bits... so I'd say yes [15:57] mterry, at least for "autopilot" itself [15:58] mterry: not sure if you're familiar with that dashboard....but there are some links to artifacts/console output that i know the QA team recently updated to hold bette/more info.... [15:58] mterry, phablet-test-run -i -p unity8-autopilot -n unity8 [15:58] maybe there's a clue in there [15:58] mterry, sets up everything-autopilot [15:59] kgunn, no useful info there [15:59] Saviq: :( [15:59] I have something weird [15:59] kgunn, all the tests basically say "didn't initialize in time" [15:59] Saviq, ooh, nice line. From the device? [15:59] mterry, no, from host [15:59] (once phablet-tools is installed) [15:59] k [16:00] mterry, it only requires ssh on the device first [16:00] within the component, just before emitting a signal called "swapped()", I put console.log("swapped called") [16:00] on the file using the component [16:00] onSwaped: console.log("swapped") [16:00] swapped [16:00] btw [16:01] I don't get 1 "swapped" for each "swapped called" [16:01] what?? [16:01] like it's not emitting signals [16:03] Cimi, maybe there's a handler in a subclass? [16:04] Saviq, sorry I don't get [16:04] Saviq, component is crossfadeimage, signal swapped in the file [16:05] Saviq, in shell.qml, inside the crossfadeimae component declaration, I have onswapped [16:05] Saviq, it gets less events :-\ [16:05] I put onSwapped within crossfadeimage.qml and it gets all of them [16:05] they are like not passed through the shell.qml [16:07] it's actually missing one event, no idea why [16:12] Saviq, ? [16:12] Cimi, dunno, did you try putting onSwapped inside crossfadeimage [16:12] Saviq, yes, and works... [16:13] Saviq, inside Shell.qml, it's missing one signal [16:13] most of them but not one [16:14] Cimi, dunno, would have to try, but gtg now [16:18] Saviq, enjoy weekend, thx [16:24] Hrm, seems like the touch object just isn't doing anything. taps or drags [16:27] mterry, I think that might be a complete input failure again [16:27] mterry, stop unity8 as phablet [16:27] mterry, sudo kill `pidof ubuntuappmanager` [16:27] mterry, and try again [16:28] Saviq, ok [16:29] mterry, to try, also check if you run indicators-client manually [16:30] mterry, also, when you reproduce the lack of input [16:32] Saviq, didn't help [16:32] mterry, interesting [16:32] mterry, does your finger work? [16:33] Saviq, yes [16:33] mterry, ok, so unity8 is stopped, right? [16:33] Saviq, I can get past the first problem, but next time it tries to use touch, it fails (well, nothing happens, then it times out) [16:33] mterry, so one test completes? [16:33] Saviq, after autopilot runs, it pops up again. I had manually stopped it like you said before [16:34] mterry, don't use -n on phablet-test-run, it won't pop up then [16:34] Saviq, no. The greeter swipes away, then the test fails the next time it tries to use touch [16:34] mterry, ah so the greeter does swipe away [16:34] mterry, so input in itself is there [16:34] If I do it manually [16:34] mterry, how do the indicator client tests behave? [16:35] Saviq, let me check, I've been focusing on one of the greeter tests [16:35] * greyback eow [16:35] mterry, real weird [16:35] bye all! [16:35] greyback, o/ [16:37] mterry, make sure ubuntuappmanager is restarted and that unity8 doesn't start [16:37] mterry, as uam sometimes gets "dirty" when the "real" unity8 starts [16:38] mterry, weird that it would only affect nexus4 :/\ [16:44] anyone with ideas? [16:44] why a signal is not passed to who is using the component? [16:45] Saviq, same behavior with indicator tests. I can manually go through the test, but as soon as it tries to use self.pointing_device, nothing happens and the next assert fails [16:47] mterry, can you go android-shell [16:47] mterry, and see with getevent [16:47] mterry, if the events even get through evdev? [16:47] mterry, https://source.android.com/devices/tech/input/getevent.html [16:47] Saviq, ok [16:49] Saviq, nothing came through getevent [16:49] mterry, and your finger does? [16:49] Saviq, yup [16:49] mterry, ok, autopilot fail [16:50] mterry, can you please file a bug [16:50] Saviq, sure. Assigned to... thomi ? [16:50] mterry, yeah [16:50] it's saturday for them already, so we won't get better until Monday, but at least he might get to it before our Monday === alan_g is now known as alan_g|EOW [17:11] larsu: ping [17:15] tedg: ping [18:05] dednick, pong [18:05] tedg: hey. was just wondering if you knew what happened to larsu today. hasnt been responding. is he away? [18:05] dednick, He's apparently on a train today. [18:06] tedg: either that or he's tired of my bitching ;) [18:06] tedg: ah right [18:06] Not sure what that means as far as when/if he'll be available. [18:06] tedg: well our days are over now, so i guess monday. no matter [18:06] dednick, Yeah, probably. [18:07] tedg: and thanks for the position update [18:07] dednick, No problem, did it work for you? [18:08] tedg: yup. all good. actually, is there a doc somewhere with the indicator position layout described? [18:08] dednick, https://wiki.ubuntu.com/StatusBar [18:09] rotation lock indicator? pttf. [18:11] have a good weekend all [18:31] tedg: quick lists? [18:31] Wellark, On the launcher [18:32] == jump lists == launcher menus [18:32] oh, ok [18:32] (sorry, really laggy connection..) [18:32] NP, yes those. [18:33] tedg: there is no service or anything as dynamic (runtime) entries are not supported before 14.04 [18:34] tedg: but I would say it would be part of the "mega" memoryspace once it's there [18:34] it's just a simple dbus interface [18:38] Wellark, Okay, but it'll be published by the Unity8 connection. [18:38] Wellark, Security is wanting a list of interfaces [18:38] Wellark, So I'll add that to it. [18:39] Would be nice if we could split it out so that Unity 7 and 8 were using the same backend. [18:41] tedg: can you send me an email on what info you need from me? [18:41] I have to EOD [18:41] or actually have already [18:41] Wellark, I don't need any other info. That's it. Have a good weekend! [18:50] tedg: oh. one thing the security team should be aware off [18:50] tedg: the unity actions [18:50] now every application exposes a set of actions to dbus [18:51] I would guess the default policy is to block any calls to the applications [18:51] but the shell has to be able to invoke them [18:51] Wellark, Yeah, they're more worried about services right now. But that is good to know. [18:51] and later we need to implement some sort of app to app IPC [18:52] Uhm, perhaps. Not sure if we want to do that. [18:52] and at that point we need to extend security to those applications also [18:52] I think we want to always go through helpers. [18:53] oh, sure. that's one way of doing it [18:53] I just don't want random apps to have access to the unity actions of other apps in the first spin [18:53] but that's probably the default [18:57] tedg: but anyway, have a great weekend! :) [18:58] Have fun! === salem_ is now known as _salem