/srv/irclogs.ubuntu.com/2014/08/07/#ubuntu-unity.txt

=== _salem is now known as salem_
=== salem_ is now known as _salem
=== Cimi_ is now known as Cimi
=== RAOF_ is now known as RAOF
tsdgeosSaviq: did you see my comment on https://bugs.launchpad.net/unity8/+bug/1316660 ?07:15
ubot5Launchpad bug 1316660 in Unity 8 "Scope preview should use SDK's Expandable*s" [High,In progress]07:15
Saviqtsdgeos, 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:16
Saviqtsdgeos, actually putting the buttons into an Expandable will be easy now that there is the rest in place07:17
Saviqoh wait07:17
Saviqyou didn't put it in an Expandable07:17
Saviqthen yeah, you're right07:17
Saviq(you should've) :P07:17
tsdgeoswell why?07:18
tsdgeosit wasn't the design07:18
tsdgeosdesign is "combobox + button"07:18
Saviqtsdgeos, well, yeah, but expansion is a common pattern, we have the expandable components in the SDK for a reason07:18
tsdgeosthe 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 stuff07:18
tsdgeosis not my problem07:18
Saviqtsdgeos, I think that's two different issues07:19
tsdgeosyes we have a combobox in the sdk now07:19
tsdgeosnot when we wrote that code07:19
Saviqtsdgeos, that's not about that07:19
Saviqtsdgeos, even the buttons we have now07:19
Saviqtsdgeos, should have been put in SDK's Expandable, and the whole Preview needs to be UbuntuListView or UbuntuColumn07:19
Saviqtsdgeos, so that when you expand something, it gets into view without you caring about it07:20
tsdgeosok, still nothing to do with my branch :D07:20
Saviqtsdgeos, I just thought you did that in there07:21
tsdgeosnope07:21
tsdgeosi just added the expandable preview widget07:21
Saviqtsdgeos, 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 branch07:21
mptWait, are you calling me two generations old07:21
* mpt shakes his walking stick at tsdgeos 07:22
Saviq;)07:22
Saviqhe's lurking!07:22
tsdgeosSaviq: that branch does that07:23
Saviqtsdgeos, oh hmm, I can't read then07:23
Saviqgood07:23
Saviqtsdgeos, ah now I can see07:24
SaviqCimi, it's time for that review when you're up ;)07:26
CimiSaviq, morning :)07:30
CimiSaviq, it's also time for my review of emblems :)07:30
CimiSaviq, but why we have the surrounding box aroung the orange?07:35
Cimiwould it look bad withou?07:35
SaviqCimi, that was what design requested, yes07:36
Cimiok07:36
SaviqCimi, otherwise it'd clash with content behind it07:36
Cimifor the same reason we are not using the ubuntushape in every single corner of the os07:36
Ciminot to clash07:37
SaviqCimi, and this doesn't use the shape, not sure what reference is that to ;)07:37
Cimihah07:37
Saviqmhr3, morning! did you guys ever get all the new square assets for dash overview?07:37
mhr3Saviq, no07:38
mhr3and morning07:38
mhr3Saviq, at the office today, so will ask esti07:38
Saviqmhr3, cool, thanks07:38
Saviqtell her we'll land this with old assets anyway, but she won't like it ;)07:38
Saviqtsdgeos, 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
tsdgeosSaviq: which shader effect?07:41
tsdgeosah07:41
tsdgeosthe copy07:41
Saviqtsdgeos, the one that copies the dash to the list07:41
tsdgeosit might07:41
Saviqit is, after all, almost a fullscreen snapshot07:41
tsdgeostbh i didn't see any lag compared to what we had07:42
SaviqI'll try it out again but we did notice some still in the office07:42
tsdgeosbut it was horrible07:42
tsdgeosso anything would be much better07:42
Saviqsure07:42
tsdgeosSaviq: sure it'd be possible to scale itself probably, but not sure if it'd be cheaper07:43
Saviqtsdgeos, well, should be, since it'd be scaling the texture itself, not a copy of it07:43
tsdgeosSaviq: but it may end up in redraws blah blah07:44
Saviqtsdgeos, trueth07:44
Saviqtsdgeos, but shouldn't :)07:44
CimiSaviq, approved, you can go in dash overview and need fix it07:45
Cimiotherwise I'll do :)07:46
Cimiwe need to export the activity indicator and use it there too07:47
Saviqtrueth07:47
tsdgeoswhat's up with cI?07:50
tsdgeosi did a commit 12 hours ago and still no run? https://code.launchpad.net/~aacid/unity8/prepend_test_xml/+merge/22976607:50
Saviqtsdgeos, it's approved07:51
Saviqtsdgeos, ci doesn't run on already top-approved branches07:52
Saviq(automagically)07:52
tsdgeosahhh07:52
tsdgeosdidn't know that07:52
tsdgeosi'll trigger a rebuild then07:53
Saviqmzanetti, welcome back ;)07:59
mzanettio/08:38
mzanettihello08:38
mzanettiSaviq: thanks08:38
tsdgeosSaviq: are we on rails again?08:39
dednickSaviq: um, any idea why indicator-network depends on unity8? seems like kind of a weird dep.08:50
mzanettiuh! dash-as-app landed. nice08:51
dednickmzanetti: pain in the arse on desktop though! :)08:52
mzanettidednick: how?08:52
mzanettidednick: you mean run.sh?08:52
dednickmzanetti: yeah. 2 windows08:52
mzanettidednick: right... well, I only use make tryShell nowadays08:53
mzanettiwould that be an option for you?08:53
dednickdash is now a screenshot in shell, with separate window for scopes :)08:53
mzanettidednick: yeah, I know... I did that08:53
dednickmzanetti: hm. could be08:53
mzanettidednick: maybe just LD_LIBRARY_PATH some indicators stuff08:53
dednickoh i'm just complaining... don't worry about it08:53
mzanettithen make tryShell should be fine for you08:53
mzanettiI guess we should get rid of run.sh...08:54
mzanettior not use upstart for it...08:54
dednickyeah08:54
mzanettidednick: what I do too is to launch the unity8 binary with qtcrator, without run.sh08:55
dednickthe upstart bit is my major gripe with it08:55
mzanettidednick: just add "-mousetouch" to the cmdline args08:55
dednickmzanetti: ah. was wondering about that. my mouse wasn't working with running binary08:55
mzanettiyep, -mousetouch08:55
tsdgeosSaviq: ping09:07
=== pete-woods1 is now known as pete-woods
Saviqtsdgeos, pong09:23
tsdgeosSaviq: where does scopeStyle come from in https://code.launchpad.net/~saviq/unity8/dash-activity-indicator/+merge/229805 ?09:23
tsdgeoscolor: scopeStyle.backgroundLuminance > 0.7 ? "#50000000" : "#50ffffff"09:23
Saviqtsdgeos, yeah, by now... from nowhere09:23
tsdgeosok09:23
tsdgeosi merged all silo1 to dash_overview09:24
tsdgeosand was wondering if that was a merge issue or not09:24
Saviqtsdgeos, yeah, not, that's broken atm09:24
Saviqas I was moving the indicator out and out, I no longer have scopeStyle indeed...09:24
Saviqmhr3, bcmwl broken == no 5GHz networks, eh?09:25
mhr3Saviq, i wouldn't know, my chip doesn't do 5ghz at all09:25
Saviqugh09:25
Saviqmhr3, fyi, downgraded to previous bcmwl version (tried previous kernel, too), didn't help :/09:34
mhr3Saviq, are you sure your driver supports 5ghz? :P09:35
Saviqmhr3, yes :P09:35
Saviqit *did*09:35
Saviqtsdgeos, yeah we are09:35
tsdgeosgooed :)09:35
tsdgeos-e09:35
Cimiold kernel?09:37
CimiI am on 3.1309:37
Cimino bcmwl issues09:37
mhr3Saviq, maybe it just pretended, and now they fixed it to stop pretending ;)09:37
SaviqCimi, do you get 5GHz networks?09:39
CimiSaviq, yes09:40
CimiSaviq, using 14.0409:40
SaviqCimi, right09:40
mhr3Saviq, reminded esti about the icons09:42
mhr3Saviq, i'll push the square ones into the existing branches at least09:42
mhr3Saviq, when do you plan to land it?09:42
Saviqmhr3, asap09:45
Saviqmhr3, doing a func/vis review right now09:45
Cimimzanetti, wanted to ping you yesterday to remove the darkening effect when only one app (dash then) is running09:48
Cimion the spread09:49
mzanettiCimi: that's on my list to discuss with Vesa when he's back09:53
mzanettiCimi: IMO we need to get rid of that effect at all09:53
mzanettiCimi: or at least make it muuuch less09:53
Cimiyeah09:53
Cimionly helps in quick flip09:53
Cimibut looks bad on spread09:54
mzanettiyeah... not even much there09:54
Saviqtsdgeos, you didn't push the whole merge of dash overview did you?09:55
tsdgeosno, i still have it here09:55
tsdgeosrunning tests and stuff09:55
tsdgeosmerge was far from trivial09:55
mhr3Saviq, do we need to modify the category template for the overviews?09:58
tsdgeosSaviq: do y'ou want me to push it to a separate branch?09:58
mhr3Saviq, or i guess you're doing magic to it already since you'll be cropping the icons?09:58
mhr3Saviq, still, we need a new template for the scope search results09:59
Saviqmhr3, yeah, we'll be overriding the aspect ratio there though10:00
Saviqtsdgeos, I'd like to have an early look at it, yeah10:02
Saviqtsdgeos, whether same or separate branch - you tell me10:02
tsdgeosSaviq: lp:~aacid/unity8/do_merged10:03
Saviqtsdgeos, thanks10:04
tsdgeosqml tests seem to be passing now10:05
tsdgeoswill run autopilot after that10:05
Saviqtsdgeos, just run ci on it10:05
Saviqtsdgeos, you can put it as WiP MP10:05
tsdgeoswill fail10:05
tsdgeosdoesn't have scopes v410:05
Saviqah v4 :/10:05
Saviqtsdgeos, hmm it just goes blank for me as I drag from the bottom :|10:07
Saviqtsdgeos, looks like the background is somewhere covering the overview?10:07
tsdgeoswops yeah wrong merge10:08
tsdgeoswill check10:08
tsdgeoswonder how the dash overview test passed10:09
tsdgeosi guess because items are still there and can be clicked10:09
Saviqyeah10:09
Saviqtsdgeos, the background should just be part of DashContent.qml don't you think?10:10
tsdgeosit is10:10
Saviqoh ok10:10
* Saviq stops10:10
tsdgeosthe wrong background is the one you made me add yesterday when i told you i didn't want to10:10
tsdgeosSaviq: pull10:10
tsdgeosi.e. just a wrong merge10:11
Saviqk10:12
Saviqmhr3, which MPs do I need to get for a silo with overview10:16
mhr3Saviq, keeping in mind that you'll need to override the search category too?10:16
Saviqmhr3, yeah, probably not straight away, but yes10:16
mhr3Saviq, https://code.launchpad.net/~unity-team/unity-scope-mediascanner/overview-icons/+merge/22825410:17
Saviqmhr3, we assumed the category name would be static10:17
mhr3Saviq, and https://code.launchpad.net/~unity-team/unity-scope-click/overview-icons/+merge/22773110:17
mhr3Saviq, it's "recommended" iirc10:17
mhr3nope10:17
mhr3"recommendations"10:17
Saviqmhr3, and we'll need the scopes scope to change template, right?10:19
Saviqtsdgeos, looks like you need to tweak data/unity8-dash.conf to drop scopes scope from favourites10:24
Saviqah wait10:24
Saviqit probabaly is10:24
tsdgeosSaviq: it's drooped10:24
tsdgeosit's just that our run system is not so great10:24
* Saviq needs to copy the .conf10:24
tsdgeosand still uses sytemwide instead ours10:24
tsdgeosneed a way to fix that10:24
tsdgeosit's borking autopilot for me now too10:24
Saviqtsdgeos, yeah, I've a branch10:24
Saviqtsdgeos, autopilot should be fine, assuming you copy the .conf file(s) to ~/.config/upstart/10:25
Saviqand make install of course10:25
tsdgeossure, i just haven't copied it ;)10:25
Saviqtsdgeos, hmm, favorites always go back to the dash, don't change scope?10:26
tsdgeosSaviq: you mean on click?10:27
Saviqtsdgeos, yes10:27
tsdgeosworks here10:27
tsdgeosdesktop or phone?10:27
Saviqtsdgeos, desktop10:27
tsdgeosthat is weird10:27
tsdgeosdo you have the latest qdeclarative?10:27
Saviqyeah10:28
tsdgeoswhat do you do exactly?10:29
Saviqtsdgeos, let me record10:31
Saviqtsdgeos, I start, drag up, click on music10:31
Saviqand am taken back to apps10:31
Saviqlike I pressed DONE10:31
Saviqit doesn't even zoom in from the right item10:31
tsdgeosSaviq: can you try make tryDash ?10:32
tsdgeosyep, nity8 seems to be failing10:33
tsdgeosqml: No match for scope with id: musicaggregator10:33
tsdgeosthat is weird10:33
tsdgeoshmmm10:33
Saviqtsdgeos, works there, yeah10:33
Saviqtsdgeos, btw, search doesn't look right in overview10:33
tsdgeosmhr3: does your sort-order branch have the things for dash_overview too?10:33
Saviqah right maybe I'm missing something there10:34
tsdgeosSaviq: what is missing in search?10:34
Saviqtsdgeos, it doesn't really search...10:34
Saviqtsdgeos, just displays All, really10:34
tsdgeosi'm going to blame the plugin10:35
tsdgeosnow that is mhr3's last day10:35
mzanettioh no! Is it?10:35
Saviqnot tomorrow?10:35
Saviqtsdgeos, yeah, /me stupid, don't have the plugin here10:36
Saviqtsdgeos, btw, we got silo 1110:37
tsdgeosawesome10:38
tsdgeosautopilot is failing for nexus 10 size10:38
tsdgeosfor some reason on resize the current scope is changing10:39
tsdgeosfrom clickscope to video10:39
tsdgeosso all the tests fail to start10:39
tsdgeosi'm confused, since doing the resize manually doesn't give me that behaviour10:39
tsdgeos            # XXX Currently we have no way to launch the application with a10:41
tsdgeos            # specific size, so we must resize it after it's launched.10:41
tsdgeos            # --elopio - 2014-06-2510:41
tsdgeoswell we could have added that way....10:41
tsdgeosinstead of doing something that now broke :D10:42
=== MacSlow is now known as MacSlow|lunch
Wellark_dednick: did it actually stay this way?10:50
Wellark_https://dl.dropboxusercontent.com/u/85539674/locked_sim/locked_sim8.png10:50
Wellark_or was that a short intermediate state ?10:51
mzanettiSaviq: hey, did you get anything regarding the lockscreen designs?10:56
SaviqWellark_, I saw an empty item like that today, yeah (no lock icon, but probably same issue then)10:56
Saviqmzanetti, yeah, my fail10:56
=== Pici` is now known as Pici
mzanettihuh?10:56
tsdgeosSaviq: 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
Saviqmzanetti, didn't get to that10:56
Saviqtsdgeos, makes sense, yeah10:56
Saviqtsdgeos, but not a high prio10:56
mzanettiSaviq: well... not your fail...10:56
mzanettiI asked the questions a thousand times already10:57
Saviqmzanetti, yeah, my fail 'cause I didn't remember to follow up on that :|10:57
tsdgeosSaviq: well the autopilot is failing :D but maybe works on CI10:57
tsdgeosso all happy10:57
tsdgeoslet's wait for silo 1 to merge10:57
Saviqtsdgeos, we're not running X11 tests on ci any more10:57
tsdgeosSaviq: when is the eta for silo1?10:57
Saviqtsdgeos, davmor2 has his hands on it, so unless he finds a blocker, soon!10:58
mzanettiSaviq: so... what shall we do? just implementing it somehow, ignoring some contradictory parts of the spec and making up missing parts ourselves?10:58
tsdgeosSaviq: greatz10:58
mzanettiI'm not confident to get the answers any more10:58
Saviqmzanetti, yeah, by now it's probably best to just do our best judgement by now10:59
mzanettiok10:59
dednickWellark_: stayed11:12
mhr3Saviq, yep, that's why i'm asking11:12
mhr3tsdgeos, overview is in distro even now11:13
Wellark_dednick, Saviq: ok. thanks11:13
tsdgeosmhr3: is it?11:13
tsdgeosmhr3: something weird is going on then11:13
Saviqmhr3, oh it is...11:13
Wellark_I have a fix coming up on the unknown networkregistration status11:13
Wellark_that would explain what you both are seeing11:14
SaviqWellark_, another issue, I renamed the SIMs, but indicator didn't get the message11:14
SaviqWellark_, even over a reboot11:14
mhr3tsdgeos, did you update dash-overview to work with v4 yet?11:14
tsdgeosmhr3: yes11:14
Wellark_Saviq: can't put everything to a single merge11:14
mhr3tsdgeos, let me check then11:14
SaviqWellark_, that's ok, just wanted to know11:14
tsdgeosmhr3: lp:~aacid/unity8/do_merged11:14
Wellark_Saviq: for now it just shows SIM 1 and SIM 2 for the users to separate them somehow :)11:15
Saviqmhr3, we're going into silo 11 btw11:15
=== Wellark_ is now known as Wellark
mhr3Saviq, cool... but they took my phone, so doesn't help really :P11:15
Saviqmhr3, you still have your laptop :P11:15
mhr3yea, and it can build stuff without silos :)11:15
tsdgeosmhr3:  ./builddir/src/Dash/unity8-dash -mousetouch11:16
Saviqpffty11:16
CimiSaviq, can I have a mid-review for this refactor? http://paste.ubuntu.com/7978870/11:17
Cimijust pasted the changed to GenericScopeView (tests pass)11:17
tsdgeosmhr3: scopes.count undefined11:17
mhr3huh?11:18
tsdgeoswe may be old11:18
tsdgeosor i may be11:18
SaviqCimi, dudue11:18
tsdgeosmhr3: which is the plugin with v4 and altnav?11:18
SaviqCimi, why don't you just use two Loaders?11:18
mhr3tsdgeos, 0.5.311:18
tsdgeosmhr3: does that have v4?11:18
mhr3yes11:19
CimiSaviq, why two loaders?11:19
SaviqCimi, why not?11:19
SaviqCimi, empty loaders are dead cheap11:19
CimiSaviq, you're always for saving code11:19
Cimiremove item, remove properties etc etc11:19
SaviqCimi, yeah, and that doesn't save any code, but makes things overly complex11:19
CimiSaviq, but supports more pages11:19
CimiSaviq, just adding another source11:19
SaviqCimi, but we only need to support the two11:20
Cimiit doesn't simply having two11:20
Cimitwo loaders11:20
SaviqCimi, and we'll need to move to a stack sooner rather than later11:20
mhr3tsdgeos, but yea, something's wrong, search no worky11:20
SaviqCimi, it's definitely more readable11:20
Cimibecause you still need the logic to unload one loader11:20
tsdgeosmhr3: which branch of scopes-shell shall i be using?11:20
mhr3tsdgeos, sort-order11:20
tsdgeosok, let me rebuild that one11:21
SaviqCimi, but that's just "active = false" or so11:21
CimiSaviq, and z index11:21
SaviqCimi, no, because you just make it invisibee11:21
SaviqCimi, only one of them will ever be visible at any given time11:21
CimiI can try another shot, but I think it will be same complexity11:22
Cimijust different code11:22
SaviqCimi, more readable for sure11:22
Cimiyou still need one variable to detect the mode11:23
Cimithe subpage opened11:23
mhr3tsdgeos, oh, i think i know what the issue is, pete-woods broke it :P11:23
Cimiand still need to make sure the other loader is completely done before showing the new one11:23
Cimi(I tried this before)11:23
Wellarkdednick: so, there was a tiny bug left that caused the unlock icon to stay visible11:24
SaviqCimi, or maybe just not use a loader, leave PreviewListView be11:24
SaviqCimi, just add SettingsPage directly, with no model set11:24
tsdgeosmhr3: makes sense :D11:24
Wellarkdednick: I've now improved the handling of the unknown network registration status11:24
SaviqCimi, aaanyway it doesn't really matter, since we need to put it in a stack asap11:24
tsdgeosmhr3: what ws the thing to tell dkpg-pabkage not to run tests?11:24
SaviqCimi, so maybe you should just think about doing that instead?11:24
mhr3tsdgeos, DEB_BUILD_OPTIONS=nocheck11:24
CimiSaviq, pagestack?11:25
CimiSaviq, with dash etc?11:25
mhr3tsdgeos, do you still have problem with the location tests?11:25
tsdgeosmhr3: yep11:25
mhr3weird11:25
mhr3must be something on your system11:25
SaviqCimi, rather Qt's StackView11:25
CimiSaviq, is it animated?11:25
SaviqCimi, http://qt-project.org/doc/qt-5/qml-qtquick-controls-stackview.html11:26
SaviqCimi, yeah, dash would be the first page in there, previews, scopes, settings pages are pushed on top11:26
SaviqCimi, Qt's is, yes11:26
Cimicool11:26
CimiSaviq, I'll spend one hour trying two loaders (if it simplifies)11:27
CimiSaviq, if it doesn't can we get this in and I use stack as next task?11:27
SaviqCimi, yeah, don't even spend the hour to use two loaders11:27
WellarkSaviq: could you plese rebuild indicator-network in silo1 ?11:27
SaviqWellark, no11:27
SaviqWellark, it's already under QA11:28
SaviqWellark, it'll land pretty soon11:28
dednickWellark: ok. I'm testing some other work now, but I'll pull your source and build it locally when i get a minute11:28
Wellarkdid you rip out indicator-network from there?11:28
SaviqWellark, so any fixes will need to come after that11:28
SaviqWellark, no I didn't11:28
Wellarkok, thanks11:28
Wellarkthere is just a small one really11:28
SaviqWellark, it's not perfect11:28
SaviqWellark, but it's not a regression, just incomplete11:28
SaviqWellark, so there's no reason to hold back really11:29
WellarkSaviq: what issues have you had?11:29
Wellarkempty item?11:29
SaviqWellark, just empty item, yes11:29
SaviqWellark, and the not-named SIMs11:29
dednickSaviq: regression being the incorrect icon being shown in the panel.11:29
Saviqbut that you said is for later11:29
Wellarkdednick: umm?11:30
Wellarkwhich icon is incorrect?11:30
Saviqdednick, didn't notice that?11:30
dednickSaviq: https://dl.dropboxusercontent.com/u/85539674/locked_sim/locked_sim8.png11:30
dednickI commented on the mp11:30
Saviqdednick, ah, empty instead of locked11:30
Wellarkthe panel icon is correct11:30
Saviqdednick, shouldn't it show a padlock?11:31
Wellarkthe "locked icon" is wrong11:31
dednickWellark: it's unlocked.11:31
dednickWellark: i had already unlocked the phone11:31
Wellarkdednick: https://code.launchpad.net/~unity-api-team/indicator-network/modeminfo/+merge/225160/comments/55810611:31
dednickWellark: and the panel is showing gsm-3g-disabled.svg11:32
Wellarkyes. as for some reason it seems you are getting unknown state from the networkregistration11:32
dednickWellark: right. but that's a regression :)11:32
Saviqdednick, are there steps to repro that?11:32
dednickSaviq: just reboot phone and unlock. It happened the second time i tried, but not subsequent tries11:33
dednickunlock sim i mean11:33
Wellarkno point doing that without my latest commit11:33
dednickSaviq: happens randomly i think.11:33
Saviqdednick, yeah, all behaves fine here, lemme try with the other phone11:33
dandradermzanetti, are you back?11:34
Saviqdandrader, he is11:35
dednickSaviq: i'm not really that concerned about the panel icon. as long as we fix it soonish. It seems to work most of the time anyway.11:37
Saviqdednick, Wellark, for a moment after putting the SIM in I got just the locked icon in an empty item, a second later Searching /| showed up and all's fine now11:38
Saviqdednick, yeah, agreed11:38
dednickSaviq: yeah. sometimes it stays empty and doesnt update. Not affecting any functions though11:38
Wellarkdednick: you have it empty how?11:39
Wellarkcould you get me the output of /usr/share/ofono/scripts/list-modems11:39
dednickWellark: no, can't seem to reproduce it now. I'm building something at the moment. will try a couple more times after11:40
Wellarkwith my latest commit instead of empty there would be 3g-disabled icon and text "Unknown"11:40
Wellarkbut the interesting part is that why it's unknown11:40
mzanettidandrader: yes11:40
Wellarkanyway, the indicator will state it more clearly, instead of just an empty item11:41
dednickWellark: what is ofono giving you for the unknown state?11:41
dandradermzanetti, welcome back!11:41
dednickWellark: might be worth doing a debug log if you can't resolve a proper state.11:42
dandradermzanetti, need some help with the spread :)11:42
mzanettidandrader: don't break it :P11:42
tsdgeosmzanetti: yeah welcome11:42
Saviqbiab11:42
mzanettitsdgeos: thanks :)11:43
=== Malsasa_ is now known as Malsasa
WellarkSaviq: so, how do we do this indicator-network landing?  there are just couple of commits that are not part of the package11:59
Wellarkin silo 112:00
Wellarkprobably just request a new silo for them straight away after silo 1 has landed12:00
mhr3tsdgeos, hm, actually, i don't see what's wrong, must be ui :P12:02
mhr3Saviq, so what's the template the scope search cat should use?12:04
=== pete-woods is now known as pete-woods|lunch
mhr3Saviq, also, the progress bar seems to always do a full back and forth, is that desired?12:06
=== MacSlow|lunch is now known as MacSlow
=== alan_g is now known as alan_g|lunch
mhr3tsdgeos, ah, the overview scope doesn't have .active = true12:14
mhr3that's why it doesn't search12:14
SaviqWellark, yeah, it will just not merge the last two commit12:16
Saviqs12:16
Saviqmhr3, yes12:16
CimiSaviq, can I ask for a triggered method for settings?12:17
Cimiwould make things easier I guess12:17
SaviqCimi, you can, but not me ;)12:17
Cimihah12:17
mhr3Saviq, it's confusing, makes it look like it's waiting for the images to load, but only sometimes :/12:17
CimiSaviq, I was wondering why someone decided not to have one12:17
Cimimhr3, can I ask for a triggered method on the settings model?12:17
Saviqmhr3, it has a delay of 200ms, and if stuff doesn't complete within that time, it shows for at least full revolution12:18
Saviqmhr3, otherwise you'd just see it flash for a split second12:18
SaviqCimi, triggered what, btw?12:18
CimiSaviq, action on the setting12:18
Cimilike, property changed12:18
mhr3Saviq, why don't we do the fade out trick?12:19
mhr3Cimi, what property?12:19
Saviqmhr3, we do12:19
Cimimhr3, any property12:19
mhr3Cimi, it's a model, you should be getting dataChanged12:19
Saviqmhr3, but there has to be a minimal time it shows12:19
Cimimhr3, how do I update properties I change in the ui?12:19
Cimistore those12:19
Saviqmhr3, otherwise it'd just be on/off within a split second at times, you wouldn't even notice it there12:20
mhr3Cimi, just write to the model?12:20
Saviqmhr3, we can make it only go one way, the current thing is what we decided with designers though12:20
SaviqCimi, the model is writable12:20
Cimiso I have to use setData?12:21
mhr3Saviq, not sure that would solve anything12:22
* mterry loves having CI runs that actually fully pass12:22
mhr3Saviq, anyway, as you said, it's desired, so i rest my case12:22
dandradermterry, it doesn't happen every day12:26
dandrader:)12:26
mterrydandrader, well today is a good day  :)12:26
SaviqCimi, model.property = foo12:28
Saviqmterry, guess what, we don't have any, yet ;012:29
Saviqmterry, check out the test count on https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/795/testReport/12:29
mterrySaviq, 704 tests?12:31
Saviqmterry, yeah, that's the real count12:31
Saviqmterry, jenkins wasn't looking at FooFooTest.xml files, just testFooBar.xml12:31
mhr3Saviq, so what's the template the scope search cat should use?12:32
Saviqmterry, which means until that branch we were not getting all the output from testing12:32
mterrySaviq, hrm12:32
Saviqmhr3, let me see12:32
Saviqmhr3, http://pastebin.ubuntu.com/7979392/ I'd say12:34
mhr3thx, using12:40
Cimimhr3, what's the difference between properties and values in the settings interface?12:43
mhr3Cimi, you asked that two days ago :P value is value, properties is valid for lists12:44
Cimimhr3, ok but I don't know what properties for list]s is12:44
mhr3Cimi, http://bazaar.launchpad.net/~unity-team/unity-scopes-shell/trunk/view/head:/tools/settings/widgets/listSettingsWidget.qml12:45
Cimimhr3, thx12:45
Ciminow lunch break12:45
mzanettiSaviq: so... what shall we do with the 4 digit autoconfirm?12:46
* mzanetti thinks it should go away12:46
Saviqmzanetti, +112:47
mzanettiack12:47
=== _salem is now known as salem_
Saviqmzanetti, or well, settings app only allows setting 4-digit PINs doesn't it?12:47
mzanettiit does atm... wouldn't prevent the lockscreen to still allow pressing OK after entering 4 digits ;)12:47
mzanettibut yeah... settings should be changed too to get away with that 4 digit limit12:48
mzanettiIMO12:48
Saviqmzanetti, well, that's not for us to say...12:48
mzanettiI know... its design... but they don't do12:48
Saviqmzanetti, if the design is still that we should to autoconfirm, and that it's always 4 digit, let's leave it at thta12:48
mzanettiSaviq: well, the new lockscreen design doesn't say anything about the autoconfirm12:49
mzanettilooks like the person who designed it didn't ever hear about that12:49
mzanettii.e. there's always a confirm button around12:50
Saviqmzanetti, heh, ok then, do not implement autoconfirm ;)12:51
mzanettiok12:51
tsdgeosis there any other way to unlock the sim than clickin on the indicator?12:52
tsdgeosbecause it doesn't work12:53
tsdgeosmaybe i need to reboot?12:53
mzanettitsdgeos: heh, same here today. works fine with silo 1 though12:53
mzanettitsdgeos: you can also use qdbus if its just for that12:53
mzanettitsdgeos: org.ofono, EnterPin12:53
tsdgeosi'd prefer my krillin to work12:54
tsdgeossince i'm supposed to do rtm tests12:55
mzanettitsdgeos: yeah... I know12:56
mzanettisame here12:56
tsdgeosinterestingly a reboot made it work12:56
mhr3tsdgeos, is the overview supposed to support overlay color yet?13:07
tsdgeosmhr3: which branch?13:07
mhr3tsdgeos, da-...13:08
mhr3tsdgeos, do-...13:08
tsdgeosmhr3: yes should be there13:08
mhr3tsdgeos, hm, when i set the overlay color to white, the text is white13:11
mhr3and when i set it to black, it's not black enough :P13:11
mhr3tsdgeos, hmm... yea, i just set it to "#000000ff" and there's no sign of black13:13
tsdgeoswell you made it totally transparent13:14
tsdgeoswhat do you expect?13:14
mhr3tsdgeos, what? it's rrggbbaa, no?13:14
Saviqtsdgeos, argb13:15
Saviqmhr3, rather ↑13:15
mhr3grrr13:15
mterrySaviq, I can't wait for this next unity8 mega merge13:18
mterryLike 20 different branches13:19
cwaynemterry: i know right?13:19
cwayneexcited :)13:19
Saviqmterry, yeah, and it's already QA-acked13:21
mterrySaviq, we're supposed to promote an image today I believe...13:21
Saviqmterry, yeah, that silo fixes 2 blockers13:22
mterrySaviq, 'course now we're lurching from traincon0 to traincon0 as we land enormous amounts of code containing a few regressions each time we exit traincon0  :-/13:22
Saviqmterry, that's why the additional QA ack13:23
Saviqmterry, is meant to make sure we're not introducing any13:23
mhr3tsdgeos, anyway, white overlay + white text not easy to read :P13:23
mterrySaviq, yeah but I'm talking about getting out of traincon0 and then landing the piles of branches that have accumulated13:23
Saviqtsdgeos, mhr3, looks like overlay doesn't do the right colour detection?13:23
Saviqmterry, ah right, yeah13:23
=== Malsasa_ is now known as Malsasa
mhr3tsdgeos, also, did you see my msg about the overview scope's .active?13:24
mhr3Saviq, right13:24
tsdgeosmhr3: no i didn't13:24
mhr3tsdgeos, ah, the overview scope doesn't have .active = true13:24
mhr3that's why it doesn't search13:24
tsdgeosSaviq: it doesn't do anything, scope just should set the right color, no?13:25
Saviqtsdgeos, it can't set the color for overlay text13:25
Saviqtsdgeos, overlay text should decide between foreground and background colours based on overlay colour luminance13:25
mhr3same as background really13:26
mhr3it's just the base color is the overlay color13:26
=== pete-woods|lunch is now known as pete-woods
Saviqyup13:26
tsdgeosSaviq: well then it's a bug13:26
tsdgeosmhr3: do i really need to do that13:27
tsdgeoscan't you know that "wll it's the damn overview scope, that was got with getOverviewScope"13:27
tsdgeosso obviously is special and can be searched all the time13:27
mhr3tsdgeos, no, it's derived from a real scope13:28
mhr3it doesn't hit the derived class if it's not active13:29
tsdgeosdednick: you aware that going in and out of the flight mode "breaks" the indicator positioning?13:29
dednicktsdgeos: er. no.13:29
dednicktsdgeos: oh. that's a crappy tab bug13:30
dednicktsdgeos: it's been removed in the design review MP13:30
=== dandrader is now known as dandrader|afk
Saviqmhr3, did you manage to ask Esti for the overview bottom hint asset? as in https://f966f709-a-c881af26-s-sites.googlegroups.com/a/canonical.com/unity8dash/scopes/dash-overview/hint-expanded.png ?13:31
mhr3Saviq, no, just the scope icons13:33
Saviqmhr3, ok, will email her13:33
Saviqmhr3, unless you can run over still? ;)13:33
mhr3Saviq, isn't that thing in the sdk?13:33
Saviqmhr3, not that I know of13:33
mhr3Saviq, she's not at her desk13:33
mhr3Saviq, and i'm pretty sure i saw it in dialer13:34
mhr3or messaging13:34
mhr3or somewhere13:34
Saviqmhr3, yeah, but there's text in there, and I don't think it's SDK, but will find out13:34
Saviqtsdgeos, https://code.launchpad.net/~aacid/unity8/dash_overview/+merge/228649/comments/55815013:40
Saviqtsdgeos, I think you mostly just missed that comment when I first sent it13:40
tsdgeosSaviq: i don't see what's missing for13:41
tsdgeosme13:41
tsdgeosother than you disagreeing on the opacity of the done text13:41
tsdgeosand someone giving me an asset13:41
tsdgeoswhich since it's probably not going to happen, i didn't do any coding for13:41
Saviqtsdgeos, what 'is not going to happen'?13:42
tsdgeossomeone producing a simple asset as that one13:42
Saviqsince when do we expect assets to not happen?13:42
tsdgeosok, let's not go there13:43
tsdgeosanyway, i'm busy playing with simcards13:43
tsdgeosif you want i can stop that and go back to dash overview13:44
Saviqtsdgeos, do you disagree on the opacity between https://docs.google.com/a/canonical.com/file/d/0B32jwBcbaPloX2hIcGZGOW9jX2s/edit vs. https://f966f709-a-c881af26-s-sites.googlegroups.com/a/canonical.com/unity8dash/scopes/dash-overview/01_Dash_scopes_nav_full_dark_v7.jpg13:44
tsdgeoswell one of them is different13:44
tsdgeosyes :D13:44
tsdgeoswhich is the good one?13:44
Saviqtsdgeos, the one where you can actually read things ;P13:44
Saviqtsdgeos, it should be full opacity white on full black13:45
tsdgeoswith 1 pixel on the bottom?13:45
tsdgeosor that's just a broken visual?13:45
Saviqtsdgeos, broken visual of course13:45
tsdgeosnow i don't even get the "unlock sim" in the network dialog13:45
tsdgeoshurray \o/13:45
=== dandrader|afk is now known as dandrader
Saviqit's publishing!13:54
tsdgeos\o/13:56
mhr3Saviq, josh said 15% opacity is desired for the overview, so that's what all the icons are setting13:56
mhr3Saviq, well.. at least in cases where they're not setting it to 0% opacity and slightly blue :P13:57
Saviqmhr3, https://f966f709-a-c881af26-s-sites.googlegroups.com/a/canonical.com/unity8dash/scopes/dash-overview/01_Dash_scopes_nav_full_dark_v7.jpg?attachauth=ANoY7crZ3-142X_ngcfTjfcyHJD-2B4hUptbj1b-1QVQPt3GsqXiOYyFECH5zacvxN2_W3-wdUYHWOzf4QnZdWi-OAcE1tKS85sKS-i7M02HrqFpchEji8CofSLT1j0vq3u3Xg22bmpRv_ZUhr8TTAdbdidXyyGKSeWD26xQVfoc29_l2exM1JmSIXGMkfD5fjAwVa_rCQZl7ddYlP-8a5NRJtHG80640Zu4WaBu6x6H44ciyhe-9MWdC54jPCOFEB5EXk6bfw81gI3kduTYh8qvObLOXOKT_g%3D%3D&attr13:57
Saviqedirects=0 suggests kinda otherwise ;)13:57
Saviqjeez google13:57
Saviqtsdgeos, sounds like we'll have to tweak ScopeStyle to actually take non-100%-opaque colors into account, I think I made it to just ~crap out when it encounters a color like that, which kinda makes sense13:58
Saviqtsdgeos, but maybe we should just ignore alpha is all13:59
tsdgeosmaybe13:59
tsdgeosnot a color physicist myself :D13:59
Saviqtsdgeos, well, it doesn't make real sense because we'd have to sample the image below13:59
Saviqtsdgeos, but maybe better this way, people will be able to "hint" at what the image color is and get the text readable on top anyway14:00
tsdgeoscan they do that per image?14:01
Saviqtsdgeos, yes14:02
tsdgeosok14:02
Saviqtsdgeos, every scope has its own overlay color14:02
tsdgeosrandom thought14:02
tsdgeoshow much faster would be things if we use === instead of == ?14:02
tsdgeosprobably not much14:02
Saviqtsdgeos, probably depends14:02
Saviqtsdgeos, but == does implicit conversions, so *somewhat* faster for sure14:03
tsdgeosmhr3: Saviq: active bit set14:03
tsdgeosplz pull and check it works14:03
Saviqtx14:03
tsdgeosSaviq: mhr3: which thing are you seeing "wrong" in overview?14:03
Saviqtsdgeos, apps14:03
Saviqtsdgeos, white on white14:03
mhr3tsdgeos, where did you push?14:04
tsdgeosmhr3: both dash_overview and do_merge14:04
tsdgeosSaviq: hmmm i may have an old apps package, i still have the old "big A" scope14:05
tsdgeosSaviq: where are you getting the new one?14:05
Saviqtsdgeos, get silo 11 (and 1 for api)14:05
Saviqtsdgeos, not store, apps14:05
tsdgeossure14:05
tsdgeosthe apps logo is an A14:05
mhr3tsdgeos, i don't see active being set14:05
Saviqtsdgeos, ah yeah, you don't have scopes14:05
Saviqtsdgeos, silo 1114:06
mhr3tsdgeos, you sure you pushed it?14:06
tsdgeosmhr3: maybe not, try now14:06
Saviqtsdgeos, searches now, after searching no scopes though14:07
tsdgeosSaviq: what do you mean after searching?14:08
Saviqtsdgeos, if I go back from search14:09
Saviqtsdgeos, favorites are empty14:09
Saviqtsdgeos, only load again after I reopen the overview14:09
tsdgeosmhr3: did you change anything in that regard?14:11
mhr3tsdgeos, http://paste.ubuntu.com/7980055/14:13
mhr3tsdgeos, no14:13
mhr3tsdgeos, apparently something resets the isActive14:14
tsdgeosmhr3: which one?14:14
mhr3tsdgeos, which what?14:15
tsdgeosmhr3: the isActive of which scope, there's more than 114:15
mhr3tsdgeos, the overview scope14:15
mhr3tsdgeos, "scopes" is the overview scope14:15
tsdgeosright, it's being changed14:16
tsdgeoshmmm, why14:16
tsdgeosit's not me14:17
mhr3tsdgeos, looks like there's only one more binding to isActive... must be that14:18
tsdgeosit's not me14:18
tsdgeoshttp://paste.ubuntu.com/7980082/14:18
tsdgeosthis is what the binding returns14:19
tsdgeosand when isActiveChanged changes14:19
tsdgeoschanges to false without my binding even being invoked14:19
tsdgeosplugin code doesn't have any setActive call :/14:20
mhr3tsdgeos, GenericScopeView has14:20
tsdgeosmhr3: but generic scope never gets the scopes overview14:21
tsdgeosah wait14:21
mhr3are you usre?14:21
tsdgeosit does for the search results :D14:21
mhr3heh14:21
tsdgeosok, will push14:22
tsdgeosbut there seems to be a huge refresh of scopes when going back14:22
tsdgeosas if the model got rebuilt or something14:23
mhr3it gets reset14:23
tsdgeoswhy?14:24
tsdgeospushed14:24
=== alan_g|lunch is now known as alan_g|tea
tsdgeosSaviq: in which of the bazillion branches do you want the fix for overlaycolor?14:27
CimiSaviq, how do I pass the field (either value, properties) from the settings widget to the listview?14:27
mhr3Cimi, so... coming to the office?14:28
mhr3Cimi, it's about time...14:28
Cimimhr3, cannot make it...14:28
Cimimhr3, really busy with this settings stuff14:29
Cimiand reviews I have to do for nick14:29
Saviqtsdgeos, just do it in overview, it's fine there14:29
mhr3Cimi, you could have done those in a bar14:29
mhr3:)14:29
SaviqCimi, you don't need to pass it to the list view, you just need it in the delegate14:30
SaviqCimi, you can just pass the whole "model" object down to the particular widget14:30
CimiSaviq, wanted to centralise the code14:31
CimiSaviq, otherwise I have to have one action per widget14:31
SaviqCimi, then signal up14:31
SaviqCimi, or alias the value14:31
CimiSaviq, I am signalling up14:31
SaviqCimi, but since you need to sync both ways, signal up, pick up in the factory and you're done14:32
CimiSaviq, but from listview I have like triggered (string settingId, var field, var newValue)14:32
MacSlowdednick, stand-up :)14:33
SaviqCimi, why settingId / field? don't you have those per model row?14:33
CimiSaviq, forget settingId14:33
CimiSaviq, I can add or not14:33
Cimiit's my signal14:33
Cimiso I have like a SettingWidgetSwitch14:33
Cimiwhere I can emit a signal14:33
CimiI am changing model.value in this case14:34
Cimisometimes I have to change model.properties14:34
=== alan_g|tea is now known as alan_g
Cimiso how do I tell the listview to change model.value or model.properties?14:34
SaviqCimi, model.value = 014:35
SaviqCimi, why would you need to change model.properties?14:35
Saviqpete-woods, you had a dummy QML using the scope settings API, do you have that around still?14:36
MacSlowtsdgeos, you've filed bugs upstream against Qt before I bet :) would the isolated qml-test and the description from https://bugs.launchpad.net/sync-monitor/+bug/1353550 be enough to be filed against Qt's upstream bug-tracker?14:44
ubot5Ubuntu bug 1353550 in sync-monitor "Contact sync notification icon contains dialer app icon in the background" [Medium,In progress]14:44
tsdgeosMacSlow: you need to set sourceSize14:44
tsdgeosso svg gets it's nice scaling rendering14:45
Saviqtsdgeos, doesn't actually help in this case14:47
tsdgeosok14:48
tsdgeosSaviq:14:48
tsdgeos        if (headerAsOverlay) {14:48
tsdgeos            color = '"white"';14:48
tsdgeos}14:48
tsdgeosso yes, it's always white14:48
tsdgeos:D14:48
Saviq;)14:48
SaviqMacSlow, in any case, you need to not use UbuntuComponents14:48
MacSlowSaviq, right14:48
SaviqMacSlow, when sending bugs up to Qt, and generally package everything up, with the assets, so that they can reproduce it without looking anywhere else14:48
MacSlowSaviq, sure14:49
Saviqtsdgeos, so yeah it needs to be luminance(overviewColor) > style.threshold ? style.dark : style.light14:49
Saviqor so14:50
tsdgeosSaviq: http://paste.ubuntu.com/7980300/14:50
Saviqtsdgeos, yeah, we should have a talk about default values for scopeStyle :P14:50
tsdgeos:D14:50
tsdgeosanyway that looks nicer14:50
tsdgeosi'll push14:50
Saviqcoolz14:50
tsdgeosSaviq: pushed to do_merged only14:52
Saviqtsdgeos, hmm hmm, still white here...14:53
tsdgeosweird14:53
tsdgeosi tried tryCard and it worked fine14:53
tsdgeosSaviq: aren't we sure the style is borked?14:53
Saviqtsdgeos, I'm not sure the overlay color makes sense, checking14:54
tsdgeosSaviq: did you make?14:54
tsdgeosthat code is stupid and needs a make to happen14:54
Saviqtsdgeos, yeah I did14:54
tsdgeosok14:54
Saviqtsdgeos, I probably don't have mhr3's latest changes14:57
tsdgeoslet's wait if tomorrow stuff settles a bit more14:57
tsdgeosSaviq: i'm not sure what to do with the hint14:57
tsdgeosdoes it have to be there all the time?14:57
tsdgeosthats what spec says14:58
Saviqtsdgeos, it's only meant to be in when header is fully in14:58
mhr3tsdgeos, pushed? still white-on-white for me14:58
Saviqmhr3, made?14:58
Saviqmhr3, you need to send a white color in the overlay for apps14:58
Saviqmhr3, or a light one at least14:58
mhr3Saviq, i am here :)14:58
Saviqtsdgeos, yeah, spec is old, I discussed it with Mike, just use a white Rectangle for now until we get the asset14:59
dednickSaviq: can you check https://code.launchpad.net/~nick-dedekind/ubuntu-settings-components/uqmlscene/+merge/229081 when you get a second please?14:59
Saviqtsdgeos, pop it in when header fully shown (if you can?), hide it otherwise14:59
Saviqdednick, will look14:59
Saviqdednick, description and commit msg probably could use an update14:59
tsdgeosSaviq: i do not understand "only meant to be in when the header is fully in"15:00
Saviqtsdgeos, page header15:00
Saviqtsdgeos, fully visible15:00
Saviqtsdgeos, then show hint15:00
Saviqotherwise hide hint15:00
tsdgeosno transition at all?15:00
tsdgeospagehader 99% visible -> no hint?15:00
Saviqtsdgeos, transition yes, but not proportional15:00
Saviqtsdgeos, you can make it header > 80% visible or something15:01
Saviqtsdgeos, but don't "connect" the two in terms of how visible they are15:01
tsdgeosthat's going to be distracting as hell15:01
Saviqtsdgeos, sure15:01
Saviqtsdgeos, but that's what we want for now15:01
tsdgeosMacSlow: what am i supposed to see in that example?15:02
tsdgeosthat Qt SVG renderer sucks?15:02
tsdgeosthat's a won't fix15:02
Saviqdednick, do we need all the changes TestCase → UbuntuTestCase?15:02
dednickSaviq: well we definately need the import Ubuntu.Test.15:03
dednickdoes it load if you dont use it?15:03
Saviqdednick, sure15:04
Saviqdednick, import gets you the singleton15:04
mhr3tsdgeos, so yea, active works fine now, but still white-on-white for me15:04
Saviqmhr3, what overlay color are you sending for apps?15:04
mhr3Saviq, ffffffffffffffff15:04
mhr3and a few more fs15:05
Saviqmhr3, you missed a #15:05
mhr3"overlay-color": "#ffffffff"15:05
tsdgeosmhr3: what about the scopeStyle?15:06
Saviqtsdgeos, in theory we should handle the defaults ourselves15:06
mhr3tsdgeos, what about it?15:06
tsdgeosSaviq: we handle defaults, but of scopestyle says white and white15:06
tsdgeoswe don't have a choice of what to use, no?15:07
dednickSaviq: hm. actually, if i use UbuntuTestCase i can get rid of the utils.js script which gives the findChild function15:07
Saviqtsdgeos, it should say grey on white grey or something15:07
Saviqdednick, yup, that you can15:07
Saviq*light grey15:07
Saviqtsdgeos, default isn't white on white15:07
Saviqmhr3, you have a branch do you>?15:07
mhr3Saviq, pushed it to the overview-icons one15:07
Saviqkk, checking15:08
mhr3Saviq, don't forget to restart scope-registry15:08
mhr3Saviq, and brace yourself for the session crash :P15:08
pete-woodsSaviq: http://bazaar.launchpad.net/~unity-team/unity-scopes-shell/trunk/files/head:/tools/settings/15:08
SaviqCimi, ↑15:08
SaviqCimi, so you still need to *set* model.value15:11
SaviqCimi, properties only gets you the list items15:11
=== jgdxx is now known as jgdx
tsdgeosSaviq: this handle thing is not trivial at all, lvwph doesn't give that info out15:15
tsdgeosi need to expose it15:15
tsdgeosand find from where to expose it15:16
Saviqtsdgeos, yeah, I didn't think it was trivial15:17
tsdgeosoh, i actually have m_headerItemShownHeight15:17
Saviqtsdgeos, another interesting question is whether it's supposed to be in Dash or GSV15:17
tsdgeosnice15:17
tsdgeosSaviq: dash since it needs to also show in the temp scope15:18
Saviqtsdgeos, well, temp scope is GSV, too15:18
tsdgeossure15:18
dednickSaviq: ok. using UbuntuTestCase for findChild, but changed rest to TestCase15:18
Saviqdednick, tx15:18
tsdgeosSaviq: but then how do you position it without access to the "overview edge"15:19
tsdgeosand having more than one "overview edge" seems clumsy15:19
Saviqtsdgeos, I don't think it should position at overview edge at all15:19
Saviqtsdgeos, it should just be stuck to the bottom of Dash15:19
tsdgeosreally?15:19
tsdgeosno movment at all?15:19
tsdgeosi understand you pull from it15:20
tsdgeosand thus moves15:20
mhr3Saviq, are the collapsible widgets in u8 yet?15:20
Saviqtsdgeos, yeah, but what do you pull from below15:20
mhr3tsdgeos, and table?15:20
Saviqmhr3, landing15:20
Saviqmhr3, table is separate, not landing yet15:20
Saviqsoon as possible15:20
Saviqtsdgeos, I feel like in this case it should just hide when you touch it15:20
tsdgeosmhr3: needs a reviewier, you can do your last minute of glory and approve it :D15:20
Saviqtsdgeos, because you're not pulling anything from below15:20
tsdgeosSaviq: well still i need the edge info15:21
mhr3pstolowski, pete-woods, thoughts on how collapsible widgets should be used on the scope side?15:21
Saviqtsdgeos, right, sure, but I think your idea was to put it in Dash, too, so what's the problem? ;D15:22
tsdgeosnone15:22
tsdgeosi'm just saying i don't see the need/benefits of GSV15:23
Saviqah yeah, there's probably even arguments against15:23
Saviqlike in GSV we'd get it in temp scopes over dash by default, too15:23
Saviq+overview15:23
pete-woodsmhr3: I don't really know what they are..15:23
pstolowskimhr3, exactly, can you elaborate / link to the design?15:24
Saviqtsdgeos, you've got mail (the tab)15:24
mhr3pstolowski, pete-woods, a container widget that hosts one or more existing widgets... and is collapsed by default15:24
pete-woodsin previews? results?15:25
SaviqIT'S MERGED!15:25
Saviqbuahahahahaha15:26
Saviqtsdgeos, can I reconf with dash_overview branch then?15:26
tsdgeosSaviq: i push do_merged over dash_overview15:26
tsdgeosso we can keep the merge reuqest?15:27
Saviqtsdgeos, ok, merge trunk, too15:27
mhr3pete-woods, yea15:27
Saviqtsdgeos, yup15:27
mhr3pete-woods, yea, previews :)15:27
=== gatox is now known as gatox_lunch
pete-woodsoh, is the thing Ruby wanted for showing more results inside the preview? (e.g. for related videos on youtube)15:28
tsdgeoswhat why did i get conflicts on merge?¿?¿?¿15:28
pstolowskimhr3, PreviewWidget::addWidget(..); PreviewWidget::addColumnLayout(..).. but I'm not sure where it leads us json-wise15:28
tsdgeosah15:29
mhr3pete-woods, nope, not that15:29
tsdgeosthe old --weave friend15:29
mhr3pete-woods, trying to find a spec with it15:29
pete-woodsokay15:29
mhr3pete-woods, pstolowski, https://f966f709-a-c881af26-s-sites.googlegroups.com/a/canonical.com/unity8dash/scopes/store/3-previews/preview-v2-info.png?attachauth=ANoY7cqxptZny07dqI-cNzSEy2VSN9PfjwDYJN5JYQeU5SNOkhI4MNxosUIPpeVMzx2TEyya3ccZyES7X3z68EGAV1sHsKXXO-NaZeNRbgSd5o-bEpbF_A-7w7HIMwQKZeYq5xGV3GYSKaiNjbBlSK4317gcUd4BVUM6AqR7E-kahto1kNNMhwlBpde5AN96_OQzJtZIYg9Ns7IP-6J5splHBzeA7eGEeXsQs3QfO8t_1IrdtDL1zMkWS4lTwuv1bbkuRSIdKO6T&attredi15:30
mhr3rects=015:30
Saviq@unity: merge trunk in your branches, you're gonna need it ;)15:30
mhr3eh15:30
mhr3pstolowski, pete-woods, http://goo.gl/5xjs3c15:30
mterry:)15:30
dednickSaviq: ?15:31
dednicksilo 1 land?15:31
Saviqdednick, yup15:31
CimiSaviq, only small issue now15:31
mhr3pstolowski, pete-woods, 3 expandable widgets, 1st has text inside, 2nd has a table, 3rd has rating+review widget15:31
CimiSaviq, on the loader onLoaded I send the binding model.value (that stores the value)15:31
Saviqs/send/set/15:32
mhr3pstolowski, pete-woods, actually, first has a few more widgets - http://goo.gl/8rYBdN15:32
CimiSaviq, when first loading the settings you see for example checkboxes animating from false to true to reflect new value15:32
Cimihow do we overcome this behaviour?15:32
SaviqCimi, SDK issue, I don't think we ever solved that TBH15:32
SaviqCimi, ask timp15:32
mhr3pstolowski, pete-woods, text, text, table, i suppose15:32
tsdgeosonto the next one!15:34
pete-woodsmhr3: okay, so we need to figure out what the preview API looks like for this? that's the important question, right?15:36
pete-woodswe seem at risk of building a full UITK layout system15:37
mhr3pete-woods, well the preview api is variant-based, so mostly a question of how to expect it from the scopes15:37
pete-woodsI do mean on the scopes end15:37
mhr3pete-woods, i'm just saying that this shouldn't affect the lib api15:37
pete-woodsthe sPreviewReplyProxy I guess15:37
mhr3unless we want some real nice support for it15:38
Saviqtsdgeos, did you push to dash_overview yet?15:38
tsdgeosSaviq: yes sir15:38
tsdgeos9 minutes ago! :D15:38
=== om26er is now known as om26er|brb
mhr3pete-woods, pstolowski, but anyway, asking you cause you've implemented previews in scopes, will know what would feel consistent15:40
pstolowskimhr3, shouldn't affect lib api? hmm.. that leaves very little options15:40
Saviqmhr3, https://code.launchpad.net/~unity-team/unity-scope-click/overview-icons/+merge/22773115:40
Saviq48+LogoOverlayColor=#2600000015:40
Saviqyou sure?15:40
Saviqthat looks quite dark to me :P15:41
pstolowskimhr3, is this for rtm?15:41
mhr3Saviq, 15% opacity, that's what josh said15:41
Saviqmhr3, of *what*15:41
mhr3Saviq, black15:41
mhr3Saviq, looks quite mild to me15:41
Saviqmhr3, and you want us to find out that we should use a dark text color over that how?15:41
mhr3Saviq, no, it's black, you should use light15:42
mhr3Saviq, apps has fffffs15:42
Saviqmhr3, grr ok, that's store15:42
Saviqmhr3, and do we really want #fff on apps?15:43
* Saviq thinks unlikely15:43
mhr3Saviq, hm, let me recheck15:43
mhr3there was a complicated rule15:43
Saviqmhr3, btw, works fine15:44
cwaynewoo big-ass merge-fest landed15:44
Saviqcwayne, indeed ;)15:44
Saviqmhr3, http://people.canonical.com/~msawicz/unity8/Zrzut%20ekranu%20z%202014-08-07%2017:44:05.png15:44
Saviqmhr3, if we wanted to change the appearance of overview (we should probably change the fore/background colors to match, otherwise stuff might be weird)15:45
Saviqmhr3, where do we do that?15:45
mhr3Saviq, scopes scope15:46
Saviqmhr3, for all/favorites, too?15:46
mhr3Saviq, yes15:46
Saviqmhr3, kk15:46
Saviqthis is starting to look usable15:47
Saviqtsdgeos, overlay text is fine now15:47
mhr3Saviq, weird, it's still white-on-white for me15:47
Saviqmhr3, you didn't build something15:47
mhr3maybe15:47
Saviqmhr3, can you change scopes scope to send white/272727 for fore/background?15:49
mhr3Saviq, so from josh:15:50
mhr3White cards only*: 100% #FFF / RGB 255, 255, 25515:50
mhr3(*This refers to the slight gradient on white e.g. eBay and Amazon ;-), but not to things like Open Library where it's just a light colour)15:50
Saviqmhr3, yeah, I don't understand that ;)15:51
mhr3and apps is neither really :P15:51
MacSlowtsdgeos, Saviq, https://bugs.launchpad.net/sync-monitor/+bug/1353550/comments/5 marked it as "won't fix".15:51
mhr3not a light colour, nor a slight gradient15:51
ubot5Ubuntu bug 1353550 in sync-monitor "Contact sync notification icon contains dialer app icon in the background" [Medium,Won't fix]15:51
tsdgeosmy computer hard locked15:51
tsdgeoshad not happened in a while15:51
tsdgeosSaviq: good stuff15:51
mhr3Saviq, and yea, changing scopes scope15:51
tsdgeosSaviq: so silo 11?15:51
Saviqtsdgeos, yup, building15:52
SaviqMacSlow, yeah, we kind of knew that15:52
mhr3mhall119, could you update scopes docs pls?15:52
mhall119mhr3: might take me a little longer today, I'm working on the API website code so my importers are our of sync with the production site15:53
SaviqMacSlow, we can't let Qt render any complicated SVGs, we're even gonna drop them from the theme altogether15:53
SaviqMacSlow, and apps need to just ship PNGs with them15:53
mhr3Saviq, http://paste.ubuntu.com/7980788/15:54
mhr3ack?15:54
Saviqmhr3, just drop the alpha15:54
Saviqmhr3, and can't you say "white"?15:54
mhr3won't it blow up?15:54
Saviqmhr3, no, it's #rgb / #argb15:54
mhr3Saviq, i just pass it to you15:54
mhr3so i guess i can :)15:54
Saviqmhr3, yup, just do white / #27272715:54
mhr3http://paste.ubuntu.com/7980796/15:54
Saviqmhr3, +115:55
mhr3Saviq, pushed15:56
tsdgeosSaviq: can you add https://code.launchpad.net/~aacid/unity8/scopeSearchHintText/+merge/228655 to the silo?15:56
tsdgeosis a very small one15:56
tsdgeosor we can leave it out if you want15:57
tsdgeosno biggie15:57
Saviqtsdgeos, I could, but it conflicted15:59
Saviqtsdgeos, merge it please and I will15:59
tsdgeosSaviq: it conflicted with what?16:01
tsdgeosit is merged on both the branches you are merging16:01
tsdgeoscan't conflict16:01
Saviqtsdgeos, with trunkl16:02
Saviq-l16:02
Saviqtsdgeos, https://ci-train.ubuntu.com/job/landing-011-1-build/142/console16:02
Saviqtsdgeos, maybe it doesn't any more16:02
tsdgeoshttp://paste.ubuntu.com/7980861/16:02
tsdgeosyeah 1093 was before my last merge16:03
tsdgeosin 109416:03
Saviqtsdgeos, sure, I just didn't know you updated it in the mean time16:04
tsdgeos:)16:04
Saviqwhat you so happy about!? ;P16:04
Saviqmhr3, so stuff works fine in silo 11 already, I just pushed a rebuild of unity8 and scope-scopes with one more MP added to unity8 and your fore/background change16:14
* Saviq gtg16:14
Saviqmhr3, so consider your last day successful, have good drinks and don't be a stranger!16:14
Saviqsee you back around in a year or so ;)16:15
mhr3Saviq, heh, thx man, it's been an honour16:15
Saviqo/16:15
olliSaviq, greyback who implemented the swipe to close in the spread?16:16
greybackolli: combinatino of tsdgeos & mzanetti I believe16:17
ollioki16:17
mzanettiyeah, me16:17
greybackmhr3: you off? In that case, best of luck!16:17
ollimzanetti, I noticed the other day that the top surface in the spread doesn't get updated16:18
mzanettimhr3: o/16:18
CimiSaviq, tags in trunk!16:18
olliwhile the lower one (with 2 apps open) does get updated when looking at the spread16:18
olliis that something known/worth a bug?16:18
mzanettiolli: well, that's our app lifecycle16:18
mzanettiolli: we freeze unfocused apps16:18
ollid'oh16:18
ollistupid me16:18
mzanettino worries16:19
mhall119mhr3: please check http://91.189.92.89/api/scopes/sdk-14.10/ carefully to make sure it's still correct, as I said my importer had gotten out ot sync with production so I had to undo some changed before running it16:19
ollimzanetti, if it wasn't for app life cycle, would both surfaces update?16:19
mzanettiolli: yes16:19
ollinice16:19
* olli envisions 2 different media players in the spread16:19
mzanettimight get a bit chaotic :) but yeah, would work as long as CPU/GPU can handle it16:20
mhr3mhall119, doesn't seem to be the latest version16:20
Cimi@unity be sure to remove tag 0.1.16 from your branches if you merged latest trunk16:21
mterryguh16:21
mhall119mhr3: ah, yes, it still had 0.5.3+14.10.20140728, I thought I pulled the latest before running16:23
mhall1190.6.0+14.10.20140806 is the latest, correct?16:23
olliSaviq, it seems a bit odd that the dash is in the spread16:23
mhr3mhall119, yep16:23
mhall119ok, re-running16:23
ollias it behaves different than everything else (can't swipe to kill;)16:24
olliI assume this is something design is aware of and decided consciously?16:24
=== om26er|brb is now known as om26er
mhall119it makes a certain kind of sense, the desktop is in alt-tab on the desktop afterall16:26
mhall119though not the window spread16:27
mhall119mhr3: please check http://91.189.92.89/api/scopes/sdk-14.10/ again16:28
mhr3mhall119, yep, that looks better16:28
mhall119mhr3: pushing to production now16:30
=== gatox_lunch is now known as gatox
mhall119mhr3: please verify http://developer.ubuntu.com/api/scopes/sdk-14.10/16:32
bregmaso the latest unity-plugin-scopes breaks dependencies in the archive and forces the removal of unity8 on the desktop16:42
bregmait's because unity-plugin-scopes no longer provides unity-scopes-impl-2 virtual package16:42
dednickSaviq: https://launchpadlibrarian.net/181743045/buildlog_ubuntu-utopic-amd64.unity8_8.00%2B14.10.20140807.1-0ubuntu1_FAILEDTOBUILD.txt.gz ?16:43
dednickSaviq: has silo1 unity8 landed?16:43
bregmaOK, the archives have been updated so the depdendency break is gone16:53
dednickbeer'o clock! cya all.16:59
=== charles_ is now known as charles
mzanettiis there a place we play music/sounds in unity8 (Note: unity8, not unity8-dash)17:12
mzanettigreyback: ^ you aware of something?17:13
mzanettitalking about this one: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/135409017:13
ubot5Ubuntu bug 1354090 in unity8 (Ubuntu) "/usr/bin/unity8:11:begin:core::Signal:core::dbus::Signal:operator:core::dbus::MessageRouter" [Undecided,New]17:13
mzanettistack trace leads to MediaPlayer backend17:13
=== dandrader is now known as dandrader|lunch
=== boiko_ is now known as boiko
elopioSaviq: do you know where can I translate the messaging user metrics that appear on the lock screen?17:36
mzanettiCimi: hey, still around?18:22
mhall119bregma: do you still want us to get this convergence survey out?18:29
bregmamhall119, yes please18:30
mhall119is that a +1 on the questions in it?18:30
bregmamhall119, yes, I believe they're fine for what we need at this point and as a basis for a possible follow-on18:31
mhall119bregma: ok, we'll get together a list of target respondents and start getting it out18:32
bregmavery exciting18:32
mhall119bregma: were you working with anybody to get these kinds of answers from existing OEM or ISV partners?18:33
bregmamhall119, I was poking Stéphane Verdy, which reminds me I need to follow up on that18:34
=== dandrader|lunch is now known as dandrader
Cimimzanetti, yup19:18
cwayneCimi: do you have any working debs for unity8 with settings?19:21
Cimicwayne, https://code.launchpad.net/~cimi/unity8/scope-settings/+merge/22999519:24
Cimicwayne, let's wait jenkins for pkgs19:24
cwayneCimi: thanks19:24
Cimicwayne, for the moment there are only scopes with boolean settings, and those should "work"19:24
Cimicwayne, tomorrow I have to write tests and I'll mock the others...19:25
SaviqCimi, just two, ones known to come around time and again ;)19:33
Saviqolli, yes, that was a conscious decision, cutting the number of different cases for the right edge from 72 I think to 2719:34
Saviqin the tablet case19:34
Saviqelopio, with the new infographics system that we're trying to land for like half a year now, it's going to be the visualizer's responsibility to give up properly localized images for the infographics19:35
Saviqelopio, see https://code.launchpad.net/~unity-team/libusermetrics/file-based-infographics/+merge/214020 for reference, there is one visualizer there already (the default one)19:35
* cwayne still wants that to land19:36
CimiI guess I should update that branch19:37
elopiowow, that's a 10000 lines diff19:37
Saviqelopio, yeah, it's a complete rework19:38
Cimielopio, mostly svg19:38
elopioSaviq: that sounds cool. But what I need now is to make sure that the labels appear in spanish. Is that possible?19:38
Saviqelopio, the apps are writing those labels19:38
Saviqelopio, in the current system19:38
Saviqelopio, but we will land this before RTM19:38
Saviqelopio, so I'm not sure it's worth the effort19:38
mzanettiCimi: can you try to copy this on to your phone and see if the spread closing behaves ok for you: http://paste.ubuntu.com/7982317/19:39
elopioSaviq: isn't the beta freeze today?19:39
mzanettiCimi: talking about this one: https://bugs.launchpad.net/unity8/+bug/135080319:39
ubot5Ubuntu bug 1350803 in Ubuntu UX "[Spread] Threashold in spread for closing apps should be increased" [High,Triaged]19:39
Saviqelopio, it was yesterday19:39
elopioI get scared to see all the things that will go into RTM and haven't landed yet.19:39
Saviqelopio, yup, plenty of things are late19:40
Cimimzanetti, path of the file to replace?19:40
mzanettiCimi: /usr/share/unity8/Stages/SpreadDelegate.qml19:40
Cimithx19:40
elopioSaviq: ok, I'll wait then. Anyway I looked everywhere on the messaging app and couldn't find the metrics labels.19:41
Saviqelopio, I think it's in the telephony service19:41
Saviqelopio, http://bazaar.launchpad.net/~phablet-team/telephony-service/trunk/view/head:/indicator/metrics.cpp19:42
elopioof course. I looked everywhere except there :)19:43
Saviqelopio, they should be there in camera as well, not sure if the new camera app got them at all thought19:43
Cimimzanetti, there is some weird behaviour19:44
elopioSaviq: so, let me see if I got it straight. Once Wellark's branch lands, these metrics.cpp file needs to be updated to use the new library?19:44
Cimimzanetti, when you just tap19:45
elopios/these/this19:45
Cimiyou see it wobbling19:45
Cimilet's talk about it tomorrow19:45
Saviqelopio, there's a compatibility layer https://code.launchpad.net/~unity-api-team/telephony-service/file-based-infographics/+merge/22410519:45
Saviqelopio, but I19:45
Saviq'm not sure the labels get through19:45
Saviqelopio, and it's pete-woods, best talk to him when he's around tomorrow19:45
Saviqelopio, I know very little about the client side of this19:46
Cimimzanetti, I see there is an improvement19:47
Cimibut also it loses the finger19:47
Cimiit like jumps19:47
mzanettiCimi: yeah... the thing is, if you swipe quickly from left to right, you make a movement up with your thumb19:48
Cimimzanetti, I can review the code eventually19:48
mzanettiCimi: There's no point in increasing the close threshold because the finger movement will always be fast/long enough19:48
Cimimzanetti, you can require a fixed amount of gu19:49
mzanettiCimi: so what I tried now is to cancel the up/down gesture when I detect a left/right movement19:49
mzanettiCimi: I already require 8gus19:49
Saviqbregma, damn, again we killed desktop session, sorry about that :|19:49
Cimilike, you cannot close if you swipe for less than 2gu19:49
Cimialready 8??19:49
mzanettiCimi: yes19:49
Cimiit feels less19:49
mzanettiCimi: that's what I'm saying19:49
elopioSaviq: ok, thanks for the info. I'm glad I asked :)19:49
CimiI was able to swipe woth a quick touch19:49
mzanettiwhen this happens you're doing fast movements19:49
Cimireally short19:49
Cimi8gu is like a pageheader?19:49
mzanettiyes19:50
Saviqbregma, looking at unity8-desktop-session, you managed to use the upstream upstart jobs by now?19:50
Cimiin that case we might want 1019:50
mzanettiCimi: doesn't matter19:50
bregmaSaviq, I'm going to have an MP soon for Unity8 to (a) fix the missing Dash and (2) switch to using the Unity8 upstart job exclusively19:50
mzanettiCimi: at some point it just get annoying to really close apps19:50
Saviqbregma, yeah I see it already, that'd be great19:50
mzanettiCimi: but increasing that threshold is not the solution19:50
bregmahopefully that can land at the same time as the unity8-desktop-session MP19:50
Cimiwell you kill less apps19:50
Saviqbregma, maybe it starts to belong in lp:unity8 by now?19:50
Cimilet's discuss tomorrow though19:51
bregmafixes a lot of divergency issues19:51
Cimii want to test vanilla better19:51
Cimiand I am quite tired19:51
mzanettiok19:51
Cimilet's do hangout/mumble19:51
Cimia domani!19:51
bregmaSaviq, there are still issues between running on desktop and running on Touch that need to be neatly resolved, so one more round of separate sources for now19:52
Saviqbregma, mhm ok19:52
bregmamostly because several iterations of tweaking won't interfere with phone work that way19:53
Saviqmterry, re: laggy volume20:25
Saviqmterry, it was quite reliable for me, until I started playing something and beating the sh$t out of the slider again20:25
Saviqmterry, it started catching up late at that point20:25
mterryplaying something.... o20:27
mterryk20:27
=== salem_ is now known as _salem
Skee_I would like to learn delvelopment. I enjoy the Ubuntu Operating System and would like to give back..I could I learn22:44

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!