[01:13] Ping any calender app devs? been a few months since I have proposed anything for merging. Was wondering if anyone wanted to give this branch a review: [01:13] http://bazaar.launchpad.net/~akiva/ubuntu-calendar-app/ubuntu-calendar-app/revision/394?start_revid=394#NewEvent.qml [01:15] or rather this one: https://code.launchpad.net/~akiva/ubuntu-calendar-app/ubuntu-calendar-app [01:15] Do I wait for someone to review it before proposing it for a merge? [01:24] ah hold on; need to recommit; forgot one expandable [01:32] okay done :) [01:38] and my computer just crashed [01:38] >:/ why do all the developers have to be sleeping at this hour! === chihchun_afk is now known as chihchun [02:45] balloons, you around? [02:45] balloons, I must have missed your ping [06:13] good morning [06:49] dholbach, hey good morning [06:50] dholbach, are you working on the calender app? I submitted a patch/merge, and its been awhile since I did it. [06:50] Just wondering if you would mind taking a look at the proposal, and tell me if I did it right? [06:51] feel free to say "busy" :) [06:53] https://code.launchpad.net/~akiva/ubuntu-calendar-app/ubuntu-calendar-app === greyback__ is now known as greyback [07:40] Akiva-Thinkpad, I don't work on calendar, sorry [07:40] np [07:40] popey should know who to ping about an MP [07:40] thanks [07:40] when he gets up then :P [07:55] Good morning :-) [07:55] nik90, morning [07:55] nik90, you work on the calender app right? [07:56] or did you just point me in that direction :P [07:56] Akiva-Thinkpad: no, I work on the clock app. But sure what's up? [07:56] nik90, oh its just been a few months since I did the ol merge thing; can you give my merge proposal a look, and just tell me if I did it right? [07:56] https://code.launchpad.net/~akiva/ubuntu-calendar-app/ubuntu-calendar-app [07:57] Akiva-Thinkpad: first thing can you rename your branch? You can do so by pushig the branch again to launchpad [07:57] Akiva-Thinkpad: bzr push lp:~launchpadid/ubuntu-calendar-app/branch-name [07:57] why do I need to rename it? [07:57] and what should I name it to? [07:58] Akiva-Thinkpad: because ubuntu-calendar-app is your current branch name which is a bit confusing. [07:58] okay [07:58] just to help the calendar devs quickly see what the branch is about [07:59] Akiva-Thinkpad: in the branch link, there is a link called "Propose for merging". Click on that, add the description, commit message and press propose. That's about it. [07:59] okay great advice [07:59] nik90, even though it hasnt been reviewed? [07:59] or does the review happen when I propose it? [07:59] Akiva-Thinkpad: the review happens when you propose a merge [07:59] Akiva-Thinkpad: it is similar to a pull request in github [07:59] nik90, and should I set the status from "Development" to something else? [08:00] Akiva-Thinkpad: nah, what it says by default should be fine [08:00] Akiva-Thinkpad: these are the current MPs for calendar app for example [08:02] nik90, bzr: ERROR: Permission denied: "~akiva/ubuntu-calender-app/new-event-pan-flickable-down/": : Project 'ubuntu-calender-app' does not exist. [08:02] using bzr explorer [08:03] Akiva-Thinkpad: did bzr push lp:~akiva/ubuntu-calendar-app/branch-name not work? [08:03] nik90, no [08:03] well [08:03] wait; i'll do it in terminal [08:06] nik90, yep worked [08:06] !cookie [08:06] Wow! You're such a great helper, you deserve a cookie! [08:06] :) [08:06] hehe [08:08] Morning all [08:09] popey, Hey busy fellow; Chris is Badmouthing ubuntu, and you arent there to defend it ! [08:09] :O [08:09] yeah, he's a silly rabbit [08:09] * Akiva-Thinkpad assumes you are swamped with the rtm [08:09] just a tiny bit [08:10] ;) [08:11] Akiva-Thinkpad: akiva/ubuntu-calender-app/new-event-pan-flickable-down/ didn't work because of a typo, you typed calender instead of calendar, that's all [08:11] ha ha shucks :P [08:12] easily done [08:12] but thank you for the contribution! [08:12] :P [08:12] so at https://code.launchpad.net/~akiva/ubuntu-calendar-app/new-event-pan-flickable-down if you're happy with the merge then click "propose for merging" [08:13] Akiva-Thinkpad: you will need to propose it [08:13] then it will get reviewed [08:13] yep; doing it right now [08:13] :) [08:13] Winner. [08:13] What an excellent way to start a friday. [08:14] Morning all o/ [08:14] DanChapman: Morning :) [08:15] nik90, https://code.launchpad.net/~akiva/ubuntu-calendar-app/new-event-pan-flickable-down/+merge/230935 [08:15] does that look good? [08:16] is there a mailing list for "app developers" or is it just ubuntu-phone? [08:16] Akiva-Thinkpad: do add a commit message otherwise jenkins will fail you :D [08:16] nik90, ah shucks; guess I have to recommit [08:16] Laney: just ubuntu-phone and ubuntu-core-apps [08:16] Akiva-Thinkpad: no no [08:16] Akiva-Thinkpad: in the MP itself [08:16] Akiva-Thinkpad: there must a set commit message for the MP [08:16] oh I think I did. [08:17] Akiva-Thinkpad: no you set the Description of the Change not the commit message [08:17] oh wait [08:17] should I just make it the same as my description? [08:18] I used to have a formal way of doing commit messages, but DPM just told me to avoid it. [08:18] just be brief [08:18] Akiva-Thinkpad: just be brief would be fine. [08:19] okay [08:19] done [08:19] thanks for the help [08:19] !cookie [08:19] Wow! You're such a great helper, you deserve a cookie! [08:19] Akiva-Thinkpad: now we wait for a calendar app dev to review it. I will test it on the device a bit later [08:20] nik90, neato [08:21] popey, Just curious; http://i.imgur.com/ZTnhvR7.png < This has been around for awhile; has there been any discussion in regards to fixing this? [08:21] the little aesthetic flaw where things in the list escape outside the ubuntu shape? [08:22] nik90, or maybe you can answer that question [08:22] Akiva-Thinkpad: I am not sure, but it is annoying to see that though [08:22] Akiva-Thinkpad: calendar app dev -----> Mirv [08:22] bah [08:22] mihir ^^ [08:22] nik90, its in the showcase too. [08:22] I know [08:23] :) [08:23] speaking of calendar app, should it work on unity7 desktops on utopic? [08:23] ha ha [08:23] nik90: hey good morning all :) [08:23] seems to hang/not display an ui here [08:23] morning [08:23] seb128: it does run on utopic [08:23] seb128, I have it working on mine [08:23] seb128: but we don't have UI change if we resize [08:23] well okay, running; I havn't tested the functionality extensively. [08:24] hum, I wonder why it's not display anything here [08:24] e_cal_client_get_view: assertion 'E_IS_CAL_CLIENT (client)' failed [08:24] wonder if that's the issue [08:24] seb128: you running from trunk ? [08:25] nik90: btw , I have fixed the Icon MR , you can review whenever you get time :) thanks for that [08:25] mihir, 0.6bzr390 from the core apps ppa, should be similar [08:26] seb128: I think that's a EDS error, not really sure how to fix it. I think it may cause the clock app to not start as well if you have the error [08:26] k [08:26] * seb128 tries a guest session [08:28] works with another user [08:28] Akiva-Thinkpad: i had never seen that [08:28] how much usually it takes to start an emulator ?? [08:28] on utopic, vm [08:28] very first run takes a lot [08:29] like 20 minutes last time for me, then it's faster for other runs of the same image [08:30] ybon: hmm okay , when i try to run from QT creator it does nothing , but when i do it from terminal it starts. [08:30] how can we change scale using Terminal ? [08:30] Good morning all; happy Friday, and happy Relaxation Day! :-D [08:30] mihir: hey can you check if icon-location@18.png and icon-contacts@8.png , avatar@8.png are required? If yes replace them with suru icons? [08:34] popey, for what its worth; filed a bug : https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1357238 [08:34] Ubuntu bug 1357238 in Ubuntu UI Toolkit "Ubuntu Shape corners escape " [Undecided,New] [08:34] thanks Akiva-Thinkpad [08:35] :) [08:35] btw ybon I'm seeing a couple of apparmor failures in osmtouch... [08:35] [Fri Aug 15 09:31:09 2014] type=1400 audit(1408091469.359:230): apparmor="DENIED" operation="chmod" profile="me.yohanboniface.osmtouch_OSMTouch_0.1.3" name="/run/user/32011/" pid=4344 comm="qmlscene" requested_mask="w" denied_mask="w" fsuid=32011 ouid=32011 [08:35] * mihir looks for the icons usage in calendar [08:35] [Fri Aug 15 09:31:09 2014] type=1400 audit(1408091470.090:231): apparmor="DENIED" operation="mkdir" profile="me.yohanboniface.osmtouch_OSMTouch_0.1.3" name="/home/phablet/.cache/QtLocation/" pid=4344 comm="qmlscene" requested_mask="c" denied_mask="c" fsuid=32011 ouid=32011 [08:35] popey: humm, thanks [08:36] I'm afraid it explains why we don't have tile caching anymore [08:36] Am I missing a securicy-policy group? [08:38] * ybon on http://developer.ubuntu.com/publish/apps/security-policy-for-click-packages/ [08:39] nik90: it is done. [08:42] oh, I see a "sensors" policy group, but I've not add it to Sensors Status, and I can still access to accelerometer or gyroscope [08:43] Just in case, I'm adding it [08:46] ybon: you'll probably need to cache somewhere else, that you have write access to [08:46] popey: I'm not sure I can control that :/ [08:47] IIRC, it's one of the QtLocation internal settings that are not exposed [08:47] but I will check that again [08:47] t1mp, kalikiana, if I want to show a popup dialog before an app shows its main UI, what's the best way to do it? I've tried to use a PopupUtils.open() call inside the MainView's Component.onCompleted handler, but that does not seem to work. [08:48] btw, I remember asking if .cache was safe, and being answered yes :) [08:48] mihir: any reason why you guys are showing the save and cancel button in the bottom instead of as header buttons? [08:48] ybon: might want to ping jdstrand, he's the apparmor guru [08:48] popey: thanks :) [08:48] mihir: that will save some precious vertical space and declutter the new event page a little [08:48] jdstrand: if you are around, I may need a bit of help to understand what's wrong on OSMTouch cache :) [08:49] nik90: you meant to say they should be on top right corner right ? [08:49] dpm: t1mp: how do you mean "before" exactly? you need to display something on screen, so normally that would mean pop over on top of your MainView [08:49] mihir: yes [08:50] kalikiana, sorry, I wasn't clear. What you're describing is exactly what I mean [08:50] nik90: i guess this was decided in Malta spring , so I am not much aware about that, popey any comments ? ^^ [08:50] mihir: the save button should be in the top right. The cancel button should replace the back button in the header. [08:50] dpm: do you have a test snippet I could have a look at? [08:51] I don't follow? [08:51] popey: in the new event page, the buttons save and cancel are shown in the bottom toolbar. [08:52] popey: any reason why this shouldnt be header buttons like the other core apps do? [08:52] oh, yes, totally should. [08:52] mihir: ^^ [08:52] nik90: noted :) [08:52] good call, this solves another issue with those buttons obscuring content [08:52] mihir: it should be an easy fix. I could do it but I don't want to mess with calendar app AP tests [08:53] nik90: yup , i know we have to change AP accoridngly too :) [08:53] mihir: make it look like https://docs.google.com/presentation/d/1Kkl7xT8BYo9mT8i3IXh6rPrDlNg6cBxTajahlRYIGlI/edit#slide=id.g18587dd40_077 [08:54] * mihir looks at presentation slides [08:55] kalikiana, I don't, I've got a full-blown app, but I think this should illustrate what I'm trying to do: http://bazaar.launchpad.net/~ubuntu-terminal-dev/ubuntu-terminal-app/require-password-screenlock/view/head:/src/app/qml/ubuntu-terminal-app.qml#L18 [08:56] trivial MR review , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1357216/+merge/230940 [08:56] kalikiana, if I put that PopupUtils.open() call in another handler, e.g. here http://bazaar.launchpad.net/~ubuntu-terminal-dev/ubuntu-terminal-app/require-password-screenlock/view/head:/src/app/qml/ubuntu-terminal-app.qml#L52 it does show up [08:56] only on Component.onCompleted handlers it seems it doesn't [08:57] mihir: hmm did you push anything into it. It looks empty to me, both MR and branch [08:57] nik90: yeah , i just saw that, the push got intrrupted re-pushing that [08:58] ah [08:59] popey, yo yo - how's life over there? [09:02] dholbach: super (on hangout) [09:04] nik90: it should be done now , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1357216/+merge/230942 ? [09:11] dpm: how does calling the popup open from mainview not work? [09:11] dpm: I fixed one of the two apps that still had wrong desktop translations. let me know if this works for you: https://code.launchpad.net/~phablet-team/dialer-app/desktop-translation-fixes/+merge/230944 [09:11] nik90: have you ever used Stroke Button > [09:11] ? [09:12] no [09:12] its quite new I think so I havent [09:13] nik90: hmm yup , we have to replace date textbox as we are using highligted property of that ,and it is going to deprciate [09:13] mihir: isn't it just a normal button with strokecolor? [09:14] dpm: the PageStack + contents is being completed before the MainView [09:14] dpm: so if you want the PageStack to be loaded later, you'll need to put it inside a Loader [09:14] nik90: yes [09:15] nik90: i am not sure . [09:15] mihir: ^^ it shouldn't be difficult [09:15] mihir: check API docs in QtC :) [09:16] /*! [09:16] \since Ubuntu.Components 1.1 [09:16] If set to a color, the button has a stroke border instead of a filled shape. [09:16] */ [09:16] property color strokeColor: Qt.rgba(0.0, 0.0, 0.0, 0.0) [09:16] hmm, docs for this are not on the webpage yet [09:16] t1mp: yeah mhall119 has to update the docs in the webpage [09:16] mhall119: can you trigger an update for http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.Button/ ? [09:16] mhall119: actually for the full uitk docs :) [09:16] t1mp: I always first look at QtC docs which get updated with every sdk update [09:17] mhall119: I think you should have a script for this to update automatically with every sdk update [09:18] Hey I recorded a 100kb (Tiny) video demonstrating a bug; how do you think should I upload it? Filebin? Youtube? This is for a bug report on the calender app. [09:18] Akiva-Thinkpad: you should be able to upload the video directly as an attachment in the bug report [09:18] nik90, okay great [09:18] will do [09:19] beuno: "Changes can not be accepted and published at this time. Please try again later." [09:19] is there a deployment happening? How do I always seem to hit those? [09:19] * nik90 is away from irc to do some coding... [09:19] o/ === nik90 is now known as nik90|BusyCoding [09:20] t1mp: got it thanks for [09:20] t1mp: i was referring docs on website instead of QTC [09:21] popey: what do you think for bug , https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1357255 ? [09:21] Ubuntu bug 1357255 in Ubuntu Calendar App "Year view has day names too condensed" [Undecided,New] [09:21] popey: do we need to handle that small resoultion as well ? [09:26] popey: quick question, when you click on an alarm sound to choose it, do you want it do a playback loop of the sound? Or just play it once? [09:29] nik90|BusyCoding: once [09:30] nik90|BusyCoding: and tapping a second stops the first and plays second. [09:30] mihir: looking... [09:35] jhodapp: ping === nik90|BusyCoding is now known as nik90 [09:39] popey: thanks :) [09:41] For the calender, are new events supposed to show up in the agenda? [09:42] or is that not implimented yet? [09:42] or do I need to be connected to google or something? [09:44] Akiva-Thinkpad: it is supposed to be show up in agenda [09:45] mihir, okay; I made a quick video showing my debug too;. will submit a bug report [09:45] Akiva-Thinkpad: sure , thanks..btw i see list of up coming events on my agenda view. [09:46] mihir, screenshot? [09:46] mihir, https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1357280 [09:46] Ubuntu bug 1357280 in Ubuntu Calendar App "Creating New Event does not show up on "Upcoming Events"" [Undecided,New] [09:50] * mihir tests agenda view [09:55] jdstrand: ping [09:59] mihir, hey; bug was actually my issue [09:59] I did not have "Personal" or "Birthdays" enabled. [10:00] mihir, Given my confusion, if the list is empty, do you think it might be a good idea to add a quick tip to an empty agenda? [10:02] say "No Upcoming Events" +
+ "Make sure you have Calendars enabled"? [10:05] Anyways; gonna head out. I marked the bug as invalid. [10:06] o/ later all [10:06] mihir: Does calendar support Trusty? [10:07] mihir: You guys are using still using toolbar items instead of the new Header API [10:07] which requires Utopic [10:19] popey: can you add me to the calendar-app-dev, I want to create a RTM milestone and put all important bugs there. [10:20] awesome [10:20] done [10:20] popey: thnx [10:21] nik90: yes it does work on trusty my machine, but haven't tested all the functionalties \ [10:22] nik90: did it break anything fo you ? [10:22] mihir: no it works on trusty [10:23] mihir: but I think you guys should move to the new headers which will requires utopic [10:23] nik90: hmmm okay :) [10:24] * mihir files a bug [10:28] dpm: messaging app should also be fixed here: https://code.launchpad.net/~phablet-team/messaging-app/more-desktop-translation-fixes/+merge/230953 [10:28] dpm: i'll email you about those two as well [10:35] nik90: there was a bug for the datetime indicator issue right? my lp foo is failing me [10:36] davmor2: yeah it doesn't show the contents anymore.. UI broke [10:37] nik90: but was there a bug? [10:37] davmor2: charles is trying to debug it. [10:37] davmor2: not that I know of...will check now..1 sec [10:37] I couldn't find one but I'm sure there was mention of one [10:38] maybe it was just discussions [10:38] davmor2: I couldn't find it either...I remember popey asking if he should report one. [10:38] popey: did you ^^ [10:38] popey: didn't he was asking me for the number :) [10:39] popey: do you want to file it or should I, I think I must of seen charles talk about this issue and throw out a bug number for another bug [10:40] charles was debugging the issue and I provided some logs..dont recollect seeing a bug report [10:40] gah [10:42] nik90: is there any way we can control text alignment of Button directly ? [10:42] mihir: no [10:42] mihir: doesn't it do it automatically? [10:43] nik90: it does set in center by default , and no other way to controll it [10:43] davmor2: nik90 bug 1357307 [10:43] bug 1357307 in indicator-datetime (Ubuntu) "Datetime indicator no longer shows events, time not updated" [Undecided,New] https://launchpad.net/bugs/1357307 [10:43] mihir: btw do you have permission to set bug priority for calendar app? [10:43] nik90: yup i do have [10:44] mihir: ok in which case please do set the status and triage the bugs as you submit them pls. This way we can focus on the important ones quicker. [10:44] mihir: also move them to the rtm milestone if they are important [10:44] nik90: sure got it, i do confirm bugs if they are reporducable on desktop as of now. [10:44] for other bugs i ask popey or dpm to confirm it [10:45] will do [10:45] davmor2, popey: I have a feeling that the datetime-indicator bug started with qtmir 0.6 [10:45] nerochiaro, awesome, thanks! I've reviewed the MP ad added a few comments [10:45] davmor2, popey: Which iimage did that land in? [10:45] because I remember charles looking at the 189 changelog and seeing absolutely no changes that could cause the bug [10:45] which makes me suspect that it could have originated in 188 [10:45] or 187 [10:46] nik90: 189 was when we all saw it [10:46] could of landed in 188 [10:47] popey: you have done it few days back , and there are no more to confirm i can see , thanks for tha t [10:47] dpm: i haven't worked on any of these apps, i think renatu, boiko and tiago will know how to fix these comments [10:47] dpm: i don't know what these strings refer to [10:50] popey: ping [10:50] popey: for the bug ,https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1315199 [10:50] Ubuntu bug 1315199 in Ubuntu Calendar App "[Desktop] Running the Calendar app from the command line (calendar-app) uses the old icon" [Low,Triaged] [10:51] i marked it low, "phone > desktop" in terms of priority [10:51] popey: i dig out into all the way , and i couldn't find anywhere the old icon is being used, is there any other location it comes in Ubuntu Desktop? [10:51] probably the debian package points to it [10:51] which is separate from the click package config [10:51] at a guess [10:51] but it's low, so I wouldn't worry too much. [10:51] popey: it should be something out of the click package , though . [10:51] popey: okay. [11:07] t1mp, thanks. So all I want is that before showing any pagestack pages, a popup dialog is shown. Would it just be a matter of putting the pagestack inside the Loader (I've never used those)? Where should the PopupUtils.open() call should be invoked, then? [11:08] dpm: what about putting pageStack.push(terminalPage) after console.log("Authenticated for full access") ? [11:09] or after that whole if-statement if that's what you want [11:11] t1mp, so I tried this for testing purposes: http://pastebin.ubuntu.com/8052914/ - but that does not load the popup, either [11:15] hey, I'm looking for anyone willing to do code reviews for my QML code for chcekbox-touch [11:15] I'm still learing basics and I'd love pointers and criticism from more experienced developers [11:16] dpm: weird that it doesn't open the popover. Is mview initialized? You can also call open() without a second parameter to have a centered popover (not pointing to an Item) [11:16] if anyone willing to spend a few minutes looking at the stuff I'm writing please ping me [11:16] zyga: link [11:17] nik90: thanks, all of the code is in in lp:checkbox, let me give you a link http://bazaar.launchpad.net/~checkbox-dev/checkbox/trunk/files/head:/checkbox-touch/ [11:17] nik90: we had a prototype but we're writing stuff from scratch, what you see there is just the main page really [11:17] nik90: now the code I'm working on here aims to introduce a simple list of items that you can select/deselect [11:17] nik90: each item is a simple string [11:18] zyga: If you can create a MP for it, I'd be willing to review and add comments to it. [11:18] nik90: ok [11:18] nik90: thanks! [11:18] zyga: say thnx after I do the review..I could ditch you ;P [11:18] t1mp, I don't know if mview is initialized, I just noticed that if I put the .open() call inside the Component.onCompleted() handler for either mainview or pagestack, it won't load the popup. Also removing mview as the caller from the .open() function does not help [11:19] zyga: to get all the latest UITK features, import Ubuntu.Components 1.1 (utopic only) [11:20] zsombi_: ping [11:20] mihir: pong [11:20] zsombi_: can we change text color of button ? [11:21] zsombi_: dpm's popover issue^ is it possible that the rootItem is not valid yet in onCompleted of an item? [11:21] mihir: lemme check, there were thing sadded recently to it... [11:21] dpm: if you can reproduce this problem with a simple (as small as possible) test program, I can play around with it [11:21] zsombi_: i couldn't see anything in doc , along with that alingment of text is also not there [11:22] zsombi_: bug https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1357303 [11:22] Ubuntu bug 1357303 in Ubuntu Calendar App "Use different component instead of using Textbox or custom component instead of textbox. " [High,Confirmed] [11:22] mihir: nope, not possible :( [11:22] dpm: the behavior seems not intuitive, so if you can reproduce it with a test program, perhaps file a bug for it [11:22] zsombi_: hmm thats bad :( for calendar :| [11:23] zsombi_: any workaround woudl you suggest ? [11:23] mihir: there [11:23] mihir: there's non really... :( [11:23] zsombi_: :( [11:23] popey, we just had this bug 1357324, are we expected to do export support as well before rtm? as we only have import as WIP at the moment? [11:23] bug 1357324 in Ubuntu Music App "Content hub integration as source/destination of music" [Undecided,New] https://launchpad.net/bugs/1357324 [11:23] mihir: I'd rather file a bug against UITK [11:24] i just did for alignment https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1357303 [11:24] Ubuntu bug 1357303 in Ubuntu Calendar App "Use different component instead of using Textbox or custom component instead of textbox. " [High,Confirmed] [11:24] zsombi_: wrong link , https://bugs.launchpad.net/bugs/1357323 [11:24] Ubuntu bug 1357323 in Ubuntu UI Toolkit "There is no way to control text alignment of the button" [Undecided,New] [11:24] could you confirm and make it possible done before RTM ? [11:24] ahayzen: wishlist [11:25] popey, ok shall i change that bug to track exporting then? and use the other one as importing [11:25] zsombi_: bug https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1357332 [11:25] Ubuntu bug 1357332 in Ubuntu UI Toolkit "There is no way to control text color of Button" [Undecided,New] [11:25] popey, this one for importing https://bugs.launchpad.net/music-app/+bug/1347444 [11:25] Ubuntu bug 1347444 in Ubuntu Music App "Music app does not expose download handler" [High,In progress] [11:25] yeah, keep the two things separate bugs [11:26] popey, cool :) and i see a silo for ms2 \o/ [11:26] mihir: is this text color bug an issue needed for rtm? [11:26] swet [11:26] +e [11:26] zsombi_: both of them actually [11:27] zsombi_: i would say first , Color then alignment [11:27] zsombi_: the reason is it will break consitetency of Event Page [11:27] mihir: I see [11:27] t1mp: thanks! [11:28] t1mp: does that inclue U.C.ListItems, we get those still at 1.0? [11:28] t1mp, something like that? -> http://bazaar.launchpad.net/~dpm/+junk/popupapp/view/head:/main.qml#L26 [11:29] zsombi_: this is how it looks like currently ,http://imgur.com/MKGhCJm [11:29] popey: http://imgur.com/MKGhCJm [11:29] zyga: listitems are still at 1.0 [11:29] k [11:29] t1mp, on that test app, I can actually see the popup for a split second before it gets hidden by the page loaded [11:30] t1mp: if the popover is created the same time as the rest of the app's components, then it can be that during onCompleted the rootItem is not yet valid [11:30] zsombi_: the text needs to be left aligned , and the color should be changed according to the other component [11:31] zsombi_, mihir: I didnt follow the issue fully, but why not use abstract button instead? [11:32] mihir: actually nik90 has a point... you can use abstractButton instead [11:33] mihir: of course in that case you need to make all teh layout, all the shading, the pressed efects ASO [11:33] Is the datetime indicator not showing the correct time on image 189 a known issue? seems to be totally frozen [11:34] dpm, t1mp: so what was the issue? [11:35] zsombi_: nik90 can I control color,alignment in AbstractButton ? [11:35] mihir: abstractButton is basically like an Item{} [11:35] zsombi_, here's a test app: http://bazaar.launchpad.net/~dpm/+junk/popupapp/view/head:/main.qml - in summary, I try to load a popup above the mainview that prevents seeing the UI below, but if I use PopupUtils.open() inside Component.onComplete, it seems the popup is loaded, but it ends up behind the main page [11:35] mihir: AbstractButton has nothing, [11:35] mihir: it offers a container where you can put the button items [11:36] zsombi_: nik90 i see [11:36] mihir: so you set the layout, color, size and son etc. [11:36] * mihir tries Abstract Button [11:37] mihir: but AbstractButton ia also stylable, so you could try to use the ButtonStyle for it... [11:38] ahayzen: yup known bug [11:38] dpm: yes, as said before, the problem you're facing is that the rootItem is not yet set by the QML system, that's why it goes like that [11:38] dpm: but a workaround ;) [11:38] ahayzen: https://launchpad.net/bugs/1357307 [11:38] Ubuntu bug 1328646 in Unity 8 "duplicate for #1357307 Clock out of sync on resume from suspend" [Critical,Triaged] [11:38] ahayzen: woops wrong bug [11:39] dpm: var popup = PopupUtils.open(.....); popup.parent = idOfTheMainView [11:39] ahayzen: nvr mind tht's the correct bug [11:39] nik90, hah but it is known at least :) [11:39] dpm where idOfYourMainView is what it says, so you need to give an ID to your MainView :) [11:40] nik90, confusing using the term 'clock' lol..not clock app but clock in the indicator [11:40] zsombi_, I did add it, I just removed it after t1mp's suggestion [11:40] dpm: and didn't work?! [11:40] ahayzen: hehe [11:41] dpm: hehh... [11:42] dpm: keep it and set popup.z = Number.MAX_VALUE [11:43] what is the best non-visual component to use for a composition of other non-visual components [11:43] apparently QtObject cannot be used as it (?) cannot hold children [11:43] * zyga was looking at http://qt-project.org/doc/qt-5/qml-qtqml-qtobject.html [11:44] zyga: what kind of non-visual components? [11:44] zyga: js functions? [11:44] nik90: a python handle (pyotherside's Python compoent) + a whole collection of high-level "handles" (other non-visual components that expose APIs of python-side objects to QML) [11:45] nik90: so for a pyotherside-based address book that might be something like [11:45] nik90: Component { Python { ... } AddressBookManager { ... } AddressBookSynchronizationMagic { ... } } [11:45] popey, hmmm the ms2 in the silo claims that 'non-existent property "onFilled"' ... but we are using that with trunk [11:46] nik90: the idea is that you then put AddressBook { } in your code and you can use various exposed APIs to talk to internals (say, find a contact by name or initialize synchronization) [11:46] nik90: it's not really an address book, I'm just using that as an example [11:46] nik90: should I just use Component? [11:46] popey, and that onFilled was added with the streaming models ...so unless they have removed it?! [11:46] zyga: Component{} are similar to defining those components in a separate file [11:47] zyga: they delay the loading of those children until they are actually needed [11:47] nik90: hmm, ok so maybe to rephrase, what is the difference between Component and QtObject? [11:47] zsombi_, I tried your suggestion: http://bazaar.launchpad.net/~dpm/+junk/popupapp/view/head:/main.qml#L30 - but still I can see the popup for a split second, to be covered by the main page afterwards === MacSlow is now known as MacSlow|lunch [11:48] nik90: in C++ I'd just subclass QtObject and shove all my other non-visual objects there [11:48] zyga: QtObject is sort of like a structure where you can store properties, functions etc that you want to access [11:48] nik90: but not children? [11:48] ahayzen: check their merge? [11:48] (or did I miss something while reading QtObject QML docsw) [11:48] popey, FYI if that lands it will break our trunk in its current state [11:48] zyga: not children [11:48] popey, yeah i will it is like it has gone backwards lol [11:48] heh [11:49] nik90: right, so for children-capable non-visual class I should go to... Component? [11:49] zyga: give it a try, although I am not that convinced [11:49] nik90: hmm, Components cannot declare new properties [11:49] zyga: here is an example of qtobject https://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/view/head:/app/alarm/AlarmUtils.qml [11:49] ogra_: seems ms2 might break the music app... [11:50] popey, i was hoping it was gonna let us land content-hub support but seems they have changed something...i'm scanning the diff at the moment [11:50] popey, bah [11:51] zyga: can I look at the code of your non-visual components in paste.ubuntu.com or somewhere. I am trying to get a better idea of it [11:51] nik90: maybe my approach is wrong, I don't want to write any C++, 90% of the code is in python, I want to provide QML interfaces to that via pyotherside, it's mostly straightforward when you just use pyotherside directly (instantiate Python and call py functions through its API), what I want to do is build a layer that hides pyotherside and just exposes my application objects as if they were from a "native" QML extension [11:51] sure [11:51] http://paste.ubuntu.com/8053127/ [11:52] popey, they have set/getStatus() now with Ready and Loading states...i assume this is what has been changed from the filled property being set to true [11:52] that's CheckboxStack, the high-level component that apps will use to use our testing library [11:52] popey, could you initiate a file manager upload to land the security checks feature? [11:52] popey, yep it is that [11:53] nik90: then other bits are mostly going to be something I call PythonHandle, a small class that has a handle to something we got from pyotherside (basically cross-runtime lifecycle manager) with a way to call methods on it [11:53] zyga: ah ok that makes better sense now [11:53] nik90: so all our boring python API objects are going to have a subclass of PythonHandle [11:53] nik90: that natively exposes the API as QML [11:53] nik90: (with API stability etc etc) [11:53] zyga: let me try to edit that and show an example [11:53] nik90: thanks [11:53] dpm: yup [11:53] nik90: feel free to rip out pyotherside [11:54] nik90: as that part works, I'm just struggling with raw qml [11:54] nik90: none of those components are visual [11:54] ybon: the chmod denial is weird and others have reported seeing it. it is a legitimate denial (ie, I won't be adding policy for it) [11:54] popey, how urgent is that ms2 landing? as i'll fix that when i get from work...and then we need to test the content-hub stuff as well [11:54] nik90: they are mostly models of some sort [11:54] ack [11:54] nik90: test definition objects, test sessions, test results, etc [11:55] jdstrand: so you mean QtLocation should not be writting on ~/.cache/ ? [11:55] ahayzen: well, i don't know, but we need to block the ms2 landing if it breaks apps. [11:55] ogra_: how do we do that? ^ [11:56] popey, yep we need todo that [11:56] ybon: jdstrand you may be interested in bug 1357327 [11:56] bug 1357327 in usensord (Ubuntu) "One app blocks anothers access to sensors" [Undecided,New] https://launchpad.net/bugs/1357327 [11:56] ahayzen, popey, did that land already ? else simply dont land it [11:56] interesting popey [11:56] ogra_: its in a silo, which one ahayzen ? [11:56] ybon: the .cache/QtLocation denial is also a legitimate apparmor denial (ie, I can't add policy for it). my guess is that either your application isn't setting 'applicationName' in your qml correctly (or setOrganizationName/setApplicationName in qt) or QtLocation isn't properly honoring these [11:57] popey, ogra_ 002 [11:57] popey, so mark that silo as not ready [11:57] then it wont land [11:57] jdstrand: okay, thanks for making this clear, I will investigate a bit more [11:57] ahayzen: got a link? I don't have it open or even have the url [11:57] popey, is music-app in the ms2 test plan? as they broke us before, if it isn't then it should be ...even if it is just check that the app starts/ap runs [11:57] I remember QtLocation was not honoring those settings time ago, but I know this was to be fixed [11:57] popey, ahayzen, probably add something to the comments [11:58] Need to check out the code of QtLocation actually used [11:58] popey, http://people.canonical.com/~platform/citrain_dashboard/#?distro=ubuntu&q=landing-002 [11:58] ta [11:58] zyga: http://paste.ubuntu.com/8053157/ [11:58] nik90: thanks [11:58] nik90: so visual Item, ok, I'll just use that :) [11:58] nik90: one thing I want to ask about as well, separately from the rest is: project code organization: http://paste.ubuntu.com/8053159/ [11:59] nik90: if you look at lines 12-14 [11:59] ogra_: I don't actually know what to edit to do that... the spreadsheet at https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFVHQ3FuMDJGLUZCamJfSjYzbWh3Wnc#gid=0 ? [11:59] nik90: Components there is a symlink to ./components [11:59] ybon: if you narrow it down to a bug in QtLocation, can you ping me with the bug-- I'd like to subscribe to it since other apps may also be affected and I can refer people to it [11:59] who's responsible for playing a sound on incoming SMS? it's not in messaging-app nor history-service, I can't find it. [11:59] nik90: hi [11:59] nik90: on lines 354+ [11:59] zyga: 1 sec [11:59] nik90: how should I arrange that so that my main.qml sees the qmldir inside [11:59] k [11:59] ogra_: i have only view access on that sheet [11:59] nik90: currently I just run qmlscene -I . main.qml [12:00] popey, well, the lander of ms2 should do that ... i thought thats ahayzen [12:00] jdstrand: https://code.launchpad.net/~nik90/ubuntu-clock-app/play-sound-preview/+merge/230959 [12:00] no, ms2 is ours, jamesh and pete-woods [12:00] jdstrand: ok [12:00] thostr owns it [12:01] zyga: you can import components in your main.qml by relative paths [12:01] ogra_, i do the music-app and was testing 'upstream' mediascanner2 [12:01] zyga: so something like import "../../components" as Components [12:02] ahayzen: maybe leave a comment on the merge at least. [12:02] jdstrand: hey I am having an apparmor issue with regards to media-hub playing the alarm ringtones from the /usr/share/sounds folder [12:02] popey, ok i'll add a comment for now... would you be able to chase up?...i should be able to fix it when i get back from work quite easily [12:02] nik90: good timing, I'd like to play a sound from my app, and for some reason, no worky :( [12:03] nik90: what is the denial? [12:03] nik90: oh, let me try, thanks! [12:03] jdstrand: https://code.launchpad.net/~nik90/ubuntu-clock-app/play-sound-preview/+merge/230959/comments/561440 [12:03] nik90: will that respect the qmldir inside? [12:03] sure ahayzen [12:03] zyga: Do you have c++ code or is this all qml and py? [12:03] popey, ahayzen, i assume you talk about line 35 ... thats not set to "testing done", talk to jamesh and pete-woods [12:03] nik90: can you paste the contents of /usr/bin/media-hub-server? [12:04] nik90: all qml and py [12:04] jdstrand: sure, 1 sec [12:04] zyga: you dont need a qmldir then [12:04] nik90: though we'd like API declarations (versioning of components) [12:04] yes ogra_ can you unset that, because i cant [12:04] zyga: ah [12:04] nik90: it's not required yet, I'm just toying with it [12:04] zyga: I don't have experience with that yet I am afraid. the sdk does it quite nicelt [12:04] nik90: later on we'd like to make the checkbox stack library a drop-in thing you can add to apps [12:05] nicely [12:05] nik90: yeah, the SDK is preinstalled though in where QML modules are looked for [12:05] nik90: so maybe I should just add it to lib/$triplet [12:05] even as symlinks [12:05] popey, i've commented on the mp [12:05] * zyga tries [12:05] thanks [12:05] though that will only work on the device [12:05] (ubuntu-app-launch is the part that understands lib/triplet) [12:05] nik90: thanks, I got enough to work with for a while now :-) [12:06] ogra_, yes line 35 [12:06] zyga: :) I will think about it [12:06] jdstrand: not sure I follow [12:06] ahayzen, right, so conact the landers listed there :) [12:06] jdstrand: the file is present there in that folder [12:06] ogra_, i gotta go back to work :/ popey can u ping them? [12:06] nik90: what is the output of 'cat /etc/apparmor.d/usr.bin.media-hub-server' [12:07] ah [12:07] nik90: :-) [12:07] ya ahayzen [12:07] popey, thanks [12:07] * ahayzen hugs popey [12:07] jdstrand: http://paste.ubuntu.com/8053241/ [12:09] ah [12:10] we have this rule:owner /tmp/orcexec* m, [12:10] but it is overriden by this: audit deny owner /** m, [12:10] nik90: can you file a bug against media-hub? [12:11] jdstrand: sure. So basically the bug is about media-hub not being to read the /usr/share/sounds folder? [12:11] nik90: is it possible to play a sound at all using MediaPlayer? or is there a known problem with that [12:11] nik90: well, the denial you have in https://code.launchpad.net/~nik90/ubuntu-clock-app/play-sound-preview/+merge/230959/comments/561440 is about mmap access to /tmp/orcexec.CLp5yf [12:12] that would prevent playback [12:12] jdstrand: ok I will file at media-hub stating permission issues :) [12:13] karni: not sure [12:13] karni: actually no it should work for you [12:13] karni: the jupiterbroadcasting app in ubuntu touch uses media hub and works fine [12:13] nik90: I see my solution was almost same, besides I'm playing my own file, and with MediaPlayer [12:13] nik90: you can test if this is the problem by doing: 'sudo mount -o remount,rw /' then adjusting /etc/apparmor.d/usr.bin.media-hub-server to remove/comment out 'audit deny owner /** m,', then doing 'sudo mount -o remount,ro /', then loading the profile with 'sudo apparmor_parser -r /etc/apparmor.d/usr.bin.media-hub-server' [12:13] nik90: I would only need media-hub to play an external file, right? [12:13] karni: i suppose yes [12:13] jdstrand: ok will give it a try [12:14] * nik90 first files a bug [12:14] nik90: thanks, please ping with the bug number [12:17] jdstrand: bug 1357348 [12:17] bug 1357348 in Media Hub "Cannot play sound files in /usr/share/sound/** folder due to apparmor permission issue" [Undecided,New] https://launchpad.net/bugs/1357348 [12:22] jdstrand: hmm that doesn't work [12:23] jdstrand: i tried your solution, while I don't see the denial I still dont hear the playback [12:28] nik90: do you see another denial? [12:28] jdstrand: no I don't see the denial [12:28] jdstrand: it work on desktop though.. [12:29] nik90: it sounds like two different issues then. you could try rebooting the device/restarting media-hub-service [12:29] nik90: I will still get the apparmor denial fixed though [12:30] jdstrand: ok [12:30] jhodapp: [12:30] jhodapp: need some help debugging an issue [12:30] nik90: I'm quite glad you pinged me on this. mediascanner2 also suffers from this, and I hadn't noticed the override deny rule [12:31] jdstrand: :) [12:31] jdstrand: Is http://paste.ubuntu.com/8053359/ the same denial I got earlier? [12:31] jdstrand: hard to see since the time stamp seems all wrong [12:34] nik90: it is not the same denial, but it is caused by the same rule and will be fixed by the same update [12:34] ah ok [12:41] nik90: media-hub uploaded with your fix [12:41] jdstrand: thnx [12:42] jdstrand: how is it so fast fix-committed? [12:42] jdstrand: we still need to wait for this to land on the image? [12:42] nik90: I did a direct upload (no code changes) [12:42] jdstrand: oh [12:42] nik90: I marked it as Fix Committed manually [12:43] (since it is building in -propsed) [12:44] jdstrand: sweet..will test again in the next image since it should be there then === dholbach_ is now known as dholbach [13:12] popey, can you review my app? :) [13:12] I fixed it! === MacSlow|lunch is now known as MacSlow [13:19] dholbach: Changes can not be accepted and published at this time. Please try again later. [13:19] ☹ [13:19] same I had later [13:20] s/later/earlier/ [13:20] wow === chihchun is now known as chihchun_afk [13:50] nik90: yeah, got the same AppArmor AccessDenied, because it's qmlscene that wants to access core.ubuntu.media.Service [13:50] happy fix for that bug moved forward so fast [13:51] karni: that bug fix alone doesnt fix the issue for me [13:51] karni: there is some other component that need fixing as well [13:51] o rly, dang. what's wrong ? [13:51] I see [13:51] nik90: is there an easy way to query media hub for notification sounds only? [13:51] nik90: I think you were talking about that today, right? [13:52] about selecting a notification sound [13:52] karni: not that I know..you need to give media-hub the url to the notification sounds [13:54] nik90: you mentioned being able to select an alarm sound -- is this implemented somewhere already? [13:54] karni: https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 [13:55] nik90: thanks a bunch man [13:55] np [14:20] balloons: Is https://code.launchpad.net/~nik90/ubuntu-clock-app/add-custom-sound-backend/+merge/225812 ready for merge? [14:20] balloons: I want to push out a store update after it is in trunk. [14:22] popey: it looks http://geoname-lookup.ubuntu.com/?query=Farnborough&release=utopic is currently down. Does that mean the migration is happening atm :D? [14:22] nvr mind it is backup now [14:22] i get 500 here [14:23] no admin2 tho [14:23] yeah no admin2 or timezoneid [14:23] * nik90 waits longer [14:28] nik90: every other refresh I get a 500 error [14:29] popey: the first time I get a 500 error, but after refreshing it works again. not sure what is happening [14:29] asked the IS guy but he's afk [14:29] so dunno if he's doing it, or what [14:30] t1mp: updating now [14:30] nik90: automating is the goal, it's pretty close already [14:30] hmm I am hoping this is in before the weekend..would love to merge that branch in [14:30] mhall119: ok [14:30] nik90: I have scripts already that pull the new docs package and process it [14:31] I just need to setup a server where I can schedule it to run regularly [14:32] nice [14:33] nik90: on it with him now [14:33] popey: ty [14:38] nik90: https://code.launchpad.net/~zkrynicki/checkbox/touch-app/+merge/230987 [14:38] nik90: criticism and other feedback welcome! :-) [14:39] nik90: going patch-by-patch is probably easier to understand [14:40] zyga: :) [14:41] nik90: feel free to ignore all of the python parts if you don't care about that [14:41] nik90: the readme should help you to get this to work if you want to try it out [14:41] yup saw it [14:41] nik90: it's pretty easy on the desktop, a bit more annoying (for now) on devices, there's a MP for getting that right but it's not ready to land yet [14:41] will test and see how it runs [14:42] nik90: thanks! [14:42] I will comment there directly when I am doe [14:42] done* [14:42] mhall119: thanks~ [14:42] ! [14:44] t1mp: done [14:48] mhall119: looks good, thanks [14:48] np [14:49] zyga: when I press "start testing" is it supposed to do anything yet? [14:51] nik90: nope [14:51] nik90: not yet [14:51] ack [14:51] nik90: I have the next bits but I wanted to know the approach is good [14:52] nik90: specifically all the QML component separation [14:52] zyga: ah ok. got it [14:52] nik90: I'm also totally not sure how to write tests for anything but that's (for now) ok as we want to do a manually tested 1.0 and iterate [14:52] (1.0 is still a few big UI screens away :-) [14:53] zyga: you can test using QTest or Autopilot. I am guessing you will prefer autopilot since it is python [14:53] zyga, autopilot autopilot :P [14:53] nik90: :-) [14:53] brendand: now I know which side you are on :D [14:54] brendand: I'll do autopilot for sure [14:54] nik90: though I'll probably do both, we'll have a moderate amount of js code as well [14:54] nik90, well qmltest has a purpose as well [14:55] brendand: qmltest? [14:55] nik90, people seem to think they do the same thing but they really don't [14:55] ooooh [14:55] yes, QTest help with unit tests quite well [14:55] I love that [14:55] qmltestrunner [14:55] \o/ [14:55] I have to enable that in our CI [14:55] nik90, QTest is for C++ though [14:56] brendand: there is a qml version for it now which is what the sdk, unity8, (recently the clock app) uses [14:57] nik90, you mean this right? http://qt-project.org/doc/qt-5/qtquick-qtquicktest.html [14:57] brendand: yup [14:57] * zyga looks [14:57] thanks for the tips guys! :_) [14:58] zyga, if you need some help with autopilot let me know. the right way to do it is not totally obvious, so best to look at a few existing app tests rather than trying to parse it from the documentation [14:59] brendand: I'd love to try to add trivial autopilot tests once we get to the second screen (next week for sure) [14:59] brendand: one thing I totally don't know is what to test [14:59] brendand: like "I click on the 'start testing' and I go to another screen?" [15:00] brendand: what's the right angle on that [15:00] brendand: so yeah, I'll definitely ask for advice [15:01] zyga, do you know what the page object model is? [15:02] brendand: a little, yeah [15:02] brendand: I read some of the sources [15:02] brendand: because they were much better than documentation [15:02] zyga, you want to write classes which represent each of your screens [15:02] zyga, and have functions that represent actual behaviour [15:03] zyga, so WelcomeScreen.continue [15:03] zyga, and that has the code to press the button and everything [15:03] brendand: what do I write those classes in? Javascript? [15:03] zyga, that way if the way of pressing the button changes, you just fix it there [15:03] zyga, no - python - this is in autopilot [15:03] oh [15:03] ohhhhh [15:03] :D [15:03] zyga, it's like a wrapper around the autopilot code [15:03] I I had a tail, it would wiggle now :> [15:04] if* [15:04] zyga, the advantage is that if the way of 'continuing' changes - to a different button or something, or the button name changes [15:04] zyga, then you fix it in the page objects method, rather than in every single test [15:05] brendand: right, that makes sense [15:05] brendand: and we can run those tests on devices and such, right? [15:05] zyga, well absolutely [15:05] brendand: can we somehow jump on part of the CI train to test our branches wrt autopilot tests on devices? [15:06] zyga, yes - you need to register your project in the 'cupstream2-distro' project [15:06] zyga, although that's changing - eventually [15:06] zyga, but for now that's how you do it [15:06] brendand: https://launchpad.net/cupstream2-distro ? [15:06] 404 for me [15:07] zyga, sorry it was one word -https://launchpad.net/cupstream2distro [15:08] brendand: I don't want to do it yet (nothing to test) but I'll ask you about how that works sometime next week or the one after, depending on our dev speed [15:09] zyga: typing up my review comments atm. [15:11] zyga: actually shall I rather just propose a MP into your MP? [15:11] :-) [15:11] nik90: whatever works better for you [15:11] nik90: I use git locally so you can push to that if you prefer git [15:11] nik90: (I can publish my native branch) [15:11] oh no ... I am a bzr guy :) [15:11] sure ;) [15:11] :-) [15:11] I will push my changed and propose an MP in 5 mins [15:12] thank you [15:14] nik90: perfect, thanks [15:14] nik90: I'll apply your changes and see how it works [15:17] zyga: quick question..you planning to show any other element in the welcome page other than welcome and the button? [15:18] nik90: no, though the welcome text can be longer [15:18] nik90: it will typically be a paragraph or two [15:19] zyga: ok [15:19] nik90: I tried your quick code and it doesn't look right as the button is not at the bottom of the page, I'll see if I can make it be there somehow [15:19] zyga: wait which code? [15:19] zyga: I didnt propose anything yet [15:19] nik90: Layout.fillHeight: true [15:19] oohh [15:19] nik90: heh, look at the merge request :) [15:19] nik90: I got another review and I thought that was you :) [15:20] nik90: :> [15:20] zyga: that's mzanetti [15:20] mzanetti: I am fixing the code atm :D [15:20] ? [15:20] mzanetti: thanks for your review :-) [15:20] zyga: I will address his comments as well [15:20] mzanetti: https://code.launchpad.net/~zkrynicki/checkbox/touch-app/+merge/230987 [15:20] right [15:20] hehe [15:22] what is checkbox btw? [15:22] googling for checkbox only brings up the obvious, which I think is unrelated to this [15:22] mzanetti: it's a testing application for ubuntu [15:23] https://code.launchpad.net/checkbox-project/ [15:23] er [15:23] https://launchpad.net/checkbox-project/ [15:23] the name is very old by now [15:23] so what is "a testing application" ? :D [15:24] mzanetti: it's an application that you can feed some tests (manual or automatic) and execute [15:25] mzanetti: btw it is installed by default on the ubuntu desktop [15:25] mzanetti: there's a UI to pick what you want to test and a large collection of existing tests for hardwre [15:25] mzanetti: search for system testing [15:25] (that UI is horrible, I take no responsibility ;) [15:25] my ubuntu desktop might be a bit special :D [15:25] lol [15:25] * nik90 lols to both statements :P [15:26] checkbox-touch is just a new "sane" touch UI for phablets, later on we'll scale it for desktops and discard checkbox-gui (the thing that is preinstalled) [15:26] if you want to know more look at the diagram here: http://checkbox.readthedocs.org/en/latest/stack.html [15:36] zyga: enjoy https://code.launchpad.net/~nik90/checkbox/clean-qml-code/+merge/230991 [15:36] nik90: thank you :-) [15:36] yw [15:52] I've got a text file containing a hexdump that I want to byteswap [15:53] surely there is some nice command line tool to do this right? [15:54] bob__: what's the word size? [15:54] 2byte words [15:57] bob__: I can write a py3k program that does that if you want [15:57] ahh no worries thanks [15:57] nik90, mzanetti: so quick question to both of you: why should I (or should not) use a ColumnLayout inside a Page? [15:57] zyga: you can... but you don't have to [15:57] zyga: you can, just that in your case you only had a label and a button [15:57] in the end the code by nik90 didn't have the extra layout [15:58] ah [15:58] is there an implicit layout on the page? [15:58] zyga: so what often happens is that you end up adding tons of anchors.whatnot properties [15:58] zyga: and with a Layout you can considerably simplify things [15:58] I see [15:58] as nik90 said, with only 2 components its not a big deal [15:58] ok [15:59] thanks, I've merged your improvements in :) [15:59] but imagine having 5, you'd grow white hair typing all the anchors stuff [15:59] credits given :-) [15:59] hehe [15:59] I've pushed --overwrite over the original branch [15:59] zyga: if you have more components use a column and it will automatically adjust the height based on the visibility of each component. [16:00] nik90: actually I'd say even ColumnLayout in this case [16:00] nik90: because you can say Layout.fillWidth/Height on one of the items and it'll expand [16:00] oh, niec [16:00] nice [16:01] mzanetti: I haven't used the Qt Layouts yet in my code, but that's looks great [16:01] will look into it [16:01] our UI is - so far - relatively simple, a few static screens, a few lists and a few interactions here and there [16:01] nik90: with a Column you can't do that and often end up with weird "height: parent.height - something.height - somethingElse.height" constructs [16:01] mzanetti: yup [16:02] should I switch to the layout then? I wonder if I make the text very long, will it hide the button or not? [16:03] zyga: well.. in your case it just doesn't make that much of a difference to bother right now :D I just added that comment so next UI you write you know the possibilties you have [16:03] mzanetti: ok :) [16:11] nik90: http://geoname-lookup.ubuntu.com/?query=Farnborough [16:11] popey: woohoo! [16:12] popey: I will fire up a click to test it [16:18] mhall119: hey, quick question. In http://geoname-lookup.ubuntu.com/?query=Springfield what entries do you need to different two springfield within the same state? [16:21] i looked through that and they have different names [16:22] popey: I notice that sometime admin1 or admin2 is empty. While at some times they are both there. This is not the fault of the server but rather the location. [16:22] e.g. there's two in Virginia, Springfield and North Springfield [16:23] popey: actually there's 3 in Virginia [16:23] but in those cases, there is no need to differentiate since in Virginia the timezone will be the same [16:23] yeah, different names [16:23] west, north, and virginia [16:23] you'd know which one you're in, surely? [16:24] exactly [16:24] Take me home, country road... West Virgina... [16:24] * popey sings [16:24] hehe [16:24] I am bit confused at what to show..let me look at the old clock app code [16:25] how hard is it for people to come up with different names for cities [16:27] heh === gatox is now known as gatox_lunch [16:42] popey: added click to test. The design spec didn't include the admin1 and admin2 being shown. So I supervised a little. [16:43] popey: but that shouldn't be hard to change once we get design spec for that [16:43] kk [16:48] mzanetti: hey, is there a way to stop QNetworkAcessManger's request? My usecase is that sometime the user clears the string which then default to local search mode. at that point any request being made should be stopped to save bandwidth. [16:49] nik90: I think you can call cancel() on the QNetworkReply* you get as the return value of the get() call [16:50] popey: as I see there is an ubuntu.com geoname server, what about a photon one? http://photon.komoot.de/ :) [16:50] mzanetti: ok [16:51] ooh [16:57] popey, thanks for the messages :) [16:57] ahayzen: which city do you live in? [16:57] sry just testing something :D [16:57] nik90, at the moment Welwyn Garden City ... [16:58] ahayzen: see the mail from jouni ? ☻ [16:58] nik90, when at university, Oxford [16:58] ahayzen: found it [16:58] popey, errr... when was it sent? [16:58] 5 mins [16:58] popey, ah my mail server is sometimes slow...or the gmail forwarder is [16:58] ah [16:59] * ahayzen blames google sending emails to NSA [17:00] popey, did ms2 make the image? or is it in the next one? [17:00] no, not in teh current image [17:00] * popey pops out, brb [17:01] ok [17:01] nik90, so what were you testing with my location? [17:01] ahayzen: online world city search in the clock reboot [17:01] nik90, or are u just tracking my movements... [17:02] nik90, \o/ cool was WGC in there then? [17:02] the latter would be a appropriate response...damn why i didnt say that [17:02] yup WGC was there ;) [17:02] I can't believe it found my parent's hometown as well..some remote place in india [17:03] nik90, yey i sometimes have to search for luton or st albans if it can't find my hometown [17:03] nik90, magic :) what service are you using? [17:03] ahayzen: http://geoname-lookup.ubuntu.com/?query=Springfield&release=utopic [17:04] ahayzen: you didn't fool for that too did you that's both of us he's tracking ;) [17:04] davmor2, yeah i did :( 'testing clock-reboot' yeah right [17:04] davmor2: what's your town? [17:04] nik90, was that an ubuntu subdomain before or did you request for it? [17:04] or u not gonna tell me [17:05] ahayzen: it is the same domain used by the ubuntu installer [17:05] nik90: wolverhampton normally but currently I'm in shrewsbury [17:05] nik90, that was gonna be my next question ;) [17:05] ahayzen: it is being there for a while by the looks...popey had to append data to it recently like the timezone info [17:05] davmor2: tracking you now [17:06] nik90, seems to have found everything i've thrown at it so far [17:06] ahayzen: i know ... its pretty awesome. [17:06] weather app will also use it [17:06] nik90, assume ur handling the case where multiple places come back from many countries? [17:06] nik90: leek [17:06] nik90, which fields are you showing in the UI? [17:06] ahayzen: I show the city name, state, country [17:06] nik90: bishops castle [17:06] nik90, cool [17:07] nik90: wyre piddle [17:07] nik90: wool [17:07] nik90: brewood [17:07] davmor2: found all except for wyre piddle [17:07] popey, just got the mail \o/ [17:10] nik90: https://maps.google.com/maps/u/0/ms?ie=UTF8&t=h&oe=UTF8&msa=0&msid=209408201867918462117.0004dfe5344545b9ace8f&dg=feature it exists though [17:10] davmor2: lol I didnt doubt you.. [17:11] * davmor2 files a bug that the clock can't find wyre piddle [17:12] * nik90 is amused [17:15] davmor2, trust you to break it ;) [17:15] ahayzen_: it's what I do [17:16] davmor2: btw I always figured I can challenge you to find bugs in clock reboot...the above is not valid :D [17:16] ahayzen_: there is upper and lower piddle too but those don't even show in google maps :D [17:16] nik90: hahahahaha [17:16] davmor2: and already reported bugs don't countr [17:16] count* [17:16] \o/ [17:17] davmor2: so you see these recent blockers are a ploy to give me more time to fix stuff [17:17] there is no Gyroscope sensor yet, right? [17:20] I just noticed that I've made a typo on Sensors Status, and using an Accelerometer in the Gyroscope page, shame on me; but now that I'm using a proper Gyroscope class, it doesn't seem connected to backend [17:20] (On Nexus 4) [17:21] ttps://wiki.ubuntu.com/Touch/SdkFeatureStatus not listed at all, bad news [17:22] ybon: one for tvoss i think. [17:24] k :) [17:25] popey, do you know if jouni has access to real devices as the music-app has changed quite a bit to the official designs? [17:26] popey, as in would it be worth sending screenshots to him of the current app and maybe the current bottom edge prototype as 'food for thought' [17:27] ahayzen_: he has one, yes [17:27] popey, ok so he'll likely be on devel with it? so not far behind :) but won't know of our bottom edge ideas hehe [17:28] yeah [17:30] popey, content-hub nearly worked lol saw all the models refresh but something bad happened [17:30] gah, will you have time to debug that? [17:31] popey, gonna try and get it working tonight :) [17:31] \o/ [17:31] popey, it needed a merge of trunk anyway :P [17:32] popey, if i ping u a click if i get it working will u be able to test it? [17:34] ahayzen_: sure [17:34] ahayzen_: do you use dropbox? [17:34] popey, thanks :) ...damn internet keeps dropping out [17:35] popey, no :/ i use gdrive or webdav [17:35] On Tabs, how do I make that the default Tab is not the #1? [17:35] k [17:35] np [17:36] popey, any particular reason? ;) [17:39] no, just that nik90 has shared a clock folder with me, when he drops a click in, i spot it, get notified === gatox_lunch is now known as gatox [17:40] popey, oh cool [17:40] * ahayzen_ thinks u can do the same with gdrive...u can share direct with people and 'notify by email' [17:41] whatever it easiest ☻ [17:41] popey, something to try later ;) [17:47] popey, got it to play something \o/ just gotta tidy up a few things and check a few dialogues are working...but i'm going out to play badminton then have dinner so will bbl to finish it o/ [17:47] popey, good news is that no more tweaks are required by any upstream peeps [17:57] kk [18:22] nik90: http://popey.mooo.com/screenshots/device-2014-08-15-192225.png [18:23] nik90: looks like you're using admin1 not admin2? [18:23] popey: no I use them conditionally. If admin1 is available I use that, if not I use admin2 and vice versa [18:23] ah [18:23] popey: there is no space for both of them, [18:23] as you see, not ideal. [18:24] yeah [18:24] I will talk to giorgio on Monday to see if he is okay with 3 rows [18:25] http://popey.mooo.com/screenshots/device-2014-08-15-192502.png [18:25] ^ weather [18:25] although that's got the admin1 and admin2 round the wrong way really [18:25] locations are hard, lets go shopping! [18:32] hehe [18:32] I will give that a try [18:32] but I will have to truncate the text in the main clock page [18:38] hey, I want to load a list of items in the background [18:38] but while it's loading I'd like to display a spinner [18:38] I think States are what I can use to achieve that [18:38] is there an example in any of the core apps that I can look at [18:38] I have all the loading working, signals and all of that [18:39] it's just the state manipulation + transitions + Page component that holds all of this that I'm not sure about [18:56] kenvandine: can you help http://askubuntu.com/questions/511645/error-while-exporting-picture-through-content-hub [18:59] mhall119, sure [19:03] mhall119, answered [19:04] thanks kenvandine [19:05] np [19:08] zyga: rather than the core apps, I think best resource would be http://qt-project.org/doc/qt-5/qtquick-statesanimations-states.html [19:09] thanks [19:09] I looked at that too [19:09] ok, I'll give it a go [19:09] nik90: what I'm after is some sensible transition [19:09] nik90: (I got a "bink" transition working) [19:09] nik90: something like one fades out and then the other fades in [19:10] zyga: oh like that [19:10] something that feels nice [19:10] zyga: I can make a sample app to help with the transition effect. [19:11] zyga: that requires design mockup first though [19:11] nik90: if you have time then sure, if you can just point me at something I'm okay with that too :) [19:11] nik90: well, we have a design but we don't have transitions specified [19:12] zyga: afaik the core apps don't have state animations..we do have animation but they more behaviour animations. [19:12] nik90: it's just a list of things to pick but there's a latency to initially load the list itself [19:12] ahh [19:12] I know what you meant by design now [19:12] did you mean that there are no "stock" designs for that that can be just used [19:12] ? [19:12] yes, you can implement this in a variety of ways [19:13] zyga: best way would be to have a state that gets triggered when: loaded, and then do a property change of a UI element to change its opacity from 0 to 1 [19:13] * zyga tries [19:14] zyga: you can then add a Behavior on Opacity { UbuntuNumberAnimation{} } which fades the element in and out gracefully [19:17] thanks, I'll give that a try [19:56] popey: found a smarter way to fix the issue :) [19:56] popey: in the place where you search for cities, I show all the info (name, admin1, admin2, country) to avoid showing duplicate results. [19:57] popey: in the main clock page however I only show name, admin2, country [20:10] nik90, i see the indicator-datetime bugs we were talking about earlier made the blocker list :/ [20:11] ahayzen__: yeah :/ [20:11] nik90, slightly better news is i think i've got content-hub working for music-app :) [20:11] ahayzen__: wow that's awesome [20:11] ahayzen__: for importing music? [20:12] nik90, just got one weird thing happening where u correctly go to the now playing then u incorrect get jumped back to the start [20:12] nik90, yeah :) [20:12] nik90, eg download a mp3 from the web browser and import then plays in music app :) [20:13] ahayzen__: does that mean the scopes like grooveshark will play in the music app now? or that is restricted by them? [20:13] nik90, err i don't think so we are just a destination for when u get the 'Open With' dialogue [20:13] nik90, for music [20:14] ahayzen__: ah like when download music from the browser. cool [20:14] yep [20:15] popey: btw I updated the clock reboot in the store. pls approve when you have time. It has the custom alarm sound and the online world city search MPs as well. [20:15] nik90, would export support be any use to u after rtm? [20:15] ahayzen__: yes [20:15] ahayzen__: for choosing a custom ringtone perhaps? [20:15] nik90, eg being able to pick a mp3 for an alarm or something [20:15] yup exactly [20:15] nik90, :) [20:16] nik90, we have a bug for export but currently we are looking at doing after rtm [20:16] nik90, just trying to think of use cases for it [20:16] ahayzen__: yeah I need to figure how to import the music from the music app :) but post-rtm it is [20:16] nik90, yeah we'll be able todo that [20:17] ahayzen__: hey would you have time in the weekend to 1-2 reviews? [20:17] nik90, maybe maybe not....i probably have a bit of time tonight inbetween writing ap tests [20:18] nik90, add me as a reviewer and i'll do them if i have time :) [20:18] ahayzen__: ok [20:25] nik90, ur asking me to review c++ code? ;) [20:25] ahayzen__: that's 6-10 lines of c++ code...sure :P [20:25] * zyga can help :-) [20:26] nik90, hah but i haven't done my c/c++ module yet ;) hehe ... that starts in september \o/ [20:26] nik90, no i've done C before it should be fine :) as u said only 6-10lines [20:27] zyga: its all QML except for 8 lines of C++ code :D. Have at it if you want https://code.launchpad.net/~nik90/ubuntu-clock-app/json-search-ui/+merge/230339 === nik90 is now known as nik90|Dinner [20:29] * zyga is better with C than QML [20:32] the C++ parts are a bit verbose [20:50] zyga: ah thnx. I will fix the inline comments [20:54] nik90|Dinner: :-) [20:54] nik90|Dinner: my pleasure [20:55] nik90|Dinner: I'm not quite sure what's the Qt version of asnpritf is, what's the "pure" way to do string formatting === nik90|Dinner is now known as nik90 [20:55] nik90: I guess http://qt-project.org/doc/qt-5/qstring.html#arg [20:56] nik90: so you can replace all of those + + with arg for easier reading [20:56] zyga: ah yes [20:56] though I suspect that proper, trully-type-safe version is not in until c+11 can be used [20:57] but whatever [20:57] Qt was always on the borderline :) [20:57] :D [21:02] nik90: will do [21:03] nik90: approved [21:03] popey: thnx [21:04] zyga: I couldn't do if(admin1) since it says cannot convert QString to bool. I guess I have to to do admin1.isEmpty() [21:04] nik90: yeah, I'm not a Qt developer so that code was more of a "guide" than a copy-paste solution [21:05] zyga: I was hoping Qt C++ would support.. I guess not [21:05] zyga: btw I don't check for country, since it was always available...should I still check to make sure? [21:06] nik90: well, it depends on what you want [21:06] nik90: what does asString() do when the json has some other type/value [21:07] nik90: does it return empty strings or raises exceptions? [21:07] brb [21:08] zyga: asString() ? [21:08] zyga: well I explicit do the conversion toString(), if it can't I suppose it will raise an error [21:09] zyga: actually if QVariant cannot return to QString() it return an empty string according to the documentation [21:10] zyga: which if happens then I will "admin1, admin2, " [21:10] where country is blank [21:29] nik90, howdy :-) I just had a cheeky peek at your jsontimezonemodel. One thing that's comes to my mind, are you always *guaranteed* to get a reply when requesting the url? As you have no handling there in the case of a QNetworkReply::NetworkError [21:30] ogra_: hey, BlueBox doesn't seem to start any more [21:30] DanChapman: It comes from http://geoname-lookup.ubuntu.com/ which can go down. So yes I suppose I should add a handling for NetworkError. [21:33] nik90: oops. sorry.. I missed that in the review [21:34] nik90 cool :-), thought i should double check. [21:34] mzanetti: ah don't worry. I suppose I will have to have another variable called error that I track in qml? If(error) display message [21:34] DanChapman: thnx for the catch :-) [21:34] nik90: no... you can check the status in the finished slot [21:35] ah right... yes... [21:35] I guess a propoerty status should do [21:35] mzanetti: similar to the loading property we have [21:35] which you change to Loading, Ready, Error [21:35] yeah, just make it an enum instead of a bool [21:35] so you can easily bind something to it... [21:36] e.g. Label { text: "D'oh"; visible: model.status == TimezoneModel.Error } [21:36] mzanetti: ooh that will be cool [21:37] and ActivityIndicator { visible: model.status == TimezoneModel.Loading } [21:37] * DanChapman is a fan of enums in qml [21:37] mzanetti: I will have to figure out how to use the enum in the c++ code [21:38] nik90, you just have to register it with Q_ENUMS and i presume you have already registered the TimeZOnemodel as a qml type [21:38] DanChapman: yeah I do..well the JsonTimeZoneModel atleast. [21:39] TimeZoneModel is a base class we derive from [21:39] nik90: I suggest you add the enum in your base timezonemodel as in theory you can use the error state in all of them [21:40] mzanetti: ok [21:40] actually the whole property should be in the base class, so you don't have to duplicate that code in all models [21:41] mzanetti: yes, I will Q_ENUMS as a property in the base class and define enums {loading, ready, error} in proctected so I can use it in the derived classes [21:41] like this http://qt-project.org/doc/qt-5/qobject.html#Q_ENUMS I suppose [21:42] nik90: what I meant is to also have the Q_PROPERTY(Status ...) in the base timeZonemodel [21:42] mzanetti: ah, will do [21:42] and a protected function setStatus(Status) in there which sets the property and does the emit [21:42] then you can just do setStatus(blabla) in the subclasses and that's it [21:43] mzanetti: that should be Q_ENUMS(Status) -> enum Status {Loading, Error, Ready} ? [21:43] yep [21:43] ok [21:44] I will have a separate MP for this since it touches all the timezone classes. Dont want to break everything here :) [22:36] mzanetti, none of the webkit apps start anymore ... there is a bug open about it === salem_ is now known as _salem [23:44] popey: http://imgur.com/a/gQ3K9 #teasing [23:44] (but now one week of boat \o/)