| === mpt_ is now known as mpt | ||
| tsdgeos | Saviq: that is very scary | 07:38 |
|---|---|---|
| Saviq | tsdgeos, the image corruption? | 07:38 |
| tsdgeos | yep | 07:38 |
| Saviq | yeah it is | 07:38 |
| Saviq | there were all kinds of issues with the image over thu/fri | 07:38 |
| Saviq | but just confirmed it's not a file corruption | 07:39 |
| Saviq | like reinstalling unity8 from .deb doesn't help | 07:39 |
| Saviq | tsdgeos, to add to the weirdness, we weren't even touching the areas that broke | 07:42 |
| Saviq | yeah, it's QML compilation | 07:42 |
| tsdgeos | ? | 07:42 |
| Saviq | tsdgeos, removing the cached precompiled files fixes the issues | 07:42 |
| tsdgeos | doh | 07:43 |
| tsdgeos | what could go wrong with caching :D | 07:43 |
| dednick | larsu: good morning. Can you take a quick look @ https://code.launchpad.net/~nick-dedekind/qmenumodel/lp1378941-disconnect-signals/+merge/238013, and top approve if you agree? | 08:49 |
| larsu | dednick: sure | 08:51 |
| larsu | dednick: approved. How does qmenumodel land? Do I need to merge manuallY? | 08:54 |
| dednick | larsu: not sure. havent done much in it in quite some time | 08:54 |
| dednick | larsu: i think it's manual. | 08:54 |
| larsu | dednick: hm, I'll wait a bit and merge if it doesn't happen automatically | 08:59 |
| larsu | thanks for the patch | 08:59 |
| seb128 | larsu, dednick: that likely go through ci landing | 08:59 |
| seb128 | larsu, dednick: I can put a landing up if you want | 09:00 |
| larsu | seb128: yes please, thanks | 09:02 |
| seb128 | yw! | 09:02 |
| larsu | one day I'll learn how all of this is set up | 09:02 |
| seb128 | larsu, they documented it recently on https://wiki.ubuntu.com/citrain/LandingProcess | 09:10 |
| seb128 | larsu, I didn't read it, but maybe have a look | 09:10 |
| larsu | ah thanks, I will | 09:10 |
| larsu | dednick: did you see the bugs about the time format in i-messages? I thought those where mine at first, but you added the relative time formatter, right? | 09:31 |
| larsu | dednick: bug #1372061 and bug #1378821 | 09:31 |
| ubot5 | bug 1372061 in Ubuntu Translations "SMS notification: time format not translatable" [Low,Triaged] https://launchpad.net/bugs/1372061 | 09:31 |
| ubot5 | bug 1378821 in indicator-messages (Ubuntu) "today/yesterday label on sms and calls is incorrect" [High,New] https://launchpad.net/bugs/1378821 | 09:31 |
| dednick | larsu: hm. yes. you send the numerical time i think. | 09:33 |
| larsu | dednick: it's definitely not the service's fault, it doesn't care about timezones or relative times at al | 09:34 |
| larsu | *all | 09:34 |
| larsu | dednick: I mean time formatting in plugins/Util/relativetimeformatter.cpp | 09:34 |
| dednick | larsu: sure. it's my fault. i pulled the code from indicator-datetime. | 09:34 |
| dednick | probably doing something wrong with tz. | 09:35 |
| larsu | dednick: do you have time to fix it or should I take a look? (probably not that hard) | 09:35 |
| larsu | I wonder if the time has come to merge those classes and put them into the SDK | 09:35 |
| larsu | Saviq reminded me of that the other day | 09:35 |
| dednick | larsu: ya. been meaning to do that for awhile.. | 09:36 |
| larsu | dednick: hm, they mix strftime and qt-style time formats ... | 09:37 |
| larsu | I guess the sdk will want qt-style, but then we'll have to re-translate all those strings | 09:37 |
| dednick | larsu: ya. i just used strtime in unity8 to keep it somewhat maintainable from indicator-datetime. but i needed to make some changes anyway, so it's probably time to change to qt. | 09:39 |
| dednick | larsu: and should be exporting raw time from indicator-datetime as well. | 09:40 |
| dednick | not formatted. | 09:40 |
| larsu | ya, of course | 09:41 |
| larsu | we should have one place to format time and it should be as close to the ui as possible | 09:41 |
| larsu | dednick: is now the time to do this or should we wait until after rtm? | 09:44 |
| seb128 | larsu, dednick, building in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/landing-008 if you want to test the deb later on | 09:44 |
| larsu | seb128: thanks | 09:50 |
| Saviq | seb128, hey, have you any idea why the language selection list scrolls so slow? | 11:02 |
| Saviq | it looks like delegate creation is awfully heavy | 11:03 |
| mzanetti | greyback: ping | 11:12 |
| === MacSlow is now known as MacSlow|lunch | ||
| dandrader | mzanetti, was testing launcher-update-on-dconf-change last Friday using the "reset launcher" button in the settings app. it din't work with or without this patch. :-/ | 11:16 |
| dandrader | mzanetti, will try with https://code.launchpad.net/~mzanetti/ubuntu-system-settings/drop-reboot-label-in-launcher-reset/+merge/236876 now | 11:16 |
| mzanetti | dandrader: define "didn't work" | 11:17 |
| dandrader | mzanetti, the launcher contents do not change, even after a reboot | 11:18 |
| mzanetti | dandrader: that system settings patch only drops the label... | 11:18 |
| mzanetti | dandrader: you need another one in systemsettings, let me find it | 11:18 |
| === shiznix_ is now known as shiznix | ||
| dandrader | mzanetti, I unpinned everything and pinned just "dropping letters" | 11:18 |
| mzanetti | dandrader: https://code.launchpad.net/~mzanetti/ubuntu-system-settings/update-reset-launcher/+merge/234309 | 11:18 |
| mzanetti | dandrader: otherwise systemsettings still resets AccountsService | 11:19 |
| * mzanetti wonders why this isn't merged yet | 11:19 | |
| mzanetti | seb128: hey, is there a reason why this isn't merged yet? https://code.launchpad.net/~mzanetti/ubuntu-system-settings/update-reset-launcher/+merge/234309 | 11:19 |
| dandrader | mzanetti, ok will add drop-reboot-label-in-launcher-reset and update-reset-launcher | 11:19 |
| greyback | mzanetti: pong | 11:20 |
| mzanetti | greyback: hey. question: if I kill an app from cmdline. what's supposed to happen? | 11:21 |
| greyback | mzanetti: did you start that app from command line too? | 11:21 |
| mzanetti | greyback: no | 11:21 |
| mzanetti | greyback: it seems I'm intentionally not getting the applicationRemoved() because appman assumes it got lifecycled | 11:21 |
| mzanetti | greyback: however, the ApplicationInfo entry goes away | 11:22 |
| greyback | mzanetti: if app in foreground, app is considered to have crashed so should be removed from app list. If in background, assumed OOM struck, so kept in app list | 11:22 |
| mzanetti | greyback: ah ok, its the focused one | 11:22 |
| mzanetti | greyback: so we have a bug that if the foreground app crashes, the dash is not woken up from suspend | 11:22 |
| dandrader | ah, drop-reboot-label-in-launcher-reset already brings update-reset-launcher | 11:23 |
| mzanetti | dandrader: yeah, one is a prereq of the other | 11:23 |
| greyback | mzanetti: I think if foreground app crashes, focusApplication is set to nullptr in qtmir | 11:23 |
| greyback | mzanetti: as that used to be the dash | 11:23 |
| mzanetti | dandrader: technically you only need the prereq as the other really just drops the label that says "you need to reboot" | 11:23 |
| mzanetti | ah ok | 11:23 |
| greyback | mzanetti: so need to change that to focus dash I guess | 11:24 |
| mzanetti | greyback: that helps, will try to fix. might come back though if failing | 11:24 |
| greyback | mzanetti: sure | 11:24 |
| dandrader | mzanetti, I didn't look at the diffs at all :) | 11:26 |
| tsdgeos | Saviq: dednick: besides the two minor comments i made sharedunitymenumodel looks good to me | 11:42 |
| tsdgeos | let's wait for the CI run and approve it | 11:43 |
| dednick | tsdgeos: cool. just uploaded fixes for those. | 11:43 |
| Saviq | cool | 11:48 |
| dednick | Saviq: i've uploaded latest changes to panel design branch as well. back in MP. | 11:49 |
| Saviq | dednick, thanks | 11:49 |
| tsdgeos | dednick: is that ready for review too? | 11:49 |
| dednick | tsdgeos: yup | 11:49 |
| tsdgeos | ok, so shall i get into that one next? | 11:50 |
| tsdgeos | Saviq: ↑? | 11:50 |
| dednick | tsdgeos: Cimi has been doing it, but could use some more eyes | 11:50 |
| Cimi | tsdgeos, yes have a look | 11:50 |
| Saviq | tsdgeos, yeah, go for it | 11:50 |
| Cimi | tsdgeos, I will re'-review after lunch | 11:50 |
| Cimi | but is tricky | 11:50 |
| tsdgeos | ok | 11:51 |
| tsdgeos | i | 11:51 |
| tsdgeos | i'm off for lunch too | 11:51 |
| Saviq | dednick, testing silo 10 now... http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-010 | 11:52 |
| Saviq | dednick, panel icons seem to get stale straight away | 11:53 |
| dednick | Saviq: stale? | 11:54 |
| Saviq | dednick, the icons don't change at all, display the first icons that were displayed | 11:54 |
| dednick | Saviq: hm. will take a look. | 11:54 |
| Saviq | dednick, any from the MRs seem relevant? | 11:54 |
| Saviq | dednick, also, datetime indicator empty | 11:55 |
| * Saviq reflashes to pristine | 11:55 | |
| dednick | Saviq: not from the list... | 11:55 |
| Saviq | dednick, yeah didn't think so | 11:55 |
| dednick | Saviq: mterrys greeter shizzle isn't there is it? | 11:56 |
| Saviq | dednick, no | 11:56 |
| seb128 | mzanetti, not that I know | 11:56 |
| mzanetti | seb128: seems its just that it was stuck in "Needs Review" and considere being finished | 11:56 |
| mzanetti | ...and *not* considered finshed | 11:57 |
| seb128 | mzanetti, well, I was waiting on the unity8 side to land, did that happen? | 11:58 |
| mzanetti | seb128: yeah, long ago | 11:59 |
| seb128 | mzanetti, ok, I'm putting it in the next landing then | 11:59 |
| mzanetti | seb128: there's multiple parts though. the fact that the launcher swiched to dconf landed a while back already | 11:59 |
| seb128 | jgdx, ^ the reset launcher changeset was ok, right? seems like you looked at it by didn't top approve | 11:59 |
| mzanetti | seb128: the dynamic updating without reboot is not landed yet | 11:59 |
| seb128 | oh | 11:59 |
| seb128 | when is that landing? | 12:00 |
| jgdx | seb128, t'was | 12:00 |
| mzanetti | seb128: hopefully soon, the code is reviewed and looks good. dandrader is doing a last test run now, so it will go into the next unity silo | 12:00 |
| seb128 | k | 12:01 |
| dandrader | mzanetti, after i do "reset launcher", should I get an empty launcher (ie, with nothing pinned)? | 12:04 |
| mzanetti | dandrader: no, you should get the default list | 12:05 |
| dandrader | mzanetti, I got an empty launcher | 12:05 |
| mzanetti | hmm, that's not good | 12:05 |
| mzanetti | dandrader: where are you trying this? | 12:06 |
| dandrader | mzanetti, hmm, I pinned contacts | 12:06 |
| mzanetti | dandrader: on the desktop? | 12:06 |
| dandrader | mzanetti, and it stayed after the reset | 12:06 |
| mzanetti | hm, ok. something wrong then | 12:06 |
| Saviq | dednick, yeah, something breaks the indicators in that silo :| | 12:06 |
| dandrader | mzanetti, but dropping letters went away | 12:06 |
| dandrader | mzanetti, as it's not in the default list, I suppose | 12:06 |
| dednick | Saviq: ok, i'm taking a look now. just finished flashing | 12:06 |
| dandrader | mzanetti, testing on the device | 12:06 |
| Saviq | dednick, thanks | 12:07 |
| mzanetti | dandrader: this is the default list: http://paste.ubuntu.com/8552071/ | 12:07 |
| Saviq | dednick, ah crap | 12:08 |
| dednick | Saviq: did you update just the packages in silo, or do upgrade? | 12:08 |
| Saviq | dednick, my fault | 12:08 |
| Saviq | dednick, I resubmitted the test fix to include greeter profiles indeed | 12:08 |
| Saviq | dednick, let me undo that | 12:08 |
| dandrader | mzanetti, tried again: had dropping letters and phone pinned. then did the reset. had a launcher with just phone pinned | 12:09 |
| dednick | ah. hm, didnt see that on list | 12:09 |
| mzanetti | dandrader: what does "gsettings get com.canonical.Unity.Launcher items" tell you? | 12:10 |
| Saviq | dednick, yeah, because I resubmitted, but didn't *add* the actual MR | 12:10 |
| dednick | Saviq: ok. | 12:10 |
| Saviq | dednick, meaning my branch included the other | 12:11 |
| Saviq | dednick, so as you were, sorry for the noise | 12:11 |
| dednick | Saviq: no prob. guess that means the greeter branch is buggy though... | 12:11 |
| Saviq | dednick, well, the indicators updates weren't in the silo though, maybe that's why? | 12:11 |
| dandrader | mzanetti, ['appid://dialer-app'] | 12:11 |
| dednick | :/ | 12:12 |
| dednick | i'll try the greeter silo again | 12:12 |
| mzanetti | dandrader: err... that's weird... seems your reset call doesn't work then | 12:12 |
| dandrader | mzanetti, which is indeed what I see pinned in the launcher | 12:12 |
| mzanetti | dandrader: try "gsettings reset com.canonical.Unity.Launcher items" | 12:12 |
| dandrader | mzanetti, I added a comment in the MP with what it seems to be happening | 12:12 |
| Saviq | dandrader, mzanetti, that branch's in silo 10, was just about to test thes | 12:12 |
| Saviq | this | 12:12 |
| dandrader | mzanetti, at least my guess about it | 12:13 |
| mzanetti | ok, need to figure what's happening... dandrader, doesn't it work even if you use gsettings command line to reset it? | 12:14 |
| dandrader | mzanetti, what's that command line? I don't know it | 12:15 |
| Saviq | mzanetti, seems to work fine here | 12:15 |
| Saviq | dandrader, what did you use? dconf? | 12:15 |
| mzanetti | dandrader: try "gsettings reset com.canonical.Unity.Launcher items" | 12:16 |
| dandrader | Saviq, the "reset launcher" button un ubuntu-system-settings | 12:16 |
| * Saviq tries | 12:16 | |
| dandrader | Saviq, under the "reset phone" option | 12:16 |
| mzanetti | Saviq: note, the settings button isn't updated yet | 12:16 |
| Saviq | mzanetti, yeah, I see that | 12:16 |
| mzanetti | Saviq: unless you pull in that branchhttps://code.launchpad.net/~mzanetti/ubuntu-system-settings/update-reset-launcher | 12:16 |
| Saviq | since it requires a restart | 12:16 |
| dandrader | mzanetti, yeah, it works with the command line! | 12:17 |
| mzanetti | Saviq: not even that... still resets AS | 12:17 |
| Saviq | mzanetti, yeah, that too | 12:17 |
| mzanetti | hmm... that's even weirder then | 12:17 |
| mzanetti | as systemsettings calls QGSettings::reset() | 12:17 |
| Saviq | mzanetti, sounds like gsettings-qt bug? | 12:17 |
| mzanetti | not ruling it out, although I would have said "unlikely" | 12:18 |
| mzanetti | but if gsettings reset works | 12:18 |
| dandrader | mzanetti, actually it does not really. if I had only phone pinned it worked | 12:18 |
| mzanetti | aha | 12:18 |
| dandrader | mzanetti, but them I unpinned everything and pinned only dropping letters | 12:18 |
| dandrader | mzanetti, then the command leaves the launcher empty | 12:18 |
| mzanetti | hmm... crap... I think I know what it is | 12:18 |
| mzanetti | Saviq: sorry then... I guess you can drop it from the silo | 12:19 |
| mzanetti | I'll fix it today | 12:19 |
| mzanetti | depends on when you want to release | 12:19 |
| Saviq | ok, dropping | 12:19 |
| === MacSlow|lunch is now known as MacSlow | ||
| dednick | Saviq: hm. greeter profiles silo doesnt work at all for me now... u8 cycles on startup. | 12:23 |
| Saviq | dednick, I don't think it's built proper | 12:23 |
| Saviq | dednick, I was waiting for silo 10 to land, can kick unity8 build if you want though | 12:23 |
| dednick | Saviq: meh. it can wait | 12:24 |
| dandrader | Cimi, started reviewing https://code.launchpad.net/~cimi/unity8/fix-1378920/+merge/238012 | 12:37 |
| dandrader | Cimi, when I press the volume buttons (with stock image) I don't hear anything | 12:38 |
| dandrader | Cimi, is that expected? | 12:38 |
| Saviq | dandrader, that's not landed yet | 12:38 |
| Saviq | dandrader, https://code.launchpad.net/~macslow/unity8/synchronous-notification/+merge/229059 | 12:39 |
| dandrader | I also cannot reproduce bug 1378920 | 12:39 |
| ubot5 | bug 1378920 in unity8 (Ubuntu) "Synchronous sound notifications pause video" [High,In progress] https://launchpad.net/bugs/1378920 | 12:39 |
| dandrader | using krillin r81 | 12:39 |
| Saviq | dandrader, yeah, because that was reverted | 12:41 |
| MacSlow | dandrader, there's http://bazaar.launchpad.net/~unity-api-team/unity-notifications/trunk/view/head:/examples/sound.py you could use to trigger a notification with sound | 12:42 |
| Saviq | mzanetti, can you please resubmit quicklist on top of touch ownership please | 12:43 |
| === davidcalle_ is now known as davidcalle | ||
| Saviq | please | 12:43 |
| MacSlow | dandrader, that would be enough to reproduce the video-playback-pause | 12:43 |
| === dandrader is now known as dandrader|afk | ||
| mzanetti | Saviq: one "please" would have been enough :) | 12:46 |
| Saviq | mzanetti, please :P | 12:46 |
| mzanetti | Saviq: https://code.launchpad.net/~mzanetti/unity8/rework-quicklist-visuals/+merge/238149 | 12:49 |
| Saviq | mzanetti, tx | 12:49 |
| mzanetti | Saviq: it's interesting that bzr thinks line 98/99 would be my change | 12:52 |
| === _salem is now known as salem_ | ||
| Saviq | mzanetti, had to pull your MP from the silo after all, it conflicted elsewhere this time (stupid bzr) | 13:10 |
| Saviq | /food | 13:10 |
| * mzanetti grumbles | 13:11 | |
| tsdgeos | dednick: https://code.launchpad.net/~nick-dedekind/unity8/expanded-panel-design/+merge/237031/comments/584355 | 13:15 |
| === dandrader|afk is now known as dandrader | ||
| dandrader | hmm, got media-hub-service eating "392%" of the phone's CPU after playing a video. | 13:23 |
| Saviq | jhodapp, ↑ | 13:24 |
| jhodapp | dandrader, Saviq: yeah that's a logged bug already | 13:24 |
| dandrader | jhodapp, good to know! | 13:25 |
| jhodapp | will be looking at that in the next several days | 13:25 |
| tsdgeos | dednick: there? | 13:54 |
| mzanetti | tsdgeos: do you also know why testShell now disappears? | 13:57 |
| tsdgeos | mzanetti: what do you mean disappears? | 14:01 |
| mzanetti | tsdgeos: I figured it... same as in testShellWithPing happened | 14:02 |
| mzanetti | it got changed to have a Row {} as main item | 14:02 |
| mzanetti | wich makes it go to 0,0 size at some point during the test run | 14:02 |
| Saviq | mzanetti, fixed in my branch | 14:05 |
| === boiko__ is now known as boiko | ||
| tsdgeos | Cimi: do you know if this is by design? https://code.launchpad.net/~nick-dedekind/unity8/expanded-panel-design/+merge/237031/comments/584390 | 14:11 |
| Cimi | tsdgeos, the panel? | 14:13 |
| Cimi | tsdgeos, yes, approved | 14:13 |
| Cimi | tsdgeos, there might be bugs in the UX, but nick is on track... | 14:13 |
| Cimi | tsdgeos, he went last week to the office and reviewed with vesa, there were things to tweak, so he might be addressing them | 14:14 |
| === artmello_ is now known as artmello | ||
| mzanetti | greyback: have time for a focus related review? https://code.launchpad.net/~mzanetti/unity8/refocus-dash-on-app-crash/+merge/238166 | 14:18 |
| greyback | mzanetti: sure | 14:19 |
| greyback | mzanetti: "interctive" - typo | 14:19 |
| mzanetti | greyback: where? | 14:20 |
| mzanetti | wah | 14:20 |
| mzanetti | how come it still works? | 14:20 |
| greyback | mzanetti: you decided against patching qtmir to not set the focused app to null - that was my first thought. Ok will test in a while | 14:20 |
| mzanetti | greyback: yeah, qtmir has a comment there: "The app disappeard. Let unity decide what to do wit it" | 14:21 |
| mzanetti | greyback: given that its our long term plan anyways, I decided to do that | 14:21 |
| greyback | mzanetti: gotcha | 14:21 |
| tsdgeos | dednick: Cimi: i'm also concenrned about the high number of timers we seem to have in there | 14:22 |
| tsdgeos | why so many? | 14:22 |
| Cimi | tsdgeos, this is probably something left from the prototype | 14:26 |
| dednick | tsdgeos: Cimi: not from the prototype. the scroll timer is used for scrolling (holding mouse on area emits scrolls ever x ms). velocity timer is used to determine when velocity decreases below a threashold. alignment timer is used for a delay. | 14:52 |
| dandrader | mzanetti, should https://code.launchpad.net/~mzanetti/unity8/fix_snap_decision_test/+merge/238015 get in or not? its description says "This proposal has been superseded by a proposal from 2014-10-10. " | 14:55 |
| mzanetti | dandrader: I just reactivated it, still need to fill in the checklist though | 14:56 |
| Saviq | tsdgeos, for your dash bottom edge branch, make the bottom tab thingy catch taps please | 15:06 |
| mzanetti | Saviq: hey, on that branch, testwhitespace fails, but doesn't tell me where and I can't find any :/ | 15:06 |
| mzanetti | seen that before? | 15:06 |
| mzanetti | https://code.launchpad.net/~mzanetti/unity8/fix_snap_decision_test/+merge/238015 | 15:06 |
| greyback | death to the whitespace test! | 15:06 |
| Saviq | checkin' | 15:07 |
| Saviq | dednick, the steps from https://code.launchpad.net/~nick-dedekind/unity8/lp1336715-checkable-bindings/+merge/234503/comments/582588 still fail for me? | 15:09 |
| mzanetti | Saviq: hmm... I deleted everything from the file, saved it, pasted everything back saved it again | 15:09 |
| mzanetti | working now | 15:09 |
| mzanetti | maybe Josh edited the file with windows :D | 15:10 |
| Saviq | mzanetti, | 15:10 |
| Saviq | /home/michal/dev/canonical/unity8/repo/tests/qmltests/tst_Shell.qml: bad whitespace in multiple lines | 15:10 |
| mzanetti | yeah | 15:10 |
| dednick | Saviq: hm. i've been meaning to test that case. | 15:10 |
| mzanetti | I know its's tst_Shell | 15:10 |
| mzanetti | but in there, also LP diff doesn't show any changes that would contain whitespaces | 15:11 |
| Saviq | mzanetti, yeah, can't see where either | 15:11 |
| dednick | Saviq: give me a minute. will try | 15:11 |
| Saviq | dednick, so yeah, the kick is that they're now cross-synced | 15:12 |
| Saviq | dednick, meaning that if I enable in settings, it gets disabled in indicators | 15:12 |
| dednick | wtf | 15:12 |
| Saviq | dednick, basically it's a toggle | 15:13 |
| larsu | dednick: did you say you'll take care of the time bugs or do you want me to take another look? | 15:13 |
| Saviq | dednick, settings sends toggle, indicator switches without checking for the real value, but maybe it's actually settings' fault | 15:13 |
| dednick | larsu: if you've got time (har har), otherwise i will try get to them later. | 15:13 |
| larsu | dednick: haha okay. I'll reassign to you for now | 15:14 |
| dednick | Saviq: it should se to the server check value rather than just switching | 15:14 |
| Saviq | dednick, or that | 15:14 |
| dednick | Saviq: i'll have to check which frontend is wrong first. | 15:15 |
| Saviq | dednick, k, I'm landing the silo anyway | 15:15 |
| Saviq | as it's not getting worse | 15:15 |
| dednick | Saviq: ok | 15:15 |
| dednick | larsu: thanks. | 15:16 |
| mzanetti | greyback: hey, you reviewed this already, I just added the test now, mind reapproving? https://code.launchpad.net/~mzanetti/unity8/fix_snap_decision_test/+merge/238015 | 15:16 |
| greyback | mzanetti: sure. | 15:17 |
| tsdgeos | Saviq: sorry was looking at the phone, what do you mean with " make the bottom tab thingy catch taps please" ? | 15:19 |
| Saviq | tsdgeos, with touchOwnership I can tap through the bottom hint | 15:20 |
| Saviq | the hint itself should be tappable I think | 15:20 |
| tsdgeos | Saviq: why? | 15:20 |
| tsdgeos | it's never been like that | 15:21 |
| Saviq | tsdgeos, because otherwise people will try and tap it and it launches an app | 15:21 |
| tsdgeos | and it's not something design requested | 15:21 |
| Saviq | tsdgeos, I don't mean it should *do* anything | 15:21 |
| Saviq | tsdgeos, just it shouldn't be transparent to input | 15:21 |
| tsdgeos | ok, we had to do some extra code with dandrader to make it happen :/ | 15:22 |
| tsdgeos | but ok | 15:22 |
| tsdgeos | Saviq: can you make a note in the MR so i won't forget while i review dednicks code? | 15:22 |
| Saviq | tsdgeos, to make the hint thingy transparent? | 15:23 |
| tsdgeos | Saviq: well to make sure it did not conflict with the scrollarea when moving up in that area | 15:25 |
| Saviq | tsdgeos, ok, I'll leave a note then | 15:25 |
| dandrader | Saviq, you mean make only that tab absorb touches, but not the whole bottom area, right? | 15:25 |
| Saviq | dandrader, yes | 15:25 |
| dandrader | tsdgeos, ^ | 15:26 |
| Saviq | dandrader, just the white thingy with ^ | 15:26 |
| tsdgeos | Saviq: ah i thought you meant the whole area | 15:26 |
| Saviq | tsdgeos, no, just the white tab hinting thingy | 15:26 |
| tsdgeos | ok | 15:26 |
| tsdgeos | that makes more sense | 15:26 |
| Saviq | thanks | 15:27 |
| dednick | Saviq: hm. installed silo + apt install unity8 = blank screen... | 15:27 |
| Saviq | I did say *tab*, *hint*, tried to explain what I mean ;) | 15:27 |
| tsdgeos | sure sure | 15:27 |
| Saviq | dednick, silo 11? | 15:27 |
| Saviq | 10 | 15:27 |
| dednick | er 10... | 15:27 |
| tsdgeos | i'm not saying it's your fault | 15:27 |
| dednick | doh | 15:27 |
| Saviq | dednick, no, 10 is good | 15:28 |
| Saviq | dednick, generally use `citrain device-upgrade 10` | 15:28 |
| Saviq | from phablet-tools-citrain | 15:28 |
| dednick | Saviq: ah. didnt know there was that... | 15:28 |
| Saviq | dednick, worked fine here every time | 15:28 |
| Saviq | so not sure what got you | 15:28 |
| dednick | Saviq: will try. perhaps there is a missing dep somewhere | 15:28 |
| Saviq | dednick, there's qtmir there as well | 15:29 |
| Saviq | dednick, just do `citrain device-upgrade 10 $password` now | 15:29 |
| Saviq | dednick, it will do the right thing | 15:29 |
| dednick | Saviq: presumably if the specific version was needed it would be deped to that? | 15:29 |
| Saviq | dednick, it was, yeah | 15:30 |
| Saviq | dednick, but maybe we do have some dependency problem still | 15:30 |
| === dandrader is now known as dandrader|lunch | ||
| dednick | hm. perhaps my install wasn't fresh or something.. | 15:30 |
| Saviq | - qtdeclarative5-qtmir-plugin (>= 0.4.3), | 15:30 |
| Saviq | + qtdeclarative5-qtmir-plugin (>= 0.4.4), | 15:30 |
| dednick | or had another silo in apt | 15:31 |
| Saviq | dednick, uugh... your 1500ms timer... plane mode takes more than that to activate... | 15:32 |
| dednick | Saviq: hm. but it should be updated by a subsequent trigger from dbus :( | 15:33 |
| dednick | unless dbus is being faaaar too clever | 15:33 |
| Saviq | dednick, yeah, *but* | 15:33 |
| Saviq | dednick, it goes on, off, on | 15:33 |
| Saviq | really not great | 15:33 |
| dednick | for it's own stupidity that is | 15:33 |
| dednick | Saviq: :/ | 15:33 |
| Saviq | aand now I'm out of sync | 15:34 |
| Saviq | dednick, I can't land that, I'm stuck in plane mode now :| | 15:35 |
| Cimi | tsdgeos, wondering if we can avoit the timers using a mousearea | 15:35 |
| dednick | ahh.. ffs | 15:35 |
| Saviq | dednick, if it decides to go back, it let's the other side know it went back | 15:35 |
| Saviq | lets | 15:36 |
| Saviq | dednick, and we basically get a feedback loop | 15:36 |
| dednick | bloody useless | 15:36 |
| tsdgeos | Cimi: how? as far as i undertand being pressed at a certain X has to trigger scroll | 15:36 |
| tsdgeos | i don't see how to do that without a timer | 15:36 |
| dednick | shouldnt do that :/ only supposed to be on user interaction! | 15:37 |
| Cimi | tsdgeos, reading mouse event x position of a mousearea on top? | 15:37 |
| Saviq | dednick, that's just my suspicion, not saying that's what happens | 15:37 |
| Saviq | dednick, pulling that MP from the silo in any case... | 15:37 |
| tsdgeos | Cimi: and how are you going to keep scrolling? | 15:37 |
| tsdgeos | Saviq: which one? | 15:38 |
| dednick | Cimi: need to keep scrolling without position changes | 15:38 |
| Saviq | tsdgeos, https://code.launchpad.net/~nick-dedekind/unity8/lp1336715.server-value-reassert/+merge/237822 | 15:38 |
| tsdgeos | ah couldn't find that 1500 reference | 15:39 |
| dednick | Saviq: Error reading https://launchpad.net/api/1.0/~ci-train-ppa-service/+archive/ubuntu: Not Found | 15:40 |
| dednick | although still seems to be doing something... | 15:41 |
| dednick | Saviq: http://paste.ubuntu.com/8553318/ | 15:41 |
| Saviq | dednick, yeah, that didn't work | 15:41 |
| Saviq | wonder if ppa:ci-train-ppa-service/ubuntu/landing-010 is correct | 15:42 |
| dednick | something is going really slow.. | 15:43 |
| Saviq | dednick, you might need to reflash, something got real weird in your sources | 15:43 |
| dednick | i just did | 15:43 |
| dednick | although not a wipe | 15:43 |
| Saviq | dednick, no no, no wipe necessary | 15:44 |
| Saviq | dednick, apt-cache policy phablet-tools-citrain? | 15:44 |
| * Saviq gotta split for now | 15:45 | |
| dednick | Saviq: 1.1+14.10.20141010-0ubuntu1 | 15:46 |
| dednick | will just do manual | 15:46 |
| MacSlow | Saviq, I guess https://code.launchpad.net/~macslow/unity-settings-daemon/max-volume-notification-warning/+merge/238180 needs and rtm branch too?! | 15:52 |
| === gatox is now known as gatox_lunch | ||
| dednick | Saviq: qtmir-anrdoid wasn't upgraded with manual silo install. | 16:03 |
| dednick | which means the rest of mir wasn't either. | 16:04 |
| greyback | mzanetti: both u8 branches approved | 16:10 |
| mzanetti | greyback: thanks a lot :) | 16:12 |
| dpm | dbarth, it seems that the FB webapp asks me for user name and password every time I start it. Is this a known bug? Or is it supposed to work like that? | 16:20 |
| dednick | dandrader|lunch: i think i sorted out some more problems with that velocity calculator now. | 17:03 |
| dednick | it should be fairly quiet now :) | 17:03 |
| === gatox_lunch is now known as gatox | ||
| === alan_g is now known as alan_g|EOD | ||
| === karni is now known as karni-afk | ||
| === karni-afk is now known as karni | ||
| === dandrader|lunch is now known as dandrader | ||
| mzanetti | dandrader: hey, the launcher dconf branch should be fixed now | 19:13 |
| mzanetti | thanks for catching that issue | 19:13 |
| === rpadovani is now known as rpadovani|Dota | ||
| dandrader | mzanetti, np. I've hit my EOD. will test it again tomorrow | 19:16 |
| === salem_ is now known as _salem | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!