[00:00] could anyone take a look over? https://code.launchpad.net/~kdub/platform-api/mir-deb-bump-0.1.3/+merge/200430 [00:00] should be pretty simple, just bumping ABI === duflu_ is now known as duflu [03:05] So I am thinking of contributing a feature to unity, patch and everything, about an idea that I just thought of. I'd like to pass the idea around here first [03:05] Its actually pretty mundane and simple, and has to do with moving windows with the mouse [03:08] When an application is full screen, and you go to the top screen bar, on the white space where no menu actions are currently situated, and then HoldLeftClick, you can move the window around [03:09] I basically want to extend this function to when windows are not full screen [03:10] One of the reasons why this is, is that I often find while using spaces, that my top menu bar is stuck on another virtual desktop [03:11] and I thus have to switch workspaces, or press alt+click the window to drag down. [03:26] Hm. I was wondering where that was going; that's an actual use-case :) [03:26] On the other hand, might this not be better solved by ensuring that the top bar is always visible? [05:51] RAOF: http://discourse.ubuntu.com/t/unity-five-suggestions-for-global-menu-bar-whitespace/1392 << sorry, library closed, and now im at the coffee shop :P. Anyways, I extended my suggestion to five [05:52] RAOF: The top bar being always visible... how would the logic work with vertical spaces? [05:52] Akiva-Mobile: You'd just move the window down when you switched workspaces. [05:53] RAOF: wouldnt that be annoying if you were trying to escape that application, and it just happened to slightly be leaking into the next workspace? [05:54] RAOF: Gnome 3 doesnt have this issue though, does it...? [05:55] Akiva-Mobile: Yeah, that would be awkward I guess. GNOME 3 gets around this by not having workspaces in a coherent space. [05:55] RAOF: If Gnome3 had HUD, I'd definitely consider using it. [06:02] RAOF: Sorry again :P [06:03] RAOF: I think my idea is reasonable, and it would be neat trying to get a patch accepted by the ubuntu project. Maybe then someone will hire me as a programmer once I have that on my mantle :) [06:03] :) [06:03] I'd give it a go. [06:04] RAOF: Any suggestion what file to start looking into? Save myself some time~ [06:04] Sorry, no. [06:05] RAOF: I'll expose my ignorance here, but will I have to do this on mir? [06:05] or will the code be transferable? [06:05] No; the Unity 8 codebase is entirely distinct. [06:05] There's not currently any window management in Unity 8 anyway :) [06:05] RAOF: For mir, how much needs to be rewritten? [06:06] We're rewriting everything; using Qt5 rather than compiz, etc. [09:07] Saviq: FYI test failing https://bugs.launchpad.net/unity-scopes-shell/+bug/1267026 [09:07] Ubuntu bug 1267026 in unity-scopes-shell "1 test failing resulting in FTBFS" [Critical,New] [09:07] mhr3, ↑ [09:08] Mirv, the unity-api issue is tricky :/ in theory we've identified the commit where it got better, but that one commit wasn't enough [09:08] Mirv, it's working fine on the stable branch, after https://bugreports.qt-project.org/browse/QTBUG-35721 has been fixed [09:09] and the tabbar was merged, let's see all the complains :D [09:09] Saviq: is stable = upcoming 5.2.1? please don't say 5.3 :) [09:09] yes [09:09] ok, yes I'd be planning to get qtbase and qtdeclarative snapshots (or maybe just qtdeclarative for starters) [09:10] "Target for Qt 5.2.1 release is at the end of January \ early February." [09:10] http://lists.qt-project.org/pipermail/releasing/2013-December/001557.html [09:12] larsu: if you could check (or delegate) gsettings-qt bug #1259145 at some point, it would be nice. maybe some tests shouldn't be run under xvfb? SDK team might have encountered something similar earlier, also [09:12] bug 1259145 in gsettings-qt "gsettings-qt fails to build (run tests) against Qt 5.2" [Critical,New] https://launchpad.net/bugs/1259145 [09:12] Mirv: as said i fixed that [09:12] it's nothing about xvfb [09:12] it's just that the test was failing [09:12] Mirv: I already did, it's fixed in qt [09:13] tsdgeos: thanks again for that :) [09:13] Mirv: i.e. the x error we get we were getting before (afaics) [09:13] tsdgeos: larsu: oh right I might be confused, there was the another bug report that was filed during RC1. the latest gsettings-qt build from 2012-12-19 failed still: https://launchpadlibrarian.net/160186213/buildlog_ubuntu-trusty-amd64.gsettings-qt_1%3A0.0%2B13.10.20130902.1-0~42.1~ubuntu14.04.1_FAILEDTOBUILD.txt.gz [09:13] Mirv: a recent enough snapshot of qtdeclarative stable branch should fix it [09:14] tsdgeos: aha, aha, so that's also one of those that needs newer qtdeclarative? [09:14] yes [09:14] they totally from QQmlPropertyMap in 5.2.0 [09:14] it's far from salvageable [09:14] right, in that case that's the first priority now (well aside from my ir testing) [09:14] you need 5.2.1 [09:14] Mir [09:15] thanks. I'll skip qtbase updating for now until there's a known need for a snapshot. [09:16] Mirv, so - we're targeting 5.2.1 then? [09:17] Saviq: we're targeting anything that can get all FTBFS:s fixed and image results as good as with 5.0.x. preferably it would have been 5.2.0 already. [09:17] Mirv, i.e. if stuff works with 5.2.1 but not 5.2.0, are we sad? [09:17] Mirv, yeah, that isn't working apparently :/ [09:17] Saviq: I think it doesn't matter what we use, as long as we can use it without regressions and the 100 source packages all compile neatly against it and everything is full of flowers :) [09:18] :/ seen that bug already [09:18] it's some kind of race [09:18] I hope the qtdeclarative will resolve some of the build failures at https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2/+packages (gsettings-qt is one pre-requirement to move forward) [09:18] Mirv, so you're snapshotting qtdeclarative from the stable branch (5.2.1) now? awesome. [09:20] elopio_, crap, sorry for yanking the bottom bar out from under your feet... :/ [09:20] elopio_, didn't know you were working on that emulator [09:20] elopio_, hopefully this new emulator will be able to use one from the sdk? is there an emulator for tabs in sdk already? [09:20] Saviq: yes [09:21] elopio_, ah, already done! ;) [09:21] Saviq, mind if we start the preview meeting 15minutes later? [09:22] mhr3, sure [09:22] mhr3, I don't [09:22] :) moving [09:24] Saviq: hey, did you come to a conclusion regarding the qml in cmake? [09:25] mzanetti, I don't think I did ;) [09:25] mzanetti, /me looks at the branch [09:25] Saviq: want me to summarize the possible options again? [09:26] mzanetti, no, I think the only last beef I had was moving it into src/ or something [09:26] Mirv, you can try to just rebuild, it might just work [09:27] ah ok. yeah, gerry had the same opinion. I still think qml sources are sources too but I don't really mind as long as I don't have to open 2 project files all the time [09:28] mhr3: well we don't allow flaky tests anyway, so it wouldn't change much even if it would work. but it seems to have failed every time on armhf: https://launchpad.net/~ubuntu-unity/+archive/daily-build/+builds?build_text=unity-scopes-shell&build_state=all [09:30] mzanetti, I think file(GLOB_RECURSE ../qml/*.qml ../qml/*.js) will do what you need [09:30] Mirv, seeing when it started happening actually helps, thx [09:31] Saviq: ok. I'll change it [09:31] mzanetti, so leave qml/ be, and just do the GLOB again in src/ [09:31] Saviq: yep [09:33] mzanetti, and drop .qmlproject then, I think? [09:33] yep [09:34] mzanetti, http://paste.ubuntu.com/6713876/ [09:34] mzanetti, that works for me [09:34] mzanetti, might want to add .png / .svg etc. [09:35] mzanetti, stuff that qmlproject itself does :/ [09:35] mhr3: ok, great, indeed there is a certain point [09:36] Saviq: on a first thought I'd say that's not needed, given that we can't really do much with them in the IDE anyways. But I'm not opposed to it [09:37] mzanetti, you can view them at least [09:37] mzanetti, and see a list of them in the project explorer [09:37] mzanetti, which is useful [09:37] ok [09:38] mzanetti, .png, .svg, .sci [09:38] mzanetti, and see if we have any other [09:38] ack [09:47] mzanetti, http://paste.ubuntu.com/6713935/ is probably enough... after all we want all files from under qml/ to show up [09:48] Saviq: true... any idea how to make the importPaths statement work? [09:48] I tried this, but doesn't seem to work: add_definitions(-DQML_IMPORT_PATH=builddir/plugins:builddir/modules:builddir/tests/plugins:builddir/tests/utils/modules) [09:49] mzanetti, do we actually need that? [09:49] its not totally essential. but for some plugins we'd get code completion [09:49] mzanetti, that's *if* QtCreator would actually honor that [09:49] :D [09:50] mzanetti, which I doubt it would [09:50] for qmake based projects apparently setting QML_IMPORT_PATH should work [09:52] Saviq: would you have some time to test qtdeclarative stable head compilation? at least when combined with our packaging + patches, I'm getting http://pastebin.ubuntu.com/6713954/ (git hash used is there in the directory) [09:53] Mirv, sure, sec [09:54] this is with normal qt 5.2.0 otherwise (qtbase + qtxmlpatterns) [09:55] Mirv, I imagine this says "you need newer qtbase" [09:57] Saviq: I wish even stable branch would be safe from the interdependencies :) [09:58] but sure, I wouldn't be surprised. the referrals failing are to qtdeclarative's own sources, but maybe something in qtbase changed about finding sources. [10:13] Saviq: https://code.launchpad.net/~mzanetti/unity8/qml-in-cmake-take2/+merge/200793 [10:15] thanks :) [10:16] mzanetti, still not using colocated branches, btw? ;) [10:16] nope [10:17] but my disk space would appreciate it I guess [10:18] Mirv, actually [10:18] Mirv, qtdeclarative compiled fine here [10:18] Mirv, clean chroot with qt5-beta2 [10:20] Saviq: where did you get the plugin from? [10:20] mzanetti, lp:~fboucault/bzr-colo/colo-push/ for some reason [10:21] mzanetti, but lp:bzr-colo is probably a better source [10:22] tsdgeos, about the horizontalJournal Q_ASSERt [10:22] tsdgeos, I thought that the signal would only be emitted if the operation was async [10:22] nope [10:22] therefore that slot would only be called in the async case [10:22] Saviq, you in a call already? [10:22] mhr3, no [10:22] dandrader: nope [10:23] dandrader: which Q_ASSERT in horizontalJournal ? [10:23] ignore me [10:23] can't read [10:23] ok [10:23] :D [10:28] is s-jenkins down? [10:29] dednick: works for me [10:29] mh. [10:30] tsdgeos, that one in itemCreated slot [10:30] yes yes [10:31] stupid dns thing again... sigh [10:31] dandrader: was lost in the way you built the sentence and how my brain read it [10:49] * mzanetti wishes we could wake up the screen by double tapping it [11:11] Saviq: hrm! === daek_ is now known as Daekdroom [11:39] mzanetti: any idea? https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/832/console [11:39] dednick: I've seen the same in one of my branches. So far I hoped it's temporary woes... seems not :/ [11:40] Saviq: ^ ? [11:41] hmmmpf [11:41] isn't that debian saying "you really need to bump the version"? [11:41] mzanetti, why does qmluitests build the package at all? [11:41] mhr3, yeah it kind of is [11:42] mhr3, but why would it suddenly start now? [11:42] TBH I never understood when dpkg starts complaining about that [11:42] error: ... binary file contents changed [11:43] Saviq: we've always built the package for qmltests [11:43] mhr3, ah, so you're saying it doesn't care until you change a binary file? === _salem is now known as salem_ [11:43] Saviq, i'm no debian pro, but seems that way [11:44] interesting... === dandrader is now known as dandrader|afk [11:46] wtf. my character map just changed, but it's still showing correct layout in region settings [11:46] mzanetti, so what we're missing probably is bumping the version before building the package [11:46] mzanetti, but why do we build the package at all? [11:46] Saviq: that was the most straight forward way to get it done in jenkins [11:46] funny thing is, i only have uk english installed... weird [11:47] mzanetti, but that doesn't build the mocks etc. does it? [11:47] Saviq: sure it does. it's installed into the unity8-fake-env package [11:49] mzanetti, I'm not sure that's true - not for *all* the mocks at least [11:49] mzanetti, anyway [11:49] hmm, ok. I might miss something [11:49] mzanetti, do you have access to the job? [11:49] yes, I do [11:50] mzanetti, the pbuilderjenkins command [11:50] mzanetti, doesn't use the hooks as defined in the job [11:51] mzanetti, if that's on purpose, add H05set_package_version there and we should be good [11:51] Saviq: added [11:52] mzanetti, dednick restarted the job, will be at http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-trusty/834 [11:52] dednick: I restarted your job. lets see what happens [11:52] oh [11:52] ta === dandrader|afk is now known as dandrader [12:00] looks better, yes [12:03] Saviq: I've changed the job to inherit hooks from upstream jobs and just add the B09qmltests etc [12:03] mzanetti, yup [12:03] lets hope I didn't break it. You'll know where to find me in that case [12:04] mzanetti, I DO KNOW WHERE TO FIND YOU! [12:04] :D [12:04] * mzanetti hides [12:11] mzanetti, http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-trusty/834/console [12:11] Started by user Saviq [12:11] Started by user mzanetti [12:11] interesting ;D [12:12] must be we triggered the exact same configuration and jenkins combined that into one [12:13] heh [12:13] I'd rather bet on a bug in jenkins :D [12:22] /food [12:43] tsdgeos, hJournal ready for merging or are you searching for the assert? [12:43] Saviq: ready for merging [12:43] i forgot to check the failure and reapprove [12:44] tsdgeos, k [12:44] test_show_hud_button_appears ? [12:44] weird [12:44] let's see next run [12:45] tsdgeos, yeah, it does fail sometimes === alan_g is now known as alan_g|afk === alan_g|afk is now known as alan_g|lunch [13:22] Saviq: one of these days I learn to attribute all mysterious immediate failures to syncqt when snapshot builds don't work [13:24] deleting the hud-service binary wasn't such a good idea :/ [13:25] Mirv, ;) [13:25] mzanetti, btw, we've missed ../tests/*qml ../tests/*js :/ [13:26] Saviq: true. I'll fix === alan_g|lunch is now known as alan_g [14:02] kgunn: you increased the dependency of unity-mir to mir* 0.1.3 but that's still not out, no? [14:07] greyback: ok, https://code.launchpad.net/~aacid/unity-mir/application_manager_tests/+merge/180898 should be ready [14:07] CI is failing to compile because the ↑↑↑ change [14:07] tsdgeos: \o/ thanks!! [14:08] if you want to try now in the phone [14:08] just do [14:08] - libmirserver-dev (>= 0.1.3), [14:08] - libmirclient-dev (>= 0.1.3), [14:08] + libmirserver-dev (>= 0.1.2), [14:08] + libmirclient-dev (>= 0.1.2), [14:08] on the control file [14:08] it worked for me [14:13] tsdgeos: ok [14:17] Saviq: I was trying to use the tab bar from ubuntu ui toolkit, but it causes a couple of problems. The biggest is that when trying to open the files & folders scope, it tries to click on the right border of the screen and nothing happens there. [14:17] I need to make a workaround on the toolkit emulator before it's usable by unity. === elopio_ is now known as elopio [14:19] tsdgeos: yes...release team is testing new mir, so they had to approve and push the bump [14:20] it should be over soon (and hopefully mir out soon) [14:20] great [14:20] ;) [14:21] elopio, ack [14:21] Saviq: but I need help from my branch, on mako there are failures that don't seem related to my branch. I was hoping to find aacid here today. [14:22] elopio, tsdgeos === aacid [14:22] he's crypto, we know [14:22] ;) [14:22] * tsdgeos hides behind his nicks [14:22] elopio: need help? [14:22] Saviq: remember we talked about the "Please enter SIM PIN" text a while ago? [14:23] tsdgeos, not really, wassup? [14:23] Saviq: basically you said "it should come from the indicator" [14:23] but it's still there [14:23] tsdgeos: \o/ I found you! [14:23] tsdgeos, right [14:23] let me get something to drink and wake up a little, and I'll start bothering you :) [14:23] tsdgeos, file a bug please? [14:23] Saviq: shall i open a bug or something so someone maybe acts on it [14:24] tsdgeos, ↑ that [14:24] Saviq: against what? unity8? or? [14:24] tsdgeos, affects unity8 and indicator-network [14:24] okidoki [14:26] man i ate too much of my fake chocolate cava bottle :D :( :-S [14:29] * mzanetti googles chocolate cava bottle [14:30] http://www.badamelos.com/1177-large_default/botellas-cava-benjamin-chocolate.jpg [14:30] mzanetti: ↑↑ [14:30] lol [14:30] leftover from new years eve? [14:31] yep [14:31] greyback, mterry_, standup [14:37] mzanetti: thanks :) When I reboot, or suspend/resume, PA thinks sound is at the usual place, but is actually 0. If I wiggle the mic volume, it then works [14:38] greyback: yeah... try a "asoundctl store" with non-muted settings === dandrader is now known as dandrader|lunch [14:42] Saviq: https://bugs.launchpad.net/indicator-network/+bug/1267135 [14:42] Ubuntu bug 1267135 in Unity 8 "network-indicator should provide the "Please enter SIM PIN" text" [Undecided,New] [14:44] I'm back. tsdgeos, please let me know when you have some time, it seems I'll need more help than what I thought. [14:45] elopio: now is a good moment [14:47] tsdgeos: the dashContentList has a currentIndex property. My problem is that the QQuickLoader and the GenericScopeViews don't have an index. [14:47] so I have no way to tell which is the currently selected scope. [14:47] I was assuming that the order of the list would give me the right index, but on mako where the music scope is to the left, my assumption breaks. [14:48] elopio: the current scope is the one of the currentIndex? [14:49] dashContent.model list the order of them [14:50] which is actually the list of visible scopes [14:50] as returned by Scopes [14:50] let me see if autopilot can access that. [14:50] elopio: what do you want to find out actually? name? or something else? [14:51] tsdgeos: well, what I need is a way to know what's the index of a scope. On the toolkit tabs I had the same problem, and they fixed it adding the a new tabIndex property to every tab. [14:52] elopio: as said the index of a scope is it's index in the list of visible scopes [14:52] i.e. whatever is sorting scopes it's not the ui [14:55] tsdgeos: right. The problem seems to be that the list of scopes returned by autopilot is not on the same order. [14:55] (Pdb) p self.dash.dash_content_list.currentIndex [14:55] 1 [14:55] (Pdb) p self.scope_loaders[1].scopeId [14:55] u'music.scope' [14:56] there's the problem, because the selected scope at this moment is 'home.scope' [14:56] so we shouldn't rely on the order of elements that autopilot returns. [14:57] what's scope_loaders ? [14:57] i.e. who provides that info? [14:58] and is self.scope_loaders[0] marked as visible or not? [14:58] tsdgeos: self.scope_loaders = self.dash.dash_content_list.select_many('QQuickLoader') [14:59] ok [14:59] tsdgeos: and all of them always have visible=True, we can't rely on the visible property. [14:59] so select_many is not giving you stuff in the corect order [14:59] (Pdb) p self.scope_loaders[1].visible [14:59] True [14:59] (Pdb) p self.scope_loaders[0].visible [14:59] True [14:59] which given the name seems even reasonable [15:02] yes. But according to thomi, autopilot just returns the elements on the same order of the QML tree it sees. [15:02] as we have seen before, and now again, it seems a bad idea to rely on the order of the elements of the tree. [15:02] so again, what do you need? [15:02] know the current scope? [15:03] maybe match it against the name of the selected tab? [15:03] or that is what you're trying to test? [15:04] tsdgeos: not just know the current scope, I could use the isCurrent property for that. [15:04] I need to know the order of them, so I scroll to the left if the index I need is lower than the currently selected one, and scroll to the right if the index is higher. [15:07] well then you can use the tabs? [15:07] i think the tabs has getters [15:07] let me see [15:07] maybe not [15:07] tsdgeos: I found the bug and branch for the same problem on the toolkit: [15:07] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1233402 [15:07] Ubuntu bug 1233402 in Ubuntu UI Toolkit "The Tab needs a tabIndex property" [Critical,Fix released] [15:07] booo [15:07] ok [15:07] :D [15:07] tsdgeos: I could use the tabs instead of scrolling, but first I need to implement some changes on its emulator. [15:08] elopio: so you want to do the change by swiping i guess, it's not ok that we add a switchToScope function [15:09] tsdgeos: sorry, I didn't get that last part. [15:10] elopio: Dash.qml has a "function setCurrentScope" [15:10] can you use that? [15:10] tsdgeos: no, with autopilot it all should be black box. [15:10] or you want to trigger the change "emulating the user touch presses"? [15:10] ok [15:10] I need to set the current scope only using the pointer. [15:11] does autopilot know how to query models? [15:11] if so i think that querying filteredScopes would be useful to get the order of things [15:12] if not, someone should make autopilot able to introspect the models :D [15:14] tsdgeos: I think that's not possible. That's information that QML doesn't expose through the testability feature, that's what autopilot uses. [15:14] tsdgeos: but if you look at zsombi's branch: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/tab-index/+merge/188769 [15:14] it doesn't seem like a big change on the model [15:14] I don't know enough QML to do it myself on this case, though. [15:16] greyback: could we sort the appmanager model in a way that the focused one is at index 0? [15:16] elopio: i'm understanding you can't access the attached properties of the listview deleagtes, no? [15:16] elopio: i.e. i can use "index" inside a QML listview item [15:16] and it'll give me the index [15:18] tsdgeos: I can access things like isCurrent and scopeId, but no, index doesn't seem to be available. [15:18] tsdgeos: have you used autopilot vis? [15:18] mzanetti: that should be the case already [15:18] i used it once when all the stars aligned [15:18] it shows the QML tree as autopilot sees it. So those are the only things we can use. [15:18] greyback: seems not... [15:18] errrrr, is everybody getting the tabs to show in unity8? [15:18] greyback: the first app I launch is always at 0 [15:18] mzanetti: but I don't see the code to do it either [15:18] they don't show here :-S [15:18] * tsdgeos does a clean build [15:19] mzanetti: suspect unity8 doing it [15:19] tsdgeos: I can see them. They work nicely here. [15:19] gnaaa [15:19] greyback: ok... I'll try to change it in my screenshotting-focusing branch [15:20] mzanetti: no objection here [15:21] ok, tabs are here now, maybe needed a clean build [15:21] elopio: let me try to run the visualizer [15:22] thanks tsdgeos. Take a look at the TabBar inside the PageHeader. It has a selectedIndex property. And then the AbstractButtons children of that TabBar have a buttonIndex property. [15:23] that's what I need. With those properties, I don't need to rely on the order of the children as autopilot sees them. [15:25] sure [15:25] adding the property is trivial [15:25] it's just that i'd prefer not to add extra code just for the testing [15:25] elopio: maybe you can use the "x" property of the loaders to sort them? [15:30] tsdgeos: I could use that. Let me try to see if it gets ugly. === alan_g is now known as alan_g|tea === rachelliu_ is now known as rachelliu === alan_g|tea is now known as alan_g [15:49] tsdgeos: that will work nicely :) thank you. [15:49] when is your EOD? [15:50] cool \o/ [15:50] in 1:10h aprox [15:50] 6pm spain time [15:58] tsdgeos: I have a meeting now, so I probably won't be able to finish my branch by then. Can you please take a look tomorrow? [15:59] https://code.launchpad.net/~elopio/unity8/open_scope/+merge/200426 [16:00] sure [16:05] Saviq: so i was thinking about the organic grid for the Dash vs the one in the gallery. Ours is always visible in full width and is designed to scrolls vertically while the one in the gallery is always full height and designed to scroll horizontally. That's not a radical problem, but introduces a few else/ifs here and there, want me to code directly for both or just catter our needs at first stage and improve later? === dandrader|lunch is now known as dandrader === boiko_ is now known as boiko [17:06] mterry_: ping [17:06] kgunn, hello [17:06] mterry_: hey hope you're feeling better.... [17:06] got a question for you that i should already know :) [17:06] kgunn, OK [17:06] so i was cleaning up some wiki stuff....for unity8 [17:07] and greeter section referenced [17:08] https://launchpad.net/unity-greeter [17:08] but isn't the greeter in unity8 now ? [17:08] mterry_: ^ [17:08] kgunn, yes. The phablet greeter is in unity8 [17:08] kgunn, unity-greeter is the old greeter [17:08] "old" [17:09] mterry_: ok...that's what i tho [17:09] t [17:09] mterry_: so even after you land the split...it'll still be the phablet greeter in use right ? [17:10] kgunn, yes [17:11] mterry_: thanks...i'll make it match [17:11] kgunn, same source package (unity8) but new binary package (unity8-greeter) [17:11] ;) [17:11] mterry_: curious...why is it delivered in a bin like that ? [17:11] at least the unity-greeter [17:11] kgunn, I don't follow [17:12] oh nvmd...i see... [17:12] mterry_: ignore me [17:18] Saviq: has there been any progress on building Unity 8 on Saucy? === jhodapp is now known as jhodapp|bbiab === jasoncwarner__ is now known as jasoncwarner [17:38] mhall119, there's more data on the failures, so hopefully it'll get better soon [18:06] fginther: hey, can I book some CI expert time for unity-mir. We've a branch that adds tests, which must run in an environment where Mir can execute. We need a hand to set up CI to deal with that === alan_g is now known as alan_g|EOD === jhodapp|bbiab is now known as jhodapp [19:12] greyback, I can help but I'll need to get back to you later, I have some fires to put out [19:13] fginther: sure, it's far from urgent [20:54] Saviq: If there are any conventions on the team I didn't follow, just lemme know :) https://code.launchpad.net/~unity-team/unity8/new-scopes-horizontal-card-layout/+merge/200910 === karni_ is now known as karni [21:03] karni, cheers [21:04] karni, jest już test "test_art_size_data" [21:04] ooh english ;) [21:05] karni, there's already test_art_size_data [21:05] * karni paczy ;D [21:05] karni, would make sense to build test_art_image_size into it [21:06] karni, "card-size": "medium" is unsupported (and ignored, for that matter) in horizontal layout - not sure we should test that... or maybe we should, but add a "small" one, too [21:07] Saviq: oh okay. I needed any size, I can replace that with small, as all sizes are covered in previous tests, right? [21:07] karni, why did you need a size? === salem_ is now known as _salem [21:08] karni, what I mean is: in horizontal layout, the card-size setting is ignored - so we should either test that setting it doesn't matter, or not set it at all [21:08] Saviq: I looked at the code, seems I could have gone with the default [21:09] Saviq: I'll remove that, yes [21:09] karni, for test_card_horizontal_layout, you're only checking for width, so integrating into test_card_size should work [21:09] Saviq: I think I wanted it to test the width/height of the artImage, but we decided that need not to be tested [21:10] Saviq: agreed on first and last comment, too. fixing. [21:11] karni, I'd rename test_*_anchors into test_*_layout, as we're not actually checking the anchors [21:11] Saviq: agree :) [21:12] karni, and now... .left/right/top/bottom are anchor lines [21:12] karni, not pixel values [21:12] karni, so test_header_anchors should actually fail [21:12] * karni looks [21:13] Saviq: the last test? I'm not actually using anchors, the test name was confusing. [21:13] karni, that's the thing - you are ;) [21:14] let me do the art test refactor, and I'll push, so we're on the same page :) [21:14] karni, header.top, header.left are anchor lines (not anchors, but something that can be attach to anchors) [21:14] or something that anchors can attach to [21:14] * karni nods [21:15] karni, so in the last test you're comparing header.top to art.bottom, for example [21:15] may have gone blindfolded a bit [21:16] + top: template && template["card-layout"] === "horizontal" ? artShape.top : artShape.bottom [21:16] karni, that's header.anchors.top, not header.top [21:16] which are two distinct anchor lines - what I mean is that if you revert the changes to Card.qml, the test will still pass (assuming it passes now) [21:16] ooooh right! [21:16] Saviq: lemme fix that :) thanks for the super fast review :D [21:16] karni, so yeah, you can't compare anchor lines of different objects [21:17] karni, so you should look at x/y/width/height instead [21:17] adding where necessary [21:17] yes, all tests pass, but I obviously failed there with that anchor bit [21:18] karni, shelve changes to qml/ and run your tests suite again - make sure that all tests that should fail - do [21:19] Saviq: regarding your first/second comment about integrating test_art_image_size into test_art_size, I'd have to name properties like "imageWidth" and "imageHeight" because most of test_art_size concerns art, while I'm testing artImage [21:19] So.. does it in fact make sense to pull this test into test_art_size? [21:20] test_art_size tests width/height of testCase.art, while test_art_image_size tests width/height of testCase.artImage [21:20] (FWIW, bit confusing terminology ;D) [21:20] good suggestion. it did feel weird to write tests after the code ;D [21:22] karni, I know, sorry for that ;) [21:22] karni, was easier to write the code than tests ;D [21:22] Saviq: no worries, I know how fun it is to improve/fix things :D [21:22] haha, happy I can help (and learn!) [21:23] karni, you shouldn't be testing artImage's size at all, as that component isn't even visible - the UbuntuShape around it is what we care for [21:24] Saviq: may have gone overboard trying to cover every line of your code ;D [21:25] karni, so yeah, there's only one additional entry to test_card_size_data that should be added [21:25] Saviq: yes, fixed that [21:25] karni, yeah, coverage for QML (or UI, for that matter) is unfortunately an arbitrary thing [21:25] haven't pushed yet, want to fix those anchors/pixels thing you noticed [21:25] karni, cheers [21:25] :) [21:25] Thank you! [21:25] karni, aren't you past EOD yourself, btw? [21:26] Saviq: You working from Poland? [21:26] karni, we're not talking about me here, are we now :P [21:26] Saviq: I wonder, why would you ask hahahahah [21:26] * karni laughs [21:26] * Saviq → food [21:27] karni, just leave it here if you have more, will reply when around the keyboard later [21:27] Saviq: thanks! :) [21:52] Saviq: updated the MP. Left one 'XXX', can't get that line in test_header_layout to pass. I'll get something to eat and will try the shelves that you suggested to make sure stuff would fail where it should. [22:04] karni, a) test_art_layout is bad, tryCompare takes three arguments: object, propertyName, timeout [22:04] so tryCompare(testCase.header.x, data.headerLeft, data.value); is bad [22:05] not to mention data.headerLeft and data.value are undefined [22:06] karni, as for the other thing... the problem is that the card is laid out only when the test is executed, the _data() function is called before that, on the default layout, so art.width is 148 then, but when you select index 5, it becomes 80 [22:07] but data.left is already evaluated with 148, so the test fails [22:22] karni, http://paste.ubuntu.com/6717510/ fixes [22:23] karni, actually make that http://paste.ubuntu.com/6717513/ [22:59] Saviq: wow.. I'm surprized I wasn't thrown at that .headerLeft was not defined (I did change the name, missed that one). and tests still passed. not reassuring :( [23:33] Saviq: fixed along your diff, appreciated https://code.launchpad.net/~unity-team/unity8/new-scopes-horizontal-card-layout/+merge/200910