| === chriadam|away is now known as chriadam | ||
| === hatchetation_ is now known as hatchetation | ||
| === nik90_ is now known as nik90 | ||
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| DanChapman_ | Good Morning | 06:34 |
|---|---|---|
| dholbach | good morning | 06:41 |
| justCarakas | good morning all | 06:56 |
| === mpt_ is now known as mpt | ||
| JamesTait | Good morning all; happy Monday and happy Nurses Day! :-D | 08:09 |
| === chriadam is now known as chriadam|away | ||
| dpm | morning all | 08:41 |
| 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:42 |
| 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:43 |
| dpm | this is from suru | 08:44 |
| 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:45 |
| 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:47 |
| 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:49 |
| dpm | ok, thanks! | 08:50 |
| === Elleo_ is now known as Elleo | ||
| 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:24 |
| t1mp_ | ok zsombi answered already :) | 09:27 |
| dpm | thanks for the info t1mp_ | 09:30 |
| t1mp_ | dpm: there was a blueprint, but it seems to have disappeared, at least I cannot find it :s | 09:31 |
| dpm | np, for the moment that clarifies things, thanks! | 09:32 |
| kalikiana_ | popey: yep. approved now | 09:33 |
| popey | thanks kalikiana_ | 09:33 |
| 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:38 |
| dpm | t1mp_, ah, cool, I had seem some movement in that direction, but I didn't know what was going on. Nice! | 09:39 |
| 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:46 |
| 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:47 |
| mzanetti | I wouldn't go that far | 09:48 |
| rpadovani | lol, the issues? The incorrect behavior? | 09:48 |
| mzanetti | :) | 09:48 |
| davidcalle | dpm, hi | 09:55 |
| dpm | hey davidcalle | 09:56 |
| davidcalle | dpm, is there any doc or code samples for the new headers? | 09:56 |
| 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:57 |
| 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:58 |
| 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 | 09:59 |
| nik90 | bzoltan: thnx | 10:00 |
| 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:01 |
| 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:02 |
| popey | nik90: on the phone, will look in a bit | 10:04 |
| nik90 | popey: np | 10:04 |
| rpadovani | mzanetti, https://code.launchpad.net/~rpadovani/reminders-app/setDefaultFontSize/+merge/219092 | 10:05 |
| 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:06 |
| rpadovani | dpm, o/ When you have time, there is also https://code.launchpad.net/~rpadovani/reminders-app/1318078/+merge/219091 | 10:09 |
| dpm | hi rpadovani, I saw it, will test it now, thanks! | 10:23 |
| 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:24 |
| rpadovani | yes yes, it was just to notify you the branch, in case you miss it :-) | 10:27 |
| nik90 | zsombi: is this bug related to the textfield? http://davmor2.co.uk/~davmor2/screenshots-phone/device-2014-05-12-111506.png | 10:32 |
| 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:33 |
| 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:34 |
| 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:36 |
| zsombi | t1mp_: ok, I'll check it | 10:37 |
| 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:38 |
| 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:39 |
| 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:42 |
| * dpm hugs rpadovani :) | 10:43 | |
| nik90 | zsombi: hmm..it is giving me a segmentation fault (core dumped) now | 10:46 |
| 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:47 |
| 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:48 |
| 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:49 |
| 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:50 |
| 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:51 |
| 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:52 |
| 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:53 |
| 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:54 |
| nik90 | https://imgur.com/AgHfFx4 | 10:55 |
| zbenjamin | nik90: ok true sorry . cat ~/.config/QtProject/qtcreator/devices.xml | grep InternalId | 10:55 |
| nik90 | zbenjamin: http://paste.ubuntu.com/7451829/ | 10:56 |
| 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:57 |
| === MacSlow is now known as MacSlow|lunch | ||
| 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 ;) | 10:59 |
| nik90 | zbenjamin: :) I clicked it but it doesn't seem to do anything (visually atleast) | 11:00 |
| zbenjamin | nik90: ok thats bad then :/ | 11:00 |
| zbenjamin | nik90: do you have a existing ubuntu kit? | 11:01 |
| 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:02 |
| 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:03 |
| nik90 | zbenjamin: erm I suppose (has no knowledge of this)...this is the same setup i had since 13.10.. | 11:04 |
| 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:07 |
| 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:08 |
| 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:09 |
| 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:10 |
| === gatox is now known as gatox_brb | ||
| 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:14 |
| 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:16 |
| dpm | fixing | 11:17 |
| dpm | mzanetti, fixed and pushed | 11:20 |
| dpm | thanks! | 11:20 |
| mzanetti | dpm: approved both, the license one not yet top-approved tho | 11:21 |
| dpm | great, thanks mzanetti | 11:26 |
| 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:33 |
| zbenjamin | nik90: you have to add the kit to your project | 11:34 |
| zbenjamin | nik90: projects -> add kit | 11:34 |
| 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:35 |
| 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:38 |
| 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:39 |
| 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:40 |
| 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:41 |
| === gatox_brb is now known as gatox | ||
| 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:43 |
| 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:44 |
| 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:45 |
| 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:46 |
| 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:47 |
| 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:49 |
| 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:50 |
| 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 | 11:51 |
| === MacSlow|lunch is now known as MacSlow | ||
| === _salem is now known as salem_ | ||
| ahayzen_ | ogra_, ping | 12:57 |
| ogra_ | ahayzen_, hey | 12:57 |
| 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:58 |
| ogra_ | (are you sure that packagename is correct ?) | 12:59 |
| ahayzen_ | ogra_, that is the QML plugin | 12:59 |
| ahayzen_ | ogra_, i believe that is correct as before you went via Grilo | 13:00 |
| ogra_ | well, grilo is still seeded ... | 13:00 |
| 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:01 |
| 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:02 |
| 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:03 |
| === jhodapp_ is now known as jhodapp | ||
| ahayzen_ | ogra_, i assume mediascanner2.0 and libmediascanner-2.0-0 are installed before of a dependency from the scopes? | 13:04 |
| 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:05 |
| 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:06 |
| 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:07 |
| 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:08 |
| ogra_ | thanks | 13:09 |
| === chihchun is now known as chihchun_afk | ||
| 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:27 |
| popey | dpm: have you tested on a recent device? | 13:36 |
| popey | recent build I mean. | 13:36 |
| 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:37 |
| Dzerillious_ | Hello | 13:39 |
| renato | popey, I will | 13:42 |
| dpm | popey, that's on Jenkins. I'm still using a promoted image, so I haven't actually experienced on the device | 13:47 |
| 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:01 |
| 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:11 |
| === chihchun_afk is now known as chihchun | ||
| elopio | thanks for the updates fginther. The tests for my branch have finally passed. | 14:40 |
| elopio | dpm: ^ | 14:40 |
| 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:45 |
| dpm | elopio, awesome, nice work! | 14:46 |
| === chihchun is now known as chihchun_afk | ||
| 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:55 |
| dpm | elopio, is it really a requirement, though? Packaging and getting it into Ubuntu might require quite a bit of effort and coordination | 14:56 |
| elopio | dpm: I can work around not having it for a while. But yes, it's definitely a testability requirement for reminders. | 14:57 |
| 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:00 |
| balloons | elopio, ahh yes the wrapper. Do we really require it being packaged though? I don't think so | 15:13 |
| balloons | we can bundle it same way as the other bindings | 15:14 |
| 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:18 |
| fginther | dpm, I re-approved it, that will retrigger the job | 15:19 |
| 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:20 |
| mzanetti | dpm: done | 15:22 |
| dpm | cool | 15:24 |
| 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:26 |
| 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:30 |
| 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:31 |
| * 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:32 |
| dpm | balloons, I'll reply to that directly, thanks for the heads up | 15:33 |
| === chihchun_afk is now known as chihchun | ||
| === chihchun is now known as chihchun_afk | ||
| === gatox is now known as gatox_lunch | ||
| 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:12 |
| 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:13 |
| 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:14 |
| 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:17 |
| 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:21 |
| dpm | perfect, thanks | 16:22 |
| jussi | !test | 16:23 |
| ubot5 | Testing... Testing... 1. 2.. 3... ( by the way, remember that you can use /join #test ) | 16:23 |
| popey | thank you jussi | 16:23 |
| jussi | :) | 16:23 |
| mzanetti | rpadovani: hey | 16:25 |
| rpadovani | mzanetti, hoy :-) | 16:26 |
| mzanetti | rpadovani: testing the fontsize branch. try exporting GRID_UNIT_PX=20 before running it | 16:26 |
| mzanetti | rpadovani: do we really need the defaultFontSize setting? | 16:27 |
| mzanetti | imo it works better without | 16:28 |
| 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:29 |
| 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:30 |
| 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:31 |
| 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:32 |
| 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:33 |
| 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:34 |
| mzanetti | rpadovani: function units.gu(x) { return GRID_UNIT_PX * x; } | 16:35 |
| mzanetti | that's what it does (more or less) | 16:35 |
| === fginther is now known as fginther|lunch | ||
| === fginther|lunch is now known as fginther | ||
| === gatox_lunch is now known as gatox | ||
| 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:36 |
| 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:37 |
| rpadovani | building | 16:38 |
| mzanetti | rpadovani: is the text in the toolbar icons too small too for you? | 16:38 |
| 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:39 |
| mzanetti | export GRID_UNIT_PX=10 | 16:40 |
| rpadovani | mzanetti, mhh ok | 16:40 |
| 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:41 |
| 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:42 |
| 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:43 |
| mzanetti | :D | 16:44 |
| rpadovani | pushed | 16:44 |
| om26er | renato, meet robotfuel :) | 16:46 |
| 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:47 |
| renato | robotfuel, and this is my error: http://paste.ubuntu.com/7453166/ | 16:48 |
| robotfuel | renato: have you used print_tree before in autopilot? | 16:50 |
| renato | robotfuel, no | 16:50 |
| renato | robotfuel, this was working before | 16:51 |
| renato | robotfuel, I am not sure what I changed that cause the problem | 16:51 |
| 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:52 |
| 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:53 |
| 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:54 |
| renato | robotfuel, I changed the base class of the ContactList QML item | 16:55 |
| 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 |
| ubot5 | Ubuntu bug 1318749 in Ubuntu Reminders app "It's not possible to delete a notebook" [Undecided,New] | 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. | 16:59 |
| 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:00 |
| 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:01 |
| 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:02 |
| 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:03 |
| nerochiaro_ | artmello: thanks | 17:04 |
| robotfuel | renato: the main window emulator isn't what you think it is, redefinition of unu sed 'ContactListPage' from line 12 | 17:07 |
| 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:08 |
| robotfuel | renato: address_book_app.emulators.page_with_bottom_edge doesn't have a 'get_contacts' method. | 17:09 |
| 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:10 |
| dpm | elopio, https://bugs.launchpad.net/reminders-app/+bug/1318751 | 17:11 |
| ubot5 | Ubuntu bug 1318751 in Ubuntu Reminders app "New notebooks edit date set to "last week"" [Low,Triaged] | 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:11 |
| robotfuel | renato: np if you get stuck feel free to ask again :D | 17:12 |
| 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:21 |
| rpadovani | dpm \o/ :D | 17:23 |
| rpadovani | but the screenshots are old! | 17:23 |
| dpm | rpadovani, I know, I'll update them later on today :) | 17:26 |
| rpadovani | :D | 17:26 |
| rpadovani | Awesome! | 17:26 |
| 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:35 |
| mzanetti | rpadovani: tried removing the sort() calls in notes.cpp? | 17:37 |
| mzanetti | rpadovani: its fixed in utopic btw | 17:38 |
| rpadovani | mzanetti, ah, thanks. Seems I need to upgrade :D | 17:39 |
| === jhodapp is now known as jhodapp|brb | ||
| === jhodapp|brb is now known as jhodapp | ||
| rickspencer3 | hey, is there a standard way in an onWhateverPropertyChanged handler to know the value of the property before it was changed? | 18:58 |
| === salem_ is now known as _salem | ||
| 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:10 |
| 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:12 |
| 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 | 19:13 |
| rickspencer3 | sweet, an update to "Random Cats" | 20:09 |
| === t1mp_ is now known as t1mp | ||
| === _salem is now known as salem_ | ||
| 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:57 |
| nerochiaro_ | artmello: ok, email me the results, i'm going offline for today | 20:58 |
| artmello | nerochiaro_: sure | 20:58 |
| nerochiaro_ | artmello: thanks | 20:58 |
| === jhodapp is now known as jhodapp|afk | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
