=== chriadam|away is now known as chriadam [00:32] balloons, jhodapp, I have some questions about a MP that the music-app has in place to test the swap-buffers bug that was fixed for Qt5.2: https://code.launchpad.net/~vthompson/music-app/proceeds-to-next [00:34] jhodapp, I haven't seen the tracklist implementation of the media-hub in any detail yet, but would I be correct in assuming that there won't be anything in place to notify/wake the music app from suspension when the track switches? [00:40] balloons, do you know if AP can retrieve component properties from an app that is suspended? [01:17] vthompson, unknown atm, that part of background playlist isn't implemented yet [01:20] jhodapp, thanks. We'll hold off on landing our AP test branch then. I assume you guys will have your own AP/unit tests for the tracklist implementation? [01:20] vthompson, yes indeed [01:21] vthompson, any reason why you can't land that now? [01:23] jhodapp, because the tests go through a sleep that should put the app into suspension. Right now we are granted a lifecycle exception so we are always on... but the goal is to remove that exception when tracklist media-hub lands. [01:23] vthompson, k [01:23] jhodapp, when that does happen, music-app will suspend like the rest and I'm not sure the AP test can get the info we want it to when it checks the suspended app [01:24] vthompson, right, so you need to rely on media-hub then to get that persistence across suspension [01:27] jhodapp, yep, but simply having that data in MediaPlayer when we awake isn't enough (I assume) to allow this test to work in our suite. Since track management is media-hub's responsibility it sounds like it's not a worthwhile test in the music-app [01:30] vthompson, yeah it'll be well tested in media-hub since it's a primary reason for even having media-hub :) === _salem is now known as salem_ [03:05] pmcgowan: xhoch3: yes, the SDK PPA has exactly the same stuff as Utopic achive === salem_ is now known as _salem === chihchun_afk is now known as chihchun [06:00] evening all [06:00] morning for some; I'm sure [06:34] good morning [06:58] Good morning all [07:24] morning nik90, around? [07:24] and good morning everyone :) [07:25] good morning dpm [07:26] hi :) === chriadam is now known as chriadam|away [08:00] Good morning all; happy No Homework Day - my boys will be thrilled! :-D [08:01] justCarakas, didn't you ask where JamesTait was? ^ :) [08:01] dholbach: I did :D [08:01] good morning JamesTait! :) [08:02] Welcome back JamesTait I missed your morning greetings [08:02] dholbach, justCarakas: o/ [08:03] JamesTait: \ o [08:09] bah this is annoying having to ask this... In Python, I can treat strings as an array, doing something like mystring = "hello world"; [08:09] and then mystring[0:3] would return "hell" [08:09] how can I do that in qml, javascript, or c++ [08:10] im not sure what language applies for a qml property [08:10] I know you can do mystring[0], and get "h", but I don't know how to define a range [08:11] dpm: hey, wanna do a couple of reviews? [08:12] mzanetti, do you have some more nice features for me? :-) [08:12] dpm: https://code.launchpad.net/~reminders-app-dev/reminders-app/new-design/+activereviews [08:12] I'm excited already by the title of the MR :) [08:13] give me a few mins to finish what I was doing and I'll have a look [08:13] dpm: don't review the one from rpadovani yet. I need to ask him to rebase it [08:13] ack [08:13] dpm: but the 2 branches of mine should be fine I hope [08:13] ok [08:15] dpm, oh hey I got the source downloaded and such. My first goal is to submit one patch. Atm I am usure if my environment is clean though, as I am having issues loading images which is causing performance issues [08:16] Aki-Thinkpad, what issues exactly? Are you seeing lots of warnings on the console output? If so, I'm seeing those too :) [08:17] dpm, yah, http://i.imgur.com/blh8v33.jpg [08:17] Aki-Thinkpad, btw, thanks for getting started on this! I saw your request to join the team, but I prefer to add members once they've done at least a contribution, I hope that's ok for you? [08:17] dpm yah I I know the drill [08:17] :) [08:18] great, thanks :) [08:18] so is that output what you are seeing? Is this a biproduct of the run environment? [08:19] mzanetti, so Aki-Thinkpad is getting his system set up to help with reminders. He could build the app already and he's seeing some warnings/errors related to images in the QtC console output. I too could see them, but I haven't bothered too much about them. Any ideas if this is something we should worry about? http://i.imgur.com/blh8v33.jpg [08:20] dpm: hmm... not sure... works on my macbook. must be the thinkpad [08:21] hahaha [08:21] no... seriously... don't know what it is [08:21] if images are missing in the notes we should worry about it, yes [08:21] one set of errors is clearly that the app cannot write to the .cache dir. Perhaps it has not been created? I don't actually know what creates the .cache/com..... dir on the desktop [08:21] I'll keep an eye on the console [08:21] for me the images are downloaded and I can see them fine, but the console messages are still there [08:22] dpm, can't miss it. I was thinking about the cache. I'll go check if its been created. [08:22] strange... I think I create the cache dir myself if its not there yetp [08:22] mzanetti, * [08:22] hi Aki-Thinkpad [08:22] hey [08:22] yeah, it's quite possible that on the desktop the .cache dir is not created automatically [08:22] I'll check where they come from [08:23] yah I think that might be it. Manually creating the directory [08:25] ok, cool [08:25] I think I might have done it too, but I can't quite remember, it's been a while since I did the initial setup [08:25] mzanetti, here are the messages I'm getting: http://pastebin.ubuntu.com/7403285/ [08:26] Images are loading fine, though [08:26] Yep, that solved part of the problem. [08:27] Aki-Thinkpad: are all the warnings/messages gone after creating the dir? [08:27] sec, uploading screenies [08:27] some [08:31] you can also paste the text... should be easier/faster :) [08:32] Good Morning [08:33] mzanetti, http://imgur.com/a/eSHJ9#0 [08:34] 0, what I was getting, 1, creating the folder, 2, in the folder, 3, running the app, 4, new output, 5 file now being written to. [08:34] hmm... need to debug it... it shouldn't print that many warnings when everything works fine [08:36] mzanetti, are you using qfile to create the directory, or cmake? [08:37] definitely not cmake [08:37] Don't know much about cmake actually. [08:39] I think I don't create it at all... [08:39] strange [08:40] nope... images fail to load here if that directory doesn't exist [08:41] I wonder how they appear for you guys... it should clearly fail in that case [08:42] mzanetti, could my debug environment be at fault here? http://i.imgur.com/Ycqvuf2.jpg [08:42] Aki-Thinkpad: no... your system is fine. its a bug in the reminders app [08:42] I just wonder where that path came from if the app doesn't create it [08:43] oh.. wait [08:51] mzanetti, will you be patching, so it creates the directory in ~/.cache, or should I file a bug report? [08:51] Aki-Thinkpad: I'm on it [08:51] k [08:52] narf... I hate that applicationName stuff [08:54] mzanetti: you should start a club :-) [08:54] * sergiusens would join [08:54] :) [08:55] dpm, mzanetti one thing I immediately notice in running the app, is lack of animations for fluidity. For example, when switching notes, it should do a fade, and should show the button being pressed. Are these things you will want implimented? [08:58] Aki-Thinkpad: yeah.. I guess we want to have that eventually [08:58] ha okay maybe I'll try that as a first patch. [09:00] sergiusens: http://paste.ubuntu.com/7403432 :D [09:01] mzanetti: I had those all over when converting to click :-P [09:09] dpm: Aki-Thinkpad: https://code.launchpad.net/~mzanetti/reminders-app/fix-image-errors-and-warnings/+merge/218368 [09:09] thanks for reporting [09:09] no probs. [09:11] Aki-Thinkpad, you might also want to look at the list of open bugs. That might give you a small task to get started, which might be easy than starting with a big feature (which could be the next contribution :) https://bugs.launchpad.net/reminders-app/ [09:12] Aki-Thinkpad, another way of easily getting started is with providing a review of the branch mzanetti just posted ^ [09:13] reviewing the work of other team members is also an important part of core apps development [09:14] reviewing implies looking at the code, testing that the branch actually works as intended and leaving a comment either pointing out what needs fixing or approving the change [09:14] you can also set the status of the comment to "Needs Fixing" or "Approve" as appropriate [09:16] dpm, Thanks. I took a look at the bugs before; there are a few of the triaged bugs I am considering looking into [09:18] Aki-Thinkpad, awesome. Once you've found one you like, feel free to assign it to yourself, so that others know that someone else is working on it already. You can then set the status to "In Progress" once you actually start working on it [09:18] for anything else, you know where we are and who to ask :) [09:18] cool beans [09:22] rock on [09:43] dpm: seen this when opening qtcreator on a cmakelists file? paste.ubuntu.com/7403548/ [09:44] * popey updates packages [09:44] popey, hm, no, I haven't. [09:46] mzanetti, on https://code.launchpad.net/~mzanetti/reminders-app/fix-image-errors-and-warnings/+merge/218368 - could you explain to me what " Don't scale images to 0,0 if sourceSize is not set." means? [09:47] That's the only part I cannot quite follow on the MP [09:51] zbenjamin: paste.ubuntu.com/7403548/ I am getting that when I bzr branch lp:~dpm/ubuntu-terminal-app/merge-plugin, and then qtcreator the cmakelists file within... [09:53] popey: wtf [09:53] popey: that looks like the chroot is broken [09:54] popey: can you build other projects? [09:54] zbenjamin: i think i need to reset my qtcreator to some kind of base [09:54] zbenjamin: clean start, spring clean, what should I rm to do that? [09:55] popey: you can remove ~/.config/QtProject/qtcreator/ that should completely reset everything [09:56] zbenjamin: including the chroots? [09:56] popey: no the chroots you have to remove either from within QtC in the Ubuntu Options page, or manually from commandline [09:59] zbenjamin: deleted that config dir, now just starting qtcreator with no project gives me the same error - paste.ubuntu.com/7403649/ [09:59] did you also remove the chroots? [09:59] no, exactly what should I remove? all of /var/lib/schroot? [10:00] popey: click chroot -a armhf -f ubuntu-sdk-14.04 destroy [10:00] thanks [10:01] ok, now I get this /usr/share/qtcreator/ubuntu/scripts/qtc_chroot_cmake2: line 19: CLICK_SDK_ARCH: Need to set CLICK_SDK_ARCH non-empty [10:01] (after deleting chroots and restarting qtc) [10:06] zbenjamin: ^ [10:09] popey: i think that can be ignored [10:09] popey: just try if its working [10:39] popey: hi [10:40] mihir: hji [10:40] -j [10:40] mihir: was just emailing you ☻ [11:03] popey: ohh got it. [11:03] popey: there are many bugs under needs-design [11:03] can we get some inputs on this ? [11:06] popey, the WIP branches: http://people.canonical.com/~dpm/core-apps-sponsoring/wip [11:06] I should probably put it all together in one page, but for now at least they won't get lost [11:12] dpm: oof, that looks bad. [11:12] mihir: see my mail ☻ [11:15] popey, it does look pretty red :) But let's focus on one thing at a time and concentrate on the "Needs review" page for now. I think the oldest WIP branches can in fact just be rejected, but let's use that page as a reminder only now, to which we can get to once we've gotten "Needs Review" under control [11:16] indeed [11:24] lool: i filed bug 1316552 - because I'm concerned that we're going to start blocking store updates soon on not having the framework. Wasn't sure you'd see it, hence pinging you with the bug number. [11:50] popey: got it :) thanks that would help a lot. === _salem is now known as salem_ [12:08] dpm: for some reason I can't see the screenshot. but anyways... strange that it doesn't work for you [12:08] dpm: works for me: http://i.imgur.com/CaXcLFC.png [12:09] nice pic :) [12:10] lol [12:10] haha [12:10] that doesn't look comfy [12:10] this is how I look like when hacking on reminders at 2am [12:10] heh [12:10] mzanetti, looking at your screen capture, it might be a locale-related thing? I use Catalan and perhaps that messes up the dates? http://i.imgur.com/NDDRjyv.png?1 [12:11] interesting [12:11] that might well be [12:11] dpm: actually no.. not really [12:11] I'm using QDateTime comparison... that should either always use catalan or never [12:11] ok [12:12] does that word start with double L? [12:12] just throwing a guess from the only difference I saw from our screenshots [12:12] which word? [12:12] Notebooks [12:13] yes, llibretes [12:17] dpm: debugging this it makes me really wonder how the heck it actually works here :D [12:18] :-) [12:18] I started debugging too, I just had to remind myself about -DCMAKE_BUILD_TYPE=debug [12:18] :) [12:19] dpm: please apply this patch and send another screenie: http://paste.ubuntu.com/7404242 [12:21] on it [12:25] popey, I added some comments on the calendar MR [12:26] thanks renato [12:26] dpm, mzanetti o/ I have a couple of hours for reminder :-) [12:26] rpadovani: yay [12:26] rpadovani, hi, awesome :) [12:26] rpadovani: I left some comments on your newDesign branch [12:27] mzanetti, I'm getting this when trying to get the app running on a device: [12:27] file:///home/phablet/dev_tmp/com.ubuntu.reminders/qml/ui/NotesPage.qml:20:1: module "QtQuick.Layouts" is not installed [12:27] import QtQuick.Layouts 1.0 [12:27] is QtQuick.Layouts not installed in the images? [12:27] dpm: apt-get install qtdeclarative5-qtquicklayouts-plugin [12:27] hmm... [12:27] mzanetti, I saw the email. There is a lot of works to do on my branch, so I'll restart from scratch after your branchs are landed [12:27] dpm: that's a bummer [12:28] mzanetti, right, but that means that we'll need to add it to the seeds before we ship the new design, yes [12:28] dpm: that, or do it still handcrafted [12:28] dpm: can you find out if its possible to include that in the image? [12:29] Mirv: ^ [12:29] Mirv: mzanetti asking if qtdeclarative5-qtquicklayouts-plugin can be added to the image? [12:29] mzanetti, my hunch is that it should be just possible to add it to the seeds, but we will need to justify why we need it too [12:30] dpm: justification is that they allow to layout items in qml with half of the code than without them [12:30] :) [12:30] finally no more typing "anchors" like crazy [12:31] seems cool [12:31] ok, let's hear what Timo has to say and then we can send a MP to update the seeds [12:31] awesome [12:32] dpm: back to the other branch for now. what does the note subtitle say after applying the above patch? [12:33] mzanetti, it's "Last week" all the way :( [12:33] really weird... [12:33] mzanetti, mhh, just tested again from scratch the notes-heading branch and I have nothing. Any idea? [12:33] I see dpm has problems too [12:33] popey: no problem, although it was called "qtdeclarative5-quicklayouts-plugin" before and now that's transitional and it's qml-module-qtquick-layouts now (we should do the other renames in seed too) [12:33] rpadovani: but dpm sees the header just fine :D [12:33] rpadovani: note: its a section header, not a header for each note [12:34] mzanetti,yes yes, I saw your screen [12:34] dpm, Curious, when clicking a delegate, is there a reason why its not exactly responsive? [12:34] rpadovani: so if you have all notes created at the same day, you would see only one [12:34] mzanetti, [12:34] ^ [12:34] popey: it's not in seeds simply because Qt Quick Controls is new in 5.2 [12:34] ah === dholbach_ is now known as dholbach [12:34] Aki-Thinkpad, not sure what you mean exactly, could you provide more context? [12:34] popey: well hmm, maybe let's bring bzoltan1 into discussion too, I fail to know how our "supported API" is exactly defined [12:35] Aki-Thinkpad: which delegates? [12:35] dpm, sure; just take the default page, "Notes", and when you click on one... sometimes it does not switch to the note. [12:35] sometimes you need to click it 3 or 4 times [12:35] Aki-Thinkpad: well, if the whole app just feels laggy and hangs all the time you're probably affected by the qt bug too [12:35] Mirv, but we are on 5.2 now, correct? [12:35] yes dpm [12:36] mzanetti, http://people.ubuntu.com/~rpadovani/touch/noteHeading.png [12:36] dpm: yes, but it was new in upstream 5.2 [12:36] Aki-Thinkpad: try commenting out the two calls to "sort(0)" in notes.cpp [12:36] mzanetti, hmmm; anything I can do about that? [12:36] hence why we didnt seed it [12:36] thanks [12:36] I have my phone setted on english US as regional format [12:36] s/phone/pc [12:36] dpm: yes we are now. the seeds / supported API just are not controlled according to any formal process, I think. [12:36] so when new things appear they don't automatically appear on images [12:37] rpadovani: the screenshot is a lie :D [12:37] rpadovani: nah, probabyl indeed the same issue dpm has [12:37] popey, Mirv, gotcha, so that explains why it's not there. And so theoretically, we could just add it [12:37] rpadovani: with some different outcome for the same root cause [12:37] dpm: yeah, I'm creating a merge request now [12:37] mzanetti, it's a screenshot, not a cake :D [12:37] awesome popey [12:37] popey is always awesome [12:37] \o/ [12:38] Aki-Thinkpad, this bug should be fixed soon on the desktop, we're in the process of SRU'ing it [12:39] Aki-Thinkpad, on the phone images it's already been fixed (for utopic) [12:39] Mirv: popey: who was the the brave to summon me? [12:39] mzanetti, worked perfectly, and it looks like my animation works. Its just a very simple fade in and out animation. I made it a separate component, that is easily configurable. ought I try and merge? It is only a few lines of code honestly. [12:39] Aki-Thinkpad: create a merge proposal for it [12:39] cool beans [12:39] bzoltan1: not me! [12:40] everyone hides now [12:40] popey: ok.. one out :) [12:40] * popey puts on his level 10 invisibility cloak [12:41] bzoltan1: are you the Keeper of the API? [12:41] dpm: rpadovani: I think I found the issue. can you retry? I've pushed to the branch [12:41] bzoltan1: you were summoned to ask whether qml-module-qtquick-layouts can be cast as part of the holy Ubuntu API [12:41] mzanetti, ok, pulling [12:42] mzanetti, commit message is interesting :D [12:42] https://code.launchpad.net/~popey/ubuntu-seeds/add-quicklayouts/+merge/218405 [12:42] hope thats right [12:42] :D [12:42] Mirv: Yes, my young fellow... I am [12:43] mzanetti, nope, no heading at all for me :/ [12:43] meh [12:43] Mirv: indeed it could be part of it... once it passes all the challenges [12:43] rpadovani: what's the debug output? [12:44] Mirv: where does it live? Who is its master? [12:44] lives upstream qt I think... [12:44] bzoltan1: it lives in Qt Quick Controls, and I assume mzanetti is a willing master since he's the one wanting it! [12:44] :D [12:44] define "master" [12:44] define "willing" :D [12:45] define define [12:45] :D [12:45] does not compute [12:45] what are the duties [12:45] ? [12:45] mzanetti: buy me beer? [12:45] Qt Quick Controls is at least part of official releases, so that makes it better than some of the other API:s we support [12:45] ok... I guess I can handle that [12:45] not in finland tho :P [12:45] mzanetti, http://paste.ubuntu.com/7404339/ [12:45] mzanetti: LOL :) I go with water in Finland [12:47] mzanetti: Mirv: I think it is fairly easy to include the Qt Quick Controls ... but I would be happier to consult with zsombi, as our layour strategy might be different from Qt's solution... and to be frank I do not want the default Qt Quick controlls sneak into our UI story [12:48] mzanetti, looks good with the new branch now, but I'd change the order to show the newest notes at the top of the list [12:48] mzanetti: bzoltan1: actually our layouting was made to complement QtQuick Controls Layoutings, so those should be usable together without any probs [12:49] zsombi: nice one ^ [12:49] dpm: its the commented away sort(0) [12:49] rpadovani: I'm thinking you're not up to date with the code :/ [12:49] rpadovani: or with the build rather [12:49] mzanetti: it was even mentioned in the tutorial if I'm not mistaken... if not, we have to update those, and even add few examples like how those work together [12:50] rpadovani: this message "NotesPage.qml:144:22: Unable to assign [undefined] to bool" is rather suspicious too [12:50] mzanetti, isn't the build 99 the last one? [12:50] mzanetti, ah, that was one of my questions on the MP, was there any reason to comment out the ordering? I guess to work around *the bug* temporarily on the desktop? :) [12:50] rpadovani: it is, but you need to call make in the top level builddir again. [12:50] rpadovani: could it be that you just typed make in the builddir/src/app subdit? [12:51] dpm: I replied to that comment on the MR [12:51] ah, ok, I'm not yet up to date, looking... [12:51] mzanetti, nope, I did it well [12:51] then I'm lost [12:51] rpadovani: I added some debug prints with the last commit too... I don't see them in your output [12:52] hola tsdgeos, I thought the "de novembre" vs. "Novembre" bug had been fixed in Qt 5.2? I'm still seeing in a current branch from the Reminders app -> http://i.imgur.com/4e6slM1.png [12:52] mzanetti, isn't this the right procedure? Maybe I understand wrong: http://paste.ubuntu.com/7404358/ [12:53] rpadovani: looks good [12:53] rpadovani: aaaah! I know what it is [12:53] rpadovani: you have account-plugin-evernote installed in your system. its using that one instead of the one from the builddir [12:53] mzanetti, ops :D [12:53] which is btw the reason I was opposed to install the plugin into the main Qt plugin dir with the account-plugin [12:54] mzanetti, ah, actually, that's something I wanted to mention: [12:54] QStringList importPathList = view.engine()->importPathList(); [12:54] importPathList.append(QDir::currentPath() + "/../plugin/"); [12:54] that does not work [12:54] it would need a prepend [12:54] to override the installed plugin [12:55] and IIRC QDir:currentPath() did not work either [12:55] somehow I had in mind it would wak them backwards, but seems I was wrong [12:55] I had to change this on filemanager to work [12:55] dpm: QDir::currentPath() wors, but you need to cd into src/app/ [12:55] let me send a quick MP for this [12:56] mzanetti, https://code.launchpad.net/~mzanetti/reminders-app/fix-image-errors-and-warnings/+merge/218368 works well for me, could I topapprove it? [12:57] fine with me, yes [13:00] mzanetti, I removed account-plugin-evernote, how can I add my account? [13:00] rpadovani: did you remove the account too? [13:00] rpadovani: the account should still be there and working [13:01] mzanetti, no, I ... wait... I did purge and not remove :/ [13:01] :D [13:02] mzanetti, rpadovani, here's a quick one for you: https://code.launchpad.net/~dpm/reminders-app/override-installed-plugin/+merge/218411 [13:02] dpm: hmmmm, tbh i don't remember what was the outcome of the discusion [13:02] dpm: i think i said it was a bug in the app [13:02] but may be wrong [13:03] tsdgeos, I think you're thinking of another conversation we had. That was about the DatePicker. This one now is about a bug you fixed upstream a while ago, and I would have thought it'd land in 5.2 [13:04] dpm: isn't that like ages ago? [13:04] yep [13:05] * tsdgeos tries to find the thing in his brain [13:05] * dpm tries to dig out the upstream bug [13:09] dpm, hey I have never proposed a merge. Is what I am doing here, pushing to lp:~akiva/reminders-app/name-of-my-proposal , and then ??? [13:09] mzanetti, ok, now there are headers but in a strange order: http://people.ubuntu.com/~rpadovani/touch/strangeOrder.png [13:10] Aki-Thinkpad, then you go to LP, click on the branch that you've pushed, and you'll find a link that says "Propose for merging" [13:10] rpadovani: yeah... I commented out the sorting to work around the qt listview bug [13:10] dpm, okay so the first step is correct then? [13:10] will keep that disabled for the new-design branch until we merge it to master... [13:10] mzanetti, ah, ok [13:11] which are next steps? [13:11] dpm: i think i'm remember something [13:11] Aki-Thinkpad, correct, and this should answer some of your questions: https://wiki.ubuntu.com/Touch/CoreApps/DevelopmentGuide#Using_Launchpad [13:11] dpm: https://codereview.qt-project.org/#change,54214 [13:11] rpadovani: I guess I'll make the new reminders delegate design [13:11] dpm, thanks [13:11] !cookie [13:11] rpadovani: while waiting for you to change the app background color and the notesview [13:11] tsdgeos, that was it! [13:12] rpadovani: can you create 2 branches, a quick one that just changes the main app color (so I can see how my stuff finally looks) [13:12] mzanetti, sure, going [13:12] rpadovani: and then the other, bigger one, for the notes delegate [13:13] mzanetti, I also tested https://code.launchpad.net/~dpm/reminders-app/override-installed-plugin/+merge/218411 and works, could I approve it? I see you approve it [13:13] dpm: let me make sure the code is still there [13:13] awesome, thanks rpadovani. I top approved [13:14] tsdgeos, cool, thanks. If it helps, here's the code where that date string on my screenshot is generated from: http://pastebin.ubuntu.com/7404464/ [13:17] mzanetti, I added a new comment to https://code.launchpad.net/~mzanetti/reminders-app/notes-headers/+merge/218328 - let me know what you think [13:18] dpm: sure, can do... [13:18] awesome, thanks. I think it will be a really nice visual improvement to have already [13:19] mzanetti, what's wrong with bzr? (besides the fact is not git) [13:19] mzanetti, https://code.launchpad.net/~rpadovani/reminders-app/newBackground/+merge/218419 [13:19] I took the newDesign branch, why the diff is so big? [13:20] I mean, it's the same branch! [13:20] a no, its my error [13:20] sorry :/ [13:21] mzanetti, new background -> https://code.launchpad.net/~rpadovani/reminders-app/newBackground/+merge/218420 [13:21] nw [13:21] Aki-Thinkpad, how are you getting on with proposing the merge? Can I help with anything else? [13:23] dpm, In my commits, I have always had a format of "Changelog: 1.... 2.... 3.... ; Todo: a.... b.... c...." Obviously employing newlines. Is this an acceptable format? [13:23] dpm: you have to do the formatting yourself, MMMM will always give you the "non standalone" month name [13:24] dpm, http://i.imgur.com/GajnYkK.jpg [13:25] dpm: i.e. QLocale::standaloneMonthName + qdate::month [13:26] Aki-Thinkpad, that should be fine, yes [13:27] tsdgeos, ok, thanks, will try that [13:29] dpm: i think what i fixed was standaloneMonthName not actually giving you the standaloneMonthName === oSoMoN_ is now known as oSoMoN [13:35] tsdgeos, so to be sure, on the original code, how should the value on L18 be actually returned to get "Novembre 2014"? -> http://pastebin.ubuntu.com/7404464/ [13:39] dpm: something like this http://paste.ubuntu.com/7404600/ [13:39] though since it's now a standalonemonthname [13:39] it's not capitalized [13:39] so you may want to capilatize it [13:39] or not [13:40] dpm, last question; "Review Type"? [13:40] tsdgeos, ok, trying that now on the app [13:41] Aki-Thinkpad, feel free to leave that blank, we don't tend to use it. Just one detail to remember -> you'll need to set the commit message in the MP. It's a bit hidden in the form, you'll need to expand the options at the bottom [13:41] yah done. [13:41] thanks [13:41] okay proposed. I set you as the reviewer [13:45] Aki-Thinkpad, awesome, thanks! No need to set anyone as the reviewer (but nothing wrong with setting it), as all team members get an e-mail notification when a new MP is submitted [13:45] heh [13:46] Bzr really is an amazing piece of technology. [13:49] tsdgeos, so how do I actually capitalize the month name? I can't find a way of doing it via http://qt-project.org/doc/qt-5/qlocale.html#standaloneMonthName ? [13:50] dpm: you'll have to play magic with qstring/qchar and ::toUpper [13:51] but now that i see http://i.imgur.com/4e6slM1.png it wasn't localized either [13:51] err localized -> uppsercase [13:51] do you really need it in uppercase? [13:51] if it's the start of a sentence, as in the header, yes [13:53] tsdgeos, so what's the point in using standaloneMonthName & co. if I have to present a translatable string to translators "%1 %2" in case they need to rearrange it and manually set the month name to uppercase? I would have expected that just presenting translators with a date string "MMMM YYYY" should have just done the trick? [13:54] dpm: http://paste.ubuntu.com/7404666/ [13:54]