| tsdgeos | Cimi: seen email about left/center align? | 08:10 |
|---|---|---|
| Cimi | tsdgeos, yup | 08:11 |
| tsdgeos | oki | 08:12 |
| Cimi | tsdgeos, still summer in barca? | 08:12 |
| tsdgeos | not really | 08:12 |
| tsdgeos | we got quite heavy rain yesterday | 08:13 |
| Cimi | tsdgeos, is quite hot in london | 08:13 |
| Cimi | tsdgeos, I was wondering how was in warmer places | 08:13 |
| tsdgeos | 19 degrees at the moment | 08:13 |
| Cimi | tsdgeos, pushed! | 08:19 |
| tsdgeos | Cimi: cool | 08:21 |
| tsdgeos | Cimi: scopes more things on memory got qmluitests passing at last | 08:21 |
| tsdgeos | Cimi: can you re-review? | 08:21 |
| Cimi | tsdgeos, sure | 08:41 |
| === seb128_ is now known as seb128 | ||
| mzanetti | tsdgeos: hey, can you take over this one from Gerry? https://code.launchpad.net/~mzanetti/unity8/async-dashcommunicator/+merge/235272 | 08:54 |
| mzanetti | tsdgeos: should be mostly ok now.. I.e. we did already a few rounds of review back and forth | 08:54 |
| mzanetti | let me know if you want me to explain why this ended up so complicated | 08:55 |
| * tsdgeos read | 08:56 | |
| tsdgeos | s | 08:56 |
| yecril71pl | How do I make an application short cut on the desktop? | 08:56 |
| tsdgeos | mzanetti: why moved the visibility thing? | 09:00 |
| mzanetti | tsdgeos: hmm... dunno tbh... that was gerry :D. I assume because of the lib | 09:01 |
| mzanetti | tsdgeos: yeah. I guess we should only have this for qml plugins, but not for the lib | 09:01 |
| mzanetti | tsdgeos: but then I don't know exactly | 09:01 |
| mzanetti | want me to investigate? | 09:02 |
| mzanetti | tsdgeos: so at some point I merged a branch from gerry that added the libunity8-private with the AbstractDBusMonitor | 09:02 |
| mzanetti | as he was kind enough to prepare that for me | 09:02 |
| mzanetti | this was in there | 09:02 |
| tsdgeos | mzanetti: yeah i mean we have more code that was previosuly covered by the visibility | 09:03 |
| tsdgeos | want to know why we need the move | 09:04 |
| tsdgeos | and if we really need to remove it that much | 09:04 |
| mzanetti | yeah | 09:04 |
| tsdgeos | mzanetti: should DashCommunicator destructor delete m_dashConnection ? | 09:07 |
| tsdgeos | i guess it's one of those long lived things that doesn't really matter in practice | 09:09 |
| tsdgeos | but just to make it "correct" | 09:09 |
| tsdgeos | and it's not hard | 09:09 |
| mzanetti | tsdgeos: yeah... will fix. also realized it needs merging with trunk | 09:12 |
| tsdgeos | mzanetti: ok, now explain me what the branch is for : | 09:21 |
| tsdgeos | :D | 09:21 |
| mzanetti | tsdgeos: basically the issue is that the dash was never suspended. so I fixed that in lp:qtmir. | 09:21 |
| tsdgeos | i see dash never being suspended as a good thing :D | 09:21 |
| mzanetti | tsdgeos: that caused the issue though that unity8 hangs when it tries to send something through dashcommunicator and the dash is sleeping | 09:21 |
| mzanetti | tsdgeos: its not | 09:21 |
| mzanetti | tsdgeos: so.. dashcommunicator needs to be async | 09:22 |
| tsdgeos | am i speaking to the guy that don't like suspending stuff?? :D | 09:22 |
| tsdgeos | j/k | 09:22 |
| mzanetti | no but the dash is actually the least useful one in terms of background activity | 09:22 |
| mzanetti | tsdgeos: and it does cause about 1% CPU per favorited scope | 09:23 |
| mzanetti | tsdgeos: anyhow... dashcommunicator needed to be async | 09:23 |
| tsdgeos | that's something we should fix in favorited scopes | 09:23 |
| mzanetti | tsdgeos: the issue now is that its not enough to use asynCall() because that would solve the issue | 09:23 |
| mzanetti | when its calling | 09:23 |
| tsdgeos | right | 09:23 |
| mzanetti | but not when its connecting | 09:24 |
| mzanetti | so the whole createdbusconnection thing needs to be threaded | 09:24 |
| mzanetti | in case it tries to connect to a suspended dash we at least only hang the thread, but not unity | 09:24 |
| mzanetti | tsdgeos: yes, I agree we should improve in the dash itself too | 09:24 |
| mzanetti | tsdgeos: still I guess we can't always as there is 3rd party stuff in there | 09:25 |
| tsdgeos | sure | 09:25 |
| mzanetti | so if a backend keeps on refreshing things we'd paint them | 09:25 |
| mzanetti | tsdgeos: also see the related branches | 09:26 |
| mzanetti | they allow waking up the dash on left edge swipe etc | 09:26 |
| tsdgeos | i'm a bit "concerned" by eitehr the dashActive naming | 09:29 |
| tsdgeos | or its valu | 09:29 |
| tsdgeos | e | 09:29 |
| mzanetti | why? | 09:29 |
| tsdgeos | i mean dashActive = launcher.shown || launcher.dashSwipe | 09:29 |
| mzanetti | yes..? still don't see the problem | 09:30 |
| tsdgeos | looks like it should be active in more cases no? | 09:30 |
| mzanetti | its also active when its focused | 09:30 |
| mzanetti | but other then that, no, I *think* that's it | 09:30 |
| tsdgeos | right, so you're using dashActive as "activateDash" function | 09:30 |
| mzanetti | yeah | 09:30 |
| mzanetti | sort of | 09:31 |
| tsdgeos | that's why i complain about the name | 09:31 |
| tsdgeos | when its focused | 09:31 |
| mzanetti | but its a property | 09:31 |
| tsdgeos | it'll be set to false | 09:31 |
| tsdgeos | which is weird | 09:31 |
| mzanetti | tsdgeos: yeah... that's weirdness caused by the cat that we still handle suspension tied to focusing in qtmir | 09:31 |
| mzanetti | tsdgeos: number 1 prio to fix after RTM for us AppManager guys | 09:31 |
| mzanetti | s/cat/fact/ :D | 09:32 |
| tsdgeos | mzanetti: well maybe it should not be a property but a function? | 09:32 |
| tsdgeos | i mean it's always dash -> qtmir no? | 09:32 |
| tsdgeos | it can be requestActivation() or something? | 09:32 |
| tsdgeos | requestDashActive | 09:32 |
| mzanetti | tsdgeos: meh | 09:32 |
| mzanetti | then I need to call shit in many places | 09:32 |
| mzanetti | or just turn the Bindiing {} into a handler that calls stuff when a intermediate property changes | 09:32 |
| tsdgeos | well, either that or you add a huge ass warning saying | 09:33 |
| tsdgeos | "dashActive" is not really when the dash is active | 09:33 |
| mzanetti | tsdgeos: there is documentation in unity-api what this property is for :D | 09:33 |
| tsdgeos | mzanetti: call it forceDashActive ? | 09:34 |
| mzanetti | tsdgeos: ok... can do | 09:34 |
| tsdgeos | it's more really waht it does, no? | 09:34 |
| mzanetti | yeah... you're probably right :) | 09:35 |
| tsdgeos | thanks :) | 09:35 |
| tsdgeos | mzanetti: i guess you need someone to review https://code.launchpad.net/~unity-team/qtmir/fix-some-lifecycle-bugs/+merge/234087 too? | 09:35 |
| mzanetti | oh... didn't see he set it to needs fixing again | 09:36 |
| mzanetti | it was approved already | 09:36 |
| mzanetti | tsdgeos: will fix first and let you know | 09:36 |
| tsdgeos | oki | 09:36 |
| tsdgeos | noooooooooooooooooooooooooooooooooo | 10:00 |
| tsdgeos | make testDefaultIndicatorPage | 10:00 |
| tsdgeos | doesn't compile! | 10:00 |
| tsdgeos | dednick: ↑↑↑ | 10:00 |
| dednick | tsdgeos: compile? | 10:01 |
| dednick | tsdgeos: what branch? | 10:02 |
| tsdgeos | dednick: trunk | 10:02 |
| dednick | tsdgeos: erm | 10:02 |
| tsdgeos | yes? | 10:02 |
| dednick | that sounds odd | 10:02 |
| tsdgeos | it does | 10:02 |
| tsdgeos | :D | 10:02 |
| dednick | and i dont get "compile". you mean the test doesnt succeed? | 10:02 |
| tsdgeos | dednick: the test can't start because there's broken qml that can't be compiled | 10:03 |
| dednick | tsdgeos: something about messaging menus? | 10:03 |
| tsdgeos | /home/tsdgeos_work/phablet/unity8/unity8/tests/qmltests/Panel/Indicators/tst_DefaultIndicatorPage.qml:32,9: Cannot assign to non-existent property "contentActive" | 10:04 |
| tsdgeos | dednick: ↑ | 10:04 |
| dednick | tsdgeos: ok. give me a sec | 10:05 |
| dednick | tsdgeos: does CI not run qmltests? | 10:06 |
| tsdgeos | dednick: it does | 10:06 |
| tsdgeos | that's why i found its failing | 10:06 |
| tsdgeos | s/why/how | 10:06 |
| dednick | tsdgeos: dont know how it landed then | 10:06 |
| tsdgeos | CI has nothing to do with landing | 10:07 |
| tsdgeos | we land broken stuff all the time | 10:07 |
| dednick | tsdgeos: silo dont run tests? | 10:07 |
| dednick | ah | 10:07 |
| tsdgeos | dednick: afaik no, no tests other than make check/test is run in silos | 10:07 |
| dednick | tsdgeos: ok, well i found the problem. give me a sec and i'll sort a branch | 10:08 |
| tsdgeos | dednick: cool | 10:09 |
| dednick | tsdgeos: https://code.launchpad.net/~nick-dedekind/unity8/broken-indicator-tests/+merge/236292 | 10:28 |
| mzanetti | tsdgeos: updated all 3 MPs related to suspending the dash | 10:36 |
| mzanetti | tsdgeos: kicking the ppa (phone-right-edge) for a test run | 10:36 |
| tsdgeos | oki | 10:37 |
| mzanetti | please vote for this :) https://bugs.launchpad.net/ubuntu/+source/lxc-android-config/+bug/1367683 | 10:40 |
| Cimi | tsdgeos, a couple of lines to edit | 10:41 |
| Cimi | tsdgeos, one indent and an updated copyright year | 10:41 |
| Cimi | tsdgeos, I would like to have this branch in ASAP | 10:41 |
| tsdgeos | dandrader: https://code.launchpad.net/~dandrader/unity8/touchOwnership/+merge/236152 fixes the "press on stuff that is too close to the edge" thing? | 10:53 |
| dandrader | tsdgeos, yes | 10:53 |
| tsdgeos | cool | 10:53 |
| facundobatista | Holas | 11:01 |
| tsdgeos | pstolowski: any idea why https://launchpadlibrarian.net/186037191/buildlog_ubuntu-utopic-i386.unity-scopes-shell_0.5.4%2B14.10.20140925-0ubuntu2_FAILEDTOBUILD.txt.gz failed? | 11:05 |
| pstolowski | looking | 11:05 |
| tsdgeos | qmlplugindump: could not find a Qt installation of '' | 11:05 |
| tsdgeos | ah | 11:05 |
| tsdgeos | it may be me | 11:06 |
| pstolowski | tsdgeos, ah, good you spotted it, i was scratching my head | 11:11 |
| Cimi | tsdgeos, how about this for the image gallery slideshow? http://paste.ubuntu.com/8454572/ | 11:11 |
| tsdgeos | boooooooooooooooo | 11:11 |
| tsdgeos | that ppa doesn't have arm | 11:11 |
| pstolowski | tsdgeos, right.. i remember mhr3 asked somebody to enable arm for his ppa | 11:12 |
| tsdgeos | Cimi: screenshots seems like the wrong text | 11:12 |
| === mpt_ is now known as mpt | ||
| Cimi | tsdgeos, yeah, we need something better | 11:21 |
| Cimi | tsdgeos, you know how we can pass the information down the image gallery? | 11:22 |
| tsdgeos | Cimi: why you need the text? we didn't have it, why the need now? | 11:22 |
| Cimi | tsdgeos, which text then? | 11:22 |
| tsdgeos | Cimi: none? | 11:22 |
| Cimi | tsdgeos, we need to fill the header with something no? | 11:23 |
| Cimi | tsdgeos, application name? | 11:23 |
| tsdgeos | Cimi: why do we need a header? | 11:23 |
| tsdgeos | is it in the spec? | 11:23 |
| Cimi | tsdgeos, they said app name | 11:23 |
| Cimi | iirc | 11:23 |
| Cimi | tsdgeos, none is actually quite good too | 11:24 |
| tsdgeos | but that's not for apps only | 11:24 |
| Cimi | tsdgeos, I'd go for none | 11:24 |
| tsdgeos | ebay has gallery of images | 11:24 |
| tsdgeos | go for none | 11:24 |
| tsdgeos | easier | 11:24 |
| === MacSlow is now known as MacSlow|lunch | ||
| tsdgeos | Cimi: which indent do you want in that "indent" comment you made? | 11:25 |
| tsdgeos | i'd have it in a single line | 11:26 |
| tsdgeos | but there's people that complain | 11:26 |
| tsdgeos | so i just wrapped it arbitrarily | 11:26 |
| tsdgeos | there's no "proper" indent imho | 11:26 |
| dandrader | right, no line should be longer than 120 chars, says our coding style | 11:31 |
| mzanetti | dednick: Did you make sure that your branch does not contain spurious tags? | 11:32 |
| tsdgeos | dandrader: can you point me to it? i always forget where it is | 11:32 |
| Cimi | tsdgeos, ouch, I don't care then | 11:33 |
| Cimi | tsdgeos, I would have the "- " under the element on top | 11:33 |
| Cimi | tsdgeos, but nevermind | 11:33 |
| tsdgeos | Cimi: but see it's part of the same thing | 11:33 |
| tsdgeos | it's not after the , or anything | 11:33 |
| Cimi | or even the - on top l;ine | 11:33 |
| tsdgeos | it's part of the same "expression" | 11:34 |
| tsdgeos | Cimi: i'll put it wherever you want | 11:34 |
| tsdgeos | keep Cimi happy :) | 11:34 |
| dandrader | tsdgeos, lp:canonical-client-development-guidelines | 11:34 |
| dandrader | tsdgeos, you have to build it in order to view it :D | 11:35 |
| tsdgeos | awesome documentaiton \o/ | 11:35 |
| tsdgeos | not | 11:35 |
| dandrader | heheheheh, yeah, it's crazy. I think there's some URL around that shows the built result but I don't know it | 11:36 |
| dandrader | it's essentially a lightly modified version of google's C++ guide | 11:36 |
| dandrader | and there's nothing on QML | 11:37 |
| Cimi | tsdgeos, you so nice on monday morning, thanks :)) | 11:38 |
| Cimi | tsdgeos, I'd put the - on the top line | 11:39 |
| Cimi | tsdgeos, and my personal taste could have cagoryview under baseItem, since they are part or Math.max | 11:40 |
| Cimi | but that's me :) | 11:40 |
| mzanetti | dednick: you here? | 11:40 |
| tsdgeos | Cimi: ok | 11:41 |
| tsdgeos | Cimi: there's a broken qmluitest now but that's on trunk too | 11:41 |
| mzanetti | meh... I'd like to go for a run... but can't start building the silo | 11:41 |
| mzanetti | wrong channel :) | 11:42 |
| tsdgeos | mzanetti: why can't start building? | 11:42 |
| mzanetti | because building the silo makes a copy of the code | 11:42 |
| mzanetti | and when merging that copy will be merged, | 11:42 |
| mzanetti | which would import tags | 11:42 |
| tsdgeos | ok | 11:43 |
| tsdgeos | mzanetti: fork it! | 11:43 |
| tsdgeos | and put it in unity-team | 11:43 |
| mzanetti | :) | 11:43 |
| mzanetti | tsdgeos: https://bugs.launchpad.net/ubuntu/+source/oxide-qt/+bug/1375215 | 11:44 |
| ubot5 | Ubuntu bug 1375215 in oxide-qt (Ubuntu) "White screen in webbrowser and webapps / system runs out of memory and kills oxide-renderer" [Critical,Confirmed] | 11:44 |
| mzanetti | tsdgeos: seems its unity8-dash | 11:44 |
| tsdgeos | meh | 11:45 |
| mzanetti | tsdgeos: mind reapproving? https://code.launchpad.net/~unity-team/unity8/broken-indicator-tests/+merge/236304 | 11:46 |
| tsdgeos | mzanetti: done | 11:47 |
| sebastian_ | Hey all. we're an ISV and getting customer complains, that our tray icon is missing from unity since 14:04 | 11:50 |
| sebastian_ | it seems that com.canonical.Unity.Panel/systray-whitelist ceased to work | 11:51 |
| tsdgeos | bregma: are you guys still the ones in charge or unity7? ↑↑↑↑ | 11:52 |
| tsdgeos | and are you awake already? | 11:53 |
| tsdgeos | :D | 11:53 |
| bregma | tsdgeos, yep, that's us | 11:53 |
| tsdgeos | bregma: see sebastian_'s question | 11:53 |
| bregma | sebastian_, the whitelist was removed in 12.10 and has not been available for years | 11:53 |
| sebastian_ | bregma: well, that doesn't help us. what can we do about that? | 11:54 |
| bregma | sebastian_, the "tray" was removed more than 4 years ago, applications that want to integrate in the menu bar need to write to the application indicator spec available on freedesktop.org and supported by most major desktops | 11:55 |
| bregma | sebastian_, here's an older wiki entry on the topic: https://wiki.ubuntu.com/DesktopExperienceTeam/ApplicationIndicators | 11:56 |
| sebastian_ | is there some kind of wrapper, that wrapps the old tray icons to ApplicationIndicators? | 11:58 |
| Cimi | tsdgeos, I am trying to implement a zoom in effect when I tap on images | 12:01 |
| sebastian_ | well the main problem ist, that we're based on Qt4 and it's QSystemTrayIcon implementation doesn't support ApplicationIndicators | 12:01 |
| Cimi | tsdgeos, there is something wrong in my code... | 12:02 |
| Cimi | tsdgeos, http://paste.ubuntu.com/8454940/ | 12:02 |
| Cimi | tsdgeos, I am using "red" to show a dot, it is in the top left of each image in the gallery, but when I am using it as origin.x and .y of a scale transform, something behaves wrong | 12:03 |
| Cimi | tsdgeos, how did you do in the overview? | 12:03 |
| bregma | sebastian_, it's possible a patch for that got dropped with the switch to Qt5, there may be a bug..... | 12:08 |
| dednick | mzanetti: ya. i did | 12:11 |
| dednick | mzanetti: hm... i know i did! | 12:11 |
| Cimi | I will use scale and transform... | 12:12 |
| dednick | oh. maybe i didnt scroll all the way to top :/ | 12:13 |
| sebastian_ | bregma: ok. then, is there a sample implementation that also works wihtout an installed unity desktop? | 12:17 |
| bregma | sebastian_, I'm poking some of the upstream developers to find out what's going on -- some of them are not online yet | 12:17 |
| sebastian_ | bregma: thanks! I'll stay here at least for the next three hours. | 12:19 |
| === MacSlow|lunch is now known as MacSlow | ||
| tsdgeos | bregma: sni-qt ? | 12:50 |
| tsdgeos | bregma: it has no qt5 version afaik | 12:50 |
| bregma | yep, wonder how that could have gotten missed? | 12:52 |
| tsdgeos | no idea | 12:53 |
| tsdgeos | https://bugs.launchpad.net/sni-qt/+bug/1347128 | 12:53 |
| ubot5 | Ubuntu bug 1347128 in owncloud-client (Ubuntu) "Owncloud client indicator has wrong position - absolutely on the left" [Undecided,Confirmed] | 12:53 |
| === karni is now known as karni-lunch | ||
| mzanetti | dednick: no worries. I've resubmitted it | 12:56 |
| mzanetti | dednick: to be able to kick the silo build | 12:56 |
| dednick | mzanetti: yup. i saw. thanks | 12:57 |
| kgunn | robotfuel: hey there, do you know where the jenkins scripts are that auto land stuff ? | 13:07 |
| kgunn | specifically if we wanted to alter unity8 landing to include a test | 13:08 |
| robotfuel | kgunn: that's cupstream2distro-config | 13:08 |
| robotfuel | kgunn: https://code.launchpad.net/cupstream2distro-config | 13:09 |
| kgunn | mzanetti: ^ | 13:09 |
| kgunn | robotfuel: who owns that if we we'd like to propose a change ? | 13:10 |
| tsdgeos | we can't run qmluitests there | 13:10 |
| kgunn | is that you? | 13:10 |
| tsdgeos | i mean we can | 13:10 |
| tsdgeos | but it needs X and stuff | 13:10 |
| tsdgeos | or xvfb-run at least | 13:10 |
| mzanetti | tsdgeos: no... make qmltests runs in xvfb | 13:10 |
| tsdgeos | may not be trivial | 13:10 |
| robotfuel | kgunn: ci | 13:10 |
| mzanetti | tsdgeos: but yeah, that's the story that triggered this request :) | 13:10 |
| kgunn | robotfuel: ack | 13:11 |
| kgunn | mzanetti: so we'd need to work with fginther | 13:12 |
| === tedg is now known as ted | ||
| === dandrader is now known as dandrader|afk | ||
| === _salem is now known as salem_ | ||
| === karni-lunch is now known as karni | ||
| === dandrader|afk is now known as dandrader | ||
| bregma | sebastian_, it looks like Qt5 integration into the Unity indicators should be accomplished by the appmenu-qt5 package -- can you check to see if that is installed and if not, install it and see if it resolves the problem? | 13:56 |
| bregma | sebastian_, you may need to set QT_QPA_PLATFORMTHEME=appmenu-qt5 | 14:01 |
| tsdgeos | pstolowski: build fails on arm :S https://launchpadlibrarian.net/186048336/buildlog_ubuntu-utopic-armhf.unity-scopes-shell_0.5.4%2B14.10.20140925-0ubuntu5_FAILEDTOBUILD.txt.gz | 14:03 |
| tsdgeos | cd /build/buildd/unity-scopes-shell-0.5.4+14.10.20140925/obj-arm-linux-gnueabihf/src/Unity && UNITY_SCOPES_NO_LOCATION=1 UNITY_SCOPES_LIST_DELAY=10000 /usr/bin/qmlplugindump -notrelocatable Unity 0.2 /build/buildd/unity-scopes-shell-0.5.4+14.10.20140925/obj-arm-linux-gnueabihf/src/Unity/../ > /build/buildd/unity-scopes-shell-0.5.4+14.10.20140925/obj-arm-linux-gnueabihf/src/Unity/plugin.qmltypes | 14:04 |
| tsdgeos | Error: SEGV | 14:04 |
| tsdgeos | Error: SEGV | 14:04 |
| tsdgeos | qemu: uncaught target signal 11 (Segmentation fault) - core dumped | 14:04 |
| tsdgeos | mzanetti: kgunn: any idea? just retry ad infinitum? ↑↑ | 14:04 |
| mzanetti | tsdgeos: not following | 14:05 |
| tsdgeos | mzanetti: have you ever seen a ppa fail on qemu for arm? | 14:05 |
| pstolowski | tsdgeos, uhm, any more details?.. | 14:06 |
| mzanetti | tsdgeos: ah... qemu segfaulting sounds like its trying to instantiate a QQuickView | 14:06 |
| mzanetti | tsdgeos: won't work | 14:06 |
| tsdgeos | so can't build unity-scopes-shell in a ppa? | 14:06 |
| mzanetti | tsdgeos: I reported a bug for that when I was still in QA :D | 14:06 |
| mzanetti | tsdgeos: you can, but the ppa needs to use real builders, not qemu | 14:06 |
| * tsdgeos does the unhappy dance | 14:06 | |
| mzanetti | tsdgeos: your personall ppa won't work, but you can use the phone-right-edge one | 14:06 |
| kgunn | or mir staging | 14:07 |
| mzanetti | kgunn: may I request a second unity-testing ppa? :) seems we're using this quite a bit | 14:08 |
| mzanetti | kgunn: somewhat low priority though | 14:08 |
| tsdgeos | mzanetti: but you're using phone-right-edge, no? | 14:08 |
| mzanetti | tsdgeos: yeah... but there is no scopes in there... | 14:09 |
| mzanetti | I guess we can mix it in this case | 14:09 |
| tsdgeos | mzanetti: well i also need unity8 | 14:10 |
| mzanetti | tsdgeos: then yeah... need to wait a little | 14:10 |
| Cimi | tsdgeos, before I jump into tests... do we need to add any smoothing or cache/loaders here? http://paste.ubuntu.com/8458732/ | 14:10 |
| tsdgeos | kgunn: ok, so no bottom list ppa then | 14:10 |
| mzanetti | tsdgeos: my stuff should be built soonish, then we test it asap and switch to your stuff | 14:11 |
| tsdgeos | Cimi: can you push it to a branch? it's easier to get :D | 14:11 |
| Cimi | ahah ok | 14:11 |
| kgunn | tsdgeos: mzanetti alternative, stuff it in a silo mark it test....easier to keep up to date imho | 14:12 |
| mzanetti | kgunn: yeah... if we're always getting so many silos... seems a bit out of scope use though | 14:13 |
| Cimi | tsdgeos, lp:~cimi/unity8/preview-image-gallery_slideshow | 14:13 |
| pstolowski | tsdgeos, i've updated my branch with tests + one minor fix | 14:15 |
| tsdgeos | oki cool | 14:15 |
| tsdgeos | Cimi: can you merge it with trunk? | 14:16 |
| tsdgeos | it doesn't compile! | 14:16 |
| Cimi | tsdgeos, sure | 14:16 |
| Cimi | @_@ | 14:16 |
| Cimi | tsdgeos, done | 14:17 |
| === dandrader_ is now known as dandrader | ||
| dandrader | mzanetti, kgunn, is it a known issue that the orientation sensor remains on while the display is off? do we have a bug for it? like is not being turned off in qtmir | 14:23 |
| kgunn | tsdgeos: mzanetti keeping a silo busy is not a big deal for big ticket stuff...but up to you, seems like phone-right-edge ppa gets us what we want...save maybe building on rtm | 14:24 |
| kgunn | dandrader: is that on n4 ? | 14:24 |
| mzanetti | dandrader: not known to me... please report a bug | 14:24 |
| mzanetti | kgunn: yeah... we use it mostly to test stuff with inter-project-dependencies before approving | 14:25 |
| mzanetti | kgunn: sometimes also to give other's a way to test some not-landed feature | 14:25 |
| mzanetti | kgunn: I think we should get more of them. at least one more | 14:25 |
| tsdgeos | Cimi: looks good ui wise (well the back thing looks a bit weird but can't suggest anything better) | 14:25 |
| mzanetti | maybe call it unity-testing instead of some-feature-from-last-year this time :) | 14:26 |
| Cimi | tsdgeos, which black thing? | 14:26 |
| Cimi | tsdgeos, header? | 14:26 |
| tsdgeos | Cimi: "back" thing | 14:26 |
| tsdgeos | sorr | 14:26 |
| Cimi | tsdgeos, or the rectangle box? | 14:26 |
| tsdgeos | y | 14:26 |
| tsdgeos | but let's not argue about that now | 14:26 |
| tsdgeos | do the tests | 14:26 |
| tsdgeos | and we review+merge | 14:26 |
| tsdgeos | and let others complain if they don't like it later | 14:26 |
| Cimi | rsalveti, how would you change? | 14:26 |
| Cimi | rsalveti, sorry | 14:27 |
| paulliu | Today I tried to update my nexus 4. And adbd doesn't want to run. It doesn't export the adb interface. Any way to recover it or I need a totally wipe from fastboot? | 14:27 |
| Cimi | tsdgeos, ? | 14:27 |
| dandrader | kgunn, seem it in krillin, but should happen with any device | 14:27 |
| tsdgeos | i don't know how i would change it, just saying it feels a bit strange | 14:27 |
| tsdgeos | my immediate feeling was "tap to close" | 14:27 |
| tsdgeos | but that is probably done nowhere | 14:27 |
| tsdgeos | in the rest of the ui | 14:27 |
| Cimi | tsdgeos, maybe I can put an X to close | 14:27 |
| tsdgeos | so not good | 14:28 |
| tsdgeos | Cimi: i think i'd like an X more | 14:28 |
| Cimi | tsdgeos, are you aware of an X in the icons for the toolbar? | 14:28 |
| tsdgeos | "back" | 14:29 |
| tsdgeos | err | 14:29 |
| tsdgeos | "close" | 14:29 |
| tsdgeos | sorry :D | 14:29 |
| pstolowski | tsdgeos, can you take a look at this trivial fix (for the problem you spotted last week)? https://code.launchpad.net/~stolowski/unity-scopes-shell/fix-delete-later/+merge/236349 ; btw, the test i added fails without that fix | 14:29 |
| tsdgeos | back is < | 14:29 |
| Cimi | close yeah | 14:29 |
| tsdgeos | close is X | 14:29 |
| Cimi | tried | 14:29 |
| Cimi | works | 14:29 |
| tsdgeos | pstolowski: sure | 14:30 |
| Cimi | tsdgeos, on my question for smooth and loaders? | 14:31 |
| Cimi | tsdgeos, which settings we want for the image? | 14:31 |
| Cimi | tsdgeos, I don't remember how you did your tests on sourcesize and that option | 14:31 |
| tsdgeos | Cimi: not sure i get what you mean, let's do the standup and come back | 14:32 |
| Cimi | ok | 14:32 |
| sebastian_ | bregma: that's a solution for Qt5. there is no such thing for our current stable release based on Qt 4? | 14:35 |
| pstolowski | tsdgeos, thanks! | 14:36 |
| sebastian_ | bregma: ahh, I'll have a look at https://launchpad.net/appmenu-qt | 14:37 |
| bregma | sebastian_, the Qt4 solution was sni-qt, which should still be working | 14:37 |
| Cimi | tsdgeos, up ^ :) | 14:42 |
| tsdgeos | Cimi: so | 14:42 |
| tsdgeos | what do you mean for smooth and loaders? | 14:43 |
| Cimi | tsdgeos, if we need smooth true in the slideshow delegates | 14:43 |
| sebastian_ | bregma: ok, thanks | 14:43 |
| Cimi | or antialiasing | 14:44 |
| Cimi | I am not 100% sure on those | 14:44 |
| Cimi | smooth true maybe yes | 14:44 |
| tsdgeos | Cimi: smooth is on by default | 14:44 |
| Cimi | tsdgeos, ok, delete me! | 14:44 |
| Cimi | :) | 14:44 |
| tsdgeos | Cimi: so that's all? | 14:45 |
| Cimi | tsdgeos, well, not sure I should use a loader for this slideshow or not | 14:47 |
| Cimi | tsdgeos, I would avoid delayes on the zoom animation when you open it | 14:47 |
| tsdgeos | Cimi: it's a listview | 14:48 |
| tsdgeos | listviews are async already | 14:48 |
| tsdgeos | Cimi: it feels good here | 14:52 |
| tsdgeos | Cimi: and Image also has another async in case you want to set it to true | 14:52 |
| Cimi | tsdgeos, nah, I don't want to slow down | 14:52 |
| Cimi | tsdgeos, I mean, seing black boxes | 14:53 |
| dandrader | dednick, the plan is that we dismiss a prompt session once its "host" session is backgrounded or suspended, right? | 15:00 |
| dednick | dandrader: yep | 15:02 |
| mzanetti | dednick: hey. I like the new indicators look :) | 15:02 |
| dednick | dandrader: for now at least | 15:02 |
| dednick | mzanetti: yeah, it's quite nice what they've done. where did you see them? | 15:02 |
| mzanetti | dednick: on devel-proposed | 15:03 |
| mzanetti | dednick: I think there's more to come which I haven't seen yet. | 15:03 |
| dednick | ah. the menus. you havent seen panel yet ;) | 15:03 |
| mzanetti | dednick: but I'm referring to the things like the green highlight on active wifi | 15:03 |
| dednick | meh. my krillin wont let me ssh on :( | 15:07 |
| dandrader | mzanetti, are you planning to take https://code.launchpad.net/~gerboland/qtmir/desktopFileReader/+merge/235223 in you next landing? | 15:11 |
| mzanetti | tsdgeos: phone-right-edge built | 15:11 |
| mzanetti | dandrader: hmm... its not in the current silo. will put it in the next one, ok? | 15:12 |
| tsdgeos | mzanetti: url is ppa:waht | 15:12 |
| tsdgeos | ? | 15:12 |
| dandrader | mzanetti, ok | 15:12 |
| mzanetti | tsdgeos: ppa:unity-team/phone-right-edge | 15:12 |
| dandrader | @unity does anyone have mako flashed with devel-proposed version 261? | 15:15 |
| dandrader | @unity oh, nevermid it's working now :) | 15:15 |
| dandrader | thought it still had the #252 issue | 15:15 |
| mzanetti | tsdgeos: have you already installed the ppa? | 15:17 |
| tsdgeos | on it | 15:17 |
| mzanetti | tsdgeos: please pay attention if it pulls unity8-fake-env on upgrade | 15:18 |
| tsdgeos | should it? | 15:18 |
| mzanetti | shouldn't happen... but there seems to be something fishy somewhere | 15:18 |
| mzanetti | tsdgeos: not in this branch, but we've seen that with some silos | 15:18 |
| mzanetti | I have hope this branch implicitly fixes it | 15:19 |
| tsdgeos | ok | 15:19 |
| dandrader | mzanetti, Is there an easy way to tell which image number I've on my device? | 15:21 |
| mzanetti | dandrader: systemsettings -> about this phone | 15:22 |
| dandrader | mzanetti, awesome, thanks! | 15:22 |
| dednick | why do we have this "waitForRendering" in our UnityTestCase touch functions? | 15:23 |
| dednick | mzanetti: ^ ? | 15:27 |
| mzanetti | dednick: hmm... not sure... I think dandrader can help there | 15:27 |
| Cimi | mzanetti, what does waitForRendering do? | 15:27 |
| dednick | Cimi: "Waits for timeout milliseconds or until the item is rendered by the renderer. Returns true if item is rendered in timeout milliseconds, otherwise returns false. The default timeout value is 5000." | 15:28 |
| Cimi | dednick, waits for the end of the rendering? | 15:28 |
| dednick | Cimi: i think it waits until the buffer switches or something | 15:29 |
| Cimi | I have animations, i want to know when this waitForRendering waits | 15:29 |
| Cimi | so maybe not my case | 15:29 |
| dednick | but I don't think it know if it's been rendered already, in which case it just waits for 5 seconds which is bloody long in test cases | 15:30 |
| dednick | i think we need to decrease the timeout at least. | 15:30 |
| dandrader | mzanetti, dednick, I don't actually remember the exact reason. But I guess it didn't work or was unstable without it | 15:30 |
| dednick | we should have a little spinner in all of our tests to get the renderer working itself out a bit. | 15:39 |
| seb128 | hey there | 15:43 |
| seb128 | is anyone planning to do a translate template update for unity8? | 15:43 |
| seb128 | it looks like it could use one | 15:43 |
| seb128 | tsdgeos, ^ you usually care about translations, do you know who is the person to ping about that one when Saviq is on vac? | 15:43 |
| tsdgeos | seb128: it is up to date | 15:44 |
| seb128 | tsdgeos, is it? | 15:44 |
| tsdgeos | seb128: looks to me http://paste.ubuntu.com/8459468/ | 15:44 |
| seb128 | tsdgeos, k, let me check a bit more then :/ | 15:45 |
| tsdgeos | seb128: what are you missing? | 15:45 |
| seb128 | tsdgeos, "Restart" in the dialog you get when you keep the power button pressed | 15:46 |
| seb128 | tsdgeos, https://translations.launchpad.net/unity8/rtm-14.09/+pots/unity8/fr/+translate?batch=10&show=all&search=Restart | 15:46 |
| seb128 | doesn't list the string | 15:46 |
| tsdgeos | weird | 15:46 |
| tsdgeos | tsdgeos_work@xps:~/phablet/unity8/unity8$ grep Restart -B 1 -A 2 po/fr.po | 15:47 |
| tsdgeos | #: qml/Components/Dialogs.qml:157 | 15:47 |
| tsdgeos | msgid "Restart" | 15:47 |
| tsdgeos | msgstr "Redémarrer" | 15:47 |
| seb128 | the launchpad template is from 08-31 | 15:47 |
| seb128 | well at least https://translations.launchpad.net/unity8/rtm-14.09 has last changed "2014-08-31" for french | 15:48 |
| seb128 | and it's fully translated | 15:48 |
| seb128 | which means it didn't get updated since | 15:48 |
| tsdgeos | there has been changes to our .pot in 18 september and 26 september | 15:48 |
| tsdgeos | i don't understand the magic behind launchpad | 15:48 |
| tsdgeos | but we are getting translations from it | 15:48 |
| seb128 | dpm, ^ is unity8 getting the template from the vcs or from the package? | 15:48 |
| tsdgeos | we got udaptes for fr on 29th | 15:49 |
| seb128 | not in rtm though? | 15:49 |
| tsdgeos | i have no idea how rtm works | 15:49 |
| seb128 | https://code.launchpad.net/~unity-team/unity8/rtm-14.09 | 15:49 |
| seb128 | hum | 15:49 |
| seb128 | that didn't get commits since 09-08 | 15:49 |
| tsdgeos | yeah what's that branch? | 15:50 |
| dpm | seb128, afaik from the vcs, unless someone set up the package | 15:50 |
| seb128 | was Saviq pushing there manually and that stopped with him being away? | 15:50 |
| tsdgeos | and who is supposed to update it? | 15:50 |
| seb128 | I guess Saviq went in vac with that knowledge | 15:50 |
| tsdgeos | is that branch magically updated by the rtm ci train? | 15:51 |
| dpm | so I guess ever since he's away releases have been made from the non-rtm branch? | 15:51 |
| tsdgeos | mzanetti: kgunn: have we been doing releases for rtm besides doing them for utopic? | 15:51 |
| seb128 | who has been doing unity8 landings? | 15:51 |
| tsdgeos | mzanetti and kgunn | 15:51 |
| seb128 | tsdgeos, I guess the landings should have been targetting the rtm vcs | 15:51 |
| tsdgeos | no | 15:52 |
| tsdgeos | both at most | 15:52 |
| tsdgeos | but we need to keep landing in trunk | 15:52 |
| seb128 | right | 15:52 |
| seb128 | but I guess if Saviq did a rtm specific branch it's because the content was different in trunk and rtm? | 15:52 |
| tsdgeos | not that i know | 15:52 |
| tsdgeos | it's just a copy | 15:53 |
| seb128 | seems wasteful to have a branch then | 15:53 |
| seb128 | we just use trunk on other components | 15:53 |
| seb128 | dpm, can we point the translations to trunk in that case? | 15:53 |
| tsdgeos | i may be speaking bullshit | 15:53 |
| dpm | seb128, yeah, just a sec | 15:53 |
| seb128 | dpm, thanks | 15:53 |
| tsdgeos | but tbh i don't remember us having anything different in trunk vs rtm | 15:53 |
| === gatox is now known as gatox_lunch | ||
| kgunn | tsdgeos: seb128 in general...yes....we're releasing to rtm as well as devel...but there's been significant lag in the time from landing on devel-utopic to rtm | 15:57 |
| kgunn | various reasons | 15:57 |
| seb128 | kgunn, shouldn't the rtm vcs serie be updated at the same time the update package hits the rtm archive? | 15:57 |
| dpm | seb128, tsdgeos, ok, translations in lp:unity point now to trunk. The lastest template update seems to have been on 2014-09-24. Which means it's recent, but it doesn't quite match the date on which tsdgeos mentioned they'd updated the .pot file (26th) | 15:58 |
| tsdgeos | confusing :D | 15:58 |
| seb128 | dpm, that's not giving us an updated template in https://translations.launchpad.net/unity8/rtm-14.09 though? | 15:59 |
| dpm | seb128, no, it doesn't, unless someone merges from trunk to rtm or updates the pot in rtm | 15:59 |
| seb128 | dpm, can we update the pot in rtm from launchpad? | 16:00 |
| === dandrader is now known as dandrader|lunch | ||
| dpm | seb128, yeah, we can either manually upload an up-to-date one directly, or if we go the vcs route someone needs to commit a new one | 16:01 |
| seb128 | kgunn, ^ can you help there? | 16:01 |
| dpm | I _think_ LP will import the latest, so if we upload manually, then it'll use that one instead of the one on the branch | 16:01 |
| seb128 | that didn't happen in practice though | 16:02 |
| seb128 | the current template on launchpad is missing strings | 16:02 |
| seb128 | e.g https://translations.launchpad.net/unity8/rtm-14.09/+pots/unity8/fr/+translate?batch=10&show=all&search=Restart | 16:02 |
| kgunn | seb128: sorry, quick context....sounds b/c way we do rtm branch (binary copy) that pot files aren't updating properly ? | 16:03 |
| kgunn | so we need to manually merge to rtm branch? | 16:03 |
| seb128 | kgunn, whoever was updating https://code.launchpad.net/~unity-team/unity8/rtm-14.09 stopped doing so | 16:03 |
| seb128 | kgunn, and that means the pot in that serie/translations are outdated | 16:04 |
| seb128 | kgunn, I guess the whoever was Saviq before being on vac and nobody has been doing that since he's away | 16:04 |
| kgunn | ah ha... thanks seb128 , yeah i would think mzanetti might know better...he did the vulcan mind meld with saviq before he left | 16:04 |
| seb128 | kgunn, can, one way or another can you get that rtm branch updated? | 16:05 |
| seb128 | we need it to get the rtm properly translated | 16:05 |
| kgunn | seb128: yep | 16:05 |
| kgunn | mzanetti: can we just manual merge trunk to unity8/rtm | 16:05 |
| kgunn | ? | 16:05 |
| mzanetti | Wellark: ping | 16:05 |
| seb128 | kgunn, why do you have a rtm serie if the content is identic to trunk? | 16:06 |
| kgunn | seb128: i think this was actually something we wanted to do in terms of running ui tests....but my memory is foggy | 16:09 |
| seb128 | kgunn, ok, no worry | 16:09 |
| seb128 | kgunn, just get the pot update in the rtm vcs please ;-) | 16:09 |
| === gatox_lunch is now known as gatox | ||
| === dandrader_ is now known as dandrader | ||
| racarr | dandrader: https://code.launchpad.net/~mir-team/qtmir/fix-touch-tooltypes fixed whitespace when you have a chance :) | 17:58 |
| racarr | Also can someone review: https://code.launchpad.net/~mir-team/qtubuntu/support-state-changing | 17:59 |
| racarr | Would like to land it with platform-api/support-state-changing | 17:59 |
| racarr | but need someone to counter gerrys old-1 review | 17:59 |
| racarr | which is fixed :) | 17:59 |
| dandrader | racarr, approved | 18:00 |
| racarr | dandrader: Thanks :) | 18:06 |
| === dandrader is now known as dandrader|afk | ||
| === dandrader|afk is now known as dandrader | ||
| === 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!