| * karni notes to do the reviews | 00:34 | |
| kklimonda | ff | 00:49 |
|---|---|---|
| Cimi | tsdgeos, ping | 08:42 |
| tsdgeos | Cimi: hi | 08:42 |
| Cimi | tsdgeos, mhr3_ told me to use the video thumbailer in few cases | 08:42 |
| Cimi | is source: widgetData["screenshot"] || widgetData["source"].startsWith("file://") ? "image://thumbnailer/..." : "" correct? | 08:42 |
| tsdgeos | Cimi: think so | 08:44 |
| tsdgeos | see uriToThumbnailerProviderString | 08:44 |
| tsdgeos | basically does | 08:44 |
| tsdgeos | if (uri.startsWith(QLatin1String("file:///"))) { | 08:44 |
| tsdgeos | thumbnailerUri = BASE_THUMBNAILER_URI; | 08:44 |
| tsdgeos | thumbnailerUri.append(uri.midRef(7)); | 08:44 |
| tsdgeos | 08:44 | |
| mhr3_ | Cimi, the "..." is supposed to be path to the video | 08:45 |
| Cimi | mhr3_, ahah ok | 08:45 |
| tsdgeos | mhr3_: the previews should be fine now | 08:45 |
| mhr3_ | Cimi, don't forget to update the factory for the video widget | 08:46 |
| Cimi | mhr3_, source: widgetData["screenshot"] || widgetData["source"].startsWith("file://") ? "image://thumbnailer/" + widgetData["source"] : "" ? | 08:46 |
| tsdgeos | mhr3_: fixed it and karni reviewed | 08:46 |
| mhr3_ | Cimi, you need to strip file:// | 08:46 |
| mhr3_ | tsdgeos, awesome, thank you | 08:46 |
| mhr3_ | does js support widgetData["source"][7:] ? | 08:46 |
| tsdgeos | Cimi: and we have a release | 08:47 |
| tsdgeos | Cimi: so remerge your stuff | 08:47 |
| tsdgeos | Cimi: no | 08:47 |
| tsdgeos | Cimi: cut the file:///, see the code i pasted | 08:47 |
| Cimi | mhr3_, dunno | 08:48 |
| * Cimi prints | 08:48 | |
| Cimi | mhr3_, widgetData["source"].substr(7) might work | 08:53 |
| * tsdgeos shows http://www.w3schools.com/jsref/jsref_obj_string.asp to Cimi | 08:54 | |
| tsdgeos | mhr3_: are you doing https://code.launchpad.net/~unity-team/unity8/unity8.preview_image/+merge/205757 ? looks good enough to me now | 08:54 |
| Cimi | tsdgeos, slice or substr ? | 08:56 |
| tsdgeos | Cimi: substr i'd say | 08:58 |
| Cimi | tsdgeos, what's diference-? | 08:58 |
| Cimi | ah | 08:58 |
| Cimi | position of end of char | 08:58 |
| tsdgeos | or slice | 08:59 |
| tsdgeos | yeah | 08:59 |
| tsdgeos | see http://stackoverflow.com/questions/2243824/what-is-the-difference-between-string-slice-and-string-substring-in-javascript | 08:59 |
| tsdgeos | it's "all do the same" kind of :D | 08:59 |
| tsdgeos | mhr3_: ping | 09:01 |
| mhr3_ | tsdgeos, pong | 09:06 |
| tsdgeos | mhr3_: just to confirm, you wanted the data passed to the actions back right? | 09:06 |
| mhr3_ | yep | 09:06 |
| tsdgeos | ok, will have a MP for you to review in a few mins | 09:07 |
| tsdgeos | mhr3_: https://code.launchpad.net/~aacid/unity8/actions_data_back/+merge/205915 | 09:13 |
| Cimi | tsdgeos, help http://paste.ubuntu.com/6919000/ | 09:14 |
| tsdgeos | lol | 09:15 |
| Cimi | tsdgeos, home/cimi/Development/video/qml/Dash/Previews/PreviewVideoPlayback.qml:40: TypeError: Object file:///path/to/local/file has no method 'startsWith' | 09:15 |
| tsdgeos | yeah no startswith in http://www.w3schools.com/jsref/jsref_obj_string.asp | 09:16 |
| Cimi | me stupid | 09:16 |
| tsdgeos | indexOf == 0 ? | 09:16 |
| tsdgeos | Cimi: i'd do a function there | 09:17 |
| tsdgeos | that'd be easier to read with proper if/else | 09:17 |
| tsdgeos | and being able to assign stuff to a var | 09:17 |
| mhr3_ | tsdgeos, i don't think the id is present in the attribute map | 09:20 |
| tsdgeos | mhr3_: the test shows it is D: | 09:20 |
| tsdgeos | and the json too | 09:21 |
| tsdgeos | { | 09:21 |
| tsdgeos | "type": "actions", | 09:21 |
| tsdgeos | "actions": [{"label": null, "icon": null, "id": null}] | 09:21 |
| tsdgeos | } | 09:21 |
| mhr3_ | oh that's the action id | 09:21 |
| mhr3_ | nvm | 09:21 |
| mhr3_ | yea, ok | 09:21 |
| tsdgeos | ep | 09:21 |
| tsdgeos | yep | 09:21 |
| sil2100 | bregma: hi! Just to mention: I disabled and removed window-mocker from cu2d | 09:44 |
| tsdgeos | sil2100: can we get a silo for unity8? | 10:05 |
| tsdgeos | we have a good bunch of stuff to land again :-) | 10:06 |
| sil2100 | tsdgeos: ;) Sure, but we apparently have some spreadsheet problems right now, so once those are done we will assign ;) | 10:06 |
| mhr3_ | tsdgeos, how's support for header looking? | 10:08 |
| tsdgeos | sil2100: ok | 10:08 |
| tsdgeos | mhr3_: should be landing soon | 10:08 |
| tsdgeos | mhr3_: we can merge it to new-scopes already if you want | 10:08 |
| mhr3_ | eh, if i knew 10minutes ago i would | 10:08 |
| tsdgeos | mhr3_: lp:~aacid/unity8/preview_header | 10:08 |
| tsdgeos | mhr3_: of course soon might mean tomorrow, since our CI train is not a high speed train :D | 10:09 |
| mhr3_ | tsdgeos, tell me about it :) | 10:10 |
| anpok | http://bazaar.launchpad.net/~phablet-team/qtubuntu/trunk/view/head:/src/platforms/ubuntu/ubuntucommon/window.cc this code querries the opaque property from the window class | 10:33 |
| anpok | writes it to flags and ignores it then.. | 10:33 |
| anpok | is the opaque property in some way configureable for applications? | 10:33 |
| anpok | i.e. through a qml property? | 10:34 |
| tsdgeos | anpok: reading the code i'd say no, but loicm should be the one that knows more | 10:38 |
| tsdgeos | mzanetti: you did the DownloadTracker thing? | 10:38 |
| tsdgeos | Saviq did | 10:39 |
| tsdgeos | it seems | 10:39 |
| mzanetti | downloadtracker? | 10:39 |
| mzanetti | never heard of that | 10:39 |
| tsdgeos | :D | 10:39 |
| tsdgeos | wonder if we have tests for it | 10:39 |
| tsdgeos | well we have a mock so may as well | 10:40 |
| tsdgeos | cool | 10:41 |
| tsdgeos | we do | 10:41 |
| tsdgeos | Cimi: you need to hook up https://code.launchpad.net/~unity-team/unity8/unity8.preview_video/+merge/205802 to the factory too | 10:43 |
| Cimi | tsdgeos, I did locally | 10:44 |
| tsdgeos | cool | 10:44 |
| Cimi | tsdgeos, I' writing tests for the image thumnnailer | 10:44 |
| tsdgeos | oki | 10:45 |
| tsdgeos | or | 10:45 |
| tsdgeos | hmmm | 10:45 |
| tsdgeos | Cimi: maybe no need | 10:45 |
| tsdgeos | Cimi: i was promised ages ago, the thumbnailer would move from our codebase | 10:45 |
| tsdgeos | so not sure it makes sense to invest time in that now | 10:45 |
| mhr3_ | thumbnailer is in uitk | 10:46 |
| tsdgeos | right, ok | 10:49 |
| tsdgeos | Cimi: so what are you testing? | 10:49 |
| mhr3_ | tsdgeos, yey, running apps in demo ppa :) | 10:49 |
| tsdgeos | mhr3_: awesome, didn't think it'd work :D | 10:50 |
| * tsdgeos hides | 10:50 | |
| mhr3_ | lol | 10:50 |
| mhr3_ | it does :P | 10:50 |
| Cimi | tsdgeos, pushed | 10:50 |
| mhr3_ | merging to new-scopes | 10:51 |
| mhr3_ | tsdgeos, the headers look bad with the white bkgr | 10:56 |
| tsdgeos | mhr3_: ok, will have to have a look at visuals | 10:57 |
| tsdgeos | mhr3_: and maybe hardcode a few things for now | 10:57 |
| tsdgeos | it's on the list | 10:57 |
| tsdgeos | * make sure visuals are correct | 10:57 |
| mhr3_ | k | 10:57 |
| mhr3_ | tsdgeos, do you see anything else majorly missing for new-scopes? | 10:57 |
| mhr3_ | besides progress widget | 10:57 |
| mhr3_ | and me providing you with all the preview stuff | 10:58 |
| tsdgeos | mhr3_: this is my todo http://paste.ubuntu.com/6919377/ | 10:58 |
| tsdgeos | progress is almost done | 10:58 |
| tsdgeos | need the tests | 10:58 |
| tsdgeos | there's reviews preview widget and rating-input preview widget that Cimi will do next? | 10:58 |
| tsdgeos | and there's a few stuff that probably needs to be tweaked | 10:59 |
| tsdgeos | but it isn't looking that bad | 10:59 |
| Cimi | tsdgeos, doing them now | 10:59 |
| tsdgeos | mhr3_: Cimi: anything you find missing in that todo list? | 10:59 |
| mhr3_ | tsdgeos, looks like you're missing some [DONE] ;) | 10:59 |
| tsdgeos | well i need to remove them :D | 10:59 |
| tsdgeos | well we first need to approve the cimi ones | 10:59 |
| tsdgeos | i'm doing the last test | 11:00 |
| tsdgeos | and will approve | 11:00 |
| mhr3_ | tsdgeos, our next major thing is the progress widget, the click guys will need it to do end-to-end test | 11:00 |
| tsdgeos | will be there in 30 mins or less hopefully | 11:00 |
| mhr3_ | that's too fast, pls slow down :P | 11:00 |
| Cimi | tsdgeos, link to MR check list? | 11:02 |
| tsdgeos | https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 | 11:02 |
| Cimi | can we just add it when answer is negative? | 11:03 |
| Cimi | what's the point? | 11:03 |
| mhr3_ | tsdgeos, once we have all this we should talk about temp dash pages | 11:03 |
| tsdgeos | Cimi: what is negative? | 11:03 |
| tsdgeos | mhr3_: that's not so easy | 11:03 |
| tsdgeos | but sure :D | 11:04 |
| mhr3_ | tsdgeos, lucky you then, i was already pushing back on that | 11:04 |
| tsdgeos | mhr3_: "so easy" -> hard | 11:04 |
| tsdgeos | but talking is "free" | 11:04 |
| tsdgeos | but yeah, i don't think we'll have time to do it tbh | 11:04 |
| mhr3_ | so.. next week ;) | 11:05 |
| mhr3_ | let's try to talk about it on fri | 11:05 |
| Cimi | tsdgeos, when I don' test, I say I did not test | 11:05 |
| Cimi | tsdgeos, it's obvious we do the checklist | 11:05 |
| Cimi | tsdgeos, I always did it | 11:05 |
| tsdgeos | Cimi: well, what do you need to test in those things that aren't hooked up anywhere? | 11:05 |
| Cimi | in the last 4 years here | 11:05 |
| tsdgeos | Cimi: i know, it's silly | 11:06 |
| tsdgeos | it's not me that came up with rules that make us look as we are children that need to reminded to do our jobs | 11:06 |
| Cimi | who did? | 11:06 |
| tsdgeos | no clue | 11:06 |
| tsdgeos | Cimi: so about your image and video stuff | 11:07 |
| tsdgeos | they conflict with lp:~aacid/unity8/preview_header | 11:07 |
| Cimi | where? | 11:07 |
| tsdgeos | can you merge lp:~aacid/unity8/preview_header into them and make lp:~aacid/unity8/preview_header be the prerequisite for lp:~unity-team/unity8/unity8.preview_image ? | 11:07 |
| tsdgeos | the factory | 11:07 |
| Cimi | ok | 11:07 |
| tsdgeos | tx | 11:08 |
| Cimi | tsdgeos, done for both | 11:13 |
| tsdgeos | tx | 11:14 |
| tsdgeos | Cimi: can you resubmit https://code.launchpad.net/~unity-team/unity8/unity8.preview_image/+merge/205757 with the correct prerequisite? | 11:16 |
| anpok | where would I enter the PIN | 11:17 |
| anpok | used n4 without a sim card up to now | 11:17 |
| mhr3_ | one of the indicators has "unlock sim" | 11:20 |
| Cimi | tsdgeos, all done | 11:20 |
| tsdgeos | mzanetti: can you add https://code.launchpad.net/~unity-team/unity8/unity8.preview_image/+merge/205937 and https://code.launchpad.net/~unity-team/unity8/unity8.preview_video/+merge/205802 to the list of MPs? | 11:28 |
| tsdgeos | mzanetti: ideally at the end and in that order | 11:28 |
| tsdgeos | since they depend on eachother and in others that are already in the list | 11:29 |
| mzanetti | tsdgeos: done | 11:29 |
| gatox | tsdgeos, here | 11:30 |
| tsdgeos | gatox: no idea about unity-scope-tool to be honest | 11:30 |
| tsdgeos | i've just used unity8 itself | 11:30 |
| tsdgeos | with the scopes of scopes that mhr3_ pointed me to | 11:30 |
| gatox | tsdgeos, how were you testing that the preview is being displayed? | 11:30 |
| tsdgeos | and it works | 11:30 |
| tsdgeos | mhr3_: what are you using unity8 or unity-scope-tool? | 11:31 |
| gatox | tsdgeos, can you tell me how to reproduce it | 11:31 |
| gatox | ? | 11:31 |
| tsdgeos | gatox: how to reproduce what? the unity8 binary? | 11:31 |
| gatox | tsdgeos, i ask because the same scope that was showing results of the search yesterday..... now it doesn't show even that | 11:31 |
| tsdgeos | hmmm | 11:32 |
| tsdgeos | we didn't change anything search related | 11:32 |
| gatox | tsdgeos, how can i execute anything unity8/unity-scope-tool or something that will allow me to see the preview working | 11:32 |
| tsdgeos | gatox: build unity8 | 11:32 |
| tsdgeos | or just install it from the demo-stuff ppa | 11:32 |
| tsdgeos | it's just easier | 11:32 |
| gatox | tsdgeos, i'm testing on the desktop | 11:32 |
| tsdgeos | gatox: yes you can do both things i mentioned (build and use the demo-stuff ppa) | 11:33 |
| tsdgeos | https://launchpad.net/~unity-team/+archive/demo-stuff | 11:33 |
| tsdgeos | on the desktop | 11:33 |
| gatox | tsdgeos, ok, i already build the new-scopes branch in the desktop | 11:33 |
| tsdgeos | gatox: and is your scope showing? | 11:33 |
| mhr3_ | tsdgeos, both | 11:33 |
| gatox | tsdgeos, i'm running it like this: UNITY_FORCE_NEW_SCOPES=1 ./builddir/tools/unity-scope-tool --scope-dir /media/gatox/leeloo/canonical/clickhack/demo | 11:34 |
| gatox | tsdgeos, how are you doing it? | 11:34 |
| tsdgeos | gatox: as i said, i am not running /unity-scope-tool, i just ran unity8 and that's it | 11:34 |
| tsdgeos | i haven't used unity-scope-tool much tbh | 11:34 |
| gatox | tsdgeos, and you are using just ./run? | 11:34 |
| tsdgeos | gatox: yes | 11:34 |
| mhr3_ | gatox, there's latest version of libunity-scopes in distro now, you need to recompile the scope | 11:34 |
| gatox | mhr3_, checking that | 11:35 |
| mhr3_ | gatox, scope-tool should print an error from scoperunner about some ABI mismatch | 11:35 |
| mhr3_ | gatox, sorry, looks like you weren't on the mail about the break | 11:37 |
| mzanetti | tsdgeos: assigning this to you, ok? https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1279298 | 11:40 |
| ubot5 | Launchpad bug 1279298 in unity8 (Ubuntu) "Header title sometimes disappears" [Undecided,Confirmed] | 11:40 |
| tsdgeos | mzanetti: sure | 11:41 |
| tsdgeos | mzanetti: it's the SDK people that don't stop messing with the tabs | 11:41 |
| mzanetti | :D | 11:41 |
| tsdgeos | and they're rewriting/killing them again ^_^ | 11:41 |
| gatox | mhr3_, does the api changed or something? i'm getting a weird issue when trying to build the scope about the create_query and preview function not being override.... which was working yesterday | 11:41 |
| mhr3_ | gatox, yes, http://bazaar.launchpad.net/~unity-team/unity-scopes-api/devel/view/head:/RELEASE_NOTES.txt | 11:42 |
| gatox | checking | 11:42 |
| mhr3_ | gatox, there's a branch that fixes all that already | 11:42 |
| mhr3_ | just merge it | 11:42 |
| mhr3_ | gatox, btw the envvar isn't necessary | 11:46 |
| mhr3_ | if you're using the new-scopes branch | 11:47 |
| gatox | ack | 11:47 |
| mhr3_ | actually it might even be breaking things | 11:47 |
| === _salem is now known as salem_ | ||
| dandrader | mzanetti, would you have time to review that? https://code.launchpad.net/~dandrader/unity8/rotatedDDA/+merge/205845 | 11:51 |
| mzanetti | dandrader: mind adding the checklist | 11:52 |
| dandrader | greyback, ^^^ that fixes the rotation issue with edge drags | 11:53 |
| dandrader | mzanetti, ah, let me find it | 11:53 |
| greyback | dandrader: nice | 11:53 |
| dandrader | greyback, I've merged it already in the unity8/rotate-n7 branch | 11:54 |
| greyback | dandrader: I know, am using it :) | 11:54 |
| tsdgeos | Cimi: mhr3_: https://code.launchpad.net/~aacid/unity8/preview_progress/+merge/205939 | 11:56 |
| Cimi | tsdgeos, kill empty double new lines :) | 11:57 |
| tsdgeos | :D | 11:58 |
| gatox | tsdgeos, mhr3_ everything working using unity-scope-tool with the scope migrated to the new api.... thx both! | 11:58 |
| tsdgeos | Cimi: there's more than one? | 11:58 |
| Cimi | tsdgeos, there are few ones | 11:58 |
| tsdgeos | really? | 11:59 |
| * tsdgeos looks again | 11:59 | |
| Cimi | tsdgeos, 25+ | 11:59 |
| Cimi | 68+ | 11:59 |
| Cimi | 147+ | 11:59 |
| tsdgeos | so there are two | 11:59 |
| tsdgeos | 68 is not a dobule line! | 11:59 |
| tsdgeos | but i'll kill it too | 11:59 |
| tsdgeos | done | 12:00 |
| mhr3_ | gatox, cool | 12:00 |
| mhr3_ | tsdgeos, looks good | 12:01 |
| mhr3_ | tsdgeos, i am wondering though - all those widgetData["foo"], are those properly bound? | 12:01 |
| mhr3_ | tsdgeos, i mean, the widgetData thing is a role in model and that thing can be updated | 12:02 |
| Cimi | tsdgeos, how about choosing Moto G instead nexus 4 as our new device? | 12:03 |
| mhr3_ | someone wants to play with new hardware :) | 12:03 |
| Cimi | tsdgeos, nexus 4 are no longer available | 12:03 |
| Cimi | mhr3_, ^ | 12:03 |
| Cimi | mhr3_, and mogo G is less than half price | 12:04 |
| Cimi | *moto G | 12:04 |
| mhr3_ | and here i thought the n4 is the cheapest smartphone ever | 12:04 |
| mzanetti | dandrader: "// NB:..." always triggers "Nicht beachten" in my head which means "disregard this:" :D | 12:04 |
| Cimi | mhr3_, you can find it for £250 N4 | 12:05 |
| Cimi | mhr3_, spent 119 for Moto G | 12:05 |
| dandrader | mzanetti, :) | 12:05 |
| Cimi | comparable specs | 12:05 |
| tsdgeos | Cimi: i'm not in position to decide that :D | 12:06 |
| Cimi | tsdgeos, I know you're not | 12:06 |
| tsdgeos | Cimi: not sure what is your concern about widgetData stuff | 12:06 |
| Cimi | tsdgeos, neither I am | 12:06 |
| Cimi | tsdgeos, but was asking your opinion | 12:06 |
| tsdgeos | Cimi: but with motorola sold to lenovo, i'm wondering how much live motog has :D | 12:06 |
| Cimi | hah | 12:06 |
| mhr3_ | Cimi, well n4 is like $200 in us, which is about 120gbp, so it's actually pretty similar | 12:06 |
| mhr3_ | wonder why the uk price is so crazy | 12:06 |
| Cimi | mhr3_, isn't it out of production? | 12:07 |
| mhr3_ | well yea, now it is | 12:07 |
| Cimi | mhr3_, 200 is a lot | 12:07 |
| Cimi | mhr3_, moto G is 99$ in us | 12:07 |
| mhr3_ | ok, nvm then :) | 12:07 |
| mzanetti | greyback: you already tested dandraders branch on a rotated N7? | 12:07 |
| greyback | mzanetti: not in isolation yet. But if you give me 5 mins, I can | 12:08 |
| mzanetti | greyback: cool. I'll test it with standard rotation | 12:08 |
| mzanetti | code looks ok to me | 12:08 |
| Cimi | mhr3_, 200$? http://www.amazon.com/s/ref=nb_sb_noss_1?url=search-alias%3Daps&field-keywords=nexus%204 | 12:09 |
| Cimi | it's 360 | 12:09 |
| mhr3_ | Cimi, understandable since google doesn't sell anymore | 12:10 |
| mhr3_ | but it was 200 when they did | 12:10 |
| Cimi | mhr3_, I know | 12:10 |
| Cimi | mhr3_, but now, we're spending lots for getting out of production phones | 12:11 |
| Cimi | mhr3_, wondering if it'll happen like galaxy nexus | 12:11 |
| mhr3_ | Cimi, problem with any hardware, no? either n4 or moto g will be obsolete in a few months | 12:12 |
| Cimi | well, moto g nope | 12:13 |
| Cimi | it's on the market since a month | 12:13 |
| Cimi | nexus 4 over a year | 12:13 |
| greyback | android 4.4 (kitkat) supports Nexus4, but the GNexus doesn't. So N4 should be good for a long while yet | 12:13 |
| Cimi | greyback, issue is that we can't buy them easily anymore | 12:14 |
| Cimi | greyback, I'd be happy to try porting ubuntu touch on my moto g (my private phone) | 12:14 |
| Cimi | it's a jewel for its price | 12:14 |
| Cimi | battery lasts 3 days with the pebble :) | 12:14 |
| mhr3_ | tsdgeos, we lost the search activity indicator btw | 12:14 |
| tsdgeos | mhr3_: in new-scopes? | 12:15 |
| mhr3_ | tsdgeos, also in trunk | 12:15 |
| greyback | Cimi: sure, it's not the newest phone, but our software stack will support it for at least another year probably. | 12:15 |
| tsdgeos | mhr3_: waaaa | 12:15 |
| tsdgeos | but we didn't merge anything that touched that :S | 12:15 |
| mhr3_ | tsdgeos, well, better double check with vanilla image, my isn't exactly vanilla :) | 12:16 |
| tsdgeos | mhr3_: seems it got lost yes :S | 12:16 |
| tsdgeos | mhr3_: can you open a bug and assign it to me plz | 12:16 |
| tsdgeos | mhr3_: did you guys change anything in the backend? | 12:16 |
| mhr3_ | nope | 12:16 |
| mhr3_ | at least i'm not aware.. maybe we did break it | 12:17 |
| mhr3_ | but it's not like we touched the old scope code | 12:17 |
| mhr3_ | tsdgeos, https://bugs.launchpad.net/unity8/+bug/1279316 | 12:19 |
| ubot5 | Launchpad bug 1279316 in Unity 8 "Search activity indicator not present" [Undecided,New] | 12:19 |
| tsdgeos | mhr3_: tx | 12:20 |
| mhr3_ | tsdgeos, something's broketh http://imgur.com/8jKw2zf | 12:24 |
| tsdgeos | booo | 12:24 |
| mhr3_ | video + header + actions | 12:25 |
| tsdgeos | i'd say video is broken | 12:25 |
| tsdgeos | header + action looks good | 12:25 |
| tsdgeos | assuming that 16xxxx is the header | 12:25 |
| mhr3_ | yea | 12:25 |
| tsdgeos | mhr3_: so it seems the search has been broken for a while | 12:27 |
| tsdgeos | just nooen noticed | 12:27 |
| tsdgeos | i know what's wrong | 12:27 |
| mhr3_ | tsdgeos, also, the carousel doesn't show anything if there are only a few items | 12:27 |
| tsdgeos | mhr3_: new-scopes? | 12:28 |
| mhr3_ | (just category header) | 12:28 |
| mhr3_ | yea | 12:28 |
| tsdgeos | should show a grid | 12:28 |
| mhr3_ | it doesn't | 12:28 |
| tsdgeos | but it may happen that the magic is not working | 12:28 |
| * tsdgeos writes to the TODO list | 12:28 | |
| === MacSlow is now known as MacSlow|lunch | ||
| mhr3_ | tsdgeos, maybe you want to check the demo ppa, there may be other issues i'm ignoring :) | 12:30 |
| tsdgeos | mhr3_: sure i know | 12:30 |
| tsdgeos | mhr3_: i'm just fixing the ones we know about first ;) | 12:30 |
| mhr3_ | k | 12:31 |
| mzanetti | tsdgeos: huh? https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1165/? | 12:31 |
| tsdgeos | mzanetti: yes? | 12:31 |
| mzanetti | tsdgeos: did we get this test released in a broken state? | 12:31 |
| tsdgeos | yes | 12:31 |
| mzanetti | and your branch from today morning fixes this, ok | 12:31 |
| tsdgeos | don't tell anyone | 12:31 |
| mzanetti | no prob | 12:31 |
| mzanetti | was just wondering how daniel's branch could fail on this now | 12:31 |
| tsdgeos | everything is failing until we merge the fix | 12:32 |
| mzanetti | ack | 12:32 |
| mzanetti | greyback: please top approve when you're happy with the DDA on Nexus 7: https://code.launchpad.net/~dandrader/unity8/rotatedDDA/+merge/205845 | 12:35 |
| * mzanetti is happy with it | 12:35 | |
| greyback | mzanetti: ack | 12:35 |
| tsdgeos | mhr3_: if you feel like reviewing https://code.launchpad.net/~aacid/unity8/bug1279316/+merge/205952 that'd be great otherwise i can try to get Cimi or mzanetti to do it | 12:37 |
| mhr3_ | tsdgeos, acked | 12:40 |
| tsdgeos | Cimi: can you quick fix https://code.launchpad.net/~unity-team/unity8/unity8.preview_video/+merge/205802 ? | 12:40 |
| tsdgeos | mhr3_: tx | 12:40 |
| Cimi | tsdgeos, mmm | 12:41 |
| Cimi | tsdgeos, I have implicitWidth | 12:41 |
| tsdgeos | height? | 12:41 |
| Cimi | tsdgeos, width | 12:42 |
| tsdgeos | yes | 12:42 |
| tsdgeos | i know | 12:42 |
| tsdgeos | yes | 12:42 |
| Cimi | tsdgeos, I got inspired by MoviePreview | 12:42 |
| tsdgeos | but what about height? | 12:42 |
| Cimi | tsdgeos, it is calculated from width | 12:42 |
| tsdgeos | the inner one | 12:42 |
| tsdgeos | not the outer one | 12:42 |
| tsdgeos | your outer widget ahs no height | 12:42 |
| Cimi | tsdgeos, but if I set height | 12:43 |
| Cimi | tsdgeos, then how does the inner widget work? | 12:43 |
| Cimi | if it is supposed to calculate from width? | 12:43 |
| tsdgeos | you're missing a height: childrenRect.implicitHeight | 12:43 |
| tsdgeos | or something along these lines | 12:43 |
| Cimi | I am not sure | 12:43 |
| Cimi | are yo? | 12:43 |
| tsdgeos | i am | 12:43 |
| tsdgeos | your widget has 0 height | 12:43 |
| tsdgeos | how is that valid? | 12:44 |
| Cimi | tsdgeos, other widget might have 0 width then | 12:44 |
| Cimi | tsdgeos, we never specify width | 12:44 |
| tsdgeos | we do | 12:44 |
| Cimi | I cannot see implicitWidth here | 12:44 |
| Cimi | on other widgets | 12:44 |
| tsdgeos | i know | 12:45 |
| Cimi | so why height and not width? | 12:45 |
| tsdgeos | because width is set by the widget creator | 12:45 |
| tsdgeos | i.e. | 12:45 |
| tsdgeos | http://paste.ubuntu.com/6919775/ | 12:45 |
| tsdgeos | this is what the Preview does | 12:45 |
| tsdgeos | creates one widget after the other | 12:46 |
| tsdgeos | and gives them widget | 12:46 |
| tsdgeos | s/widget/width | 12:46 |
| tsdgeos | but can't give them height because it doesn't know | 12:46 |
| tsdgeos | Cimi: basically see http://imgur.com/8jKw2zf | 12:46 |
| tsdgeos | the video doesn't define height | 12:47 |
| tsdgeos | so stuff can't know where to add itself | 12:47 |
| Cimi | tsdgeos, I see | 12:47 |
| Cimi | tsdgeos, so it's basically to fit our use case | 12:47 |
| tsdgeos | and to do the right thing :-) | 12:47 |
| tsdgeos | every widget should either define width/height/implicitWidth/implicitHeight correctly | 12:48 |
| tsdgeos | if there are that do not | 12:48 |
| tsdgeos | well we have to fix them, but if the missing thing is width/implicitWidth it's less pressing since our current use case doesn't need them | 12:48 |
| Cimi | tsdgeos, but we don't for width | 12:48 |
| Cimi | yeah | 12:48 |
| tsdgeos | Cimi: in which one we don't? | 12:48 |
| Cimi | tsdgeos, look at others, we don't add implicitWidth iirc | 12:50 |
| Cimi | tsdgeos, btw, make tryPreviewWidgetFactory is empty | 12:50 |
| tsdgeos | yaeh it's a bit useless as try | 12:50 |
| tsdgeos | Cimi: you're right, we don't set width because they are done thinking on this usecase | 12:51 |
| mzanetti | tsdgeos: what was the reason why we changed the return value of findChild from undefined to null? | 12:51 |
| tsdgeos | where their width won't matter | 12:51 |
| tsdgeos | mzanetti: Saviq said it was "more proper" | 12:51 |
| mzanetti | ok | 12:51 |
| tsdgeos | mhr3_: in that screenshot which is the order that you have? movie, header, action? | 12:52 |
| Cimi | tsdgeos, I pushed, can you test if works for you now? | 12:52 |
| mhr3_ | tsdgeos, think so | 12:53 |
| tsdgeos | ok | 12:53 |
| === alan_g is now known as alan_g|lunch | ||
| tsdgeos | Cimi: looks good | 12:55 |
| Cimi | tsdgeos, but works? | 12:56 |
| tsdgeos | Cimi: in my testing yes, the screenshot is from mhr3_, it'll be easier for him to really test | 12:56 |
| Cimi | tsdgeos, also, the 16/10 multiplication isn't really ideal | 12:56 |
| dandrader | what do I do when I get an error like that on a "apt-get dist-upgrade"? http://paste.ubuntu.com/6919807/ | 12:56 |
| tsdgeos | mhr3_: can you remerge lp:~unity-team/unity8/unity8.preview_video and test that? | 12:56 |
| Cimi | tsdgeos, as might not be aligned to gu | 12:56 |
| tsdgeos | Cimi: what wait | 12:56 |
| tsdgeos | Cimi: why did you do that? | 12:56 |
| Cimi | tsdgeos, this was in moviepreview | 12:57 |
| tsdgeos | ah | 12:57 |
| tsdgeos | it's not new | 12:57 |
| tsdgeos | ok | 12:57 |
| tsdgeos | i guess it's the "cinema aspect ratio" | 12:57 |
| tsdgeos | need to get food | 12:57 |
| tsdgeos | back later | 12:57 |
| Cimi | I'll go cooking as well then | 12:57 |
| anpok | mhr3_: hm no pin entry possible in the indicators.. | 13:03 |
| mhr3_ | tsdgeos, yep, fixed | 13:04 |
| mhr3_ | tsdgeos, btw all the widgets still disappear sometimes when swiping through the previews and going back | 13:04 |
| mhr3_ | tsdgeos, also from the designs it looks like they want to put art-less cards into an ubuntushape | 13:24 |
| mhr3_ | not sure if that's even supported | 13:24 |
| mhr3_ | and mascot shouldn't be in ubuntushape | 13:24 |
| === MacSlow|lunch is now known as MacSlow | ||
| === dandrader is now known as dandrader|afk | ||
| === alan_g|lunch is now known as alan_g | ||
| === dandrader|afk is now known as dandrader | ||
| tsdgeos | mhr3_: the disapearring is because the listview chnages, i'll add it to the list to see how hard or not is to fix | 14:01 |
| tsdgeos | mhr3_: to be honest i haven't done much regarding the card layouts, not sure what is supported or not, if you give me an specific example i can dig on it | 14:02 |
| mhr3_ | tsdgeos, i'm trying to get cards like https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgQ002UmQxbjJmMHc/edit | 14:02 |
| mhr3_ | minus the custom background color | 14:03 |
| tsdgeos | so mascot, title et all | 14:03 |
| tsdgeos | mhr3_: doesn't work at all? | 14:03 |
| tsdgeos | mhr3_: if you give me the json you're using i can make it work if needed | 14:03 |
| mhr3_ | tsdgeos, http://imgur.com/PxvSV1q | 14:04 |
| mhr3_ | grid, medium, using title, subtitle, mascot | 14:04 |
| tsdgeos | ok, i'll have a look | 14:04 |
| mhr3_ | tsdgeos, note that mascots in previews do have the ubuntushape | 14:05 |
| tsdgeos | mhr3_: you're sure they never have to have it? | 14:05 |
| tsdgeos | or? | 14:05 |
| mhr3_ | ^ | 14:05 |
| mhr3_ | tsdgeos, rosie will know if never ever :) | 14:06 |
| mhr3_ | anyway, /me lunch | 14:06 |
| tsdgeos | err | 14:06 |
| tsdgeos | mzanetti: wtf happened on the train ci document? | 14:06 |
| tsdgeos | sil2100: didrocks: any idea why it replaced all our stuff we had written in there? | 14:07 |
| didrocks | tsdgeos: seems that google is having some issues | 14:07 |
| didrocks | tsdgeos: the spreadsheet was reverted + some random failures when editing cells | 14:07 |
| tsdgeos | awesome | 14:08 |
| didrocks | I opened a RT for that, being fighting it for 5 hours | 14:08 |
| didrocks | yeah awesome, really having fun :/ | 14:08 |
| tsdgeos | that'll teach us of depending on google for crucial stuff | 14:08 |
| mzanetti | tsdgeos: huh? | 14:08 |
| tsdgeos | mzanetti: the MP have gone away and there's the old content, and a nice warning syaing we are doing it wrong | 14:08 |
| tsdgeos | well they are doing it wrong for overwriting the stuff we had | 14:09 |
| mzanetti | spreadsheets yay :) | 14:09 |
| mzanetti | tsdgeos: hmm... not sure... maybe kgunn resumed his machine and that implicitly saved the old state again or something like that | 14:10 |
| didrocks | mzanetti: I wonder if there isn't any crazy client, yeah | 14:11 |
| kgunn | tsdgeos mzanetti i'm on it | 14:11 |
| didrocks | even if you edit the spreadsheet, it's telling "last edit was 10 hours ago" | 14:11 |
| mzanetti | didrocks: side question: is it important to put links to merge requests there or are branches ok too? | 14:12 |
| tsdgeos | mzanetti: look at the red cell | 14:12 |
| mzanetti | hah. right | 14:12 |
| mzanetti | yeah | 14:12 |
| didrocks | mzanetti: MP requests is important, as it's what is being checked for following the MP procedure | 14:12 |
| mzanetti | yeah, that's what I thought | 14:13 |
| mzanetti | could it be that using branches in there messed it up? | 14:13 |
| didrocks | I don't think so, it's free-form text for the scripts | 14:14 |
| mzanetti | kgunn: jfi: I've updated it to contain all the approved branches from there: https://code.launchpad.net/~unity-team/unity8/trunk/+activereviews | 14:15 |
| didrocks | the issue is global on the spreadsheet, even showing/hiding a column is resulting in an error | 14:15 |
| mzanetti | kgunn: so if you're updating it again, I guess that's what it should be | 14:15 |
| mzanetti | + any unity-mir stuff | 14:15 |
| didrocks | mzanetti: see, I see your changes, but it's telling "last edit was 10 hours ago" | 14:15 |
| kgunn | mzanetti: when you say "it"...do you mean cell F75 ? | 14:16 |
| mzanetti | kgunn: yes | 14:16 |
| kgunn | i just updated based on the list albert gave me...i'll go check against approved | 14:16 |
| mzanetti | kgunn: yeah, albert and me approved a bunch of branches today morning | 14:16 |
| kgunn | mzanetti: it appeared "old"....i didn't see your changess | 14:16 |
| didrocks | juts tell me once it's ready | 14:17 |
| mzanetti | didrocks: yeah, that seems wrong. should be 4 - 5 hours by now | 14:17 |
| kgunn | mzanetti: i'll sort it | 14:17 |
| mzanetti | kgunn: so yeah, its the existing unity-mir stuff + all approved in unity8 | 14:17 |
| kgunn | mzanetti: when you say "existing unity-mir" stuff am i safe to just take all the approved MP's there ? or are you after something specific ? | 14:18 |
| mzanetti | kgunn: I mean those already In the document. all I did for unity-mir was to convert them to point to MPs instead of branches | 14:18 |
| mzanetti | not added any new ones | 14:19 |
| kgunn | mzanetti: ok, i just reloaded the spreadsheet...and looking at cell F57 i see no unity-mir mp's | 14:20 |
| mzanetti | hmm... damn... I could swear there was one... but maybe I'm wrong then | 14:21 |
| tsdgeos | mzanetti: i think there was none | 14:24 |
| tsdgeos | unity-mir branches i mean | 14:25 |
| tsdgeos | kgunn: video call? | 14:30 |
| tsdgeos | @unity: video call? | 14:30 |
| dandrader | oh no | 14:31 |
| tsdgeos | dandrader: what no? | 14:33 |
| dandrader | video call. takes me a while to get the laptop etc | 14:34 |
| greyback | mterry: as part of the qml compositor work, notification of frame ready to draw was important (includes first frame ready) - this was my rough approach: http://bazaar.launchpad.net/~gerboland/+junk/mir-for-qpamirserver/revision/1198 | 14:44 |
| greyback | mterry: do have a chat with racarr, as he's working on trying to add that notification to Mir properly | 14:45 |
| mterry | greyback, thanks will look | 14:45 |
| mterry | oh nice | 14:45 |
| mhr3 | tsdgeos, the spec was updated https://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1#heading=h.9298l1cke028 | 14:55 |
| mhr3 | (ubushape stuff) | 14:55 |
| tsdgeos | ok | 14:56 |
| tsdgeos | mzanetti: can you press the build button on sili 9? | 15:09 |
| tsdgeos | kgunn: or you? | 15:09 |
| tsdgeos | sili -> silo | 15:10 |
| kgunn | tsdgeos: you got it | 15:10 |
| tsdgeos | mhr3: i'm going to have look at the carousel/grid thing | 15:12 |
| tsdgeos | mhr3: do yo have an easy way to make it happen? | 15:12 |
| mhr3 | tsdgeos, i'm getting it on the phone when i search in videos | 15:16 |
| mhr3 | cause it's not like i had hundreds of videos there | 15:16 |
| tsdgeos | mhr3: it works for me in the unity-scope-tool | 15:21 |
| tsdgeos | mhr3: i search for maps | 15:21 |
| tsdgeos | there's two results | 15:21 |
| tsdgeos | then override the thing to carousel | 15:22 |
| tsdgeos | and i still get a grid | 15:22 |
| tsdgeos | mhr3: can you check if that works for you? | 15:22 |
| === alan_g is now known as alan_g|tea | ||
| mhr3 | tsdgeos, when doing that yes it does | 15:23 |
| tsdgeos | :/ | 15:23 |
| mhr3 | i think the "go from carousel to grid" is broken | 15:23 |
| tsdgeos | mhr3: is there a scope for the desktop that would give me a carousel by default? | 15:25 |
| mhr3 | tsdgeos, unity-scope-mediascanner2 | 15:26 |
| om26er | tsdgeos, Hey! how to test the new-scopes branch ? I ran unity from it and nothing showed up | 15:28 |
| tsdgeos | om26er: how did you install it? | 15:29 |
| om26er | tsdgeos, on desktop ran ./build and for a nexus 4 build a deb from jenkins | 15:29 |
| tsdgeos | om26er: use the demo-stuff ppa since you need to get the new scopes too | 15:29 |
| tsdgeos | om26er: https://launchpad.net/~unity-team/+archive/demo-stuff | 15:30 |
| om26er | tsdgeos, great, testing now | 15:31 |
| === alan_g|tea is now known as alan_g | ||
| tsdgeos | mhr3: so we need a new way for the json to give us a color (regarding the shaping)? | 15:36 |
| tsdgeos | mhr3: any clue how to make unity-scope-mediascanner2 find anything in my pc? | 15:36 |
| mhr3 | tsdgeos, it should search your ~/Music | 15:36 |
| tsdgeos | ok i'll put something in that folder then | 15:37 |
| * tsdgeos creates the folder | 15:37 | |
| tsdgeos | Caught exception from create_query()Caught exception from create_query()Caught exception from create_query()Caught exception from create_query()Caught exception from create_query() | 15:38 |
| tsdgeos | ** (process:13771): WARNING **: Unable to allocate query: S'ha esgotat el temps d'espera | 15:38 |
| tsdgeos | doesn't look good | 15:38 |
| mhr3 | tsdgeos, what does the last thing mean? | 15:41 |
| tsdgeos | that's what searching for music gives me :D | 15:41 |
| tsdgeos | on unity8 | 15:41 |
| tsdgeos | ah sorr | 15:41 |
| tsdgeos | you mean the catalan part :D | 15:41 |
| mhr3 | yes | 15:41 |
| tsdgeos | basically timeout | 15:41 |
| tsdgeos | there's no waiting time left | 15:41 |
| mhr3 | tsdgeos, start mediascanner2; restart scope-registry | 15:41 |
| tsdgeos | would be the rough translation | 15:41 |
| tsdgeos | same thing | 15:43 |
| tsdgeos | ah wait mediascanner is using my cpu | 15:43 |
| tsdgeos | let it do stuff then | 15:43 |
| mhr3 | tsdgeos, latest libunity-scopes0 & unity-plugin-scopes? | 15:43 |
| tsdgeos | from demo-stuff yes | 15:43 |
| mhr3 | latest is in distro :) | 15:43 |
| mhr3 | the question is whether it will even finish, iirc it wasn't much tested on the desktop | 15:44 |
| tsdgeos | well yes, i'm totally dist-upgraded | 15:45 |
| tsdgeos | ah nice | 15:45 |
| tsdgeos | it's back at 99% | 15:45 |
| tsdgeos | i remember why i deleted it | 15:45 |
| * tsdgeos kills it again | 15:45 | |
| mhr3 | tsdgeos, but this is mediascanner2! | 15:45 |
| mhr3 | it's supposed to like... work and stuff | 15:46 |
| mhr3 | tsdgeos, on, wait got better idea how you can get carousel | 15:49 |
| mhr3 | tsdgeos, the onlinemusic scope installs a renderer.json, just change that | 15:50 |
| mhr3 | should take effect on next search | 15:50 |
| === dandrader is now known as dandrader|lunch | ||
| === gatox is now known as gatox_lunch | ||
| mzanetti | anpok: ping | 16:03 |
| tsdgeos_ | mzanetti: build is done | 16:03 |
| mzanetti | tsdgeos_: nice | 16:03 |
| * mzanetti prepares a device | 16:03 | |
| tsdgeos_ | mzanetti: i'll run autopilot twice and qmluitests too, can you do one autopilot and the manual piloting? | 16:03 |
| mzanetti | tsdgeos_: awesome. seems we developed a quite efficient workflow with this :D | 16:04 |
| tsdgeos_ | mhr3: tx, will check | 16:04 |
| anpok | mzanetti: pong | 16:06 |
| mzanetti | anpok: hey. I've just seen this bug you reported: https://bugs.launchpad.net/indicator-network/+bug/1279335 | 16:06 |
| ubot5 | Launchpad bug 1279335 in Network Menu "no pin entry possible" [Undecided,New] | 16:06 |
| mzanetti | anpok: this _should_ be working... | 16:07 |
| mzanetti | if not, then something is broken | 16:07 |
| mzanetti | anpok: so the pin entry dialog sits in unity. what happens is that the network indicator triggers a notification and unity displays that | 16:08 |
| mzanetti | did it not trigger the dialog for you? | 16:08 |
| tsdgeos_ | Cimi: do not understand this question "Question, is total of Ubuntu.DownloadDaemonListener 0.1 always > 0?" | 16:11 |
| Cimi | tsdgeos_, 'total' | 16:12 |
| Cimi | tsdgeos_, if it's 0, we divide by 0 | 16:12 |
| Cimi | tsdgeos_, was just thinking of corner cases | 16:12 |
| mzanetti | its javascript. what can happen :P | 16:12 |
| Cimi | tsdgeos_, btw, you appear as a pink coloured nick | 16:12 |
| tsdgeos_ | i'm with mzanetti | 16:13 |
| Cimi | tsdgeos_, very girlie, sorry | 16:13 |
| tsdgeos_ | it's javascript | 16:13 |
| tsdgeos_ | nothing will crash | 16:13 |
| mzanetti | actually that was a joke | 16:13 |
| tsdgeos_ | Cimi: besides this code is just a copy & paste from what we have in the other preview | 16:13 |
| tsdgeos_ | Cimi: so i will guess it's working | 16:13 |
| Cimi | tsdgeos_, no reason why we should not fix if there are issues | 16:13 |
| tsdgeos_ | Cimi: sure, what is the issue? | 16:14 |
| Cimi | apart that /me remembers a bug with progress | 16:14 |
| Cimi | tsdgeos_, 55+ var percentage = parseInt(received * 100 / total); | 16:14 |
| Cimi | 56+ progressBar.value = percentage; | 16:14 |
| mzanetti | hmm... true... I think it doesn't crash but might cause percentage to be infinite or something like that | 16:14 |
| Cimi | can we rely on total? | 16:14 |
| tsdgeos_ | hmmmm | 16:14 |
| tsdgeos_ | wait where is that parseInt comming from | 16:14 |
| Cimi | tsdgeos_, your code | 16:14 |
| Cimi | hah | 16:15 |
| tsdgeos_ | Cimi: my copied code ;) | 16:15 |
| Cimi | +1 :P | 16:15 |
| Cimi | apparently it's something coming down from dbus | 16:15 |
| tsdgeos_ | ok, that's javascript proper | 16:15 |
| Cimi | I had a quick look before on the plugin code, briefly | 16:15 |
| Cimi | it looked to me it was coming from somewhere else | 16:15 |
| tsdgeos_ | parseInt i am asking | 16:16 |
| tsdgeos_ | not total | 16:16 |
| tsdgeos_ | total comes from dbus yes | 16:16 |
| tsdgeos_ | Cimi: so what is the solution if total is 0 | 16:16 |
| Cimi | mzanetti, https://bugs.launchpad.net/unity8/+bug/1230204 ? | 16:16 |
| ubot5 | Launchpad bug 1230204 in Ubuntu UX "[dash] When installing an app, progress shows 200%" [High,Triaged] | 16:16 |
| Cimi | tsdgeos_, well, it's a division by 0 | 16:16 |
| tsdgeos_ | Cimi: yes | 16:16 |
| mzanetti | Cimi: no. that's not the one I remember. But I think I had something like this in xbmcremote | 16:16 |
| Cimi | tsdgeos_, not sure what js does | 16:17 |
| tsdgeos_ | and that returns a NaN | 16:17 |
| tsdgeos_ | Cimi: so what should percentage be | 16:17 |
| tsdgeos_ | 0 | 16:17 |
| tsdgeos_ | 100? | 16:17 |
| Cimi | I'd say 0 | 16:17 |
| Cimi | well, received has to be more than total | 16:17 |
| mzanetti | shouldn't it actually paint the unknown duration progressbar? | 16:17 |
| Cimi | sorry, less | 16:17 |
| mzanetti | with the continuous animation | 16:17 |
| mzanetti | Cimi: check the uitk gallery -> Progress and acitvity -> infinite progress bar | 16:19 |
| Cimi | tsdgeos_, ^ | 16:20 |
| Cimi | mzanetti, it's not my branch :P | 16:20 |
| mzanetti | sorry... thought you asked the question | 16:20 |
| Cimi | mzanetti, I'm just being the saviq of the situation | 16:20 |
| mzanetti | aaahhh | 16:20 |
| Cimi | mzanetti, needs fixing everything I see | 16:20 |
| Cimi | ahaha | 16:20 |
| mzanetti | :D | 16:20 |
| Cimi | mzanetti, needs fixing! | 16:20 |
| Cimi | mzanetti, it should be :-D | 16:20 |
| mzanetti | I'm afraid so | 16:20 |
| tsdgeos_ | Cimi: pushed | 16:20 |
| mzanetti | tsdgeos_: seems silo 009 doesn't boot successfully | 16:21 |
| tsdgeos_ | mzanetti: : that's baaaad :D | 16:21 |
| * mzanetti tries with a clean flash, just to be sure | 16:21 | |
| tsdgeos_ | what did we break? | 16:21 |
| mzanetti | dunno... unity8 doesn't start... sensorservice on 100% since a while now | 16:22 |
| mzanetti | actually 103.8% | 16:22 |
| mzanetti | :P | 16:22 |
| mzanetti | Needs fixing! | 16:22 |
| tsdgeos_ | mzanetti: works here | 16:23 |
| tsdgeos_ | i had a clean image | 16:23 |
| tsdgeos_ | added the ppa | 16:23 |
| tsdgeos_ | update, dist-upgrade, reboot | 16:23 |
| tsdgeos_ | and i have stuff on screen | 16:23 |
| mzanetti | yeah... did have clean image + yesterday's silo which should be exactly what the lasted upgrade would give me anyways | 16:23 |
| mzanetti | but just to make sure, flashing now | 16:23 |
| anpok | mzanetti: yes didnt happen | 16:24 |
| mzanetti | anpok: do you have an item "Enter sim pin" below the list of WiFi networks? | 16:25 |
| anpok | mzanetti: sim card in - boot - unity8 starts -> pin entry pops up on its own? | 16:25 |
| anpok | nope | 16:25 |
| mzanetti | anpok: hmm... then there seems to be an issue indeed. but not that it's not implemented, but more that it's broken. | 16:26 |
| Cimi | tsdgeos_, can we use anchors instead width and height? :P | 16:26 |
| mzanetti | anpok: can you please add that info to the bug report (i.e. Enter Sim Pin" entry doesn't show in network indicator etc) | 16:26 |
| anpok | mzanetti: since I never saw that I was not sure where to report it :) | 16:26 |
| tsdgeos_ | Cimi: we can, don't see a need tbh | 16:26 |
| anpok | sure | 16:26 |
| mzanetti | anpok: no problem. thanks for reporting :) | 16:27 |
| Cimi | tsdgeos_, saviq asked on my reviews | 16:27 |
| Cimi | tsdgeos_, maybe he has his own reasons | 16:27 |
| anpok | mzanetti: another thing | 16:27 |
| tsdgeos_ | Cimi: he asked on mine too, i told him to stop asking silly stuff :D | 16:27 |
| Cimi | tsdgeos_, btw, test name is PreviewHeaderTest | 16:27 |
| tsdgeos_ | Cimi: wops, too much c&p | 16:27 |
| Cimi | tsdgeos_, ahahah | 16:27 |
| anpok | mzanetti: I still want to avoid the alpha channel were possible.. | 16:28 |
| Cimi | tsdgeos_, I'duse light grey as rectangle color, in line with dash | 16:28 |
| tsdgeos_ | Cimi: where? | 16:28 |
| Cimi | tsdgeos_, test | 16:28 |
| tsdgeos_ | instead of the red? | 16:28 |
| tsdgeos_ | or the white? | 16:28 |
| Cimi | tsdgeos_, actually, since you import ubuntu components, use the palette | 16:28 |
| tsdgeos_ | Cimi: it's a test man, it just has to show the stuff on screen, not be pretty | 16:29 |
| mzanetti | :D | 16:29 |
| anpok | mzanetti: the window class in qtubuntu has an opaque property | 16:29 |
| anpok | but I could not see anybody using that | 16:29 |
| mzanetti | anpok: mhm... yeah might be | 16:29 |
| mzanetti | should we make use of that? can we default it to true? | 16:30 |
| anpok | mzanetti: we cannot dafeult all application windows to true | 16:30 |
| anpok | some popups need to be transparent as far as i can see | 16:30 |
| anpok | *default | 16:30 |
| mzanetti | ah right... | 16:30 |
| mzanetti | well, right now they're inside the shell surface anyways | 16:30 |
| anpok | i.e. when composing messages .. or incoming calls seem to overlay with transparent surfaces.. | 16:31 |
| mzanetti | so yeah, unity8 needs to be transparent | 16:31 |
| mzanetti | but for the rest I don't think we have anything transparent (yet) | 16:31 |
| anpok | ah.. and on screen keyboards? | 16:31 |
| mzanetti | hmm... that's not transparent either as per current design | 16:32 |
| anpok | ok then I will dig into unity8 code | 16:32 |
| anpok | but maybe not this week | 16:32 |
| mzanetti | anpok: should be in unity-mir | 16:32 |
| anpok | k thanks for the pointers | 16:32 |
| mzanetti | np | 16:32 |
| mzanetti | anpok: however, you might have a chat with mterry about it as he's starting to split the unity8 surfaces into different pieces | 16:33 |
| * mterry looks up | 16:34 | |
| mzanetti | mterry: do you need any transparent surface from mir? | 16:35 |
| mzanetti | like notifications for example | 16:35 |
| mterry | mzanetti, yeah, the greeter uses a transparent surface. It's main surface is transparent so you can see the user session beneath | 16:35 |
| mzanetti | (I don't know how much the split greeter will dive into that - but I figured you might need such a thing) | 16:35 |
| mterry | mzanetti, but as far as internally, not so much | 16:36 |
| mzanetti | anpok: so yeah, one more consumer for a transparent surface ^ | 16:36 |
| anpok | in any case I dont see that just changing the default will be sufficient - so there will be an API | 16:37 |
| anpok | https://code.launchpad.net/~andreas-pokorny/platform-api/opaque_by_default/+merge/205259 | 16:37 |
| anpok | so here the OSK seems to make issues - where would I find the code for that one? | 16:38 |
| mzanetti | anpok: oh... reading tvoss's comment I might have been wrong on the OSK | 16:38 |
| mzanetti | anpok: it's called Maliit. the actual code is on github. but I don't know where the glue code to mir is happening. | 16:38 |
| mzanetti | dandrader|lunch: I think you can tell anpok where to find the OSK code... | 16:39 |
| Cimi | tsdgeos_, it looks like that in order to provide custom url for stars | 16:42 |
| Cimi | tsdgeos_, I'll have to write a custom style for rating to use in the dash | 16:43 |
| Cimi | this rating widget requires work to do it properly | 16:43 |
| tsdgeos_ | Cimi: so maybe drop it for now? not sure how critical it is tbh | 16:44 |
| tsdgeos_ | mhr3: ↑↑↑ ? | 16:44 |
| tsdgeos_ | mzanetti: first autopilot run sccessful, running second | 16:44 |
| Cimi | tsdgeos_, drop what? | 16:44 |
| tsdgeos_ | Cimi: i mean fix/do other things that may be more critical for now | 16:44 |
| Cimi | tsdgeos_, I only have rating on my plate now | 16:45 |
| tsdgeos_ | Cimi: i.e. you that like pretty things, run new-scopes and fix all the uglyness | 16:45 |
| Cimi | and I'm happy to finish it | 16:45 |
| tsdgeos_ | like for example no clue what happened with the greeter | 16:45 |
| tsdgeos_ | but it's ultra black | 16:45 |
| mhr3 | Cimi, tsdgeos_, if by default you don't have to provide any icons, i'm only happy | 16:45 |
| tsdgeos_ | ok | 16:45 |
| mhr3 | and if the non-default doesn't work... meh tiny bug :P | 16:46 |
| Cimi | mhr3, if rating-label or review-label is null, we use a default? | 16:47 |
| Cimi | or submit label | 16:48 |
| Cimi | like | 16:48 |
| mhr3 | Cimi, yes, that's what the spec says | 16:48 |
| Cimi | gotcha | 16:48 |
| mzanetti | tsdgeos_: rebooting freshly flashed image + silo 9 packages... | 16:49 |
| mzanetti | yep. looking better | 16:50 |
| * mzanetti walks through the checklist | 16:50 | |
| mzanetti | dednick: hmm... seeing something weird in the indicators... | 16:53 |
| dednick | ? | 16:53 |
| mzanetti | if you pull down and then move left/right to switch tabs, it doesn't always follow the finger | 16:53 |
| mzanetti | as in: if you move the finger while another move animation is running, it doesn't get that secnd movement any more | 16:53 |
| dednick | hm | 16:54 |
| mzanetti | could that be something introduced by the 2 indicator branches? | 16:54 |
| mzanetti | I don't remember seeing that in trunk... but I'm not sure | 16:54 |
| dednick | mzanetti: so it's not getting to the correct indicator? | 16:55 |
| mzanetti | dednick: yeah | 16:55 |
| dednick | mzanetti: which image? | 16:56 |
| mzanetti | freshly flashed trusty-proposed + silo 009 packages | 16:56 |
| mzanetti | whcih contain: https://code.launchpad.net/~nick-dedekind/unity8/indicator-client-profile-switch/+merge/203985 | 16:56 |
| mzanetti | and https://code.launchpad.net/~nick-dedekind/unity8/plugin-qml/+merge/203810 | 16:57 |
| mzanetti | ok. its probably not the latter one | 16:57 |
| dednick | mzanetti: hm. i doubt its either of those. | 16:58 |
| === gatox_lunch is now known as gatox | ||
| mzanetti | dednick: its not completely broken... but I remember this used to be working better | 16:58 |
| dednick | possibly lp:~nick-dedekind/unity8/indicators.better-visible-items or lp:~nick-dedekind/unity8/lp1264678-indicator.misalignment | 16:58 |
| mzanetti | anyways... ignoring for now as its really just a small visual glitch | 16:58 |
| tsdgeos_ | mzanetti: second autopilot successful too, going to run the qmluitests now | 16:58 |
| Cimi | can I make a button half transparent when it's disabled? | 16:58 |
| Cimi | in the previews... want to make the submit button transparent when the review is required and you didn't add it | 16:59 |
| Cimi | oh actually, there's a palette! | 16:59 |
| dednick | mzanetti: does the swiping just feel a bit "sticky"? or is there something more specific? | 16:59 |
| mzanetti | dednick: it just feels a bit sticky so to say... sometimes it misses a finger movement if its still busy animating the previous finger movement | 17:00 |
| dednick | by swiping i mean tab switching by left/right move | 17:00 |
| mzanetti | but moving the finger a bit more makes it recover again | 17:00 |
| dednick | mzanetti: any vertical movement while swiping? | 17:01 |
| mzanetti | hmm... maybe a little bit... but not really | 17:01 |
| dednick | mzanetti: hm. it does feel a bit sticker than it used to i think | 17:02 |
| dednick | mzanetti: i'll have to go back through revisions to figure it out though | 17:02 |
| mzanetti | yeah... its actually not even easy to reproduce manually... but if you just use it, every once in a while it ends up at the wrong one | 17:02 |
| mzanetti | dednick: in any case. not blicking the release on it | 17:03 |
| dednick | ok, can you log a bug? | 17:03 |
| dednick | mzanetti: ^ | 17:03 |
| mzanetti | ok | 17:03 |
| dednick | thanks | 17:03 |
| === dandrader|lunch is now known as dandrader | ||
| mzanetti | tsdgeos_: silo9 works for me. in the manual testing. let me know when the second AP run is done and I'll approve it | 17:06 |
| tsdgeos_ | mzanetti: second autopilot is done, i want to run qmluitests once | 17:06 |
| tsdgeos_ | will take a bit | 17:06 |
| mzanetti | oh right | 17:06 |
| mzanetti | tsdgeos_: you doing qmltests on the phone? | 17:06 |
| tsdgeos_ | i'm going to risk it :D | 17:07 |
| tsdgeos_ | mzanetti: why? | 17:07 |
| mzanetti | tsdgeos_: last time I tried I think there were sme issues because of the stuff we paint around it (controls etc) | 17:07 |
| mzanetti | they were to big for the screen and caused issues | 17:07 |
| tsdgeos_ | ah | 17:07 |
| tsdgeos_ | may happen | 17:07 |
| dandrader | anpok, mzanetti: OSK code: lp:ubuntu/maliit-framework and lp:ubuntu-keyboard | 17:07 |
| tsdgeos_ | ok, let me run them on the PC then | 17:07 |
| mzanetti | thanks danie | 17:07 |
| mzanetti | l | 17:07 |
| mzanetti | tsdgeos_: well, ideally that would work too | 17:08 |
| mzanetti | so wouldn't hurt pasting the failures somewhere | 17:08 |
| tsdgeos_ | ok | 17:10 |
| mhr3 | robru, ideas when will unity-scope-scopes get from the new queue? | 17:12 |
| robru | mhr3, didrocks is here today so it's up to him to fix it... should be soonish | 17:12 |
| tsdgeos_ | mzanetti: qmluitests fine on PC | 17:28 |
| mzanetti | tsdgeos_: ack. will push the button | 17:30 |
| tsdgeos_ | ok | 17:30 |
| mzanetti | didrocks: Testing done on Silo 9 :) | 17:31 |
| didrocks | thanks mzanetti | 17:31 |
| Cimi | mmm rating widget | 17:32 |
| Cimi | I have enough :) | 17:32 |
| Cimi | hah | 17:32 |
| karni | tsdgeos_: from blueprint "[saviq] greeter with paper overlay - with dash - Saviq: INPROGRESS" -- do you know anything about this? | 17:39 |
| karni | It does in fact look bit strange to have this vilolet/aubergine greeter, and paper dash background, perhaps I could fix it later today. | 17:40 |
| karni | mterry: actually, you might know something ^ | 17:40 |
| mterry | karni, I haven't been following the paper design changes | 17:41 |
| mterry | karni, I haven't heard what design wants for the greeter | 17:41 |
| karni | I assume they want greeter with paper overlay, since that's what Saviq ^ seemed to have worked on | 17:42 |
| karni | And since he's not here, we might want to pick this up (might be me, just going through list of items we need to wrap some work) | 17:42 |
| mterry | karni, do we have a mockup? | 17:48 |
| karni | of the greeter? don't know, will check | 17:49 |
| karni | mterry: can't see it in design drive. will contact design. | 17:50 |
| karni | mterry: btw what have you been working on with then? for the multiple user greeter? | 17:51 |
| karni | mterry: sent e-mail, Cc'ed you | 17:52 |
| mterry | karni, cool thanks | 17:53 |
| mterry | karni, I've been working on splitting the greeter out into a separate process managed by lightdm, run under a different user | 17:54 |
| mterry | karni, this is needed to eventually have actual security for the user session | 17:54 |
| karni | mterry: I see. so, little to do with UI | 17:55 |
| mterry | karni, the basics of the split are easy, but there are so many points of communication between the user session and greeter and hitting odd Mir cases since I'm pushing its use cases | 17:55 |
| mterry | karni, right, that's why I haven't been paying much attention | 17:55 |
| karni | I need leave for ~3 hours to the airport, I'll be back after that time. Sorry for my spotty presence, guys :/ | 17:55 |
| karni | mterry: ack | 17:55 |
| mterry | karni, if I do my job right, no one will notice a darn thing. Which is sort of frustrating ;) | 17:55 |
| karni | hahaha | 17:55 |
| karni | :) | 17:55 |
| === Trevinho_ is now known as Trevinho | ||
| kgunn | mzanetti: do you know if anyone is promoting the packages to archive from the unity8 landing attempt ? (moments ago) | 17:56 |
| dandrader | tsdgeos_, still there? | 17:58 |
| mzanetti | kgunn: Is there anything special we need to do? | 18:01 |
| kgunn | mzanetti: nope just checking | 18:01 |
| mzanetti | kgunn: I've set "Testing done" to yes and pinged the release team about it. afaik that should make it happen soon | 18:01 |
| kgunn | mzanetti: who on rel team ? | 18:01 |
| mzanetti | didrocks: | 18:01 |
| kgunn | ack | 18:02 |
| didrocks | mzanetti: it's released bt (well, migrating to the archive) | 18:04 |
| didrocks | you will just need to merge and cleaned once it reaches the release pocket | 18:04 |
| mzanetti | didrocks: perfect. thanks! | 18:05 |
| mzanetti | kgunn: ^ | 18:05 |
| didrocks | yw | 18:05 |
| didrocks | hum, seems someone rerun the build job after it though | 18:05 |
| didrocks | kgunn: it's you, what have you done? :) | 18:05 |
| mzanetti | :D | 18:06 |
| kgunn | didrocks: its why i was asking around... | 18:06 |
| didrocks | kgunn: this didn't had any impact, just run "merge and clean" as expected | 18:06 |
| didrocks | ah ok | 18:06 |
| kgunn | i accidently hit the 009 instance | 18:06 |
| kgunn | i figured | 18:06 |
| kgunn | cause you use ppas right didrocks | 18:06 |
| kgunn | ? | 18:06 |
| didrocks | kgunn: no worry, you will just have some RED on the spreadsheet, color is nice as well! | 18:06 |
| didrocks | yep | 18:06 |
| kgunn | didrocks: you should have seen my face when i realized what i had hit | 18:07 |
| didrocks | kgunn: let me run "merge and clean" right now, it will be another kind of red | 18:07 |
| didrocks | ahah :) | 18:07 |
| didrocks | red as well? ;) | 18:07 |
| kgunn | something like that | 18:07 |
| didrocks | kgunn: so, I just run "merge and clean" right now | 18:07 |
| didrocks | (a little bit early, of course, unity8 isn't in the release pocket yet) | 18:07 |
| didrocks | but at least, the red in the spreadsheet will make more sense | 18:07 |
| didrocks | it's telling you it's in proposed) | 18:08 |
| * kgunn unlocks a way to get release guys to hit buttons sooner :) | 18:12 | |
| didrocks | kgunn: well, not really, you will still have to run merge and clean (and the publication was done already :p) | 18:15 |
| didrocks | kgunn: however, you just made me more satisfied to have done the "check previous step" security :) | 18:15 |
| tsdgeos | karni: mterry: yeah someone needs to make the new-scopes greeter look not totally ugly, if you have time for that, awesome | 18:36 |
| === dandrader is now known as dandrader|bbl | ||
| * greyback eod | 19:08 | |
| anpok | greyback: | 19:16 |
| anpok | ignore /me | 19:16 |
| karni | mterry: I guess no change to greeter background (see e-mail_ | 20:34 |
| mterry | karni, well, it is our brand colors at least. Could have been green & orange | 20:34 |
| * karni falls of the chair :D | 20:34 | |
| === salem_ is now known as _salem | ||
| kgunn | its gonna make this bug stand out like a turd in a puch bowl mterry | 20:58 |
| kgunn | https://bugs.launchpad.net/ubuntu/+source/mir/+bug/1279078 | 20:58 |
| ubot5 | Launchpad bug 1279078 in mir (Ubuntu) "application flashes before showing infographic after turning off screen" [Medium,Confirmed] | 20:58 |
| kgunn | its was kind of camouflaged when dash & greeter were similar | 20:59 |
| kgunn | that high contrast is gonna get some comments :) | 20:59 |
| mterry | kgunn, if you say so... I still shiver every time I turn my device on | 20:59 |
| kgunn | mterry: i know | 21:00 |
| mterry | kgunn, so that might arguably be a dup of bug 1279422, depending on how Mir folks want to solve it | 21:00 |
| ubot5 | bug 1279422 in Mir "permit clients to perform prep logic while screen is blanked" [High,Triaged] https://launchpad.net/bugs/1279422 | 21:00 |
| kgunn | mterry: yeah....partly, but i think not totally the same right? | 21:00 |
| mterry | kgunn, yeah they may have different technical solutions. But similar symptoms | 21:02 |
| kgunn | for sure...gonna try to generate some love for it | 21:02 |
| mhr3_ | bad kgunn, bad! you broke new-scopes | 21:24 |
| mhr3_ | http://bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/619?start_revid=622 causes | 21:24 |
| mhr3_ | https://launchpad.net/~unity-team/+archive/demo-stuff/+build/5582171 | 21:24 |
| kgunn | mhr3_: but i tested locally...and we need that | 21:25 |
| mhr3_ | it's just whitespace test.. shouldn't be exactly hard to fix :) | 21:26 |
| kgunn | mhr3_: i see | 21:26 |
| kgunn | i was like "oh crap what now" | 21:26 |
| mhr3_ | kgunn, i like scaring people :P | 21:28 |
| kgunn | mhr3_: your a sick puppy | 21:28 |
| mhr3_ | lol | 21:32 |
| mhr3_ | thanks? :) | 21:32 |
| === dandrader|bbl is now known as dandrader | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!