[06:34] <DanChapman_> Good Morning
[06:41] <dholbach> good morning
[06:56] <justCarakas> good morning all
[08:09] <JamesTait> Good morning all; happy Monday and happy Nurses Day! :-D
[08:41] <dpm> morning all
[08:42] <dpm> hi zsombi, I'm not sure who's the icons/toolbarbuttons expert from you guys, so I'll pick you :) Do you happen to know why that is not working, and whether I should file a bug or I'm doing something wrong? http://pastebin.ubuntu.com/7451341/
[08:43] <zsombi> dpm: looks like the theme the iconName is taken either is not having the icon or the path the toolkit uses is wrong
[08:44] <dpm> this is from suru
[08:45] <zsombi> dpm: the toolkit uses the ubuntu-mobile theme, so if this is not there, then it won't find it
[08:45] <dpm> aha
[08:47] <dpm> zsombi, do you happen to know if there is a particular reason why the toolkit has not migrated to suru? Or should I ask the designers?
[08:49] <zsombi> dpm: perhaps :) we had problems with this, as ubuntu-mobile weren't having all the icons in the beginning, neither suru, nor ubuntu-desktop. Ideally we should use the mode specific icons (Touch/Desktop) but as none of those were complete, we had this hardcoded.
[08:50] <dpm> ok, thanks!
[09:24] <t1mp_> dpm: iconName is still using the old theme
[09:24] <t1mp_> dpm: ubuntu-mobile. There are still some issues (with unity8?) that need to be fixed before we can switch to suru
[09:24] <popey> kalikiana_: is https://code.launchpad.net/~elopio/ubuntu-ui-toolkit/datepicker-autopilot_helper/+merge/218909 good now elopio answered your question?
[09:27] <t1mp_> ok zsombi answered already :)
[09:30] <dpm> thanks for the info t1mp_
[09:31] <t1mp_> dpm: there was a blueprint, but it seems to have disappeared, at least I cannot find it :s
[09:32] <dpm> np, for the moment that clarifies things, thanks!
[09:33] <kalikiana_> popey: yep. approved now
[09:33] <popey> thanks kalikiana_
[09:38] <t1mp_> dpm: an upcoming MR will show all the available icons (in the standard icon theme, so ubuntu-mobile at the moment) in the UITK gallery https://code.launchpad.net/~kalikiana/ubuntu-ui-toolkit/demoIcons/+merge/217433
[09:39] <dpm> t1mp_, ah, cool, I had seem some movement in that direction, but I didn't know what was going on. Nice!
[09:46] <rpadovani> mzanetti, o/ I took a look, but I have no idea how to fix https://code.launchpad.net/~rpadovani/reminders-app/setDefaultFontSize/+merge/219092/comments/522611
[09:46] <rpadovani> Sorry!
[09:46] <mzanetti> rpadovani: basically there's a commit in the history we just need to revert :)
[09:46] <mzanetti> rpadovani: I think its rev 71
[09:47] <mzanetti> rpadovani: can you try reverting that and test is notes created with reminders look ok on the website?
[09:47] <mzanetti> rpadovani: right now, if we create a note on the phone it has a huge font on the website
[09:47] <rpadovani> mzanetti, ah ok, yes, it's the one I found that can be causes the damage, I'll try asap
[09:47] <mzanetti> damage :D
[09:48] <mzanetti> I wouldn't go that far
[09:48] <rpadovani> lol, the issues? The incorrect behavior?
[09:48] <mzanetti> :)
[09:55] <davidcalle> dpm, hi
[09:56] <dpm> hey davidcalle
[09:56] <davidcalle> dpm, is there any doc or code samples for the new headers?
[09:57] <dpm> davidcalle, I think they've not yet landed in trunk, they are in staging, but t1mp_ should be able to tell you
[09:57] <nik90> bzoltan: can you push the latest uitk updates to the SDK PPA pls? I want to test the new headers on trusty desktop.
[09:58] <davidcalle> dpm, thanks :)
[09:58] <nik90> fginther: can you check why jenkins hasn't merged https://code.launchpad.net/~nik90/ubuntu-clock-app/disable-clock-hand/+merge/217907. The MP is 5 days old and top approved.
[09:59] <rpadovani> mzanetti, you're right, but now text in 16 is toooo small, what's the best size? Maybe in units.gu()?
[09:59] <bzoltan> nik90:  sure
[10:00] <nik90> bzoltan: thnx
[10:01] <mzanetti> rpadovani: hmm, yeah... make it units.gu(2) or similar
[10:01] <zbenjamin> dpm: does this still happen to you? https://bugs.launchpad.net/ubuntu/+source/qtcreator/+bug/1255462
[10:02] <nik90> popey: Hi. There are 2 MPs submitted by an external contributor which are related to design. Who do I check with before approving them? They are rather small design changes.
[10:02] <nik90> popey: https://bugs.launchpad.net/ubuntu-clock-app/+bug/1318344
[10:02] <nik90> popey: https://bugs.launchpad.net/ubuntu-clock-app/+bug/1318248
[10:04] <popey> nik90: on the phone, will look in a bit
[10:04] <nik90> popey: np
[10:05] <rpadovani> mzanetti, https://code.launchpad.net/~rpadovani/reminders-app/setDefaultFontSize/+merge/219092
[10:06] <rpadovani> mzanetti, I did the revert with bzr merge . -r 71..70
[10:06] <mzanetti> rpadovani: nice. looks good. But I need more time to test this propoerly on various screen sizes and resolutions. can't do that right now
[10:06] <rpadovani> sure, np!
[10:09] <rpadovani> dpm, o/ When you have time, there is also https://code.launchpad.net/~rpadovani/reminders-app/1318078/+merge/219091
[10:23] <dpm> hi rpadovani, I saw it, will test it now, thanks!
[10:24] <rpadovani> thanks :-)
[10:24] <dpm> actually, let me fix the openssl copyright one first, that's the main blocker for getting the new design to the store
[10:27] <rpadovani> yes yes, it was just to notify you the branch, in case you miss it :-)
[10:32] <nik90> zsombi: is this bug related to the textfield? http://davmor2.co.uk/~davmor2/screenshots-phone/device-2014-05-12-111506.png
[10:33] <nik90> zsombi: I noticed this in the contacts app as well. the orange dot is at the wrong position due to the icon on the left of the textfield
[10:33] <zsombi> nik90: yes, and we already have a fix for that in teh staging
[10:34] <nik90> zsombi: awesome
[10:34] <zsombi> nik90: now that you are here, could you check this once more pls? https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-visibility/+merge/218916
[10:34] <nik90> zsombi: yes..on it
[10:36] <t1mp_> zsombi: fyi, I have an MR where pages inside PageStack are not active by default (if they are not pushed), https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/80-betterActive/+merge/219093
[10:37] <zsombi> t1mp_: ok, I'll check it
[10:38] <zsombi> nik90: unfortunately CI did not pick up the changes I made, so you need to build it on teh device if you wanna test there
[10:39] <nik90> zsombi: are you referring to https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/layouts-visibility/+merge/218916 ?
[10:39] <nik90> zsombi: if yes, I am testing on the desktop not on the phone
[10:39] <zsombi> nik90: yes, and ok :)
[10:42] <t1mp_> nik90: you asked about Action.visible being deprecated last week
[10:42] <t1mp_> nik90: I have an MR that makes it not deprecated anymore https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/actionVisibility/+merge/219109
[10:42] <dpm> rpadovani, don't worry, I don't miss them, I get e-mail notifications on each, and we go through http://people.canonical.com/~dpm/core-apps-sponsoring/ each morning :)
[10:42] <rpadovani> dpm :-) Ok, so I'll leave to your work: I code, you fix it :P
[10:42] <nik90> t1mp_: yup I noticed it yesterday
[10:43]  * dpm hugs rpadovani :)
[10:46] <nik90> zsombi: hmm..it is giving me a segmentation fault (core dumped) now
[10:47] <nik90> zsombi: same code no changes that I used previously to test laast week
[10:47] <zsombi> nik90: that's weird... do you have a branch I can test with?
[10:47] <nik90> zsombi: yeah ... one second, let me grab the link
[10:48] <nik90> zsombi: https://code.launchpad.net/~nik90/cliffhanger/convergence-1
[10:48] <zsombi> nik90: ok, I'll test with it! thx!
[10:48] <nik90> zsombi: np
[10:49] <zsombi> nik90: perhaps there is some corner case in your branch for which I had not prepared teh Layouts with ;)
[10:49] <nik90> zsombi: hehe :)
[10:50] <nik90> zbenjamin: ping
[10:50] <zbenjamin> nik90: yo
[10:50] <nik90> zbenjamin: hey..I am having some issues with the new Devices Tab...this is the screen I see https://imgur.com/vjadDvY
[10:51] <nik90> zbenjamin: why are there 7 ubuntu devices?
[10:51] <zbenjamin> nik90: check in tools->options->devices
[10:51] <zbenjamin> nik90: are there 7 too?
[10:51] <zbenjamin> nik90: note that emulators are also listed as devices
[10:51] <nik90> zbenjamin: yup there are 7 too
[10:52] <zbenjamin> nik90: do they all have the same serial ID?
[10:52] <nik90> zbenjamin: I created 3 emulators
[10:52] <zbenjamin> ok , how many devices do you have?
[10:52] <zbenjamin> i mean real devices
[10:52] <nik90> zbenjamin: 1 physical device
[10:53] <zbenjamin> did you sometimes run more than 1 emu at the same time?
[10:53] <zbenjamin> check the serial ID's please
[10:53] <nik90> zbenjamin: oh I have genyMotion (emulator for Android tablets) installed, would that affect it in anyway?
[10:53] <nik90> zbenjamin: where is the serial ID?
[10:53] <zbenjamin> you can see it on the screenshot, its the first in the list
[10:54] <zbenjamin> directly under Device Status
[10:54] <nik90> zbenjamin: ah. ok
[10:54] <nik90> zbenjamin: well except for Ubuntu Device5 that you see on the screenshot all the others are not connected..hence I cannot see the device serial ID.
[10:55] <nik90> https://imgur.com/AgHfFx4
[10:55] <zbenjamin> nik90: ok true sorry . cat ~/.config/QtProject/qtcreator/devices.xml  | grep InternalId
[10:56] <nik90> zbenjamin: http://paste.ubuntu.com/7451829/
[10:57] <zbenjamin> ouch
[10:57] <zbenjamin> if you open tools->options->devices you can go over the devices and see the IDs, remove the ones that make no sense
[10:59] <nik90> zbenjamin: done..Now how do I create a kit for it? Otherwise I wont be able to Ctrl+F12 to run my app on the device
[10:59] <zbenjamin> nik90: press the autocreate kit button on the new device page :)
[10:59] <zbenjamin> nik90: its even in your screenshot ;)
[11:00] <nik90> zbenjamin: :) I clicked it but it doesn't seem to do anything (visually atleast)
[11:00] <zbenjamin> nik90: ok thats bad then :/
[11:01] <zbenjamin> nik90: do you have a existing ubuntu kit?
[11:02] <zbenjamin> nik90: if yes then go to tools->options->build & run -> kits, clone the kit there, rename it and assign the device you want to it
[11:03] <nik90> zbenjamin: https://imgur.com/aOoLRw0
[11:03] <nik90> zbenjamin: what I did was go to devices and set the ubuntu device5 as default..then ctrl+f12 seems to run on my nexus phone.
[11:03] <nik90> is that okay
[11:03] <zbenjamin> nik90: aaa you have no builder chroot
[11:04] <nik90> zbenjamin: erm I suppose (has no knowledge of this)...this is the same setup i had since 13.10..
[11:07] <zbenjamin> nik90: ok for the kit support you need a click chroot, go to tools -> options -> ubuntu -> click -> create click target
[11:07] <zbenjamin> nik90: this will take a while and after it you have to restart QtC
[11:08] <nik90> zbenjamin: which arch do I choose?
[11:08] <zbenjamin> nik90: arm
[11:08] <zbenjamin> nik90: that will allow you to build also c++ projects btw
[11:09] <zbenjamin> nik90: otoh you don't need that for qml only projects. For qml only projects the device selected on the devices page is used to start
[11:09] <nik90> zbenjamin: ok. So once this is complete I should be able to specify the kit and then pressing run should run it on the specific device directly?
[11:09] <nik90> ok
[11:09] <zbenjamin> nik90: for cmake projects you can use run/debug
[11:09] <zbenjamin> nik90: for the qml projects we are not at that point yet
[11:10] <nik90> zbenjamin: well the clock app is a qml project but with a CMAKELIST.txt file...which should make it a cmake project, no?
[11:10] <zbenjamin> exactly, then you need the kit
[11:10] <nik90> wonderful
[11:14] <dpm> rpadovani, mzanetti, when you've got a minute, could you have a look at https://code.launchpad.net/~dpm/reminders-app/clarify-reminders-take2/+merge/219170 and https://code.launchpad.net/~dpm/reminders-app/fix-1317977-openssl-exception-take2/+merge/219167 ? I'll then have a look at Riccardo's branches and once these 4 are approved we should be good to go to upload the new design to the store
[11:16] <mzanetti> dpm: \o/
[11:16] <dpm> really looking forward to that :)
[11:16] <mzanetti> dpm: shouldn't this
[11:16] <mzanetti> install(FILES COPYRIGHT DESTINATION ${CMAKE_INSTALL_PREFIX})
[11:16] <mzanetti> be
[11:16] <mzanetti> install(FILES COPYING DESTINATION ${CMAKE_INSTALL_PREFIX})
[11:16] <mzanetti> ?
[11:16] <dpm> ah, shit, yes
[11:16] <dpm> typo
[11:17] <dpm> fixing
[11:20] <dpm> mzanetti, fixed and pushed
[11:20] <dpm> thanks!
[11:21] <mzanetti> dpm: approved both, the license one not yet top-approved tho
[11:26] <dpm> great, thanks mzanetti
[11:33] <nik90> zbenjamin: the click target completed. I am now able to create a kit in the devices tab.
[11:33] <nik90> zbenjamin: How do I choose to have the phone as the default kit instead of the desktop?
[11:34] <zbenjamin> nik90: you have to add the kit to your project
[11:34] <zbenjamin> nik90: projects -> add kit
[11:35] <zbenjamin> nik90: then you can select the kit on the left side right over the run button
[11:35] <nik90> zbenjamin: https://imgur.com/LEzFYjE
[11:35] <zbenjamin> nik90: apt-get install gdb-multiarch
[11:38] <nik90> zbenjamin: hmm..I installed it and then restarted Qtc. But now I get "Device type not Desktop" when I try to add it as the kit
[11:38] <zbenjamin> nik90: i thought you openend the project as a cmake project?
[11:39] <zbenjamin> nik90: you opened the cmakelists.txt in qtc?
[11:39] <nik90> zbenjamin: ah ok..this is the first I did that since the transition :P
[11:39] <nik90> zbenjamin: I used to open the .qmlproject file
[11:39] <zbenjamin> :)
[11:40] <nik90> zbenjamin: although when I open the cmake file, the file tree doesn't show the source code https://imgur.com/6RB6oYY
[11:40] <nik90> zbenjamin: sry for all the trouble...
[11:40] <zbenjamin> nik90: seems like your qml files are not added to a target
[11:41] <nik90> zbenjamin: that means the error is in the Cmakelist file?
[11:41] <zbenjamin> nik90: yeah, it will still build correctly, but QtC can not know that there are files if they are not part of a target :/
[11:43] <zbenjamin> nik90: thats what the project templates do: add_custom_target(untitled234_QMlFiles ALL SOURCES ${QML_JS_FILES})
[11:43] <nik90> zbenjamin: I will try to get the help from dpm and balloons to help me add the qml files to the target
[11:44] <zbenjamin> nik90: http://pastebin.com/phYZn0yz
[11:44] <zbenjamin> nik90: basically you will need that in every CMakeLists.txt that should include qml files
[11:45] <dpm> nik90, what's that for, are we transitioning clock from a qmlproject to a cmake project?
[11:45] <nik90> dpm: didnt we already do that?
[11:46] <nik90> dpm: to help with the qa testing?
[11:46] <nik90> zbenjamin: the clock app has some subfolders where the qml js files live..so I need to look closer at the cmake file
[11:46] <zbenjamin> nik90: k
[11:47] <dpm> nik90, did we? I thought only the cmakelists.txt files were added for CI, but the project could still be edited as a .qmlproject
[11:47] <dpm> in any case, happy to help, but I need a bit more of context
[11:49] <nik90> dpm: ah yes..so instead of having both, can we transition clock app to a cmake project? The reasoning for this is to align with upstream qtc improvements which have landed recently. For a cmake project, qtc now offers a nice option to switch the run targets from desktop to phone seamlessly.
[11:50] <nik90> dpm: this way we also do not need to have the temporary .desktop file we added as a temporary hack
[11:50] <dpm> nik90, yeah, sure
[11:51] <dpm> I'm happy to help on that one, but not sure I can get to it this week. But do not block on me: you can have a look at reminders, filemanager or terminal for the cmake setup
[11:51] <nik90> dpm: will do
[11:51] <dpm> cool
[12:57] <ahayzen_> ogra_, ping
[12:57] <ogra_> ahayzen_, hey
[12:58] <ahayzen_> ogra_, I'm trying to add a dependency for the music-app, so that it will work with mediascanner2. Would you mind checking over this mp? Also should this be in touch file or one of the others such as sdk-libs? https://code.launchpad.net/~andrew-hayzen/ubuntu-seeds/ubuntu-touch-utopic-add-mediascanner2/+merge/219122
[12:58] <ogra_> ahayzen_, hmm, you want to use mediascanner2 so you add a seed entry for mediascanner1 ?
[12:59] <ogra_> (are you sure that packagename is correct ?)
[12:59] <ahayzen_> ogra_, that is the QML plugin
[13:00] <ahayzen_> ogra_, i believe that is correct as before you went via Grilo
[13:00] <ogra_> well, grilo is still seeded ...
[13:01] <ogra_> the qtdeclarative stuff should go into sdk-libs instead of touch btw
[13:01] <ogra_> (touch pulls sdk-libs in anyway)
[13:01] <ahayzen_> ogra_, yeah we plan to have a few transition images with both installed so that we can test, then remove grilo totally
[13:02] <ahayzen_> ogra_, ok i'll move it to sdk-libs, and looking at the packages that are created from mediascanner2 it states 0.1 https://launchpad.net/ubuntu/+source/mediascanner2
[13:03] <ogra_> right, so please move it to sdk-libs ... and dont forget to remove mediascanner1 asap ... so we dont bloat the images
[13:03] <ogra_> ah, snap :)
[13:04] <ahayzen_> ogra_, i assume mediascanner2.0 and libmediascanner-2.0-0 are installed before of a dependency from the scopes?
[13:05] <ogra_> yeah, i see both on the image ... but we need to get rid of one ... the image is way past the allowed 500MB due to many of such duplications
[13:06] <ogra_> mediascanner is only one of them ... there are more
[13:06] <ogra_> we need to drop all duplicated stuff before june/july ... to not ship that on the final images for the phones in summer
[13:06] <ahayzen_> ogra_, moving off grilo is our top priority... we're almost there just a few bugs left to resolve and rewriting the autopilot tests (which should be fun)... so we should be talking in terms of days not weeks
[13:07] <ogra_> cool
[13:07] <popey> renato: I added a comment to https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/CalManagement/+merge/213355 - tsdgeos ran under valgrind and found it still crashing in qtorganized-eds - can you take a look please?
[13:08] <ahayzen_> ogra_, i've moved to to sdk-libs https://code.launchpad.net/~andrew-hayzen/ubuntu-seeds/ubuntu-touch-utopic-add-mediascanner2/+merge/219122
[13:09] <ogra_> thanks
[13:27] <dpm> fginther, balloons, elopio, it seems all reminders MPs are failing in Jenkins, seems to be related to online accounts. Do you know what's going on there?
[13:36] <popey> dpm: have you tested on a recent device?
[13:36] <popey> recent build I mean.
[13:37] <popey> online accounts changed recently, now uses oxide I believe
[13:37] <fginther> dpm, elopio mentioned a recent bug in the signon-plugin-oauth2 plugin. He has a fix in his email that I need to try today
[13:39] <Dzerillious_> Hello
[13:42] <renato> popey, I will
[13:47] <dpm> popey, that's on Jenkins. I'm still using a promoted image, so I haven't actually experienced on the device
[14:01] <dpm> fginther, ok, thanks. Would you mind pinging me or popey once you've had the chance to try that? I'd like to push an update to the store with the new Reminders design today
[14:11] <fginther> dpm, I've update the package on the utopic test runner
[14:11] <fginther> dpm, I re-ran the most recent reminders-app ci run and it passed
[14:11] <dpm> fginther, excellent, thanks a lot for the superquick response
[14:40] <elopio> thanks for the updates fginther. The tests for my branch have finally passed.
[14:40] <elopio> dpm: ^
[14:45] <nerochiaro_> loicm: artmello: bfiller: sent you an email with the info about the latest patches for camera and qtmultimedia
[14:45] <nerochiaro_> loicm: artmello: bfiller: please let me know how well it works on your system
[14:45] <bfiller> nerochiaro_: great, thanks
[14:46] <dpm> elopio, awesome, nice work!
[14:55] <elopio> balloons, dpm: we need this python package to make pretty reminders tests:
[14:55] <elopio> https://github.com/evernote/evernote-sdk-python
[14:55] <elopio> can you get somebody to package it and include it into ubuntu?
[14:56] <dpm> elopio, is it really a requirement, though? Packaging and getting it into Ubuntu might require quite a bit of effort and coordination
[14:57] <elopio> dpm: I can work around not having it for a while. But yes, it's definitely a testability requirement for reminders.
[15:00] <dpm> elopio, ack, thanks. Let's try to get this release with the new design out of the door and then look at this when we've got more tests
[15:13] <balloons> elopio, ahh yes the wrapper. Do we really require it being packaged though? I don't think so
[15:14] <balloons> we can bundle it same way as the other bindings
[15:18] <dpm> fginther, would it be possible to retrigger Jenkins on this one too? I think it's failing on the same libaccounts issue: https://code.launchpad.net/~dpm/reminders-app/clarify-reminders-take2/+merge/219170
[15:19] <fginther> dpm, I re-approved it, that will retrigger the job
[15:20] <dpm> great, thanks!
[15:20] <dpm> mzanetti, rpadovani, would you mind top-approving https://code.launchpad.net/~dpm/reminders-app/fix-1317977-openssl-exception-take2/+merge/219167 ? I got a licensing expert to review it
[15:22] <mzanetti> dpm: done
[15:24] <dpm> cool
[15:26] <elopio> balloons: well, I would hate to include it on reminders if we can get it packaged and useful for everybody in a decent amount of time.
[15:30] <balloons> dpm, so Leo's code is merged, I'll merge Carla's 2 tests as well. Have you gone further with adding tihs to the image?
[15:31] <dpm> balloons, we're still waiting for Evernote's feedback, but that shouldn't be a blocker for the tests
[15:31] <dpm> nice work with getting the tests ready balloons and elopio :)
[15:31] <balloons> that was all Leo, major kudos. He's getting a beverage of choice on me in Malta ;-)
[15:32]  * elopio chooses beer :)
[15:32] <balloons> dpm, I ask because https://bugs.launchpad.net/ubuntu-ci-services-itself/+bug/1307542. And Paul is ready to put it in :-)
[15:33] <dpm> balloons, I'll reply to that directly, thanks for the heads up
[16:12] <dpm> popey, once Jenkins lands https://code.launchpad.net/~rpadovani/reminders-app/1318078/+merge/219091 we should be good to go for a store upload soon. Do you think you'd have time for that today?
[16:12] <rpadovani> \o/
[16:13] <popey> dpm: sure, technically balloons will do the upload, I just accept it ☻
[16:13] <popey> but I'll chase him.
[16:13] <dpm> :)
[16:14] <balloons> dpm, popey I wanted to land Carla's tests too
[16:14] <balloons> I'll need a little time to do it since she's not around
[16:14] <dpm> balloons, that's fine, but do they need to be in the store too? I'd like to get the new design out of the door first
[16:17] <popey> mhall119: wanna try and reproduce bug 1318735 ☻
[16:17] <popey> we lost our bot again
[16:17] <balloons> dpm, not persay no. But atm you'll already be getting leo's test hehe
[16:17] <dpm> nice :)
[16:17] <popey> mhall119: https://bugs.launchpad.net/software-center-agent/+bug/1318735
[16:21] <dpm> fginther, on https://code.launchpad.net/~rpadovani/reminders-app/1318078/+merge/219091 I'm assuming the top-approval will trigger Jenkins to do the autolanding without need for any manual retrigger to successfully run?
[16:21] <fginther> dpm, yes
[16:22] <dpm> perfect, thanks
[16:23] <jussi> !test
[16:23] <popey> thank you jussi
[16:23] <jussi> :)
[16:25] <mzanetti> rpadovani: hey
[16:26] <rpadovani> mzanetti, hoy :-)
[16:26] <mzanetti> rpadovani: testing the fontsize branch. try exporting GRID_UNIT_PX=20 before running it
[16:27] <mzanetti> rpadovani: do we really need the defaultFontSize setting?
[16:28] <mzanetti> imo it works better without
[16:29] <rpadovani> mzanetti, without it the font is little
[16:29] <rpadovani> I don't understand about exporting GRID_UNIT_PX=20
[16:29] <mzanetti> rpadovani: that scales the UI for high dpi screens
[16:30] <rpadovani> mzanetti, ah, ok, and where have I to set it?
[16:30] <mzanetti> just before running something
[16:30] <elopio> balloons: Carla's tests need a couple of fixes.
[16:30] <mzanetti> rpadovani: GRID_UNIT_PX=20 ./reminders
[16:31] <mzanetti> rpadovani: you can also use "export GRID_UNIT_PX=20" to make it persistent for this terminal session
[16:31] <mzanetti> rpadovani: works with every ubuntu touch app
[16:31] <elopio> balloons: that's where we need the evernote sdk python package, to create a note on the server that will be downlowded by the client.
[16:31] <balloons> elopio, indeed. I think I'm going to let her take care of it actually
[16:32] <elopio> balloons: I'm writing a couple of tests to add notebook and note. For that we don't need the sdk.
[16:32] <balloons> elopio, ohh, that kind of fix you mean?
[16:32] <rpadovani> mzanetti, thanks for the explanation. I have a 1920x1080 and it's too big with grid unit px 20
[16:32] <mzanetti> rpadovani: sure... just saying, my desktop uses 18 for example.
[16:32] <mzanetti> rpadovani: and with your branch it looks wrong.
[16:33] <balloons> she's just downloading notebooks and notes.. but by default I believe you have one so it works
[16:33] <mzanetti> rpadovani: the font in the noteview is much bigger than the rest of the ui
[16:34] <mzanetti> rpadovani: what i mean is: you should be able to change the GRID_UNIT_PX and everything looks bigger/smaller, but it should keep the same proportions. everything needs to grow/shrink the same
[16:34] <rpadovani> mzanetti, sorry, I'm not understanding: isn't units.gu() job to adapt to monitor? So, if is too big, we can not simply change the units.gu of font size?
[16:35] <mzanetti> rpadovani: function units.gu(x) { return GRID_UNIT_PX * x; }
[16:35] <mzanetti> that's what it does (more or less)
[16:36] <mzanetti> rpadovani: the Nexus for just has GRID_UNIT_PX set to 18 (or 16, don't remember)
[16:36] <rpadovani> mzanetti, if I leave the font setting the font is bigger as in the preview, but IMO is too small: one thing is to have a small text for preview, one thing is to have small text to read notes
[16:37] <mzanetti> rpadovani: on which device do you think it is too small?
[16:37] <rpadovani> mzanetti, on px
[16:37] <mzanetti> nexus4?
[16:37] <rpadovani> *pc
[16:37] <mzanetti> rpadovani: then test it on the nexus4. it'll be too big there
[16:38] <rpadovani> building
[16:38] <mzanetti> rpadovani: is the text in the toolbar icons too small too for you?
[16:39] <rpadovani> mzanetti, a bit, yes: to be clear, I read it without problems, but seems too small
[16:39] <mzanetti> rpadovani: then you might want to put something like this in your ~/.bashrc
[16:39] <mzanetti> export GRIDUNIT_PX=10
[16:40] <mzanetti> export GRID_UNIT_PX=10
[16:40] <rpadovani> mzanetti, mhh ok
[16:41] <rpadovani> mzanetti, ok, indeed on mako is too big. I'll remove the font size preference, so the branch is only to revert #71
[16:41] <mzanetti> rpadovani: no, keep the font to Ubuntu
[16:42] <rpadovani> mzanetti, the font size preference :-)
[16:42] <mzanetti> rpadovani: oh...
[16:42] <mzanetti> I thought you would have set the default font type to ubuntu too :D
[16:42] <mzanetti> apparently you haven't :D
[16:43] <mzanetti> ok... let me just verify what that bug was it was supposed to fix
[16:43] <rpadovani> mzanetti, it's in another branch the font family :-)
[16:44] <mzanetti> :D
[16:44] <rpadovani> pushed
[16:46] <om26er> renato, meet robotfuel :)
[16:47] <renato> robotfuel, hi
[16:47] <robotfuel> renato: hi
[16:47] <renato> robotfuel, I am having problems with autopilot and emulator classes
[16:47] <robotfuel> renato: I can help you with that :D
[16:47] <robotfuel> renato: which project?
[16:47] <renato> robotfuel, this is my branch ~renatofilho/address-book-app/new-header
[16:48] <renato> robotfuel, and this is my error: http://paste.ubuntu.com/7453166/
[16:50] <robotfuel> renato: have you used print_tree before in autopilot?
[16:50] <renato> robotfuel, no
[16:51] <renato> robotfuel, this was working before
[16:51] <renato> robotfuel, I am not sure what I changed that cause the problem
[16:52] <renato> robotfuel, the object is returned but is not a instance of the emulator class
[16:52] <robotfuel> renato: did the object name change?
[16:52] <renato> robotfuel, no
[16:52] <mzanetti> rpadovani: I think we need parts of the old fix
[16:53] <mzanetti> rpadovani: and some enhancements too
[16:53] <robotfuel> renato: hmm I will have to take a look, I use print_tree on the object and sometimes the parent of the one that is failing so I can see what autopilot sees when it fails.
[16:53] <rpadovani> mzanetti, ok, so I'll leave the bug to you, I red the code but I don't understand (almost) anything
[16:54] <mzanetti> rpadovani: oh really... well, let me know if you want me to explain it
[16:54] <robotfuel> renato: http://unity.ubuntu.com/autopilot/api/introspection.html?highlight=print_tree
[16:54] <rpadovani> mzanetti, I'll read your edit then I'll ask you, thanks :-)
[16:55] <renato> robotfuel, I changed the base class of the ContactList QML item
[16:59] <elopio> dpm, balloons: we have no way to delete notebooks on the app
[16:59] <elopio> https://bugs.launchpad.net/reminders-app/+bug/1318749
[16:59] <elopio> the alternative would be to delete them with the api, but we don't have that package available yet. So the tests will fill up the account.
[17:00] <dpm> elopio, I think the API does not let us. mzanetti, do you remember whether it's possible to delete notebooks with the Evernote API?
[17:00] <mzanetti> dpm: no, its not
[17:00] <dpm> I remember we had some constraints on the things we could destroy :)
[17:00] <mzanetti> dpm: well, there is an api call, but aren't allowed to use it
[17:00] <dpm> elopio, so there you go ^
[17:00] <dpm> thanks mzanetti
[17:01] <mzanetti> dpm: you said you would ask the evernote guys if we could have a api key that is allowed to do this
[17:01] <mzanetti> but it really wasn't a priority back then
[17:01] <mzanetti> maybe now with the transition to the production servers its time to clarify that
[17:02] <renato> robotfuel, I did not understand the problem :D
[17:02] <dpm> mzanetti, no, I don't see it as much of a priority still, given the other things to do. I agree, I'll bring it up as part of the transition to production servers discussion
[17:02] <nerochiaro_> artmello: loicm: bfiller: did any of you get any chances to test these patches ?
[17:03] <artmello> nerochiaro_: will test it right now
[17:03] <robotfuel> renato: I am looking at the changes, I'll have a suggestion shortly
[17:03] <renato> robotfuel, thanks
[17:04] <nerochiaro_> artmello: thanks
[17:07] <robotfuel> renato: the main window emulator isn't what you think it is, redefinition of unu    sed 'ContactListPage' from line 12
[17:08] <elopio> mzanetti or dpm: when I add a notebook it says: Last edited last week.
[17:08] <elopio> shouldn't it say: Last edited today?
[17:08] <elopio> or never edited
[17:08] <mzanetti> elopio: yeah, it probably should
[17:08] <dpm> elopio, known bug, I mentioned it on a MP but never filed a bug for it, sorry
[17:09] <robotfuel> renato: address_book_app.emulators.page_with_bottom_edge doesn't have a 'get_contacts' method.
[17:10] <renato> robotfuel, hummm, ok
[17:10] <elopio> I'll report the bug.
[17:10] <renato> robotfuel, my fault, the page_widh_bottom_edge should derive from the ContactListPage emular
[17:10] <renato> emultaor
[17:11] <dpm> elopio, https://bugs.launchpad.net/reminders-app/+bug/1318751
[17:11] <robotfuel> renato: do you have it from here?
[17:11] <elopio> dpm: great, thanks.
[17:11] <renato> robotfuel, yes, thanks, and sorry for this :D
[17:12] <robotfuel> renato: np if you get stuck feel free to ask again :D
[17:21] <dpm> rpadovani, mzanetti, check out your nearest Ubuntu Software Store for a nice update of Reminders with a stunning new design ;)
[17:21] <mzanetti> dpm: *\o/*
[17:21] <dpm> indeed :) nice work guys, here's to you!
[17:23] <rpadovani> dpm \o/ :D
[17:23] <rpadovani> but the screenshots are old!
[17:26] <dpm> rpadovani, I know, I'll update them later on today :)
[17:26] <rpadovani> :D
[17:26] <rpadovani> Awesome!
[17:35] <rpadovani> dpm, mzanetti mhh, could you try a thing? On desktop, If I have more than 5 notes, some notes aren't on the screen. When I try to pull up the list to see others notes, listview flicks and returns. Are you able to reproduce this? to top
[17:35] <rpadovani> *and returns top
[17:37] <mzanetti> rpadovani: tried removing the sort() calls in notes.cpp?
[17:38] <mzanetti> rpadovani: its fixed in utopic btw
[17:39] <rpadovani> mzanetti, ah, thanks. Seems I need to upgrade :D
[18:58] <rickspencer3> hey, is there a standard way in an onWhateverPropertyChanged handler to know the value of the property before it was changed?
[19:10] <rickspencer3> beuno, I uploaded a new version of one of my apps, and got a very poorly formatted error message :(
[19:10] <rickspencer3> the body of hte page seems to be json
[19:12] <beuno> rickspencer3, oops
[19:12] <beuno> rickspencer3, pastebin?
[19:12] <rickspencer3> beuno, it worked the second time
[19:12] <rickspencer3> beuno, oops
[19:12] <rickspencer3> my bad, I retried and it worked
[19:12] <rickspencer3> looked like a run of the mill error 500
[19:12] <beuno> rickspencer3, that's ok, we have logs and error reports
[19:12] <beuno> I'll track it down
[19:13] <rickspencer3> beuno, yeah, really I was thinking you might want to make sure that the error pages get rendered properly
[19:13] <beuno> rickspencer3, yes we will
[20:09] <rickspencer3> sweet, an update to "Random Cats"
[20:57] <nerochiaro_> artmello: any news in testing these branches ?
[20:57] <artmello> nerochiaro_: sorry, still doing some changes on the gallery MR
[20:57] <artmello> nerochiaro_: but I will test it today
[20:58] <nerochiaro_> artmello: ok, email me the results, i'm going offline for today
[20:58] <artmello> nerochiaro_: sure
[20:58] <nerochiaro_> artmello: thanks