| === kalikiana__ is now known as kalikiana_ | ||
| === alblasch is now known as ablasche | ||
| === wgrant_ is now known as wgrant | ||
| dholbach | good morning | 08:09 |
|---|---|---|
| === dholbach_ is now known as dholbach | ||
| === lan3y is now known as Laney | ||
| === Laney is now known as Guest1882 | ||
| === Guest1882 is now known as Laney | ||
| JamesTait | Good morning all! :-D | 09:59 |
| === greyback_ is now known as greyback | ||
| nerochiaro | zsombi: good morning. hope you had a good start of the year. I have a quick question for you, how does the StateSaver work when elements are loaded by a Loader ? | 10:43 |
| zsombi | nerochiaro: morning sire!!! indeed a good start, but got few bugs to solve :) the StateSaver works in the same way as usual. | 10:44 |
| zsombi | nerochiaro: Loader must have an ID as it will be the parent of the loaded element | 10:45 |
| nerochiaro | zsombi: interesting. it looks like in some case it doesn't work even if the loader has an id. i'll investigate further | 10:46 |
| zsombi | nerochiaro: so you have a component loaded by the Loader and the component uses the StateSaver? | 10:46 |
| nerochiaro | zsombi: yes | 10:47 |
| zsombi | nerochiaro: we have test cases for that, check them | 10:47 |
| nerochiaro | zsombi: ok | 10:48 |
| nerochiaro | zsombi: i seem to be doing all correctly, same as the test case, but I don't see the state being restored | 11:22 |
| nerochiaro | zsombi: in the app, of course it works in a separate case | 11:22 |
| zsombi | nerochiaro: so you don't see the state restored in your case? | 11:27 |
| nerochiaro | zsombi: no | 11:27 |
| zsombi | do you see any warnings? | 11:27 |
| nerochiaro | zsombi: i put a onSavedPropertyChanged: console.log(savedProperty) and the value is not hte value i saved. no warnings from statesaver either | 11:28 |
| nerochiaro | zsombi: i know the saved value is different because if i cat the file .config the right value is there | 11:28 |
| zsombi | nerochiaro: so no warnings on the console from StateSaver? | 11:28 |
| nerochiaro | zsombi: no | 11:28 |
| zsombi | nerochiaro: weird... show me the branch | 11:29 |
| nerochiaro | zsombi: one moment | 11:29 |
| nerochiaro | zsombi: lp:~amanzi-team/gallery-app/gallery-app-listview-state/ | 11:33 |
| nerochiaro | zsombi: check Components/MediaViewer.qml | 11:33 |
| nerochiaro | zsombi: (i will have to run out for 15 minutes soon) | 11:34 |
| nerochiaro | zsombi: so take your time | 11:34 |
| zsombi | nerochiaro: ok, will check it | 11:34 |
| nerochiaro | zsombi: thanks | 11:35 |
| === _salem is now known as salem_ | ||
| zsombi | nerochiaro: cannot see such a QML file... | 11:41 |
| zsombi | nerochiaro: ah, under Components/MediaViewer there is one | 11:42 |
| mzanetti | popey: hi | 12:08 |
| mzanetti | popey: could you please test again if this works with a smaller image? https://code.launchpad.net/~mzanetti/reminders-app/add-image-support/+merge/199031 | 12:08 |
| popey | mzanetti: ok | 12:09 |
| nerochiaro | zsombi: news ? | 12:21 |
| zsombi | nerochiaro: on standup, but couldn't find your StateSaver there... | 12:21 |
| mzanetti | popey: and I've added the cmake dep to the script in here: https://code.launchpad.net/~mzanetti/reminders-app/switch-to-cmake/+merge/199377 | 12:22 |
| nerochiaro | zsombi: rc/qml/Components/MediaGrid.qml line 48 | 12:25 |
| nerochiaro | zsombi: sry I had pointed you to the wrong file before | 12:27 |
| zsombi | timp: you completely forgot this! https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/pickerpanel/+merge/199250 ;) | 12:44 |
| timp | zsombi: no it is 4th on my to review list ;) | 12:44 |
| zsombi | timp: last year it was the first on your list :D | 12:44 |
| zsombi | you downgraded it???! | 12:44 |
| nerochiaro | zsombi: done with standup ? | 12:55 |
| zsombi | nerochiaro: still | 12:58 |
| zsombi | in | 12:58 |
| nik90 | zsombi: btw nice touch on the SDK dialer component. I am currently transitioning the clock app to use them :) | 13:15 |
| zsombi | nik90: it has been there for a while ;) | 13:16 |
| nik90 | zsombi: I know, but only recently I really took notice of that component. | 13:16 |
| === matiasb1 is now known as matiasb | ||
| zsombi | nik90: glad that you'll take it in use :) | 13:20 |
| nik90 | zsombi: I did find two small bugs, will report them a bit later when I am done with the MP. But otherwise it has helped remove quite a bit of code from the clock app. | 13:21 |
| zsombi | nik90: +1 | 13:24 |
| nerochiaro | zsombi: i've worked around the problem for now, but it would be still better if we could figure out what's going on | 13:39 |
| zsombi | nerochiaro: ended the standup, lemme check the file | 13:39 |
| zsombi | nerochiaro: who loads this MediaGrid.qml? | 13:42 |
| nerochiaro | zsombi: rc/qml/PhotosOverview and that one is loaded by rc/qml/MainScreen | 13:42 |
| zsombi | nerochiaro: hmm... weird... | 13:49 |
| nerochiaro | zsombi: fails for you too ? | 13:49 |
| zsombi | nerochiaro: I have checked the code, all seems to be OK, I'll try to build it | 13:50 |
| nerochiaro | zsombi: ok. i'm testing on desktop at the moment | 13:50 |
| zsombi | uhh... dependencies... | 13:50 |
| nerochiaro | zsombi: just apt-get install gallery-app | 13:50 |
| nerochiaro | zsombi: i mean build-dep gallery-app | 13:51 |
| nerochiaro | zsombi: sucess @ build ? | 14:10 |
| zsombi | nerochiaro: still builds | 14:10 |
| zsombi | nerochiaro: done, success | 14:10 |
| zsombi | nerochiaro is there any script to help it running on desktop? | 14:13 |
| nerochiaro | zsombi: just run src/gallery-app | 14:14 |
| zsombi | nerochiaro: actually there is something that overwrites the savedDistance to 0 upon next start... | 14:20 |
| nerochiaro | zsombi: what is that ? | 14:20 |
| nerochiaro | zsombi: it has first to be restored, then overwritten, if the StateSaver is working the way it should. and i never see it being resotored | 14:20 |
| zsombi | nerochiaro: what is what? the one which overwrites? don't know... something that happens after the state is restored... | 14:21 |
| zsombi | nerochiaro: will have to dive more into it | 14:21 |
| zsombi | nerochiaro: but will check it tomorrow... | 14:22 |
| nerochiaro | zsombi: i don't think it ever gets restored. if i put an onChanged handler on it I never see the value being restored and hten overwritten | 14:22 |
| nerochiaro | zsombi: i won't be around tomorrow, but can you please send me an email ? | 14:22 |
| zsombi | nerochiaro: right, I see that. There;s somewhere between the time the state is about to be restored, but it is overruled by something... | 14:22 |
| zsombi | nerochiaro: sure | 14:22 |
| nerochiaro | zsombi: perfect, i'll check once in a while and come up if there's more to discuss | 14:23 |
| zsombi | nerochiaro: cool!! | 14:23 |
| nerochiaro | zsombi: thanks | 14:23 |
| zsombi | nerochiaro: ywc | 14:23 |
| nerochiaro | zsombi: i am trying with other lists that are loaded in the same way, same problem | 14:30 |
| nerochiaro | zsombi: just to confirm | 14:30 |
| zsombi | nerochiaro: then it's worth reporting a bug!!! | 14:37 |
| nerochiaro | zsombi: even if i can't repro in a simple exaple ? | 14:37 |
| zsombi | nerochiaro: well, not then... give me few days to hunt for it, ok? | 14:38 |
| nerochiaro | zsombi: ok, i'll let you upgrade from tomorrow to "a few days" :) but maybe then you can also look into this at the same time: you will notice that if you scroll around the list, especially past the edges, you will get binding loops for preferredHighlightEnd having a loop with height. We tried to figure this out with tim but couldn't get to the bottom of it. | 14:40 |
| nerochiaro | zsombi: since you suggested to use preferredHighlightEnd in the first place ... | 14:40 |
| zsombi | nerochiaro: few days was before I found the solution to the bug in DatePicker, so I'll recall my few days :D | 14:42 |
| nerochiaro | zsombi: ok then :) | 14:49 |
| mhall119 | Wellark: ping me when you're around | 14:55 |
| nerochiaro | oSoMoN: what do you think we should do regarding this MR that has been pending for quite a while now ? https://code.launchpad.net/~amanzi-team/gallery-app/gallery-app-save-state/+merge/198364 | 15:00 |
| oSoMoN | nerochiaro, are you able to successfully run the tests for this MR on your device? | 15:16 |
| nerochiaro | oSoMoN: they were all passing on my device last i tried. i think the crashes were on mako ? | 15:18 |
| oSoMoN | nerochiaro, I’m unable to retrieve the debs from the links in the MR comments posted by the CI job (getting 404s), I’ll trigger a new CI run | 15:35 |
| nerochiaro | oSoMoN: ok. i'm upgrading the phone to the latest ubuntu-system from channel trusty | 15:38 |
| oSoMoN | nerochiaro, you should probably use trusty-proposed to test against the latest stuff | 15:40 |
| om26er | oSoMoN, who is working on gallery-app now ? | 15:41 |
| nerochiaro | om26er: i'm doing some work on it, but it's not "my app" | 15:41 |
| oSoMoN | om26er, noone is actively working on it, I’m doing AP test fixes if needed | 15:42 |
| om26er | nerochiaro, oSoMoN got it. Do we have someone filling gusch place ? | 15:43 |
| om26er | soonish ? | 15:43 |
| nerochiaro | om26er: not yet | 15:43 |
| oSoMoN | om26er, not yet, but we’re in the process of hiring | 15:43 |
| om26er | nerochiaro, oSoMoN ok. Thanks | 15:43 |
| === gatox is now known as gatox_lunch | ||
| Wellark | mhall119: ping | 16:28 |
| mhall119 | Wellark: pong | 16:29 |
| Wellark | please use the "other" IRC server if you need to reach me urgently. I sometimes miss stuff on Freenode :) | 16:29 |
| Wellark | mhall119: what's up? | 16:30 |
| mhall119 | Wellark: I spoke to ted a couple weeks ago about enabling the HUD toolbar icons from QML apps, and he said that the QML Actions API needs to be able to send the toolbar-item-hint property over DBUS for the HUD to bind it to an icon, is there a roadmap for implementing that? | 16:30 |
| Wellark | mhall119: short answer: no | 16:31 |
| mhall119 | can we get one? Having unuseable icons in the HUD is annoying | 16:31 |
| Wellark | mhall119: agreed. | 16:31 |
| mhall119 | Wellark: so who do I need to bribe to get this on the schedule for this cycle? | 16:32 |
| Wellark | mhall119: thostr_ | 16:32 |
| mhall119 | ok, I'll send him an email | 16:33 |
| mhall119 | thanks | 16:33 |
| Wellark | mhall119: np. | 16:33 |
| Wellark | mhall119: other option would just be removing the icons until we have API for them. would be easier. | 16:34 |
| mhall119 | would be, but making them useful would be better | 16:34 |
| balloons | nik90, ping | 16:34 |
| Wellark | mhall119: well, sure. it's just that we must make compromises on features vs. man-power | 16:35 |
| Wellark | and unfortunately you are the first person to ask about the toolbar icons in 9 months or so. | 16:36 |
| mhall119 | man, we have *got* to get that engineer cloning program working | 16:36 |
| timp | Wellark: what's the toolbar-item-hint? | 16:36 |
| Wellark | just saying that it has not been a priority feature :( | 16:36 |
| balloons | nik90, does this have the online search in it? https://code.launchpad.net/~nik90/ubuntu-clock-app/more-stopwatch-tests/+merge/193922 | 16:36 |
| timp | Wellark: it may be simpler to add an iconName property to the Action. No need to pass the icon itself | 16:36 |
| timp | but the only icons from the theme can be used | 16:36 |
| timp | (just an idea, mybe a bad one) | 16:37 |
| Wellark | timp: the hud toolbar items are a bit special | 16:37 |
| timp | mhall119: nobody has time to make the engineer cloning program work ;) | 16:37 |
| mhall119 | timp: toolbar-item-hint AIUI, is a DBUS property that tells the HUD to bind an action to an icon in the HUD, rather than (or in addition to) the list | 16:37 |
| Wellark | they are basically predefined actions that an application may or may not define for it self | 16:37 |
| === gatox_lunch is now known as gatox | ||
| nik90 | balloons: pong | 16:58 |
| nik90 | balloons: no it doesn't have online search in it | 16:58 |
| balloons | nik90, see above about the merge.. kk, just wanted to double check that indeed it did not :-) I didn't see it but ;-) | 16:58 |
| nik90 | balloons: i removed it since when I looked at it more closely, I realised it was quite similar to the local search | 16:59 |
| oSoMoN | nerochiaro, CI job finished, you can grab the debs for testing at http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/1949/artifact/work/output/*zip*/output.zip | 16:59 |
| balloons | nik90, right I remember our conversation.. perfect | 17:00 |
| balloons | ty | 17:00 |
| nik90 | :) | 17:00 |
| nik90 | popey: ping | 17:00 |
| nerochiaro | oSoMoN: i'm already installing it | 17:01 |
| nerochiaro | oSoMoN: thanks for the heads up though | 17:01 |
| popey | nik90: pong | 17:01 |
| popey | nik90: just on the phone for an hour now, but type at me anyway | 17:01 |
| nik90 | popey: sure | 17:01 |
| nik90 | popey: I was just wondering if the core apps PPA updates trickle down to the phone automatically? | 17:02 |
| nik90 | popey: or if I should request it specifically? | 17:02 |
| popey | nik90: they do not | 17:02 |
| popey | we request click packages to be updated by sergio and I confirm in the store | 17:03 |
| balloons | ^^ indeed, I push things to be landed, like your changes.. It's already been requested | 17:04 |
| nik90 | balloons: ah okay | 17:04 |
| nik90 | I will remember that | 17:05 |
| popey | beuno: Changes can not be accepted and published at this time. Please try again later. | 17:12 |
| popey | halp! | 17:12 |
| beuno | popey, you're so impatient | 17:13 |
| beuno | let's see | 17:13 |
| popey | I know right? | 17:13 |
| beuno | popey, I'll have to bring down some logs, which will take a bit | 17:16 |
| popey | k | 17:17 |
| nerochiaro | oSoMoN: all tests seems to pass correctly on my maguro device | 17:20 |
| oSoMoN | nerochiaro, running them on my device now | 17:21 |
| beuno | popey, found the broken service, chasing it with IS | 17:25 |
| popey | thanks beuno | 17:30 |
| oSoMoN | nerochiaro, I’m still getting the same errors I was getting in Dec | 17:32 |
| oSoMoN | nerochiaro, gotta go now, can you describe how exactly you run the tests on your device in a comment to the MR? | 17:32 |
| === davmor2_ is now known as davmor2 | ||
| === balloons_ is now known as balloons | ||
| === bfiller is now known as bfiller_afk | ||
| === daker_ is now known as daker | ||
| === kalikiana__ is now known as kalikiana_ | ||
| === balloons is now known as Guest99854 | ||
| beuno | popey, this happened to you when doing what? | 18:07 |
| popey | approving this app https://myapps.developer.ubuntu.com/dev/click-apps/173/changerequest/ | 18:07 |
| beuno | popey, k, so there's a bug somewhere | 18:11 |
| beuno | I'll need to chase that tomorrow with a dev | 18:11 |
| popey | ok. | 18:12 |
| popey | kenvandine: ^ | 18:12 |
| kenvandine | thx | 18:12 |
| ahayzen | balloons_, ping | 18:13 |
| balloons_ | ahayzen, pong | 18:14 |
| ahayzen | balloons_, happy new year! ... i'm having an issue with the jenkins bot wondered if u could interpret the output for me https://code.launchpad.net/~andrew-hayzen/ubuntu-ui-toolkit/fix-swipe-delete-001/+merge/199906 | 18:14 |
| balloons_ | ahayzen, most certainly | 18:14 |
| balloons_ | ahayzen, at first glance it seems some things failed while running | 18:16 |
| balloons_ | but the test output is cryptic :-) I don't have super power on the toolkit | 18:16 |
| ahayzen | balloons_, i looked at it and didnt have a clue wht was going on :/ | 18:16 |
| balloons_ | you could ask Tim to re-run it, but my guess is something you've changed broke it | 18:17 |
| balloons_ | this is where I'm looking: https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/4417/ | 18:17 |
| * ahayzen wonders wht could be broken with such a small change... | 18:17 | |
| ahayzen | balloons_, thx | 18:17 |
| balloons_ | all the tests failed.. doesn't seem like it ran properly | 18:18 |
| balloons_ | ahayzen, ask tim to re-run, and if the results look funny again, we'll ping ci | 18:18 |
| === balloons_ is now known as balloons | ||
| ahayzen | balloons, ok thx... think timp has gone by now? | 18:19 |
| balloons | mm.. perhaps | 18:19 |
| balloons | ahayzen, I'll try and re-run it | 18:20 |
| ahayzen | balloons, thx :) | 18:20 |
| beuno | popey, kenvandine, fixed the problem and approved the update | 19:09 |
| kenvandine | beuno, thx | 19:09 |
| popey | thank you | 19:10 |
| balloons | ahayzen, it passed, but we need someone from that team to approve. I can't do it | 19:32 |
| balloons | might have to wait | 19:32 |
| ahayzen | balloons, cool... ah ok i was wondering wht tht message meant | 19:50 |
| === salem_ is now known as _salem | ||
| === bfiller_afk is now known as bfiller | ||
| nik90 | timp: thnx for getting started on https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1259917. The timing is impeccable. | 20:34 |
| ubot2 | Launchpad bug 1259917 in Ubuntu UI Toolkit "Using a Qml Loader in combination with Pagestack and Tabs results in incorrect positioning" [High,In progress] | 20:34 |
| nik90 | balloons: got a moment? | 20:49 |
| balloons | nik90, shoot | 20:49 |
| nik90 | balloons: https://bugs.launchpad.net/ubuntu-clock-app/+bug/1266841 | 20:49 |
| ubot2 | Launchpad bug 1266841 in Ubuntu Clock App "1 test error, when running in the emulator" [Medium,Incomplete] | 20:49 |
| nik90 | balloons: how long does it take for the click package to be updated? | 20:50 |
| balloons | nik90, ahh, sergio hasn't landed it yet | 20:50 |
| nik90 | balloons: can you confirm if my asserting in that bug report is correct? | 20:50 |
| nik90 | assertion* | 20:50 |
| balloons | you are correct | 20:51 |
| nik90 | balloons: what is sergio's nick? I like to ask him some questions related to core apps click packages version numbering and stuff. | 20:53 |
| balloons | serguisens | 20:54 |
| nik90 | thnx | 20:54 |
| === Ursinha is now known as Guest50378 | ||
| === Ursinha-afk is now known as Ursinha | ||
| nik90 | popey: do you happen to know who the best person would be to talk to about the wake/unlock devices when an alarm notification is triggered? | 23:11 |
| popey | nik90: tvoss i think. | 23:11 |
| nik90 | popey: ok. I will check with him tomorrow. | 23:12 |
| nik90 | popey: and accordingly update the progress in the email thread | 23:12 |
| === Ursinha is now known as Ursinha-afk | ||
| === Ursinha-afk is now known as Ursinha | ||
| === _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!