kgunn | elopio: its these three | 00:00 |
---|---|---|
kgunn | https://code.launchpad.net/~unity-team/unity8/dash-as-app/+merge/228534 | 00:00 |
kgunn | https://code.launchpad.net/~mzanetti/unity8/drop-running-apps-from-dash/+merge/227707 | 00:00 |
kgunn | https://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/229123 | 00:00 |
elopio | kgunn: the first one is already merged, so I guess it shouldn't be part of the silo. | 00:16 |
elopio | it shouldn't affect either. | 00:17 |
elopio | I have merged trunk with the other two and it has no conflicts. | 00:17 |
elopio | maybe the latest merge I did improved things? Can you try again?? | 00:17 |
kgunn | elopio: hmmm, musta been merged with saviqs | 00:18 |
elopio | kgunn: sorry, it's the second one the already merged. | 00:18 |
kgunn | elopio: can you merge back trunk once more ? i bet you probably did it right before all the unity8 stuff migrated... | 00:19 |
kgunn | we caught it on the edge | 00:19 |
elopio | kgunn: http://paste.ubuntu.com/7919046/ | 00:20 |
elopio | also http://paste.ubuntu.com/7919049/ | 00:21 |
elopio | this 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 | ||
Saviq | elopio, apps not launching make sense because of https://code.launchpad.net/~unity-team/unity-scopes-shell/drop-appid-workaround/+merge/228427 | 07:47 |
Saviq | elopio, we need a corresponding branch from the click scope | 07:47 |
Saviq | that will send us proper appid:///, applications:/// urls | 07:47 |
tsdgeos | holy merge batman | 07:55 |
tsdgeos | too many branches take a while to merge :D | 07:55 |
Saviq | :) | 07:59 |
Saviq | it's been 13 MPs... | 08:00 |
Saviq | mhr3, FYI: bcmwl builds again | 08:11 |
mhr3 | yey, so i can reboot again :) | 08:11 |
Saviq | mhr3, yeah, upgrade first ;) | 08:12 |
tsdgeos | Saviq: who do we get to merge https://bugs.launchpad.net/ubuntu/+source/qtdeclarative-opensource-src/+bug/1349705 ? | 08:15 |
ubot5 | Launchpad bug 1349705 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Flickable gets confused when there's other flickables on top" [Undecided,New] | 08:15 |
tsdgeos | otherwise merging in the overview is going to be a mess | 08:15 |
tsdgeos | Saviq: 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 |
dandrader | tsdgeos, leo has a branch for it that fixes the AP tests | 08:18 |
tsdgeos | ah cool | 08:18 |
Saviq | tsdgeos, there's the branch after it that fixes all those | 08:19 |
Saviq | tsdgeos, let me find out | 08:19 |
Saviq | (re: qt) | 08:19 |
tsdgeos | still 1 test broken | 08:19 |
tsdgeos | https://code.launchpad.net/~canonical-platform-qa/unity8/dash-as-app-autopilot/+merge/229123 | 08:19 |
tsdgeos | but much more acceptable | 08:19 |
Saviq | tsdgeos, yeah, Leo reported that's a bug, shell locks up when you try to swipe dash in when app is still loading | 08:20 |
tsdgeos | oki | 08:20 |
Saviq | tsdgeos, for bug #1349705 I'll just prepare a package and put it in there | 08:20 |
ubot5 | bug 1349705 in qtdeclarative-opensource-src (Ubuntu) "[PATCH] Flickable gets confused when there's other flickables on top" [Undecided,New] https://launchpad.net/bugs/1349705 | 08:20 |
Saviq | same silo with overview, only problem is we'll need to smoke-test it | 08:21 |
tsdgeos | dandrader: 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 |
tsdgeos | Saviq: cool | 08:22 |
tsdgeos | and not cool at the same time :D | 08:22 |
tsdgeos | Saviq: ah now that you're here | 08:23 |
tsdgeos | shouldn't | 08:23 |
tsdgeos | if (!rendererLoader.expanded && !seeAllLabel.visible && target.collapsedItemCount > 0) { | 08:23 |
tsdgeos | be | 08:23 |
tsdgeos | if (!rendererLoader.expanded && seeAllLabel.visible && target.collapsedItemCount > 0) { | 08:23 |
=== popey_ is now known as popey | ||
tsdgeos | in qml/Dash/GenericScopeView.qml ' | 08:23 |
tsdgeos | ? | 08:23 |
Saviq | tsdgeos, lookin' | 08:26 |
Saviq | tsdgeos, no, it's only meant to filter the results if see all isn't there | 08:26 |
tsdgeos | ah | 08:27 |
tsdgeos | ok | 08:27 |
Saviq | tsdgeos, because you can't see all the results in the grid (because of collapsed-rows: 2 for example) | 08:27 |
tsdgeos | makes sense | 08:27 |
Saviq | and you can't expand it because you have no see all (because you have a header link, which "invalidates" see all) | 08:27 |
tsdgeos | ok :) | 08:29 |
tsdgeos | Saviq: so what's our next target for landing stuff? dash_overview + friends? | 08:29 |
tsdgeos | do we have a silo | 08:29 |
tsdgeos | ? | 08:29 |
Saviq | tsdgeos, we're doing dash as app first | 08:30 |
Saviq | tsdgeos, then overview | 08:30 |
tsdgeos | right | 08:31 |
tsdgeos | makes sense | 08:31 |
Saviq | tsdgeos, do you have dash overview handy btw? | 08:38 |
tsdgeos | Saviq: on the desktop yes, phone not up to date | 08:38 |
Saviq | tsdgeos, desktop's fine | 08:38 |
Saviq | tsdgeos, I'll forward you images for the favorites, could you please do a few screenshots at different aspect ratios? | 08:39 |
tsdgeos | Saviq: what you mean different aspect ratios? | 08:40 |
tsdgeos | change the "screen" size? | 08:40 |
Saviq | tsdgeos, yeah, landscape, portrait etc. | 08:42 |
Saviq | tsdgeos, try out three or four sizes please | 08:43 |
tsdgeos | Saviq: favorites and all or just favorites? | 08:43 |
Saviq | tsdgeos, just favourites, in All we're not forcing aspect ratio yet (are we?) | 08:45 |
tsdgeos | Saviq: no we're not | 08:49 |
tsdgeos | Saviq: do you happen to know the paths of the files i have to overwrite? | 08:49 |
tsdgeos | because they are not unity8's | 08:50 |
tsdgeos | Saviq: i think i may need some extra packages | 08:53 |
tsdgeos | mhr3: ping | 08:53 |
mhr3 | tsdgeos, pong | 08:53 |
tsdgeos | mhr3: which files do i need to overwrite to show my own dash overview images | 08:54 |
tsdgeos | for applications, music, video | 08:54 |
mhr3 | tsdgeos, $ scopes-client musicaggregator "" | head | 08:54 |
mhr3 | icon field | 08:55 |
tsdgeos | oki, tx | 08:56 |
mhr3 | tsdgeos, alt nav should be fully working now | 10:19 |
mhr3 | ...ish :) | 10:19 |
tsdgeos | mhr3: :D | 10:23 |
tsdgeos | ish? | 10:23 |
mhr3 | might be a bug here and there :) | 10:23 |
tsdgeos | mhr3: tests don't pass | 10:29 |
tsdgeos | can't dpkg pacakge it | 10:29 |
mhr3 | tsdgeos, did you pull? | 10:30 |
tsdgeos | ah you just changed it | 10:30 |
tsdgeos | meh | 10:31 |
tsdgeos | still failing | 10:31 |
Saviq | tsdgeos, 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 |
Saviq | tsdgeos, creating/destroying delegates (and whole categories, for that matter) will always kick us in the a$$ | 10:34 |
mhr3 | tsdgeos, can you pastebin LastTest.log? | 10:34 |
tsdgeos | Saviq: yeah :/ | 10:35 |
Saviq | tsdgeos, and well, if you're looking at a dash page, we should not refrain from committing memory to it | 10:35 |
tsdgeos | mhr3: http://paste.ubuntu.com/7922784/ | 10:36 |
Saviq | tsdgeos, 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 categories | 10:36 |
Saviq | tsdgeos, let's have a think/sleep & chat about it next week | 10:37 |
tsdgeos | :( | 10:37 |
tsdgeos | :) | 10:37 |
tsdgeos | i mean | 10:37 |
Saviq | lol | 10:37 |
Saviq | tsdgeos, btw, does that make any sense to you http://pastebin.ubuntu.com/7922748/ | 10:37 |
mhr3 | tsdgeos, weird, it's failing in location test... i didn't even touch that :P | 10:37 |
Saviq | tsdgeos, that's what I got out of a local build with your patch | 10:37 |
tsdgeos | mhr3: i need to more somewhere else :D | 10:37 |
tsdgeos | Saviq: beats me | 10:38 |
mhr3 | tsdgeos, do you have python-tornado installed? | 10:38 |
tsdgeos | i do | 10:38 |
mhr3 | pete-woods, why would that fail ^? | 10:38 |
mhr3 | tsdgeos, or you know... DEB_BUILD_OPTIONS="nocheck" bzr bd | 10:39 |
tsdgeos | taht too | 10:40 |
pete-woods | mhr3: well the location tests only require tornado, and don't do anything clever outside | 10:42 |
Saviq | tsdgeos, mhr3 is right, it built with 4.9 | 10:43 |
Saviq | (at least probably) | 10:43 |
Saviq | we might need to force it to 4.8 for now and do a whole-set transition to 4.9 separately | 10:43 |
tsdgeos | Saviq: what built with 4.9? | 10:44 |
Saviq | tsdgeos, qtdeclarative, the symbol changes | 10:44 |
tsdgeos | ahhhh | 10:44 |
tsdgeos | ok, thought you were talking about unity-scopes-shell and the test issues :D | 10:44 |
Saviq | sry | 10:44 |
dpm | hi 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-woods | dpm: yep, I'm happy to take on that task when I have a minute | 10:45 |
pete-woods | I'd basically be gutting the one of the video scopes | 10:45 |
pete-woods | I'll probably keep a basic test case, too | 10:45 |
=== vila is now known as vila-lunch | ||
dpm | pete-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 scopes | 10:46 |
dpm | pete-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-woods | dpm: yep, will do it with json | 10:47 |
dpm | sounds good | 10:47 |
pete-woods | dpm: I'll put a basic python based http server in the test case to test it all | 10:47 |
dpm | oh, that'd be a huge improvement from what we've got now, yes | 10:48 |
pete-woods | dpm: well I have that already in the video scope tests, so it should be very little effort to copy it over | 10:48 |
dpm | great | 10:50 |
facundobatista | Hola | 10:51 |
dpm | pete-woods, do you have a rough idea of when you could have this template ready? | 10:51 |
pete-woods | dpm: it's a matter of priorities, if thostr_ says to do it right away, then pretty quick really | 10:52 |
dandrader | Saviq, 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-woods | I'm currently managing a release of unity-scopes-api with ABI breaks, though, so after that | 10:52 |
Saviq | dandrader, "any time soon" meaning until we fix it (we need to) | 10:54 |
Saviq | dandrader, 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 |
dandrader | Saviq, I mean that this will likely come before dash-as-app: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1350917 | 10:56 |
ubot5 | Launchpad bug 1350917 in Unity 8 "UI freezes when switching to an app which is opening." [Critical,In progress] | 10:56 |
dandrader | Saviq, or should, as it's a critical bug... | 10:57 |
Saviq | dandrader, it can come *with* it, not necessarily before | 10:57 |
dandrader | Saviq, ah, the silo thing | 10:57 |
Saviq | dandrader, yup ;) | 10:57 |
dpm | thanks pete-woods | 11:04 |
thostr_ | dpm: about pete's priorities | 11: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 |
dpm | thostr_, 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 say | 11:09 |
dpm | thostr_, 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 |
dpm | ok, let's follow up next week, then. Thanks! | 11:20 |
=== MacSlow is now known as MacSlow|lunch | ||
=== vila-lunch is now known as vila | ||
tsdgeos | mhr3: ping | 11:27 |
mhr3 | tsdgeos, poing | 11:27 |
tsdgeos | mhr3: can we kill visible in scopves-v4 ? | 11:28 |
tsdgeos | or is anyone using it? | 11:28 |
tsdgeos | Saviq: ↑↑ | 11:29 |
mhr3 | tsdgeos, visible what? | 11:29 |
tsdgeos | mhr3: Scope::visible | 11:29 |
mhr3 | tsdgeos, oh yea, let's drop it | 11:29 |
tsdgeos | seems like a bastard favorite | 11:29 |
tsdgeos | to me | 11:29 |
Saviq | +1 | 11:33 |
mhr3 | tsdgeos, and that means you can drop the filter proxy model in shell right? | 11:48 |
mhr3 | yey! | 11:48 |
tsdgeos | mhr3: hmmmm | 11:48 |
tsdgeos | we may need to add some stuff | 11:49 |
tsdgeos | mhr3: we are using the filter proxy model to ask for the index of a scope in the model | 11:49 |
tsdgeos | mhr3: do you think we can add to ScopesInterface ? or at least a count | 11:50 |
tsdgeos | so i can just iterate over getScope(int row) | 11:50 |
mhr3 | it doesn't have that? | 11:50 |
tsdgeos | count not | 11:51 |
tsdgeos | getScope(row) yes | 11:51 |
tsdgeos | but without count i don't know where to stop | 11:51 |
tsdgeos | well i guess i could stop on null | 11:51 |
mhr3 | adding it, we have it on all models | 11:52 |
tsdgeos | cool | 11:52 |
tsdgeos | lunchy lunchy | 11:52 |
mhr3 | same | 11:56 |
jgdx | larsu, 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 |
seb128 | jgdx, is that crossbuild? | 12:08 |
jgdx | seb128, yes | 12:11 |
seb128 | jgdx, likely an archive issue rather a gsettings-qt one then... | 12:15 |
jgdx | seb128, yeah, and jenkins build it seemingly fine for armhf. | 12:18 |
seb128 | jgdx, well, jenkins doesn't crossbuild | 12:19 |
jgdx | seb128, it is all clear now | 12:19 |
seb128 | weird | 12:19 |
=== alan_g is now known as alan_g|lunch | ||
jgdx | seb128, no, "it's all clear to me now [why it passes on jenkins but fails for me]" | 12:20 |
seb128 | oh, ok | 12:20 |
jgdx | seb128, this is new to me, how is archive issues like these resolved? | 12:20 |
jgdx | s/is/are | 12:21 |
seb128 | by somebody looking what the issue is and fixing it | 12:21 |
seb128 | likely buggy multiarch support somewhere in the depends of what you try to install | 12:21 |
seb128 | xnox might be able to help you/give more hints on how to debug | 12:21 |
jgdx | right, thanks | 12:21 |
xnox | hm?! =) | 12:24 |
* xnox reads back | 12:24 | |
seb128 | xnox, gsettings-qt doesn't seem to be crossbuildable and jgdx is not sure how/where to report the issue | 12:24 |
xnox | seb128: by default, one should assume that cross-building has not been enabled. Most packages need tweaks to get it working. | 12:25 |
seb128 | xnox, well, in that case jgdx says he got gio not installable, that should work | 12:26 |
xnox | seb128: my laptop seems to lack cross-sbuild chroot, please wait =) | 12:27 |
seb128 | k | 12:27 |
seb128 | thanks for looking ;-) | 12:27 |
xnox | seb128: 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 |
seb128 | oh, ok | 12:39 |
xnox | seb128: jgdx: please use CMake, or some other sensible build-system like autotools. | 12:39 |
seb128 | we have better things to do for rtm than changing build systems | 12:40 |
seb128 | so I guess that's just not cross buildable | 12:40 |
seb128 | it's not like we were changing it every week anyway, so shouldn't be a real issue | 12:40 |
xnox | jgdx: 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 | ||
jgdx | xnox, seb128, k, thank you | 12:48 |
seb128 | jgdx, yw! | 12:49 |
=== _salem is now known as salem_ | ||
tsdgeos | mhr3: you removed visible but did not add count | 13:05 |
mhr3 | tsdgeos, just got back from lunch | 13:06 |
tsdgeos | same here :) | 13:06 |
tsdgeos | mhr3: ping me when you're done | 13:06 |
mhr3 | tsdgeos, pushed | 13:09 |
tsdgeos | cool tx | 13:09 |
Saviq | tsdgeos, ah, test reports should be good now... it was a lettercase issue | 13:21 |
tsdgeos | Saviq: ?¿ | 13:21 |
Saviq | tsdgeos, Test vs test | 13:21 |
tsdgeos | lol | 13:22 |
tsdgeos | really? | 13:22 |
Saviq | yeah | 13:22 |
tsdgeos | mterry_: answered your question | 13:22 |
tsdgeos | Saviq: in our side or on CI side? | 13:23 |
Saviq | tsdgeos, well, depends ;) | 13:23 |
Saviq | tsdgeos, CI expected test, we had Test | 13:23 |
tsdgeos | i mean there's no standard, is there? | 13:23 |
Saviq | tsdgeos, but got fixed CI side | 13:23 |
Saviq | nope | 13:23 |
tsdgeos | did they tell us? | 13:23 |
mterry_ | tsdgeos, cool updated | 13:23 |
Saviq | tsdgeos, it was just miscomm, fginther added Test now, too | 13:24 |
tsdgeos | cool :) | 13:24 |
Saviq | tsdgeos, *but* it still didn't end up in http://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/730/testReport/%28root%29/qmltestrunner/ yet | 13:24 |
Saviq | tsdgeos, so we might need to follow up | 13:24 |
kgunn | did we find another bug in qtdeclarative ? (dash as app silo) | 13:26 |
Saviq | kgunn, found it earlier than that | 13:26 |
Saviq | kgunn, and related to overview rather than dash as app | 13:26 |
Saviq | kgunn, but we thought we can just put it in anyway, we're working on a fix for dash as app anyway | 13:27 |
Saviq | or dandrader is working on a fix | 13:27 |
kgunn | sure | 13:27 |
* Saviq is about to fly out | 13:27 | |
tsdgeos | kgunn: it was there for a while, caused using the "old combo in header" not work sometimes, never was able to pin point it | 13:27 |
tsdgeos | it is this one | 13:27 |
* kgunn hopes Saviq's arms are well rested for the flappign | 13:27 | |
Saviq | o/ | 13:32 |
Saviq | \o/ —o— \o/ —o— \o/ | 13:32 |
kgunn | lol | 13:34 |
tsdgeos | /°\ | 13:35 |
tsdgeos | weird guy | 13:35 |
tsdgeos | mhr3: for mocking purposes can i make navigationid() and query() be the same? | 13:35 |
mhr3 | tsdgeos, do whatever you want in mocks | 13:36 |
tsdgeos | well i need to make them do similar something to what you do :D | 13:36 |
tsdgeos | in 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 it | 13:37 |
cwayne | tsdgeos: hey, any progress on table widget for previews? | 13:38 |
tsdgeos | cwayne: it's done | 13:39 |
mhr3 | tsdgeos, oh that reminds me, i'm not doing anything with currentAltNavId | 13:39 |
tsdgeos | cwayne: needs to be reviewed and merged :D | 13:39 |
tsdgeos | mhr3: meaning? | 13:39 |
mhr3 | tsdgeos, that i got a bug :) | 13:39 |
cwayne | tsdgeos: :D nice, thanks! | 13:39 |
tsdgeos | cwayne: i can give you a branch if you want to try it | 13:40 |
mhr3 | tsdgeos, but we'll need to think things through, to deal with the bug we've been talking about for the past few days | 13:40 |
cwayne | tsdgeos: that'd be great | 13:40 |
tsdgeos | cwayne: https://code.launchpad.net/~aacid/unity8/tablePreviewWidget/+merge/229006 | 13:41 |
mhr3 | tsdgeos, for one, you should consider that performquery for a particular navId will change currentNavId, but asynchronously | 13:44 |
mhr3 | tsdgeos, and maybe it won't if something goes wrong (query fails or something) | 13:45 |
cwayne | tsdgeos: awesome, thanks man | 13:45 |
tsdgeos | mhr3: i see | 13:46 |
mhr3 | tsdgeos, 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/*.ini | 13:48 |
tsdgeos | mhr3: so what the current department code does is | 14:01 |
tsdgeos | on pressed, get the departmentid the role returns and assume that is the next department id we will go | 14:02 |
tsdgeos | is that wrong? | 14:02 |
mhr3 | tsdgeos, sounds good to me | 14:02 |
tsdgeos | mhr3: so what's performquery doing then? | 14:03 |
tsdgeos | it goes there | 14:03 |
tsdgeos | ok | 14:03 |
mhr3 | asks the scope to get data for that dep_id | 14:03 |
mhr3 | and that can fail | 14:03 |
tsdgeos | but i'll be showiing that department already | 14:04 |
mhr3 | tsdgeos, but i think the bug was that you assumed that when a scope page is shown the first time, you're in root department | 14:04 |
mhr3 | and that's not always the case | 14:04 |
tsdgeos | it's the only way it can work | 14:05 |
tsdgeos | i can't go more left than first | 14:05 |
tsdgeos | kind of tricky :D | 14:05 |
mhr3 | tsdgeos, unfortunately you might need to | 14:05 |
tsdgeos | unfortuantely i might need to rewrite the whole thing then | 14:05 |
mhr3 | tsdgeos, 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 left | 14:06 |
tsdgeos | sure | 14:06 |
tsdgeos | there were no scope links | 14:06 |
tsdgeos | and noone told me i should have to go left | 14:06 |
tsdgeos | so it's not developed for that | 14:06 |
mhr3 | tsdgeos, you know things change and get extended all the time | 14:07 |
tsdgeos | i'm just saying "that small bugfix" may mean re-coding everything | 14:07 |
mhr3 | ouch | 14:07 |
tsdgeos | it's not your fault nor for you to care | 14:07 |
tsdgeos | but it's what it is | 14:07 |
tsdgeos | just complaining to the air | 14:14 |
mterry | tsdgeos, in ~aacid/unity8/implementOverlayColor, how does one test the branch? | 14:15 |
tsdgeos | mterry: make tryCard | 14:16 |
greyback | tsdgeos: was that supposed to rhyme? Cuz it sounded good :) | 14:16 |
mterry | tsdgeos, I tried altering the default #99000000 value, but it didn't give any visible chang | 14:16 |
tsdgeos | and do some stuff | 14:16 |
tsdgeos | mterry: let me see | 14:16 |
greyback | tsdgeos: even has good rhythm | 14:16 |
tsdgeos | greyback: :D | 14:16 |
tsdgeos | no it was not | 14:16 |
mterry | will try tryCard | 14:16 |
tsdgeos | mterry: ah you need to be in overlay | 14:16 |
MacSlow | What was the reason again for unity8 not starting on the desktop and just idling around after run.sh? | 14:17 |
tsdgeos | mterry: are you there? | 14:17 |
mterry | tsdgeos, I don't know unity8 terminology outside of the greeter ;) | 14:17 |
mterry | tsdgeos, where is the overlay? | 14:17 |
tsdgeos | mterry: use the combo, select overlaid | 14:17 |
tsdgeos | then go to the lower editable part | 14:18 |
mterry | tsdgeos, are you talking about tryCard? | 14:18 |
tsdgeos | mterry: yes | 14:18 |
tsdgeos | mterry: there's a combo topright, no? | 14:18 |
mterry | tsdgeos, oh I was talking about looking around unity8 while it's running | 14:18 |
tsdgeos | mterry: ah, then you look at the carousel | 14:18 |
tsdgeos | but tryCard is just easier | 14:18 |
mterry | tsdgeos, I thought we dropped carousels | 14:18 |
tsdgeos | you can change stuff live | 14:18 |
tsdgeos | select art, header, summary - overlaid | 14:19 |
tsdgeos | and then in the lower editable box | 14:19 |
tsdgeos | put "overlayColor":"#99990000", | 14:19 |
tsdgeos | after title for example | 14:19 |
tsdgeos | and change it to | 14:19 |
tsdgeos | "overlayColor":"#99009900", | 14:19 |
tsdgeos | and see how it changes | 14:19 |
mterry | k | 14:20 |
mterry | tsdgeos, brilliant | 14:22 |
tsdgeos | works by chance | 14:23 |
tsdgeos | i just played with the shader code until it did what i wnted | 14:23 |
tsdgeos | :D | 14:23 |
mhall119 | mhr3: ping | 14:30 |
mhall119 | nvm, I can look it up myself | 14:31 |
mhall119 | actually I'll ping again, do you know of any tutorials for using net-cpp? | 14:34 |
tsdgeos | mterry: if you're doing small reviews, you could do https://code.launchpad.net/~aacid/unity8/pageHeaderBackgroundOverflow/+merge/228881 | 14:44 |
tsdgeos | mterry: very easy to test, run make tryDash, go to the third scope (the one with the ubuntu logo) and click on the search icon | 14:45 |
tsdgeos | see difference with and without patch | 14:45 |
tsdgeos | Cimi: 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 |
Cimi | not doing reviews until I got mine approved :O | 14:47 |
Cimi | :P | 14:47 |
Cimi | it's give and take! | 14:48 |
cwayne | Cimi: any eta on settings UI support landing? | 15:02 |
mterry | tsdgeos, will review that one | 15:02 |
Cimi | cwayne, I will start monday | 15:02 |
Cimi | cwayne, might take a week | 15:02 |
=== gatox is now known as gatox_lunch | ||
dednick | tsdgeos: ping | 15:42 |
tsdgeos | dednick: yep? | 15:42 |
dednick | tsdgeos: hey. did you do the work on scopeStyle? | 15:42 |
tsdgeos | dednick: it's mostly saviq's | 15:42 |
tsdgeos | but i know some | 15:42 |
tsdgeos | why? | 15:42 |
dednick | tsdgeos: 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 |
dednick | all i need to do it define QtObject with known params right? | 15:43 |
tsdgeos | i remmeber arguing with saviq about that yeah | 15:44 |
tsdgeos | yeah a QtObject would do | 15:44 |
tsdgeos | now | 15:44 |
tsdgeos | you're using a PageHeader in indicators? | 15:44 |
dednick | tsdgeos: yes. getting rid of tabs | 15:45 |
tsdgeos | dednick: why you need PageHEader? | 15:45 |
tsdgeos | and not the SDK's one? | 15:45 |
tsdgeos | i mean PageHeader gives you the search thing | 15:46 |
tsdgeos | do you need search? | 15:46 |
dednick | tsdgeos: no. | 15:46 |
tsdgeos | then you should probably use SDK one | 15:46 |
dednick | tsdgeos: doesnt the sdk header require mainview crap? | 15:46 |
dednick | integrated with the pages | 15:46 |
tsdgeos | well you can use the "private" stuff like our PageHeader does | 15:46 |
tsdgeos | PageHeadStyle | 15:47 |
tsdgeos | see inside pageheader | 15:47 |
tsdgeos | PageHeadStyle is "most" of the stuff | 15:47 |
tsdgeos | and then there's all the saerch and etc | 15:47 |
tsdgeos | but if you don't need that | 15:47 |
tsdgeos | PageHeadStyle should be good for you | 15:47 |
dednick | tsdgeos: hm. does it look the same as the dash headeR? | 15:47 |
tsdgeos | dednick: well it's what the PageHeader is using ;) | 15:47 |
Saviq | dednick, yeah, the PageHeader component is actually dash-specific... | 15:47 |
Saviq | dednick, it should actually be moved to Dash/ | 15:48 |
Saviq | and you just use the SDK one | 15:48 |
dednick | tsdgeos: Saviq: ok, thanks. i'll take a look | 15:50 |
cwayne | Saviq: 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-tool | 15:53 |
Saviq | cwayne, nope, as you can see in https://code.launchpad.net/~paulliu/unity8/attribute/+merge/223242 around line 49 | 15:55 |
Saviq | cwayne, we just force height to 2gu and use natural width | 15:55 |
cwayne | Saviq: any idea why mine wouldn't show up on device, but does show up in scope-tool? | 15:56 |
Saviq | cwayne, paulliu might be able to help, he wrote that | 15:56 |
Saviq | cwayne, but nothing comes to mind, any interesting unity8.log output? | 15:56 |
cwayne | or am i abusing this, and we're expecting an actual icon name instead of a .png... | 15:56 |
dednick | krieky. we should really get a basic header from sdk... | 15:57 |
mhr3 | cwayne, is your device updated? :) | 15:57 |
Saviq | cwayne, no, any valid URL is supported | 15:57 |
Saviq | cwayne, now mhr3's is an interesting question ;) | 15:57 |
cwayne | mhr3: yessir, my other attribute is showing up | 15:58 |
Saviq | cwayne, look in unity8.log for any relevant errors/warnings | 15:58 |
mhr3 | cwayne, how many of them do you have? | 15:58 |
cwayne | mhr3: 2 | 15:58 |
mhr3 | hm | 15:58 |
cwayne | could 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 |
tsdgeos | yeah the sdk went crazy | 15:59 |
Saviq | cwayne, unlikely | 15:59 |
tsdgeos | but that is "ok" i think | 15:59 |
Saviq | cwayne, that's a spurious warning in SDK | 16:00 |
Saviq | gtg | 16:00 |
Saviq | boarding in 5 | 16:00 |
Saviq | o/ | 16:00 |
cwayne | mhr3: so if i switch the order, I do get the icon, but not the label, or the second attribute | 16:01 |
mhr3 | whaaat | 16:01 |
mhr3 | cwayne, how are you pushing it? | 16:02 |
cwayne | mhr3: http://paste.ubuntu.com/7925147/ | 16:04 |
mhr3 | cwayne, hm, wonder if it's something to do with go binding | 16:05 |
cwayne | mhr3: i thought that, but then why would it work on scope-tool at all? | 16:05 |
mhr3 | oh it works there? | 16:06 |
mhr3 | hmmm | 16:06 |
mhr3 | ok weird | 16:06 |
cwayne | yeah, worked like a charm in scope-tool, just not device :/ | 16:06 |
mhr3 | cwayne, could you screenshot both the device and the scope-tool? | 16:13 |
cwayne | mhr3: yep, was just doing that | 16:13 |
cwayne | http://people.canonical.com/~cwayne/device.png | 16:13 |
cwayne | http://people.canonical.com/~cwayne/scope-tool.png | 16:14 |
cwayne | mhr3: ^ | 16:14 |
mhr3 | cwayne, could you make a click out of the scope and send it to paul? | 16:15 |
mhr3 | cwayne, but i'm afraid it will have to wait till monday, we're just wrapping up here | 16:16 |
cwayne | mhr3: sure | 16:17 |
mhr3 | thx | 16:17 |
* mhr3 out | 16:17 | |
=== gatox_lunch is now known as gatox | ||
Wellark | JohnLea: I need design review for this: | 16:45 |
Wellark | https://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159 | 16:45 |
Wellark | JohnLea: the attacched pictures should be enough | 16:52 |
Wellark | Saviq: charles is testing my modeminfo branch | 16:55 |
Wellark | all of the individual ci tests are either SUCCESS or UNSTABLE | 16:55 |
Wellark | and jenkins gives overall status as FAILED | 16:55 |
Wellark | is that expected? | 16:55 |
Wellark | https://code.launchpad.net/~unity-api-team/unity8/modeminfo/+merge/225159/comments/555347 | 16: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!