=== _salem is now known as salem_ === salem_ is now known as _salem === Cimi_ is now known as Cimi === RAOF_ is now known as RAOF [07:15] Saviq: did you see my comment on https://bugs.launchpad.net/unity8/+bug/1316660 ? [07:15] Launchpad bug 1316660 in Unity 8 "Scope preview should use SDK's Expandable*s" [High,In progress] [07:16] tsdgeos, well, kind-a, that was just one example. I'm not saying your branch fixes all of the cases, but it does make Previews use Expandables ;) [07:17] tsdgeos, actually putting the buttons into an Expandable will be easy now that there is the rest in place [07:17] oh wait [07:17] you didn't put it in an Expandable [07:17] then yeah, you're right [07:17] (you should've) :P [07:18] well why? [07:18] it wasn't the design [07:18] design is "combobox + button" [07:18] tsdgeos, well, yeah, but expansion is a common pattern, we have the expandable components in the SDK for a reason [07:18] the fack that design decided to not use a well known word just because people 2 generations ago in design hated the word combbox and now is colliding with other stuff [07:18] is not my problem [07:19] tsdgeos, I think that's two different issues [07:19] yes we have a combobox in the sdk now [07:19] not when we wrote that code [07:19] tsdgeos, that's not about that [07:19] tsdgeos, even the buttons we have now [07:19] tsdgeos, should have been put in SDK's Expandable, and the whole Preview needs to be UbuntuListView or UbuntuColumn [07:20] tsdgeos, so that when you expand something, it gets into view without you caring about it [07:20] ok, still nothing to do with my branch :D [07:21] tsdgeos, I just thought you did that in there [07:21] nope [07:21] i just added the expandable preview widget [07:21] tsdgeos, on that note we still need to collapse the visible widgets in the expandable widget to their collapsedHeight, but it's fine in a separate branch [07:21] Wait, are you calling me two generations old [07:22] * mpt shakes his walking stick at tsdgeos [07:22] ;) [07:22] he's lurking! [07:23] Saviq: that branch does that [07:23] tsdgeos, oh hmm, I can't read then [07:23] good [07:24] tsdgeos, ah now I can see [07:26] Cimi, it's time for that review when you're up ;) [07:30] Saviq, morning :) [07:30] Saviq, it's also time for my review of emblems :) [07:35] Saviq, but why we have the surrounding box aroung the orange? [07:35] would it look bad withou? [07:36] Cimi, that was what design requested, yes [07:36] ok [07:36] Cimi, otherwise it'd clash with content behind it [07:36] for the same reason we are not using the ubuntushape in every single corner of the os [07:37] not to clash [07:37] Cimi, and this doesn't use the shape, not sure what reference is that to ;) [07:37] hah [07:37] mhr3, morning! did you guys ever get all the new square assets for dash overview? [07:38] Saviq, no [07:38] and morning [07:38] Saviq, at the office today, so will ask esti [07:38] mhr3, cool, thanks [07:38] tell her we'll land this with old assets anyway, but she won't like it ;) [07:41] tsdgeos, on dash overview, the shader effect itself does still result in some lag when you open the dash, think it'd be possible (later) to just scale the real thing down? [07:41] Saviq: which shader effect? [07:41] ah [07:41] the copy [07:41] tsdgeos, the one that copies the dash to the list [07:41] it might [07:41] it is, after all, almost a fullscreen snapshot [07:42] tbh i didn't see any lag compared to what we had [07:42] I'll try it out again but we did notice some still in the office [07:42] but it was horrible [07:42] so anything would be much better [07:42] sure [07:43] Saviq: sure it'd be possible to scale itself probably, but not sure if it'd be cheaper [07:43] tsdgeos, well, should be, since it'd be scaling the texture itself, not a copy of it [07:44] Saviq: but it may end up in redraws blah blah [07:44] tsdgeos, trueth [07:44] tsdgeos, but shouldn't :) [07:45] Saviq, approved, you can go in dash overview and need fix it [07:46] otherwise I'll do :) [07:47] we need to export the activity indicator and use it there too [07:47] trueth [07:50] what's up with cI? [07:50] i did a commit 12 hours ago and still no run? https://code.launchpad.net/~aacid/unity8/prepend_test_xml/+merge/229766 [07:51] tsdgeos, it's approved [07:52] tsdgeos, ci doesn't run on already top-approved branches [07:52] (automagically) [07:52] ahhh [07:52] didn't know that [07:53] i'll trigger a rebuild then [07:59] mzanetti, welcome back ;) [08:38] o/ [08:38] hello [08:38] Saviq: thanks [08:39] Saviq: are we on rails again? [08:50] Saviq: um, any idea why indicator-network depends on unity8? seems like kind of a weird dep. [08:51] uh! dash-as-app landed. nice [08:52] mzanetti: pain in the arse on desktop though! :) [08:52] dednick: how? [08:52] dednick: you mean run.sh? [08:52] mzanetti: yeah. 2 windows [08:53] dednick: right... well, I only use make tryShell nowadays [08:53] would that be an option for you? [08:53] dash is now a screenshot in shell, with separate window for scopes :) [08:53] dednick: yeah, I know... I did that [08:53] mzanetti: hm. could be [08:53] dednick: maybe just LD_LIBRARY_PATH some indicators stuff [08:53] oh i'm just complaining... don't worry about it [08:53] then make tryShell should be fine for you [08:54] I guess we should get rid of run.sh... [08:54] or not use upstart for it... [08:54] yeah [08:55] dednick: what I do too is to launch the unity8 binary with qtcrator, without run.sh [08:55] the upstart bit is my major gripe with it [08:55] dednick: just add "-mousetouch" to the cmdline args [08:55] mzanetti: ah. was wondering about that. my mouse wasn't working with running binary [08:55] yep, -mousetouch [09:07] Saviq: ping === pete-woods1 is now known as pete-woods [09:23] tsdgeos, pong [09:23] Saviq: where does scopeStyle come from in https://code.launchpad.net/~saviq/unity8/dash-activity-indicator/+merge/229805 ? [09:23] color: scopeStyle.backgroundLuminance > 0.7 ? "#50000000" : "#50ffffff" [09:23] tsdgeos, yeah, by now... from nowhere [09:23] ok [09:24] i merged all silo1 to dash_overview [09:24] and was wondering if that was a merge issue or not [09:24] tsdgeos, yeah, not, that's broken atm [09:24] as I was moving the indicator out and out, I no longer have scopeStyle indeed... [09:25] mhr3, bcmwl broken == no 5GHz networks, eh? [09:25] Saviq, i wouldn't know, my chip doesn't do 5ghz at all [09:25] ugh [09:34] mhr3, fyi, downgraded to previous bcmwl version (tried previous kernel, too), didn't help :/ [09:35] Saviq, are you sure your driver supports 5ghz? :P [09:35] mhr3, yes :P [09:35] it *did* [09:35] tsdgeos, yeah we are [09:35] gooed :) [09:35] -e [09:37] old kernel? [09:37] I am on 3.13 [09:37] no bcmwl issues [09:37] Saviq, maybe it just pretended, and now they fixed it to stop pretending ;) [09:39] Cimi, do you get 5GHz networks? [09:40] Saviq, yes [09:40] Saviq, using 14.04 [09:40] Cimi, right [09:42] Saviq, reminded esti about the icons [09:42] Saviq, i'll push the square ones into the existing branches at least [09:42] Saviq, when do you plan to land it? [09:45] mhr3, asap [09:45] mhr3, doing a func/vis review right now [09:48] mzanetti, wanted to ping you yesterday to remove the darkening effect when only one app (dash then) is running [09:49] on the spread [09:53] Cimi: that's on my list to discuss with Vesa when he's back [09:53] Cimi: IMO we need to get rid of that effect at all [09:53] Cimi: or at least make it muuuch less [09:53] yeah [09:53] only helps in quick flip [09:54] but looks bad on spread [09:54] yeah... not even much there [09:55] tsdgeos, you didn't push the whole merge of dash overview did you? [09:55] no, i still have it here [09:55] running tests and stuff [09:55] merge was far from trivial [09:58] Saviq, do we need to modify the category template for the overviews? [09:58] Saviq: do y'ou want me to push it to a separate branch? [09:58] Saviq, or i guess you're doing magic to it already since you'll be cropping the icons? [09:59] Saviq, still, we need a new template for the scope search results [10:00] mhr3, yeah, we'll be overriding the aspect ratio there though [10:02] tsdgeos, I'd like to have an early look at it, yeah [10:02] tsdgeos, whether same or separate branch - you tell me [10:03] Saviq: lp:~aacid/unity8/do_merged [10:04] tsdgeos, thanks [10:05] qml tests seem to be passing now [10:05] will run autopilot after that [10:05] tsdgeos, just run ci on it [10:05] tsdgeos, you can put it as WiP MP [10:05] will fail [10:05] doesn't have scopes v4 [10:05] ah v4 :/ [10:07] tsdgeos, hmm it just goes blank for me as I drag from the bottom :| [10:07] tsdgeos, looks like the background is somewhere covering the overview? [10:08] wops yeah wrong merge [10:08] will check [10:09] wonder how the dash overview test passed [10:09] i guess because items are still there and can be clicked [10:09] yeah [10:10] tsdgeos, the background should just be part of DashContent.qml don't you think? [10:10] it is [10:10] oh ok [10:10] * Saviq stops [10:10] the wrong background is the one you made me add yesterday when i told you i didn't want to [10:10] Saviq: pull [10:11] i.e. just a wrong merge [10:12] k [10:16] mhr3, which MPs do I need to get for a silo with overview [10:16] Saviq, keeping in mind that you'll need to override the search category too? [10:16] mhr3, yeah, probably not straight away, but yes [10:17] Saviq, https://code.launchpad.net/~unity-team/unity-scope-mediascanner/overview-icons/+merge/228254 [10:17] mhr3, we assumed the category name would be static [10:17] Saviq, and https://code.launchpad.net/~unity-team/unity-scope-click/overview-icons/+merge/227731 [10:17] Saviq, it's "recommended" iirc [10:17] nope [10:17] "recommendations" [10:19] mhr3, and we'll need the scopes scope to change template, right? [10:24] tsdgeos, looks like you need to tweak data/unity8-dash.conf to drop scopes scope from favourites [10:24] ah wait [10:24] it probabaly is [10:24] Saviq: it's drooped [10:24] it's just that our run system is not so great [10:24] * Saviq needs to copy the .conf [10:24] and still uses sytemwide instead ours [10:24] need a way to fix that [10:24] it's borking autopilot for me now too [10:24] tsdgeos, yeah, I've a branch [10:25] tsdgeos, autopilot should be fine, assuming you copy the .conf file(s) to ~/.config/upstart/ [10:25] and make install of course [10:25] sure, i just haven't copied it ;) [10:26] tsdgeos, hmm, favorites always go back to the dash, don't change scope? [10:27] Saviq: you mean on click? [10:27] tsdgeos, yes [10:27] works here [10:27] desktop or phone? [10:27] tsdgeos, desktop [10:27] that is weird [10:27] do you have the latest qdeclarative? [10:28] yeah [10:29] what do you do exactly? [10:31] tsdgeos, let me record [10:31] tsdgeos, I start, drag up, click on music [10:31] and am taken back to apps [10:31] like I pressed DONE [10:31] it doesn't even zoom in from the right item [10:32] Saviq: can you try make tryDash ? [10:33] yep, nity8 seems to be failing [10:33] qml: No match for scope with id: musicaggregator [10:33] that is weird [10:33] hmmm [10:33] tsdgeos, works there, yeah [10:33] tsdgeos, btw, search doesn't look right in overview [10:33] mhr3: does your sort-order branch have the things for dash_overview too? [10:34] ah right maybe I'm missing something there [10:34] Saviq: what is missing in search? [10:34] tsdgeos, it doesn't really search... [10:34] tsdgeos, just displays All, really [10:35] i'm going to blame the plugin [10:35] now that is mhr3's last day [10:35] oh no! Is it? [10:35] not tomorrow? [10:36] tsdgeos, yeah, /me stupid, don't have the plugin here [10:37] tsdgeos, btw, we got silo 11 [10:38] awesome [10:38] autopilot is failing for nexus 10 size [10:39] for some reason on resize the current scope is changing [10:39] from clickscope to video [10:39] so all the tests fail to start [10:39] i'm confused, since doing the resize manually doesn't give me that behaviour [10:41] # XXX Currently we have no way to launch the application with a [10:41] # specific size, so we must resize it after it's launched. [10:41] # --elopio - 2014-06-25 [10:41] well we could have added that way.... [10:42] instead of doing something that now broke :D === MacSlow is now known as MacSlow|lunch [10:50] dednick: did it actually stay this way? [10:50] https://dl.dropboxusercontent.com/u/85539674/locked_sim/locked_sim8.png [10:51] or was that a short intermediate state ? [10:56] Saviq: hey, did you get anything regarding the lockscreen designs? [10:56] Wellark_, I saw an empty item like that today, yeah (no lock icon, but probably same issue then) [10:56] mzanetti, yeah, my fail === Pici` is now known as Pici [10:56] huh? [10:56] Saviq: so that handmade resize is broken for me too in master when using autopilot+nexus10, shall i add a -windowgeometry to unity8-dash like the one we have in unity8? [10:56] mzanetti, didn't get to that [10:56] tsdgeos, makes sense, yeah [10:56] tsdgeos, but not a high prio [10:56] Saviq: well... not your fail... [10:57] I asked the questions a thousand times already [10:57] mzanetti, yeah, my fail 'cause I didn't remember to follow up on that :| [10:57] Saviq: well the autopilot is failing :D but maybe works on CI [10:57] so all happy [10:57] let's wait for silo 1 to merge [10:57] tsdgeos, we're not running X11 tests on ci any more [10:57] Saviq: when is the eta for silo1? [10:58] tsdgeos, davmor2 has his hands on it, so unless he finds a blocker, soon! [10:58] Saviq: so... what shall we do? just implementing it somehow, ignoring some contradictory parts of the spec and making up missing parts ourselves? [10:58] Saviq: greatz [10:58] I'm not confident to get the answers any more [10:59] mzanetti, yeah, by now it's probably best to just do our best judgement by now [10:59] ok [11:12] Wellark_: stayed [11:12] Saviq, yep, that's why i'm asking [11:13] tsdgeos, overview is in distro even now [11:13] dednick, Saviq: ok. thanks [11:13] mhr3: is it? [11:13] mhr3: something weird is going on then [11:13] mhr3, oh it is... [11:13] I have a fix coming up on the unknown networkregistration status [11:14] that would explain what you both are seeing [11:14] Wellark_, another issue, I renamed the SIMs, but indicator didn't get the message [11:14] Wellark_, even over a reboot [11:14] tsdgeos, did you update dash-overview to work with v4 yet? [11:14] mhr3: yes [11:14] Saviq: can't put everything to a single merge [11:14] tsdgeos, let me check then [11:14] Wellark_, that's ok, just wanted to know [11:14] mhr3: lp:~aacid/unity8/do_merged [11:15] Saviq: for now it just shows SIM 1 and SIM 2 for the users to separate them somehow :) [11:15] mhr3, we're going into silo 11 btw === Wellark_ is now known as Wellark [11:15] Saviq, cool... but they took my phone, so doesn't help really :P [11:15] mhr3, you still have your laptop :P [11:15] yea, and it can build stuff without silos :) [11:16] mhr3: ./builddir/src/Dash/unity8-dash -mousetouch [11:16] pffty [11:17] Saviq, can I have a mid-review for this refactor? http://paste.ubuntu.com/7978870/ [11:17] just pasted the changed to GenericScopeView (tests pass) [11:17] mhr3: scopes.count undefined [11:18] huh? [11:18] we may be old [11:18] or i may be [11:18] Cimi, dudue [11:18] mhr3: which is the plugin with v4 and altnav? [11:18] Cimi, why don't you just use two Loaders? [11:18] tsdgeos, 0.5.3 [11:18] mhr3: does that have v4? [11:19] yes [11:19] Saviq, why two loaders? [11:19] Cimi, why not? [11:19] Cimi, empty loaders are dead cheap [11:19] Saviq, you're always for saving code [11:19] remove item, remove properties etc etc [11:19] Cimi, yeah, and that doesn't save any code, but makes things overly complex [11:19] Saviq, but supports more pages [11:19] Saviq, just adding another source [11:20] Cimi, but we only need to support the two [11:20] it doesn't simply having two [11:20] two loaders [11:20] Cimi, and we'll need to move to a stack sooner rather than later [11:20] tsdgeos, but yea, something's wrong, search no worky [11:20] Cimi, it's definitely more readable [11:20] because you still need the logic to unload one loader [11:20] mhr3: which branch of scopes-shell shall i be using? [11:20] tsdgeos, sort-order [11:21] ok, let me rebuild that one [11:21] Cimi, but that's just "active = false" or so [11:21] Saviq, and z index [11:21] Cimi, no, because you just make it invisibee [11:21] Cimi, only one of them will ever be visible at any given time [11:22] I can try another shot, but I think it will be same complexity [11:22] just different code [11:22] Cimi, more readable for sure [11:23] you still need one variable to detect the mode [11:23] the subpage opened [11:23] tsdgeos, oh, i think i know what the issue is, pete-woods broke it :P [11:23] and still need to make sure the other loader is completely done before showing the new one [11:23] (I tried this before) [11:24] dednick: so, there was a tiny bug left that caused the unlock icon to stay visible [11:24] Cimi, or maybe just not use a loader, leave PreviewListView be [11:24] Cimi, just add SettingsPage directly, with no model set [11:24] mhr3: makes sense :D [11:24] dednick: I've now improved the handling of the unknown network registration status [11:24] Cimi, aaanyway it doesn't really matter, since we need to put it in a stack asap [11:24] mhr3: what ws the thing to tell dkpg-pabkage not to run tests? [11:24] Cimi, so maybe you should just think about doing that instead? [11:24] tsdgeos, DEB_BUILD_OPTIONS=nocheck [11:25] Saviq, pagestack? [11:25] Saviq, with dash etc? [11:25] tsdgeos, do you still have problem with the location tests? [11:25] mhr3: yep [11:25] weird [11:25] must be something on your system [11:25] Cimi, rather Qt's StackView [11:25] Saviq, is it animated? [11:26] Cimi, http://qt-project.org/doc/qt-5/qml-qtquick-controls-stackview.html [11:26] Cimi, yeah, dash would be the first page in there, previews, scopes, settings pages are pushed on top [11:26] Cimi, Qt's is, yes [11:26] cool [11:27] Saviq, I'll spend one hour trying two loaders (if it simplifies) [11:27] Saviq, if it doesn't can we get this in and I use stack as next task? [11:27] Cimi, yeah, don't even spend the hour to use two loaders [11:27] Saviq: could you plese rebuild indicator-network in silo1 ? [11:27] Wellark, no [11:28] Wellark, it's already under QA [11:28] Wellark, it'll land pretty soon [11:28] Wellark: ok. I'm testing some other work now, but I'll pull your source and build it locally when i get a minute [11:28] did you rip out indicator-network from there? [11:28] Wellark, so any fixes will need to come after that [11:28] Wellark, no I didn't [11:28] ok, thanks [11:28] there is just a small one really [11:28] Wellark, it's not perfect [11:28] Wellark, but it's not a regression, just incomplete [11:29] Wellark, so there's no reason to hold back really [11:29] Saviq: what issues have you had? [11:29] empty item? [11:29] Wellark, just empty item, yes [11:29] Wellark, and the not-named SIMs [11:29] Saviq: regression being the incorrect icon being shown in the panel. [11:29] but that you said is for later [11:30] dednick: umm? [11:30] which icon is incorrect? [11:30] dednick, didn't notice that? [11:30] Saviq: https://dl.dropboxusercontent.com/u/85539674/locked_sim/locked_sim8.png [11:30] I commented on the mp [11:30] dednick, ah, empty instead of locked [11:30] the panel icon is correct [11:31] dednick, shouldn't it show a padlock? [11:31] the "locked icon" is wrong [11:31] Wellark: it's unlocked. [11:31] Wellark: i had already unlocked the phone [11:31] dednick: https://code.launchpad.net/~unity-api-team/indicator-network/modeminfo/+merge/225160/comments/558106 [11:32] Wellark: and the panel is showing gsm-3g-disabled.svg [11:32] yes. as for some reason it seems you are getting unknown state from the networkregistration [11:32] Wellark: right. but that's a regression :) [11:32] dednick, are there steps to repro that? [11:33] Saviq: just reboot phone and unlock. It happened the second time i tried, but not subsequent tries [11:33]