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

kgunnelopio: its these three00:00
kgunnhttps://code.launchpad.net/~unity-team/unity8/dash-as-app/+merge/22853400:00
kgunnhttps://code.launchpad.net/~mzanetti/unity8/drop-running-apps-from-dash/+merge/22770700:00
kgunnhttps://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/22912300:00
elopiokgunn: the first one is already merged, so I guess it shouldn't be part of the silo.00:16
elopioit shouldn't affect either.00:17
elopioI have merged trunk with the other two and it has no conflicts.00:17
elopiomaybe the latest merge I did improved things? Can you try again??00:17
kgunnelopio: hmmm, musta been merged with saviqs00:18
elopiokgunn: sorry, it's the second one the already merged.00:18
kgunnelopio: can you merge back trunk once more ? i bet you probably did it right before all the unity8 stuff migrated...00:19
kgunnwe caught it on the edge00:19
elopiokgunn: http://paste.ubuntu.com/7919046/00:20
elopioalso http://paste.ubuntu.com/7919049/00:21
elopiothis one I did it when I got your ping. It added some changes but was a clean merge.00:21
=== shiznix_ is now known as shiznix
=== _salem is now known as salem_
=== salem_ is now known as _salem
Saviqelopio, apps not launching make sense because of https://code.launchpad.net/~unity-team/unity-scopes-shell/drop-appid-workaround/+merge/22842707:47
Saviqelopio, we need a corresponding branch from the click scope07:47
Saviqthat will send us proper appid:///, applications:/// urls07:47
tsdgeosholy merge batman07:55
tsdgeostoo many branches take a while to merge :D07:55
Saviq:)07:59
Saviqit's been 13 MPs...08:00
Saviqmhr3, FYI: bcmwl builds again08:11
mhr3yey, so i can reboot again :)08:11
Saviqmhr3, yeah, upgrade first ;)08:12
tsdgeosSaviq: who do we get to merge https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/1349705 ?08:15
ubot5Launchpad bug 1349705 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Flickable gets confused when there's other flickables on top" [Undecided,New]08:15
tsdgeosotherwise merging in the overview is going to be a mess08:15
tsdgeosSaviq: dandrader: are we fine with top approving https://code.launchpad.net/~unity-team/unity8/dash-as-app/+merge/228534 that has every single autopilot test not passing?08:17
dandradertsdgeos, leo has a branch for it that fixes the AP tests08:18
tsdgeosah cool08:18
Saviqtsdgeos, there's the branch after it that fixes all those08:19
Saviqtsdgeos, let me find out08:19
Saviq(re: qt)08:19
tsdgeosstill 1 test broken08:19
tsdgeoshttps://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/22912308:19
tsdgeosbut much more acceptable08:19
Saviqtsdgeos, yeah, Leo reported that's a bug, shell locks up when you try to swipe dash in when app is still loading08:20
tsdgeosoki08:20
Saviqtsdgeos, for bug #1349705 I'll just prepare a package and put it in there08:20
ubot5bug 1349705 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Flickable gets confused when there's other flickables on top" [Undecided,New] https://launchpad.net/bugs/134970508:20
Saviqsame silo with overview, only problem is we'll need to smoke-test it08:21
tsdgeosdandrader: if there's https://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/229123 i guess you can top approve https://code.launchpad.net/~unity-team/unity8/dash-as-app/+merge/228534 again and put https://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/229123 as requisite in the description?08:21
tsdgeosSaviq: cool08:22
tsdgeosand not cool at the same time :D08:22
tsdgeosSaviq: ah now that you're here08:23
tsdgeosshouldn't08:23
tsdgeos                        if (!rendererLoader.expanded && !seeAllLabel.visible && target.collapsedItemCount > 0) {08:23
tsdgeosbe08:23
tsdgeos                        if (!rendererLoader.expanded && seeAllLabel.visible && target.collapsedItemCount > 0) {08:23
=== popey_ is now known as popey
tsdgeosin qml/Dash/GenericScopeView.qml '08:23
tsdgeos?08:23
Saviqtsdgeos, lookin'08:26
Saviqtsdgeos, no, it's only meant to filter the results if see all isn't there08:26
tsdgeosah08:27
tsdgeosok08:27
Saviqtsdgeos, because you can't see all the results in the grid (because of collapsed-rows: 2 for example)08:27
tsdgeosmakes sense08:27
Saviqand you can't expand it because you have no see all (because you have a header link, which "invalidates" see all)08:27
tsdgeosok :)08:29
tsdgeosSaviq: so what's our next target for landing stuff? dash_overview + friends?08:29
tsdgeosdo we have a silo08:29
tsdgeos?08:29
Saviqtsdgeos, we're doing dash as app first08:30
Saviqtsdgeos, then overview08:30
tsdgeosright08:31
tsdgeosmakes sense08:31
Saviqtsdgeos, do you have dash overview handy btw?08:38
tsdgeosSaviq: on the desktop yes, phone not up to date08:38
Saviqtsdgeos, desktop's fine08:38
Saviqtsdgeos, I'll forward you images for the favorites, could you please do a few screenshots at different aspect ratios?08:39
tsdgeosSaviq: what you mean different aspect ratios?08:40
tsdgeoschange the "screen" size?08:40
Saviqtsdgeos, yeah, landscape, portrait etc.08:42
Saviqtsdgeos, try out three or four sizes please08:43
tsdgeosSaviq: favorites and all or just favorites?08:43
Saviqtsdgeos, just favourites, in All we're not forcing aspect ratio yet (are we?)08:45
tsdgeosSaviq: no we're not08:49
tsdgeosSaviq: do you happen to know the paths of the files i have to overwrite?08:49
tsdgeosbecause they are not unity8's08:50
tsdgeosSaviq: i think i may need some extra packages08:53
tsdgeosmhr3: ping08:53
mhr3tsdgeos, pong08:53
tsdgeosmhr3: which files do i need to overwrite to show my own dash overview images08:54
tsdgeosfor applications, music, video08:54
mhr3tsdgeos, $ scopes-client musicaggregator "" | head08:54
mhr3icon field08:55
tsdgeosoki, tx08:56
mhr3tsdgeos, alt nav should be fully working now10:19
mhr3...ish :)10:19
tsdgeosmhr3: :D10:23
tsdgeosish?10:23
mhr3might be a bug here and there :)10:23
tsdgeosmhr3: tests don't pass10:29
tsdgeoscan't dpkg pacakge it10:29
mhr3tsdgeos, did you pull?10:30
tsdgeosah you just changed it10:30
tsdgeosmeh10:31
tsdgeosstill failing10:31
Saviqtsdgeos, I was thinking... when you look at a dash page, meaning you focused the dash and you are on a certain page... maybe we should just create all the delegates on that page, maybe delaying just the image fetching until they're on screen (to potentially save data)10:33
Saviqtsdgeos, creating/destroying delegates (and whole categories, for that matter) will always kick us in the a$$10:34
mhr3tsdgeos, can you pastebin LastTest.log?10:34
tsdgeosSaviq: yeah :/10:35
Saviqtsdgeos, and well, if you're looking at a dash page, we should not refrain from committing memory to it10:35
tsdgeosmhr3: http://paste.ubuntu.com/7922784/10:36
Saviqtsdgeos, as we unfocus, we should destroy all offscreen delegates (at least those vertically off-screen, not sure if we should destroy all that are not visible in the dash right now), but maybe not the categories10:36
Saviqtsdgeos, let's have a think/sleep & chat about it next week10:37
tsdgeos:(10:37
tsdgeos:)10:37
tsdgeosi mean10:37
Saviqlol10:37
Saviqtsdgeos, btw, does that make any sense to you http://pastebin.ubuntu.com/7922748/10:37
mhr3tsdgeos, weird, it's failing in location test... i didn't even touch that :P10:37
Saviqtsdgeos, that's what I got out of a local build with your patch10:37
tsdgeosmhr3: i need to more somewhere else :D10:37
tsdgeosSaviq: beats me10:38
mhr3tsdgeos, do you have python-tornado installed?10:38
tsdgeosi do10:38
mhr3pete-woods, why would that fail ^?10:38
mhr3tsdgeos, or you know... DEB_BUILD_OPTIONS="nocheck" bzr bd10:39
tsdgeostaht too10:40
pete-woodsmhr3: well the location tests only require tornado, and don't do anything clever outside10:42
Saviqtsdgeos, mhr3 is right, it built with 4.910:43
Saviq(at least probably)10:43
Saviqwe might need to force it to 4.8 for now and do a whole-set transition to 4.9 separately10:43
tsdgeosSaviq: what built with 4.9?10:44
Saviqtsdgeos, qtdeclarative, the symbol changes10:44
tsdgeosahhhh10:44
tsdgeosok, thought you were talking about unity-scopes-shell and the test issues :D10:44
Saviqsry10:44
dpmhi pete-woods, so thostr_ told me you're the man for adding a snippet for basic networking to the Qt Creator scopes template :)10:44
pete-woodsdpm: yep, I'm happy to take on that task when I have a minute10:45
pete-woodsI'd basically be gutting the one of the video scopes10:45
pete-woodsI'll probably keep a basic test case, too10:45
=== vila is now known as vila-lunch
dpmpete-woods, awesome, yes, that sounds like a plan. So if that helps, this is essentially the code we've got now http://bazaar.launchpad.net/~dpm/+junk/scopetemplate/files - you might want to use it as a basis or as you say, start from one of the video scopes10:46
dpmpete-woods, in addition to networking, it'd be good to have some basic code to process what the remote service returns (e.g. perhaps json, if that's the most common format?)10:47
pete-woodsdpm: yep, will do it with json10:47
dpmsounds good10:47
pete-woodsdpm: I'll put a basic python based http server in the test case to test it all10:47
dpmoh, that'd be a huge improvement from what we've got now, yes10:48
pete-woodsdpm: well I have that already in the video scope tests, so it should be very little effort to copy it over10:48
dpmgreat10:50
facundobatistaHola10:51
dpmpete-woods, do you have a rough idea of when you could have this template ready?10:51
pete-woodsdpm: it's a matter of priorities, if thostr_ says to do it right away, then pretty quick really10:52
dandraderSaviq, so dash-as-an app is not getting merged any time soon because of that AP test failure, right (which pointed to a regression)10:52
pete-woodsI'm currently managing a release of unity-scopes-api with ABI breaks, though, so after that10:52
Saviqdandrader, "any time soon" meaning until we fix it (we need to)10:54
Saviqdandrader, there's two issues I know of: stuff getting stuck when you switch between dash/app, and failure to launch apps from dash (I believe a click scope issue)10:55
dandraderSaviq, I mean that this will likely come before dash-as-app: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/135091710:56
ubot5Launchpad bug 1350917 in Unity 8 "UI freezes when switching to an app which is opening." [Critical,In progress]10:56
dandraderSaviq, or should, as it's a critical bug...10:57
Saviqdandrader, it can come *with* it, not necessarily before10:57
dandraderSaviq, ah, the silo thing10:57
Saviqdandrader, yup ;)10:57
dpmthanks pete-woods11:04
thostr_dpm: about pete's priorities11:05
thostr_dpm: so, first he needs to finish the unity-scopes release and afterwards he can spend some time on it... would it be ok for you if he'd just "throw" you some code over?11:06
dpmthostr_, I think so, I'm talking to Zoltan about when it'd be good for this to land on a QtC release. My hunch is that it'd be good to have the template early next week if possible, but let me see what he has to say11:09
dpmthostr_, pete-woods, so either after the unity-scopes release is finished (if that's today) or early next week would work well to include the template in QtC. Would that work for you?11:18
thostr_dpm: next week :)11:18
dpmok, let's follow up next week, then. Thanks!11:20
=== MacSlow is now known as MacSlow|lunch
=== vila-lunch is now known as vila
tsdgeosmhr3: ping11:27
mhr3tsdgeos, poing11:27
tsdgeosmhr3: can we kill visible in scopves-v4 ?11:28
tsdgeosor is anyone using it?11:28
tsdgeosSaviq: ↑↑11:29
mhr3tsdgeos, visible what?11:29
tsdgeosmhr3: Scope::visible11:29
mhr3tsdgeos, oh yea, let's drop it11:29
tsdgeosseems like a bastard favorite11:29
tsdgeosto me11:29
Saviq+111:33
mhr3tsdgeos, and that means you can drop the filter proxy model in shell right?11:48
mhr3yey!11:48
tsdgeosmhr3: hmmmm11:48
tsdgeoswe may need to add some stuff11:49
tsdgeosmhr3: we are using the filter proxy model to ask for the index of a scope in the model11:49
tsdgeosmhr3: do you think we can add to ScopesInterface ? or at least a count11:50
tsdgeosso i can just iterate over getScope(int row)11:50
mhr3it doesn't have that?11:50
tsdgeoscount not11:51
tsdgeosgetScope(row) yes11:51
tsdgeosbut without count i don't know where to stop11:51
tsdgeoswell i guess i could stop on null11:51
mhr3adding it, we have it on all models11:52
tsdgeoscool11:52
tsdgeoslunchy lunchy11:52
mhr3same11:56
jgdxlarsu, are you able to build gsettings-qt for armhf? I get dep issues I am unable to solve (first mesa, then gio2.0 dev)12:06
seb128jgdx, is that crossbuild?12:08
jgdxseb128, yes12:11
seb128jgdx, likely an archive issue rather a gsettings-qt one then...12:15
jgdxseb128, yeah, and jenkins build it seemingly fine for armhf.12:18
seb128jgdx, well, jenkins doesn't crossbuild12:19
jgdxseb128, it is all clear now12:19
seb128weird12:19
=== alan_g is now known as alan_g|lunch
jgdxseb128, no, "it's all clear to me now [why it passes on jenkins but fails for me]"12:20
seb128oh, ok12:20
jgdxseb128, this is new to me, how is archive issues like these resolved?12:20
jgdxs/is/are12:21
seb128by somebody looking what the issue is and fixing it12:21
seb128likely buggy multiarch support somewhere in the depends of what you try to install12:21
seb128xnox might be able to help you/give more hints on how to debug12:21
jgdxright, thanks12:21
xnoxhm?! =)12:24
* xnox reads back12:24
seb128xnox, gsettings-qt doesn't seem to be crossbuildable and jgdx is not sure how/where to report the issue12:24
xnoxseb128: by default, one should assume that cross-building has not been enabled. Most packages need tweaks to get it working.12:25
seb128xnox, well, in that case jgdx says he got gio not installable, that should work12:26
xnoxseb128: my laptop seems to lack cross-sbuild chroot, please wait =)12:27
seb128k12:27
seb128thanks for looking ;-)12:27
xnoxseb128: jgdx: all build-dependencies are coorectly installable for cross-compilation. However gsettings-qt appears to be using qmake, which is not supported build-system for cross-compilation.12:39
seb128oh, ok12:39
xnoxseb128: jgdx: please use CMake, or some other sensible build-system like autotools.12:39
seb128we have better things to do for rtm than changing build systems12:40
seb128so I guess that's just not cross buildable12:40
seb128it's not like we were changing it every week anyway, so shouldn't be a real issue12:40
xnoxjgdx: if you want to build it locally, $ mk-sbuild --arch armhf utopic; sbuild --arch armhf *.dsc should work to do "native" compilation locally, powered by qemu-user-static.12:41
=== alan_g|lunch is now known as alan_g
jgdxxnox, seb128, k, thank you12:48
seb128jgdx, yw!12:49
=== _salem is now known as salem_
tsdgeosmhr3: you removed visible but did not add count13:05
mhr3tsdgeos, just got back from lunch13:06
tsdgeossame here :)13:06
tsdgeosmhr3: ping me when you're done13:06
mhr3tsdgeos, pushed13:09
tsdgeoscool tx13:09
Saviqtsdgeos, ah, test reports should be good now... it was a lettercase issue13:21
tsdgeosSaviq: ?¿13:21
Saviqtsdgeos, Test vs test13:21
tsdgeoslol13:22
tsdgeosreally?13:22
Saviqyeah13:22
tsdgeosmterry_: answered your question13:22
tsdgeosSaviq: in our side or on CI side?13:23
Saviqtsdgeos, well, depends ;)13:23
Saviqtsdgeos, CI expected test, we had Test13:23
tsdgeosi mean there's no standard, is there?13:23
Saviqtsdgeos, but got fixed CI side13:23
Saviqnope13:23
tsdgeosdid they tell us?13:23
mterry_tsdgeos, cool updated13:23
Saviqtsdgeos, it was just miscomm, fginther added Test now, too13:24
tsdgeoscool :)13:24
Saviqtsdgeos, *but* it still didn't end up in http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/730/testReport/%28root%29/qmltestrunner/ yet13:24
Saviqtsdgeos, so we might need to follow up13:24
kgunndid we find another bug in qtdeclarative ? (dash as app silo)13:26
Saviqkgunn, found it earlier than that13:26
Saviqkgunn, and related to overview rather than dash as app13:26
Saviqkgunn, but we thought we can just put it in anyway, we're working on a fix for dash as app anyway13:27
Saviqor dandrader is working on a fix13:27
kgunnsure13:27
* Saviq is about to fly out13:27
tsdgeoskgunn: it was there for a while, caused using the "old combo in header" not work sometimes, never was able to pin point it13:27
tsdgeosit is this one13:27
* kgunn hopes Saviq's arms are well rested for the flappign13:27
Saviqo/13:32
Saviq\o/        —o—        \o/         —o—           \o/13:32
kgunnlol13:34
tsdgeos /°\13:35
tsdgeosweird guy13:35
tsdgeosmhr3: for mocking purposes can i make navigationid() and query() be the same?13:35
mhr3tsdgeos, do whatever you want in mocks13:36
tsdgeoswell i need to make them do similar something to what you do :D13:36
tsdgeosin my head query and navigationid are just the same since i don't do weird jumps, so performquery will just use navigationid to internally change the current navigationid and that's it13:37
cwaynetsdgeos: hey, any progress on table widget for previews?13:38
tsdgeoscwayne: it's done13:39
mhr3tsdgeos, oh that reminds me, i'm not doing anything with currentAltNavId13:39
tsdgeoscwayne: needs to be reviewed and merged :D13:39
tsdgeosmhr3: meaning?13:39
mhr3tsdgeos, that i got a bug :)13:39
cwaynetsdgeos: :D nice, thanks!13:39
tsdgeoscwayne: i can give you a branch if you want to try it13:40
mhr3tsdgeos, but we'll need to think things through, to deal with the bug we've been talking about for the past few days13:40
cwaynetsdgeos: that'd be great13:40
tsdgeoscwayne: https://code.launchpad.net/~aacid/unity8/tablePreviewWidget/+merge/22900613:41
mhr3tsdgeos, for one, you should consider that performquery for a particular navId will change currentNavId, but asynchronously13:44
mhr3tsdgeos, and maybe it won't if something goes wrong (query fails or something)13:45
cwaynetsdgeos: awesome, thanks man13:45
tsdgeosmhr3: i see13:46
mhr3tsdgeos, btw if you want to test it against a scope, you can build my sort-order branch and point scope-tool to ./builddir/tests/data/mock-scope-double-nav/*.ini13:48
tsdgeosmhr3: so what the current department code does is14:01
tsdgeoson pressed, get the departmentid the role returns and assume that is the next department id we will go14:02
tsdgeosis that wrong?14:02
mhr3tsdgeos, sounds good to me14:02
tsdgeosmhr3: so what's performquery doing then?14:03
tsdgeosit goes there14:03
tsdgeosok14:03
mhr3asks the scope to get data for that dep_id14:03
mhr3and that can fail14:03
tsdgeosbut i'll be showiing that department already14:04
mhr3tsdgeos, but i think the bug was that you assumed that when a scope page is shown the first time, you're in root department14:04
mhr3and that's not always the case14:04
tsdgeosit's the only way it can work14:05
tsdgeosi can't go more left than first14:05
tsdgeoskind of tricky :D14:05
mhr3tsdgeos, unfortunately you might need to14:05
tsdgeosunfortuantely i might need to rewrite the whole thing then14:05
mhr3tsdgeos, consider that you click on a scope link that takes you to amazon's "Music DVDs" right away, there's quite a lot of stuff to the left14:06
tsdgeossure14:06
tsdgeosthere were no scope links14:06
tsdgeosand noone told me i should have to go left14:06
tsdgeosso it's not developed for that14:06
mhr3tsdgeos, you know things change and get extended all the time14:07
tsdgeosi'm just saying "that small bugfix" may mean re-coding everything14:07
mhr3ouch14:07
tsdgeosit's not your fault nor for you to care14:07
tsdgeosbut it's what it is14:07
tsdgeosjust complaining to the air14:14
mterrytsdgeos, in ~aacid/unity8/implementOverlayColor, how does one test the branch?14:15
tsdgeosmterry: make tryCard14:16
greybacktsdgeos: was that supposed to rhyme? Cuz it sounded good :)14:16
mterrytsdgeos, I tried altering the default #99000000 value, but it didn't give any visible chang14:16
tsdgeosand do some stuff14:16
tsdgeosmterry: let me see14:16
greybacktsdgeos: even has good rhythm14:16
tsdgeosgreyback: :D14:16
tsdgeosno it was not14:16
mterrywill try tryCard14:16
tsdgeosmterry: ah you need to be in overlay14:16
MacSlowWhat was the reason again for unity8 not starting on the desktop and just idling around after run.sh?14:17
tsdgeosmterry: are you there?14:17
mterrytsdgeos, I don't know unity8 terminology outside of the greeter ;)14:17
mterrytsdgeos, where is the overlay?14:17
tsdgeosmterry: use the combo, select overlaid14:17
tsdgeosthen go to the lower editable part14:18
mterrytsdgeos, are you talking about tryCard?14:18
tsdgeosmterry: yes14:18
tsdgeosmterry: there's a combo topright, no?14:18
mterrytsdgeos, oh I was talking about looking around unity8 while it's running14:18
tsdgeosmterry: ah, then you look at the carousel14:18
tsdgeosbut tryCard is just easier14:18
mterrytsdgeos, I thought we dropped carousels14:18
tsdgeosyou can change stuff live14:18
tsdgeosselect art, header, summary - overlaid14:19
tsdgeosand then in the lower editable box14:19
tsdgeosput   "overlayColor":"#99990000",14:19
tsdgeosafter title for example14:19
tsdgeosand change it to14:19
tsdgeos  "overlayColor":"#99009900",14:19
tsdgeosand see how it changes14:19
mterryk14:20
mterrytsdgeos, brilliant14:22
tsdgeosworks by chance14:23
tsdgeosi just played with the shader code until it did what i wnted14:23
tsdgeos:D14:23
mhall119mhr3: ping14:30
mhall119nvm, I can look it up myself14:31
mhall119actually I'll ping again, do you know of any tutorials for using net-cpp?14:34
tsdgeosmterry: if you're doing small reviews, you could do https://code.launchpad.net/~aacid/unity8/pageHeaderBackgroundOverflow/+merge/22888114:44
tsdgeosmterry: very easy to test, run make tryDash, go to the third scope (the one with the ubuntu logo) and click on the search icon14:45
tsdgeossee difference with and without patch14:45
tsdgeosCimi: and maybe you can do https://code.launchpad.net/~aacid/unity8/collapsingPreviews/+merge/228857 and https://code.launchpad.net/~aacid/unity8/tablePreviewWidget/+merge/229006 since they have to do with previews?14:47
Ciminot doing reviews until I got mine approved :O14:47
Cimi:P14:47
Cimiit's give and take!14:48
cwayneCimi: any eta on settings UI support landing?15:02
mterrytsdgeos, will review that one15:02
Cimicwayne, I will start monday15:02
Cimicwayne, might take a week15:02
=== gatox is now known as gatox_lunch
dednicktsdgeos: ping15:42
tsdgeosdednick: yep?15:42
dednicktsdgeos: hey. did you do the work on scopeStyle?15:42
tsdgeosdednick: it's mostly saviq's15:42
tsdgeosbut i know some15:42
tsdgeoswhy?15:42
dednicktsdgeos: hm. ok. don't think it belongs in PageHeader since it's in Components, not Dash. Causing me pain since i want to use it in indicators.15:43
dednickall i need to do it define QtObject with known params right?15:43
tsdgeosi remmeber arguing with saviq about that yeah15:44
tsdgeosyeah a QtObject would do15:44
tsdgeosnow15:44
tsdgeosyou're using a PageHeader in indicators?15:44
dednicktsdgeos: yes. getting rid of tabs15:45
tsdgeosdednick: why you need PageHEader?15:45
tsdgeosand not the SDK's one?15:45
tsdgeosi mean PageHeader gives you the search thing15:46
tsdgeosdo you need search?15:46
dednicktsdgeos: no.15:46
tsdgeosthen you should probably use SDK one15:46
dednicktsdgeos: doesnt the sdk header require mainview crap?15:46
dednickintegrated with the pages15:46
tsdgeoswell you can use the "private" stuff like our PageHeader does15:46
tsdgeosPageHeadStyle15:47
tsdgeossee inside pageheader15:47
tsdgeosPageHeadStyle is "most" of the stuff15:47
tsdgeosand then there's all the saerch and etc15:47
tsdgeosbut if you don't need that15:47
tsdgeosPageHeadStyle should be good for you15:47
dednicktsdgeos: hm. does it look the same as the dash headeR?15:47
tsdgeosdednick: well it's what the PageHeader is using ;)15:47
Saviqdednick, yeah, the PageHeader component is actually dash-specific...15:47
Saviqdednick, it should actually be moved to Dash/15:48
Saviqand you just use the SDK one15:48
dednicktsdgeos: Saviq: ok, thanks. i'll take a look15:50
cwayneSaviq: is there a size limit on the icon for an attribute?  I'm trying to use an image I get from yelp API showing the ratings, and it's not showing up on the device, but working in scope-tool15:53
Saviqcwayne, nope, as you can see in https://code.launchpad.net/~paulliu/unity8/attribute/+merge/223242 around line 4915:55
Saviqcwayne, we just force height to 2gu and use natural width15:55
cwayneSaviq: any idea why mine wouldn't show up on device, but does show up in scope-tool?15:56
Saviqcwayne, paulliu might be able to help, he wrote that15:56
Saviqcwayne, but nothing comes to mind, any interesting unity8.log output?15:56
cwayneor am i abusing this, and we're expecting an actual icon name instead of a .png...15:56
dednickkrieky. we should really get a basic header from sdk...15:57
mhr3cwayne, is your device updated? :)15:57
Saviqcwayne, no, any valid URL is supported15:57
Saviqcwayne, now mhr3's is an interesting question ;)15:57
cwaynemhr3: yessir, my other attribute is showing up15:58
Saviqcwayne, look in unity8.log for any relevant errors/warnings15:58
mhr3cwayne, how many of them do you have?15:58
cwaynemhr3: 215:58
mhr3hm15:58
cwaynecould this be it? file:///usr/lib/arm-linux-gnueabihf/qt5/qml/Ubuntu/Components/Icon10.qml:49:9: QML Connections: Cannot assign to non-existent property "onSourceChanged"15:59
tsdgeosyeah the sdk went crazy15:59
Saviqcwayne, unlikely15:59
tsdgeosbut that is "ok" i think15:59
Saviqcwayne, that's a spurious warning in SDK16:00
Saviqgtg16:00
Saviqboarding in 516:00
Saviqo/16:00
cwaynemhr3: so if i switch the order, I do get the icon, but not the label, or the second attribute16:01
mhr3whaaat16:01
mhr3cwayne, how are you pushing it?16:02
cwaynemhr3: http://paste.ubuntu.com/7925147/16:04
mhr3cwayne, hm, wonder if it's something to do with go binding16:05
cwaynemhr3: i thought that, but then why would it work on scope-tool at all?16:05
mhr3oh it works there?16:06
mhr3hmmm16:06
mhr3ok weird16:06
cwayneyeah, worked like a charm in scope-tool, just not device :/16:06
mhr3cwayne, could you screenshot both the device and the scope-tool?16:13
cwaynemhr3: yep, was just doing that16:13
cwaynehttp://people.canonical.com/~cwayne/device.png16:13
cwaynehttp://people.canonical.com/~cwayne/scope-tool.png16:14
cwaynemhr3: ^16:14
mhr3cwayne, could you make a click out of the scope and send it to paul?16:15
mhr3cwayne, but i'm afraid it will have to wait till monday, we're just wrapping up here16:16
cwaynemhr3: sure16:17
mhr3thx16:17
* mhr3 out16:17
=== gatox_lunch is now known as gatox
WellarkJohnLea: I need design review for this:16:45
Wellarkhttps://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/22515916:45
WellarkJohnLea: the attacched pictures should be enough16:52
WellarkSaviq: charles is testing my modeminfo branch16:55
Wellarkall of the individual ci tests are either SUCCESS or UNSTABLE16:55
Wellarkand jenkins gives overall status as FAILED16:55
Wellarkis that expected?16:55
Wellarkhttps://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159/comments/55534716:55
=== alan_g is now known as alan_g|EOW
=== salem_ is now known as _salem
=== _salem is now known as salem_
=== salem_ is now known as _salem

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