/srv/irclogs.ubuntu.com/2014/08/11/#ubuntu-app-devel.txt

=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== shuduo is now known as shuduo_afk
mihirGood Morning :)06:51
dholbachgood morning07:19
justCarakasgood morning :)07:21
justCarakaswhere can I help for the dutch translation ?07:23
justCarakasO was running dutch on my phone but I didn't even notice because there was so much english07:23
dholbachhttps://translations.launchpad.net/ubuntu-touch-preview07:27
dholbachhttps://translations.launchpad.net/ubuntu-phone-coreapps07:27
justCarakasthx dholbach07:28
justCarakasI'm getting a lot of timeout errors on launchpad07:51
=== chriadam is now known as chriadam|away
justCarakasdholbach: the dash on the phone shows up as an app in the app overview but unlike the other apps you can't close it whit swiping up, but you also don't get a notification about it, found it rather confusing07:58
justCarakaswhere should I report that ?07:58
dholbachon unity8 perhaps?07:58
ogra_i think thats the wanted design08:01
justCarakasok did that :)08:01
ogra_(i agree it is confusing at first)08:01
justCarakasI can understand that that is the wanted design08:01
justCarakasbut yea, you should than get a notification about it at least08:02
justCarakasmaybe even prevent the movenment08:02
ogra_yes, thats what i would do08:03
ogra_make it immovable to confuse less08:04
justCarakasindeed08:04
justCarakasmaybe even both08:04
justCarakasimmovable and a small notification that you can't close it08:04
JamesTaitGood morning all; happy Monday, and happy Ingersoll Day! :-D08:34
mzanettidholbach: fixed08:52
dholbachmzanetti, approved08:53
mzanettidholbach: cheers08:53
dholbachrock on!08:54
mzanettit1mp: hey, ToobarItems.back doesn't seem to work for me... Is that a known bug? I tried to override the back icon just like the exampe in the docs, didn't change a thing08:55
t1mpmzanetti: do you have the url for those docs at hand?09:03
t1mpmzanetti: are you sure you want to be implementing a toolbar?09:03
mzanettit1mp: I pressed F1 on ToolbarItems in qtc09:03
mzanettit1mp: ToolbarItems has (according to the docs if you press F1) a "back" property which points to the back button09:04
nik90popey: ping09:05
mzanettit1mp: the doc example overrides it with a cancel button which is pretty much what I wanted to do09:05
mzanettit1mp: but whatever I did with that "back" property, it wouldn't matter09:05
t1mpmzanetti: the example works for me, see https://www.dropbox.com/s/49phikdl26yrkko/Screenshot%202014-08-11%2011.05.05.png09:05
popeynik90: pong09:05
t1mpmzanetti: it is not aligned pretty, and it doesn't look pretty to put a "normal" button in a toolbar (and toolbars are deprecated), but it does what it should09:06
mzanettit1mp: hmm... I see09:07
mzanettit1mp: useDeprecatedToolbar: false :)09:07
nik90popey: hey did you get an update from ev about the geonames situation?09:07
nik90popey: I am working now on implementing the feature in the new clock app and need the data to be extended to include timezoneid as mentioned in the bug report09:07
t1mpmzanetti: do you want to get a toolbar or header?09:07
mzanettit1mp: well, header, with actions, and I would like to override the back action09:08
t1mpmzanetti: with the header, I recommend to use the new API that is here http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.PageHeadConfiguration/09:08
t1mpmzanetti: if you don't want to switch to that yet and use the old (ToolbarItems API) with the new header,09:08
t1mpmzanetti: then define the back button using an action. So back: Button { action: Action { text: "cancel" }}09:09
popeynik90: no, i need to look at that again today09:10
t1mpmzanetti: code + screenshot: https://www.dropbox.com/s/jtg8l4375aa6ufp/Screenshot%202014-08-11%2011.10.02.png09:10
mzanettit1mp: hmm... ok... I'll try again... I wouldn't mind using the new api, but I didn't succeed with that either... but I need to try more to be certain what's wrong09:10
nik90popey: ok09:10
mzanettit1mp: I'm very sure I tried this... but ok... seems its working. thanks a lot09:11
mzanettit1mp: btw, new fahrplan app now seems to support netherlands if you're still interested (I remember we talked about this like a year ago :D)09:11
t1mpmzanetti: new API is like this: https://www.dropbox.com/s/q9kfsbdo49cjq8r/Screenshot%202014-08-11%2011.11.32.png09:11
t1mpmzanetti: I just move to spain 4 days ago :)09:12
mzanettiah ok :)09:12
t1mpmzanetti: I'll check it out when I'm in Netherlands :) if I remember09:12
mzanettit1mp: oh, that's how you use the new one! that looks good, thanks! will move over to the really new one then09:12
nik90mzanetti: ooh thnx for that. I can use it :D09:12
mzanettinik90: :)09:12
nik90mzanetti: how the hell did you get the API for 9292ov.nl. I have been looking for a long time09:12
t1mpmzanetti: sure. Let me know if you get stuck with it, I should be able to help09:13
mzanettit1mp: well, now that I know it *is* working and have a code example I'm sure I'll find my way. thanks again.09:13
mzanettinik90: wasn't me... I just do the ubuntu ui09:13
nik90mzanetti: ah ok09:14
mzanettinik90: this app has been rated as one of the top 10 BBX apps of 2013 :D09:14
mzanettiwhile in ubuntu it has one rating with 1 star...09:14
* mzanetti wonders if he messed up so hard with the ui :D09:14
nik90mzanetti: wow it returns the results that I expect :) This is awesome09:15
popeynik90: you can see the code yourself, it's at https://launchpad.net/ubuntu-geonames - there's a script there which sets it up09:15
nik90mzanetti: where do I report bugs?09:15
t1mpmzanetti: perhaps ubuntu users are more critical, and 1 star is super good ;)09:15
nik90popey: sweet. I will look into it09:15
mzanettinik90: https://github.com/smurfy/fahrplan09:17
mzanettinik90: if you report bugs related to the ubuntu ui, please ping me too. I'm not really watching that github project all the time09:20
nik90mzanetti: will do. I would like to contribute too :)09:21
mzanettinik90: ah ok, sure... that guy is really open to contributions.09:21
mzanettinik90: one thing that would be great:09:21
nik90mzanetti: is it easy to get it and build it?09:21
mzanettinik90: the backend code supports location stuff. so it autoselects stations according to location. I didn't add that in the ubuntu ui yet09:22
mzanettinik90: just clone it, open it in qtc and run it (desktop only)09:22
mzanettinik90: for the phone there's a run_on_ubuntu_touch.sh script09:22
mzanettinik90: to create a click package use run_on_ubuntu_touch.sh -c09:22
nik90mzanetti: ah ok09:22
mzanettinik90: but finish off tht clock app first... RTM is close ;)09:23
nik90mzanetti: hehe ofc :)09:23
nik90mzanetti: only 1 feature away from completion which is a wip09:24
mzanettinik90: and then I'm going to test it - muahahaha :D09:24
nik90lol09:24
mzanettinik90: reminds me... started to use the alarm now in production mode... turns out it doesn't support snoozing09:25
mzanettiwill that come with the reboot?09:25
nik90mzanetti: yup, its being worked by charles kerr. It will land in indicator-datetime and then come in the clock reboot09:25
nik90mzanetti: ETA ~1-2 weeks09:25
mzanettiawesome09:25
=== chihchun is now known as chihchun_afk
nik90anyone up for reviewing https://code.launchpad.net/~nik90/ubuntu-clock-app/add-multiselection-mode/+merge/230140, it adds a nice multiselect mode for alarms :D09:54
ahayzenpopey, o/ this landed \o/ https://code.launchpad.net/~andrew-hayzen/music-app/listitem-actions-take-2/+merge/22953609:57
popey\o/09:57
popeytime for a store update? ☻09:57
ahayzenpopey, yep :)09:57
ahayzenpopey, we updated the .pot as well09:57
popeysweet, will set some time aside to update the app in a little bit09:58
ahayzenpopey, thanks09:58
popeyno, thank *you* !09:58
ahayzenpopey, now onto bottom edge and ap refactor :)09:58
ahayzenpopey, hah no problem just glad to get it merged after 2-3months lol09:58
popey09:59
ahayzenpopey, how to extend it in the future with multiselect etc but it has feature parity to what we had before so we merged :)09:59
ahayzen*going to extend09:59
ahayzenpopey, oh and FYI, as u weren't in the meeting last wk, the last thing we are waiting for before content-hub support lands is https://code.launchpad.net/~jamesh/mediascanner2/model-auto-update/+merge/22990310:01
ahayzenpopey, bug tracking here https://bugs.launchpad.net/music-app/+bug/134744410:01
ubot5Ubuntu bug 1347444 in Ubuntu Music App "Music app does not expose download handler" [High,In progress]10:02
popeythanks10:03
=== chihchun_afk is now known as chihchun
popeynik90: got a click? ☻10:57
nik90popey: not yet. one min10:57
mercury-hi everyone11:00
nik90popey: https://www.dropbox.com/s/bih4zlrun7f7yyc/com.ubuntu.clock.devel_0.1_all.click11:02
dholbachhey nik90, do you know if the package branch got anywhere?11:03
dholbachnik90, does it fully work for you now?11:03
nik90dholbach: I asked sergiusens to look at it on friday to finish any remaining bits. I haven't heard from him yet :/11:03
dholbachmaybe we should start pinging other folks11:04
dholbachmaybe Mr Zanetti can help11:04
dholbachor anyone else11:04
nik90dholbach: I don't know who else is familiar with cmake11:04
dholbachis https://bugs.launchpad.net/ubuntu-clock-app/+bugs?field.tag=reboot a good reflection of what still doesn't work with the branch?11:04
popeynik90: thanks11:04
nik90dholbach: I think it is better to ask someone to take your branch and then fix the necessary stuff to get it to build a deb and click successfully.11:05
nik90dholbach: deb part is complete. I guess so only the cmake stuff needs fixing to get the click packaging done. I can report bugs about it not finding the backend when running a device11:06
nik90on device*11:06
popeynik90: love the bounce when you choose alarms to delete11:06
dholbachnik90, that would be good11:06
dholbachnik90, I updated https://bugs.launchpad.net/ubuntu-clock-app/+bug/135437711:06
ubot5Ubuntu bug 1354377 in Ubuntu Clock App "Unbreak 'make autopilot'" [Medium,Confirmed]11:06
popeywe need a BOOIINNNNGGGG sound effect11:07
dholbachballoons, ^ do you think you can help find somebody who can try to fix the bug?11:07
nik90popey: lol11:07
popeynik90: it doesn't actually delete them when I select and then tap delete11:07
dholbachpopey, it'd be good if we could find somebody else who can fix the remainder of https://bugs.launchpad.net/ubuntu-clock-app/+bugs?field.tag=reboot to make https://code.launchpad.net/~dholbach/ubuntu-clock-app/reboot-packaging work11:07
dholbachpopey, I think I did all I could so far11:07
nik90popey: wat, it doesn't delete? let me try again on my end11:08
popeyk11:08
popeyoh, i lied, it probably does but the app locked up11:08
popeyprobably because I have another clock reboot app open11:08
popeyyeah, it did, no alarms left11:09
* nik90 sighs a relief11:09
nik90popey: btw your bug about alarms showing 00:00 has been approved. Will start on it soon11:10
nik90popey: I might need to report a bug about showing an activity indication when an alarm operation is being done.11:11
nik90dholbach: btw balloons suggested some changes to the ap-setup for trunk. Could you check if it will merge into your branch as well without much conflicts.11:14
nik90dholbach: I can first merge to trunk which you can then update to get it automatically if you want11:15
dholbachnik90, let me check11:19
dholbachI'll merge it11:20
nik90ok, it lgtm. Shall I approve and merge to trunk first?11:21
davmor2popey: goto blue fin via big ben at midday :)11:22
popeywat wat?11:22
davmor2you'll be able to get a nice recording of the boooooiiiinnnnngggggggssss11:23
nik90popey: can you approve (device testing) https://code.launchpad.net/~nik90/ubuntu-clock-app/add-multiselection-mode/+merge/230140 and https://code.launchpad.net/~nik90/ubuntu-clock-app/add-vibration-support/+merge/22932711:24
nik90popey: you already tested those branches11:24
nik90I will find someone to do the qml code review and merge it when top approved11:24
davmor2popey: like this http://www.youtube.com/watch?v=juT1zsim6es11:24
popeynik90: done11:25
nik90popey: thnx11:25
davmor2popey: is that the kinda boing you were thinking of11:26
popeyno11:26
popeythats bong, not boing11:26
popeyhttps://www.youtube.com/watch?v=Tnh8IGwvL9U11:27
davmor2popey: there is bound to be a boing in here you can use11:30
davmor2http://www.youtube.com/watch?v=PMpu8jH1LE811:30
davmor2popey: oh for that boing you just need a mouth harp11:30
nik90popey: if you want I can release a special clock reboot click which does that for you :P11:31
popeyhaha11:31
popeyno.11:31
popey11:31
popeyThank you.11:31
mihirpopey: thanks for bug prioritize :)11:36
seb128is there a way to make a ListItem element use an UbuntuShaped icon?11:37
seb128rather than a square one11:37
ahayzenseb128, do you mean like the music app does?11:40
seb128ahayzen, not sure, I've not been trying the music application11:40
nik90seb128: are you referrring to the icon in a listitem to use the ubuntushape?11:40
seb128nik90, yes11:40
nik90seb128: doesn't it use the ubuntushape automatically?11:41
ahayzenah yes11:41
nik90seb128: either way there is a property called "iconFrame". Set that to true if you want the ubuntushape border around the icon11:41
seb128nik90, thanks11:42
seb128that does it11:42
nik90seb128: yw11:42
seb128if that property is false then the icons are squared11:43
nik90yes indeed11:43
davmor2nik90: you can't reboot popey ;)11:44
nik90davmor2: wat nonsense...you didnt try with sudo :P11:44
davmor2nik90: hahahaha11:45
dholbachnik90, I merged it - bug http://91.189.93.70:8080/job/generic-mediumtests-utopic/1466/console doesn't look like it's fixed by merging the branch yet11:53
nik90dholbach: ack. I expected it just remove some hacks for the OSK and improve local test runs.11:54
dholbachok11:54
dholbachnik90, unrelated, but when I run "make autopilot" in the autopkgtest, I get this: http://paste.ubuntu.com/8016541/11:56
nik90dholbach: on utopic?11:56
dholbachyes11:56
dholbachshall I file a bug for it?11:56
nik90dholbach: sure, but I think it is a autopilot related error and not due to the clock app11:57
dholbachok, should be good to get an autopilot expert on it11:57
nik90yes11:57
nik90we use dbus in our Alarm.Settings plugin, however that doesn't even get loaded in those tests.11:58
dholbachI have no idea11:58
nik90report the bug and I will check with elopio and balloons about it11:59
dholbachdone12:00
=== _salem is now known as salem_
nik90mzanetti: quick question. Is there a way I can detect when the jsonModel request is fully received? My use case is to show a activity indicator when the data is being gatthered and hide it when it is finished receiving everything.12:40
nik90mzanetti: should I emit a signal in the networkReplyFinished() function?12:40
mzanettinik90: yeah, the replyFinished is the one that tells you when its finished12:42
mzanettinik90: but I'd do this:12:42
mzanettinik90: add a Q_PROPERTY(bool loading ...)12:42
mzanettinik90: so you don't have to deal with signals in the ui12:43
mzanettinik90: but just can do activityIndicator.running: backend.loading12:43
nik90mzanetti: ah ok. and set loading as false in the replyFinished part.12:43
nik90mzanetti: sweetr12:43
nik90mzanetti: thnx12:43
mzanettinik90: yeah, you need to do a emit loadingChanged() in order to tell QML that the property has changed12:43
mzanettinik90: but yeah, inside QML just use the property, not the signal12:44
nik90mzanetti: yeah as usual .12:44
mzanettiyep12:44
ahayzenpopey, have u got a moment just to try an early prototype? no worries if u haven't12:59
popeyahayzen: does it need rw phone?12:59
ahayzenpopey, just a click for music app bottom edge support12:59
popeyoh sweet, sure13:00
ahayzenpopey, https://docs.google.com/file/d/0B3XynHVKfrvMM1B3OXlWQ0wxdHc/edit13:00
ahayzenpopey, i've only just moved it from experimental to development so it has some rough edges but u'll get the idea :)13:00
* ahayzen hopes it works13:00
popeylets see...13:00
popeyahayzen: thats quite neat!13:02
ahayzen\o/13:02
ahayzenpopey, a found bounds a mouse areas need tweaking but it is getting there13:03
* ahayzen hopes victor approves of his hacks to reparent the same page13:03
ahayzenpopey, yey https://code.launchpad.net/~andrew-hayzen/music-app/bottom-edge-implementation-001/+merge/230303 \o/ we'll see what jenkins thinks of that13:30
=== chihchun is now known as chihchun_afk
popeydholbach: can we add an option to click-reviewers-tools to only spit out warnings/errors?14:20
zsombipopey: ping14:40
zsombipopey: this seems to be an ancient bug, and I guess it is not valid anymore... https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/120687014:40
ubot5Ubuntu bug 1206870 in Ubuntu UI Toolkit "OSK doesn't automatically hide when fields no longer on screen" [Undecided,New]14:40
dholbachpopey, yes, it should be easy to write a script which does that14:42
dholbachmaybe you can file bug report?14:43
popeyzsombi: will take a look14:52
popeydholbach: already done ㋛14:52
popeydholbach: https://bugs.launchpad.net/click-reviewers-tools/+bug/135521514:52
ubot5Ubuntu bug 1355215 in Canonical Click Reviewers tools "Should have an option to only show errors/warnings" [Undecided,New]14:52
dholbachgreat14:53
dholbachballoons, do you think some QA hero could take lp:~dholbach/ubuntu-clock-app/reboot-packaging and make autopilot bits, autopkgtest and everything related to that work?15:20
dholbachballoons, I did as much as I could, but I just don't know how to proceed15:20
dholbachhttps://bugs.launchpad.net/ubuntu-clock-app/+bugs?field.tag=reboot being the list of related bugs I know of15:21
balloonshey dholbach :-) I actually did an MP that nik90 was going to review last week to make launching work a bit saner. I believe it landed15:21
balloonsdholbach, making your branch ubuntu-clock-dev might be helpful and I contrib straight to that branch15:21
nik90balloons: he tried it with your branch as well. I told dholbach about it15:21
balloonsbut yea, definitely can help15:22
dholbachballoons, bug 1354377 and bug 135514515:24
ubot5bug 1354377 in Ubuntu Clock App "Unbreak 'make autopilot'" [Medium,Confirmed] https://launchpad.net/bugs/135437715:24
ubot5bug 1355145 in Ubuntu Clock App ""make autopilot" fails in autopkgtest" [Undecided,New] https://launchpad.net/bugs/135514515:24
dholbachballoons, I merged your branch15:25
dholbachballoons, ... with a very small modification (using 1355145() in tests/autopilot/ubuntu_clock_app/tests/__init__.py)15:25
dholbachsorry, ... using find_local_path() in tests/autopilot/ubuntu_clock_app/tests/__init__.py15:26
balloonsdholbach, yea, I saw that and wasn't sure how I felt about it.. it's an interesting issue with local15:26
dholbachballoons, the problem is that "builddir" is not what is used during the deb package build15:27
balloonsdholbach, yea, we should talk about it after our call if you have a moment15:27
dholbachsure15:27
balloonsnik90, do you plan on merging down stuff or you wanting to land dholbach's mp first?15:30
nik90balloons: I was thinking of merging your branch since daniel already merged your branch into his15:31
dholbachI was thinking that it'd be nice if stuff was working and then could be merged into trunk15:31
dholbachbut I'm not sure - maybe it'll help to get stuff fixed if it's trunk already :)15:31
nik90dholbach: so what's the verdict? Should I wait or merge?15:31
nik90:)15:31
dholbachnik90, it's your project :)15:32
balloonsI don't see any harm in pushing dholbach's change before merging15:32
nik90dholbach: well I don't want your life even more harder by changing trunk so frequently (atleast the parts that afffect you)15:32
nik90balloons: I cannot merge dholbach's branch into trunk yet since that breaks the click packaging. I would no longer be able to test on device.15:33
dholbachnik90, can you file a bug tagged with 'reboot' about how it breaks click packaging?15:34
balloonsnik90, ahh.. this is something we want to fix before merging then eh?15:34
balloonsI'm saying while we have dholbach helping out it makes sense to square everything away15:34
nik90balloons: definitely. I am still implementing some features and testing on device is crucial for me15:34
nik90dholbach: sure, let me test again15:34
dholbachcool15:34
nik90dholbach, balloons: Can we quickly make a decision on balloons's ap-setup branch. Wait or push?15:35
dholbachnik90, sure - feel free to merge15:35
nik90dholbach: ok15:35
nik90balloons: merged. Can you ping me later in the evening when you are free? I want to merge 2 other branches I have.15:49
dholbachthank god we have the unstoppable balloons!15:59
balloonshaha.. nik90 dholbach is going to finish his magic and I think we'll be good to go15:59
* nik90 is puzzled16:00
nik90what magic happened?16:00
balloonsdholbach, btw, those hardcoded paths were supposed to die, ohh I don't know, last July? :-)16:01
dholbachballoons, they weren't my idea16:01
dholbach:)16:01
balloonsso ideas for more elegant ways to do this are welcome from you16:01
balloonswell, the idea is we intended to upstream this complexity, but it's never really fit for various reasons16:02
balloonswe have cmake and autopkgtest now.. in theory it's more doable to change the setup16:02
balloonsand nik90 is used to being the guinea pig for these experiments16:02
balloons:-016:02
dholbachballoons, added my work-flow to https://bugs.launchpad.net/ubuntu-clock-app/+bug/135437716:03
ubot5Ubuntu bug 1354377 in Ubuntu Clock App "Unbreak 'make autopilot'" [Medium,Confirmed]16:03
nik90balloons: I am kinda out of sync with what's happening :P16:03
nik90but yes :D16:03
dholbachnik90, ^ too - if you should ever want to test autopkgtest, etc16:03
balloonsdholbach, ahh you are testing it still as a deb.. interesting16:03
nik90dholbach: oh new way to run ap test.16:04
dholbachballoons, yeah, generate a source package from the branch, then feed it into autopkgtest16:04
balloonsdholbach, so pitti added support for running clicks directly.. Let me also comment on the bug16:04
dholbachballoons, hum, this is not in the click mode16:04
balloonsdholbach, right, but again, pretend this wasn't debian packaged.. we can still run it16:04
balloonswhich is something we couldn't do before.. and it's why we don't need to package tests anymore16:05
dholbachballoons, sure, it'll exercise a second path of the code16:05
* balloons checks his workflow, well, works16:08
balloonsso anyways, let's look at the output of http://91.189.93.70:8080/job/generic-mediumtests-utopic/1466/16:12
balloonsso the -autopilot package looks good.. sticks stuff into /usr/lib/python3/dist-packages/ubuntu_clock_app/16:13
balloonsbah-hah.. so dholbach you have been foiled by jenkins.. Notice it's running the tests as python216:15
balloonsthat's the only issue methinks16:15
dholbachoh?16:15
=== gatox is now known as gatox_lunch
dholbachballoons, how and where do I fix it? :)16:17
balloonsdholbach, well from the jenkins side, that's for CI to run.. but I need to check something16:20
balloonsdholbach, just for kicks, is it possible for you to package python2 tests as well?16:22
dholbachballoons, where do we have python2 tests in the branch?16:22
balloonsassuming they are compatible, etc, etc16:22
balloonsaka, the same tests16:22
dholbachballoons, how would they be different then?16:24
dholbachyou mean, there should be one package     ubuntu-clock-app-autopilot-python2      shipping files in /usr/lib/python2...... and another one for python3?16:25
balloonsdholbach, they wouldn't.. I was just going to check on the other apps to see how the packaging was going16:25
balloonsdholbach, yes.. jenkins would work as-is if we put them in /usr/lib/python216:25
balloonsmaybe not even 2 packages.. anyways I need to check16:25
dholbachnotes-app-autopilot uses ./usr/lib/python3/dist-packages/notes_app/......16:25
balloonsyes, dholbach technically jenkins should be updated to run as py316:26
balloonsthey all should be running as py3, but I see many are not16:26
dholbachhum16:26
dholbachmaybe we should ask somebody from the the team16:26
balloonsyea, seems like the packaging is also python2 on the other core apps16:27
balloonsanyways, yes the solution is to have CI update the jenkins job16:27
dholbachok, I see16:27
dholbachall right - I think with that I call it a day now :)16:28
dholbachthanks for helping getting this fixed!16:28
balloonsdholbach, for sure.. thanks. I think everything is good. I'll fix jenkins16:28
dholbachrock and roll :-D16:28
dholbachnik90, do you think you can still file this other bug with your click work-flow?16:29
dholbachnik90, maybe with instructions for how you use it16:29
nik90dholbach: yeah, sry was triaging bugs16:29
dholbachawesome, thanks!16:29
dholbachhave a great rest of your day - see you tomorrow! :-)16:29
nik90dholbach: cu tomorrow :)16:30
nik90mzanetti: hey finished implementing https://code.launchpad.net/~nik90/ubuntu-clock-app/json-search-ui/+merge/230339 based on your whereyourissisat. The bool loading property did the trick with showing the activity indicator.16:36
nik90mzanetti: try searching for cities to see if you get results16:36
=== bfiller is now known as bfiller_afk
balloonsfginther, can you switch clock-app jenkins merges to rn as python3 instead of python2?16:51
nik90balloons: are we sure the old clock app tests pass with python3?16:52
nik90balloons: or is it possible to only change to python3 for the reboot branch alone?16:53
balloonsnik90, that's what I was going to ask you.. if the old clock branch will receive any development or not16:55
nik90balloons: not by me atleast..I have been marking certain bugs as Won't fix for the old clock app simply because we fixed in the new clock app and cannot backport due to lack of time16:56
nik90balloons: so to me, old clock app is frozen. I will need to talk to popey about it in my meeting this week16:56
balloonsas far as the tests go they do run py316:57
nik90ok16:57
ahayzenballoons, o/16:59
balloonsahayzen, allo16:59
ahayzenballoons, FYI listitem actions landed!...so now may be the time to start the ap refactor17:00
ahayzenballoons, the only 'big' landing i see is the bottom edge stuff i'm working on...but that has only broken 3 tests (which i think are all due to it not waiting for animations to complete)17:00
ahayzenballoons, how would i go about starting this rewrite?17:01
balloonsahayzen, yes I saw the bottom edge bit17:04
ahayzen:) bottom edge is fun17:04
balloonsahayzen, I would take a Page at a time and redo the tests17:04
ahayzenballoons, should i copy a structure from somewhere? eg clock tests?17:04
balloonspull them out of test_music.py and make a seperate file. Then make a helper class inside emulator.py. I would also rename emulators to helpers.17:05
ahayzenballoons, i was gonna make it owned by ~music-app-devs so any of us could work on it17:05
balloonsthere's a few other cleanups you can do, but redoing the tests themselves is a good start17:05
balloonsahayzen, brillant17:05
balloonsahayzen, reminders is even newer than clock17:05
balloonsI suppose I would copy it17:05
ahayzenballoons, ok i'll look into doing that... shall i just convert one test and then send to u so u can tell me what i'm doing right/wrong ?17:05
=== chihchun_afk is now known as chihchun
balloonsahayzen, if you put it into a branch we can all touch, feel free to ping17:06
ahayzenballoons, awesome thanks :) ... once we have migrated the tests i wanna add more for url-handler as well ... do u know if we can do ap tests for content-hub?17:06
ahayzenballoons, ...and would we be able to emulate a blank library?17:07
ahayzenballoons, ^^ are the places the current tests don't cover17:07
balloonsahayzen, feel free to file some bugs so we know about those gaps if you haven't done so17:08
balloonsahayzen, reminders has the urldispatcher tests in it you might find interesting17:08
ahayzenballoons, we have some old bugs... but basically this is my list of references/things to do at the moment https://trello.com/c/2p7zUffL/278-music-refactor-ap-tests-to-use-python3-correct-objectname-test-var-names17:09
ahayzenballoons, awesome i'll check them out as well17:09
mihirballoons: ping!17:13
balloonsmihir, pong17:13
mihirballoons: this MR should be ready , but as per kunal last comment ,17:17
mihirhttps://mail.google.com/mail/u/0/#inbox/14721a12ea11cf5e17:17
=== gatox_lunch is now known as gatox
mihiri have que, why onExpandedChanged is not getting tirgged on OptionSelector17:17
mihirany ida?17:17
=== bfiller_afk is now known as bfiller
nik90mihir: you do realise we do not have access to your gmail account :D17:23
mihirnik90: sorry , forgot to paste the code, http://paste.ubuntu.com/8018891/17:24
mihirI am trying when openSelector get expanded hide the OSK17:25
fgintherballoons, can you provide a link to a job or build that needs to be changed? I just want to be certain on which clock tests to chane17:30
fgintherchange17:30
balloonsfginther, sure. http://91.189.93.70:8080/job/generic-mediumtests-utopic/1466/, http://91.189.93.70:8080/job/ubuntu-clock-dev-ubuntu-clock-app-utopic-3.0-ci/17:45
balloonsmihir, :-) so you think what you have in the MP should work as-is? I wasn't sure17:45
balloonsThe pickers work on the device again17:45
mihirballoons: okay , let me know when you have time we can test my MR on device.17:46
balloonsmihir, I can do it now.. let me try17:46
mihirballoons: okay, but i guess , onExpandedChanged is not working as expected17:47
mihirballoons: sorry got dc17:57
balloonsmihir, no worries18:04
mzanettinik90: heh, you missed part of my inline comment before :)18:14
mzanettinik90: technically it was 2 inline comments, although just one sentence18:15
nik90mzanetti: the inline comments feature is a bit confusing and goes away after I do a push. But luckily I didn't refresh the page, so I caught the second part of your inline comments :)18:17
mzanettinik90: there's a combobox above the diff where you can go back to previous versions and inline comments18:17
mzanettinik90: but I agree it's not totally awesome yet18:18
nik90mzanetti: oh I totally missed that..nice18:18
nik90balloons: got some time to spare?18:25
balloonsnik90, sure thing18:49
nik90balloons: can you top approve https://code.launchpad.net/~nik90/ubuntu-clock-app/add-vibration-support/+merge/229327 and https://code.launchpad.net/~nik90/ubuntu-clock-app/add-multiselection-mode/+merge/23014018:53
nik90balloons: they were tested by me and popey and device18:54
balloonsnik90, that was simple :-0 done18:55
nik90balloons: :-)18:56
popey\o/19:04
davmor2balloons: don't trust that nik90 I know him and popey but who the hell is this device bloke he is on about ;)19:10
nik90lol19:10
balloonsdavmor2, I stopped reading after the first line19:10
* balloons notes word wrapping 'and device' ftw19:10
* nik90 eyes suspiciously at davmor2 wondering if his account was hacked :P19:11
nik90impersonation is not tolerated..what happened to the davmor2 I met at malta?19:11
davmor2hahahahahahaha.19:13
davmor2nik90: I have to say nice work on the clock reboot by the way, looking really nice :)19:13
nik90davmor2: thnx ;-) I don't get to hear about your bug reports anymore :P19:14
nik90smooth flying under the radar19:14
davmor2nik90: no reflashing devices too often to keep non stock app on for any length of time.  Wait till it lands :D19:15
nik90davmor2: trust me by then I will have fixed all bugs I have in mind 😃19:16
davmor2that's ok I have a different mind, I'll file loads ;)19:17
nik90:P19:17
ybonI've created an armfh 14.10 kit, and it was showing up yesterday while hacking on my GPSStatus app, but it doesn't shows up when I try to run OSMTouch on my Nexus19:30
ybonI can't say I understand everything around kits19:30
ybonbut I don't remember doing something specific to link a kit to an app19:30
ybonoh19:30
ybonmaybe on the project page19:30
ybonyes, thanks you rubber duck :)19:31
ybonany good tutorial about using the new header actions?19:34
popeyheh19:35
ybonpopey: I was just jocking for answering the tweet ;)19:35
popey19:36
nik90ybon: the API docs are pretty clear actually19:36
nik90ybon: and as usual take a look at how core apps use it19:36
ybonyep :)19:36
ybonbut when tutorials exist, they are usually good :)19:37
ybonso I was wondering rather before working than after ;)19:37
nik90true19:37
* nik90 is too lazy for that :D19:37
absk007which is lightweight Geany or Bluefish? I need a lightweight programming text editor.19:44
nik90absk007: hard to say, I have only used Geany and seemed pretty ligthweight to the point where I opened that rather than gedit19:48
absk007nik90, thanx. Currenly installing.19:49
nik90yw19:49
absk007wow! Geany is fast like leafpad19:51
nik90yup ;)19:51
absk007nik90, any quick geany configs. to make it way faster?19:52
nik90ahayzen: afaik no. I tend to not do anything special and just use the tool. Helps with migration between systems19:53
mihirrpadovani: ping !!19:53
ybonwhere do I look to get the list of iconName values possible?19:53
mihirpopey: ping19:54
popeymihir: pong19:54
ybonok got it, in suru/actions/scallable19:54
mihirpopey: for calendar Sync Account as of now we should allow only Google, is that correct?19:55
popeyyes, we don't have account providers for anything else, why? What alternative did you have?19:55
popeyor is this for the "launch online accounts" bug?19:55
mihirpopey: nope was just confirming that, i am looking at bug# 134227519:56
mihirpopey: yeah..19:56
mihirpopey:  so should we just navigate users to Accounts , or to google accounts , i just need suggestion :19:56
mihirbug https://bugs.launchpad.net/ubuntu-calendar-app/+bug/134227519:57
ubot5Ubuntu bug 1342275 in Ubuntu Calendar App "Provide a quick way to setup a sync account" [Medium,Confirmed]19:57
popeyit should go direct to google19:57
mihirpopey: okay.20:00
absk007nik90, how to choose theme. I've installed monokai.conf20:09
mihirpopey: any idea where could I found list of provider Ids,  http://developer.ubuntu.com/api/qml/sdk-1.0/Ubuntu.OnlineAccounts.AccountServiceModel/#provider-prop ?20:27
mihirI am just using "Google" but it doesn't work20:28
mihirpopey: I am using this , http://paste.ubuntu.com/8020345/20:28
popeyhm20:29
mihirpopey: i observed, If i use evernote , it does work well  but if i do type google it doesn't popup web authentication20:31
popeyhmm, wonder if there's something different in the provider20:31
popeyI'll have a word with dpm in the morning, he will know20:32
mihirpopey: okay thanks.20:33
ybonhumm, seems that I cannot show the header without a page title?20:41
ybonMine would be "Map", but it's useless, so I'm tempted to remove it20:41
ybonok, title: ' ' does the trick20:44
ybonbut maybe it's not a good idea20:44
ahayzenballoons, ping20:52
balloonsahayzen, pong20:52
ahayzenballoons, i'm trying to figure out what is going on with some failing ap tests mind helping me figure out what is wrong?20:52
ahayzenhttps://code.launchpad.net/~andrew-hayzen/music-app/bottom-edge-implementation-001/+merge/230303 http://91.189.93.70:8080/job/generic-mediumtests-utopic/1482/?20:52
ahayzenballoons, i've tried adding waits in, and locally i've tried adding a wait_for to ensure that the animations are complete but it still occurs20:53
ahayzenballoons, any ideas what is going on?20:53
balloonsahayzen, works on desktop for you?20:53
ahayzenballoons, doesn't work on device for me (i don't have a desktop to test music app... or autopilot on)20:54
balloonsahayzen, ahh.. my point was, it's not a jenkins thing20:54
ahayzenballoons, just wondering if u have any idea what is the issue, i assumed it was todo with the new animations because of bottom edge...20:54
balloonsso i can reproduce easily just pulling the branch20:54
ahayzenballoons, yeah no jenkins... just its failing ... yeah pull and run it'll fail :)20:54
balloonsahayzen, it's a 2.7k diff ;-) anything is possible20:54
ahayzenballoons, i'll push my latest one that the wait_for actually makes sense that i added to test_artists_tab_album20:55
ahayzenballoons, thats a small diff for me ;) hehe20:55
ahayzenballoons, pushed a later one20:56
mihirpopey: i have proposed that MR, and kept it WIP ,  https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/1342275/+merge/23038620:56
popeygood call20:56
balloonsahayzen, working off rev53721:01
ahayzenballoons, yep thats the 'latest' :)21:02
ahayzenballoons, i wonder if the way i'm reparenting the now playing page may not be helping, but then i would expect more tests to be failing21:02
balloonsahayzen, heh, things all pass here . . .21:03
ahayzenballoons, wha!21:03
ahayzenballoons, what about on your device?21:03
* ahayzen suspects timing could be an issue with the animations21:03
balloonsok, got one failure21:03
ahayzenballoons, which one?21:04
balloonsmusic_app.tests.test_music.TestMainWindow.test_swipe_to_delete_song21:04
ahayzenok so the same as jenkins21:04
balloonsohh boy ahayzen .. these tests.. you can add a select_many_rety to get_queue_now_playing_artist21:07
balloonsthat moves it along a bit more21:07
ahayzenballoons, ah ok ... so does that mean it could be due to timing then?21:08
balloonshaving it pass after doing that means yea, timing21:08
ahayzenballoons, get_queue_now_playing_artist?21:08
ahayzenballoons, but it is stopping at get_queue_track_count() for me on device and on jenkins21:09
ahayzenballoons, so for test_artists_tab_album i added a wait_for(False) on the bottomedge animation running property ... but tht didn't seem to resolve the issue in jenkins/device21:13
balloonsthe waitfor false is good21:13
balloonsmine still fails even accounting for timing.. some of these selects are pretty nuts21:14
ahayzenballoons, yeah they are lol21:14
balloonsthere really shouldn't be any select_many21:14
balloonsnot when we want a specific track21:14
ahayzenballoons, every time i look at emulators.py i wanna rewrite it lol....21:14
ahayzenballoons, yeah21:14
ybonpopey: who should I bug to know more about the process of freezing https://launchpad.net/ubuntu/utopic/+source/qtlocation-opensource-src and where can I submit issues / help?21:14
balloonshmm, ok, so adding a sleep fixes the test. It seems the animation on page load is interferring with things21:15
popeyhttps://bugs.launchpad.net/ubuntu/+source/qtlocation-opensource-src/+bugs21:15
popeyybon: ^^ should be the right place to file bugs21:15
ahayzenballoons, i'm not surprised knowing what is going behind21:16
ahayzenballoons, as in not surprised with the load interfering ...21:16
ahayzenballoons, but can that really be an acceptable solution?21:17
balloonsahayzen, well, I mean things need to be redone. You want to land this first right?21:17
ybonthanks popey :)21:17
popeynp21:17
balloonsif we can find a simple way to account and wait for the page to load, let's do it21:17
popeyhows it going anyway ybon ?21:17
ahayzenballoons, most likely... gotta get my hacks past victor first :) hehe21:17
ahayzenballoons, agreed21:17
balloonsyou need to eventually account for it21:18
balloonsso . . .21:18
ybonpopey: just pushed a first version of using new header21:18
ybonnot yet release though21:19
ybonI'm thinking about how to use bottom edge thing21:19
ybonmaybe with the last place searched21:19
ybonplaces*21:19
ybonand I'm still looking how to make possible for the user to choose the background map21:19
ybonbut it seems it's not possible without going the C++ way, which is why is not done yet21:20
ahayzenballoons, so how can we tell when the page is done?21:20
balloonsthere should be a page moving property21:21
popey"Nearby" would be a good thing to have on the bottom edge, especially if you could have a set of categories..21:21
popeyybon: like, if i only really am ever interested in pubs or restaurants I could leave those ticked in some settings screen.21:21
ahayzenballoons, hmmm on which autopilot property?21:21
popeyybon: then, wherever I am, swipe up, and instantly get a list of pubs in the area.21:21
ybonthat's a nice idea21:21
popeyon another day I might be out walking, and be more interested in some other land mark, so set that in settings, and swipe up, see those21:22
ybonso I need to manage sort of "favorite" of nearby items21:22
popeyyeah, a settings screen with a basic list of categories21:22
ybonand to "recent searches" at all?21:23
ybonI mean in the bottom edge21:23
ybonnot sure how to mix both idea by the way21:23
popeyuse a multi-select to tick them. could even have profiles. "eating out" profile could be pubs & restaurants, "Travelling" could bring up bus stops and train stations.21:23
balloonsahayzen, exploring21:23
ahayzenballoons, thanks21:24
ybonI think it's better if user choose them, because if I do category I will miss the exact configuration one wants in many cases I think21:24
popeyya21:24
ahayzenballoons, i had thought about waiting until the header text became "Now Playing"21:24
balloonsok, hmm21:24
ybonbut if I get you well, the idea is: bottom edge step 1 => show my favorite nearby items; then step 2 => go to full list to pick one21:24
ybondoes that make sense?21:24
balloonsahayzen, what's your qml look like? does it have an animationcontainer?21:25
ahayzenballoons, but wasn't sure where to get that from in ap ... guess i could expose the Tabs {} ?21:25
popeyybon: let me do a simple mock up...21:25
ahayzenballoons, errr ... this bottomedge component is the upstream one...but slightly hacked/modified to our needs21:25
ahayzenballoons, diff line 2481 is the animation that brings the bottom edge up21:26
balloonsahayzen, ok, so let's try21:26
ahayzenballoons, that is the one i added an id to and then tried to wait for running to be false21:26
ahayzenballoons, in the PageWithBottomEdge.qml everywhere u see // CUSTOM is where i've changed/added something ;)21:27
ybonpopey: https://bugs.launchpad.net/ubuntu/+source/qtlocation-opensource-src/+bug/1355459 fyi21:33
ubot5Ubuntu bug 1355459 in qtlocation-opensource-src (Ubuntu) "Make tile URL configurable" [Undecided,New]21:33
rpadovani mihir: sorry, I'm on holiday until Wednesday night21:35
mihirrpadovani: no issues :)21:36
balloonsahayzen, hmm.. I was expecting something different21:36
balloonsyour implementation must be different enough21:36
ahayzenballoons, i blame mainly the reparenting of the Nowplaying page21:36
popeyybon: http://imgur.com/men0ron21:37
balloonsyes. I'm dumping the page to see what it has21:37
balloonsok, so it does have isReady21:37
balloonswhy isn't it becoming true?21:37
ahayzenballoons, basically the issue was we want the same component as the bottom part everywhere, and it to be the same one so it remembers states and things...so it uses the same Page but cahnges the parent depending where it is21:37
ahayzenballoons, that property isn't linked to my part...21:38
popeyybon: so (1) you're on the map, (2) you pull up a little way, and a couple of pre-defined searches appear, if you let go when one his highlighted it searches for those. (3) pull all the way up, you get the usual search21:38
ahayzenballoons, oh no that 'should'21:38
ybonpopey: search or nearby? :)21:38
ahayzenballoons, anything that talks to edgeLoader may not as i have a customLoader21:39
balloonsahayzen, so typically we would use the isready to wait for the edge21:39
popeyybon: nearby21:39
popeysearch for nearby ㋛21:39
ybonpopey: I wonder if I can just manage a concept of "favorites", and one put a search OR a nearby in favorite21:39
balloonswhy so much custom21:39
ahayzenballoons, interesting...also note that this test does not bring up the bottom edge by dragging, but by setting the state to 'expanded'21:39
balloonsohh.. ugh21:40
ahayzenballoons, because of the reason ^^ about the need of reparenting ... also we need a 3rd state because of the panel at the bottom21:40
ybonso when you pull up, you have your last favorite, and if you pull up again, all your favorites21:40
ahayzenballoons, yeah ugh :/21:40
balloonsI'm not sure I like the hackery21:40
popeyybon: maybe 2 faves, have you see the dialer app recently?21:41
ybonyes21:41
ahayzenballoons, but i would still expect those properties to be set correctly...i'll investigate what is happening21:41
balloonsso basically you can invoke the bottom edge by dragging, but also by doing other things21:41
popeywhere you pull up and quickly access something21:41
ybonyou mean like pulling of 1 make the 1 hightlighed, then two, etc.?21:41
ybonyes21:41
ybonthat's what I have in mind :)21:41
popeyyes21:41
ybonbut I was focusing on search place only21:41
ybonand your suggestion is good21:41
popeyyay21:41
ahayzenballoons, well yeah we can get to the now playing by clicking on a track...so it emulates clicking on it by doing what the click would have done....setting the bottom edge state to expanded21:41
ybonbut I think favorites places are nice to have too21:41
balloonsahayzen, well, I mean really your bottom edge is just the nowplaying page I guess21:41
ybonlike "home", "work"21:42
ybonand also recent searches21:42
ahayzenballoons, yeah sortof21:42
ybonso I need to clear this out :p21:42
ybonclear in the sense of "make clear"21:42
balloonsahayzen, yea, that's probably no good.. From the test perspective we should wait_for isReady and expect it to get set to true21:42
balloonsI think the ball falls back into your court21:42
ahayzenballoons, the hacks make 'sense' lol21:42
balloonscan you set the property inside your qml?21:43
ahayzenballoons, ok i'll see whats going on with isReady ... oh ...21:43
ahayzenballoons, you are checking the isReady from the original page not the now playing page?21:43
ahayzenballoons, as it is the original page which the property would change? ...21:43
ybonalso, popey, one thing nice is that the bottom edge should be useful even without any configuration21:44
balloonsahayzen, if you have a custom loader, can you set property, just like setting the 'expanded'21:44
ahayzenballoons, let me check if i think its working from my end21:44
balloonsahayzen, i'm checking the nowplaying page21:44
ahayzenballoons, u want to be checking the page that was clicked21:44
ahayzenballoons, the now playing page doesn't have a bottom edge itself it is disabled21:44
ahayzenballoons, but as i said just let me double check it does get set21:45
balloonsahayzen, whoa.. this is getting more weird21:45
ahayzenballoons, why? lol21:45
balloonsthe page, the nowplayingpage, disappears after things load.. it's werid21:46
ahayzenballoons, maybe because it gets reparented ?21:46
ahayzennowPlaying.reparent(mainPageStack)  // CUSTOM21:46
ahayzen        mainPageStack.push(nowPlaying)  // CUSTOM21:46
ahayzenand then...21:46
ahayzenfunction reparent(newParent) {21:46
ahayzen        if (parentCache !== newParent) {21:46
ahayzen            nowPlaying.parent = newParent21:46
ahayzen            nowPlaying.anchors.fill = newParent21:46
ahayzen            parentCache = newParent21:46
ahayzen        }21:46
ahayzen    }21:46
balloonsahayzen, let me pass you the tree output21:48
ahayzenballoons, hah this will be fun21:48
balloonshttp://paste.ubuntu.com/8020901/21:48
balloonsso look at the object right on top21:48
ahayzenballoons, when would u expect isReady to be true, when the bottom edge becomes expanded ?21:48
balloonsbottomEdgeState: 'collapsed'21:48
balloonstat's all weird stuff21:49
balloonsit's reading as collapsed21:49
ahayzenballoons, i would say that is 'expected'21:49
ahayzenballoons, look at the SongsPage what does that say?21:49
ahayzenballoons, also note line 16 'bottomEdgeEnabled: False'21:49
balloonssure I suppose21:49
ahayzenballoons, it is because this is the page inside the bottom edge...so it doesn't have a bottom edge itself21:50
ahayzenballoons, u want to be looking at the page which started the bottom edge ... i think21:50
ybongasp, I've typed "bzr view 33" hoping to see the rev 33 content, but now I've messed up my bzr status :/21:50
balloonsahayzen, sure.. but since you've invoked it in a different way, we need a way to account for the page load21:50
balloonsahayzen, yes I agree with you..21:50
ahayzenballoons, qml: IS READY Album SET TO true21:50
ahayzenqml: IS READY Album SET TO false21:50
ahayzenqml: IS READY Album SET TO true21:50
ahayzenballoons, so i selected an album and ^^ happened when the bottom edge was shown21:51
ahayzenballoons, 'Album' is the title of the page it was on... which shows we want to be looking at SongsPage21:51
ahayzenballoons, or whereever u clicked the track to play21:51
ahayzenballoons, i'll try changing the wait_for in my ap to isReady rather than the one i alias'd21:52
ahayzenballoons, it also depends what u mean by 'loaded' ?21:53
ahayzenballoons, are u meaning all the animations have been completed? or are u meaning when the loader is ready? ...because if the latter i don't think that is a case for my implementation as the page should always be loaded21:54
balloonsmy brain..21:54
balloonsI'm going to try adding something to populate_and_play_queue_from_songs_tab21:54
ahayzenballoons, ...imagine hacking on it...my brain hurt earlier lol21:54
ahayzenballoons, hmm put the wait_for in and it got 'stuck' and didn't do any animation then timed out after 10s21:55
ahayzenballoons, ah the wait_for maybe in the wrong place...doh21:57
balloonswow.. the songspage properties hardly change21:59
ahayzenballoons, Ran 1 test in 46.820s21:59
ahayzenOK21:59
ahayzenOMGOMGOMG21:59
ahayzenballoons, gotcha  ... but it is funny in ap the page behind the now playing disappears so u can see the dash through the back of the now playing page lol ... but the main thing is it is passing \o/22:00
balloonswhat did you change?22:00
ahayzenballoons, i'll try it on the other tests and if they pass as well i'll push22:00
ahayzenballoons, i added page.isReady.wait_for(True)22:00
balloonsto?22:00
ahayzenballoons, after clicking on the track22:00
balloonssure.. but.. whatever, push it22:01
ahayzenballoons, basically http://pastebin.ubuntu.com/8021014/22:01
balloonsthat's what I was after you to do , but I'm not seeing it22:01
ahayzenballoons, just need to add to the other 2 failing tests22:01
balloonsahayzen, I think that should be in populate_and_play_queue_from_songs_tab22:02
ahayzenballoons, probably as well i'll add there22:02
ahayzenballoons, will need a get_tracks_page() as well22:02
balloonsbut if you do, it of course fails.. because, it never changes state22:02
balloonshence my confusion22:02
ahayzenballoons, ah thats why the other one was failing22:03
ahayzenballoons, we'll see ... i may just be a magician ;)22:03
balloonsand why would it, because you are on the nowplaying page, and you aren't using the bottom edge22:03
ahayzenballoons, i'm looking at the page u clicked on from ?22:05
ahayzenballoons, i think we're confusing each other22:05
balloonsyep, you go ahead.. I don't see it22:05
ahayzenhah22:05
ahayzenballoons, right all the tests that were failing for me are now passing...pushing :)22:09
balloonsheh.. ok magicman22:10
ahayzenballoons, ah damn i can remove that extra property alias i added now...22:10
ahayzenballoons, this diff if ur interested http://bazaar.launchpad.net/~andrew-hayzen/music-app/bottom-edge-implementation-001/revision/53822:10
ahayzenballoons, yeah but u helped me get there i would probably have lost most of my hair otherwise...22:11
nik90popey, ybon: In popey's mockup, I would move the search textfield into the header which you can show by pressing on the search icon on the top right side of the header (similar to the contacts app)22:11
balloonsahayzen, awesome..22:12
balloonsnext time I see those tests, I want to be helping in the re-write :-)22:12
ahayzenballoons, same22:12
ahayzenballoons, it is next on my task list... assuming upstream ms2 fix for content-hub doesn't land as we are blocked on that...but the code at our end is 'done' for that anyway22:13
ahayzenballoons, i hope to either tonight/tomorrow start a branch to work on the rewrite i'll ping it across to u when i do22:13
ybonnik90: watch out that there is search AND find nearby points of interest22:14
ybonthose are two kinds of action22:14
nik90ybon: what's the difference?22:15
nik90ybon: doesn't search and find near by points of interest the same?22:15
ahayzenballoons, time for a mini break while i wait for jenkins to catch up... thanks for ur help :)22:15
ybonsearch => means searching like "paris" or "10 downing street", etc.22:15
ybonnearby => find all restaurants around me22:15
ybonor => find all bars around me, or supermarker, or toilets, etc.22:15
nik90ah22:16
nik90ybon: well you can show both of them as buttons in the bottom edge in popey's mockup22:16
nik90ybon: when clicking on search, show the search field in the header and hide the other button. If user presses back, go back to normal state and show both the buttons22:17
nik90makes sense?22:17
ybonno I don't follow you :s22:18
ybonwhat do you mean by "buttons in the bottom edge"?22:18
nik90when you drag the bottom edge all the way to the top, you will see a header for that page22:19
ybonI'm really sad that there seems to be still no way to handle double touch to zoom on a map: http://developer.ubuntu.com/api/qml/sdk-14.10/QtLocation.MapGestureArea/ :(22:19
nik90in that header add the two button as header actions22:19
ybonoh, ok22:19
ybongot it22:19
ybonI think both buttons will be in the main header22:19
ybonfrom the first view22:19
nik90well I thought you didnt want to show a header in the main view22:20
ybonso one doesn't need to pull up the all thing to access those buttons22:20
ybonthat was my start point yes :)22:20
nik90ah yes22:20
elopioboiko: could you add this branch to the silo 4? https://code.launchpad.net/~canonical-platform-qa/dialer-app/py3autopilot-debug2/+merge/23032222:20
nik90makes sense22:20
elopiothat way we can start testing the integration between the dialer app and the other apps.22:20
ybonbut then I thought that bottom edge was too long to give access to those actions22:20
nik90ahayzen: something awesome just landed in the latest image. I can feel it :)22:20
nik90ahayzen: relates to dash overview22:21
ybonI mean, the bottom edge is nice because it's a quick way to access some lists for example22:21
boikoelopio: sure22:21
ybonbut if you need to pull up all the screen to access all the actions, it becomes a pain I think22:21
nik90ybon: I think your idea to add them in the main view header makes sense22:21
nik90ybon: we can use the bottom edge for something else22:21
ybonso i've pushed a header for now https://github.com/yohanboniface/OSMTouch/commit/c6e087505ac6e0b591e5de386d9bf46721516db322:21
ybonyes22:21
ybonbottom edge should be for the thing you want to access quickly and often22:22
ybonthis is why the last idea is to manage some favorites22:22
ybonso you can put some searches in favorites22:22
ybonor some nearby searches22:22
nik90+122:22
ybonand by default we just show the last searches you've made22:22
ybonor some nearby suggestions, dunno :p22:23
nik90ybon: why do you set the title to " " if you are setting the header actions in the main view?22:23
nik90might as well set a title22:23
ybonbut "Map" as a title when you see a map is... useless, no? :)22:23
nik90ybon: may be OSMTouch?22:23
ybonyeah, can be that22:24
nik90ybon: I just feel leaving to blank seems like a waste22:24
ybonbut in the future I think I will have more than one page22:24
ybonincluding settings for example22:24
ybonhumm22:24
ybonbut this can still be OSMTouch for the main page22:24
ybonnik90: blank is a better information than a useless information :)22:25
nik90I suppose22:25
ahayzennik90, o/ oh has the dash overview landed?22:25
nik90ahayzen: yup indeed22:25
ahayzennik90, which image?22:25
nik90ahayzen: I had a sneak preview at the mockups22:25
nik90ahayzen: it just landed..not sure if they triggered a new image yet for that22:26
dakerybon: anyidea how we can get offline maps ?22:26
ahayzennik90, hehe ah .. yeah i've been spying on silos22:26
nik90ahayzen: exactly22:26
ybondaker: the short answer is C++22:26
nik90ahayzen: I saw the dash overview MP which showed the design mockups22:26
ahayzennik90, i'm also hoping this in silo 003 https://code.launchpad.net/~stolowski/thumbnailer/request-high-res/+merge/229940 improves the low res images we get22:26
ahayzennik90, i just read the summary didn't bother to read the mp....i'll let it be a surpise when it lands ;)22:27
nik90ahayzen: ooh sweet22:27
dakerybon: what do you mean ?22:27
nik90ahayzen: :P22:27
ybondaker: are you talking btw about caching tiles or being able to download a whole area and then consult the map without any internet connection?22:27
dakerybon: caching tiles22:28
ybonso this is a bit easier22:28
ybonit was working until some versions btw22:28
ybonoh22:28
dakerybon: yes i remember that22:28
ybonyou mean caching the tiles you have already seen, or you mean pre-caching for further navigation?22:28
dakercaching the tiles i have already seen22:29
dakerseen/loaded22:29
ybonok22:29
ybonso that was working22:29
ybonI don't know atm why it's not working anymore22:29
ybonit's on my todo list though22:29
dakerok thanks for that22:30
ahayzennik90, too many awesome things landing lately22:32
ahayzennik90, i managed to use the GPS on the phone to track me back and watch the traffic from Oxford to home earlier today :)22:32
nik90ahayzen: wat!22:32
nik90ahayzen: how fast was it?22:32
ahayzennik90, yeah ... really fast!22:33
nik90ahayzen: do you think it was AGPS or just GPS?22:33
ahayzennik90, i was outside in a car...but like instant lock ... it felt like AGPS but i could have been wrong lol22:33
nik90ybon: what do you think ^^22:33
ybonaGPS is not landed AFAIK22:33
ahayzen:(22:33
ahayzennik90, that means it could be even faster though \o/22:34
dakerme too, that's why i was caching title22:34
ybonsee https://github.com/yohanboniface/GPSStatus to have a GPS status :)22:34
nik90ybon: btw do you have sample code I can use to get the user's location using QtLocation?22:34
yboni.e. lat/lng, speed, and source22:34
nik90ybon: did you read my mind?22:34
ybonhuhu :)22:34
ahayzenybon, cool thanks :) i noticed the location-service CPU usage was a bit lower in the recent image as well ~2% vs ~10%22:34
nik90ahayzen: test it tomorrow to see if it comes up quickly again as well22:35
nik90ahayzen: which app did you use for testing?22:35
ybonstill less than the 3G22:35
ahayzennik90, google maps web app and google maps in browser22:35
ahayzennik90, downloaded the webapp halfway through as i remember there was one22:35
dakernik90: i used ybon app22:35
ahayzennik90, i was mainly using for traffic reports not navigation22:36
ahayzenballoons, success \o/ https://code.launchpad.net/~andrew-hayzen/music-app/bottom-edge-implementation-001/+merge/23030322:36
nik90ok22:36
ahayzennik90, ybon there seem to be quite a few gps/mapping apps now which one should i be using/is best?22:36
ahayzennik90, ybon  i just assumed the google maps published by canonical would be a good shout22:37
nik90ybon: are you using QtLocation to already get the location in osmtouch?22:37
nik90ahayzen, ybon: I just opened OSMTouch and it got my rough location in a few seconds22:37
ybonahayzen: my religion doesn't allow me to use Evilgle products ;)22:37
ahayzennik90, ah yes OSMTouch :) ... does that give u traffic though? .. i use that for general walking maps22:38
ybonnik90: if your phone is on since a moment, it can have fetch your location22:38
ahayzenybon, ah.... OSMTouch is awesome btw22:38
ybonnik90: try to install GPSStatus, you will have a more raw and debugable info22:38
nik90ybon: is it on the store?22:38
nik90ybon: or through qtc?22:38
ybonhumm, it's not22:38
ybonnot sure it makes sense22:38
ybonpopey: ^^22:39
nik90no worries I know what you meant22:39
popeywhats gpsstatus?22:39
ybonlet me make a screenshot22:40
ybonbasically it displays all informations available from the GPS22:40
popeyoh, android?22:40
popeyyeah, i have used that on android, but i dont have android on my phone now22:40
popeywould love an app like that on ubuntu22:40
nik90ybon: it says it found a support backend. Method was satellite. But all other fields are blank22:41
ahayzenpopey, ur OpenStreetMap webapp....'whats new'... 'Bump version number for no explicable reason' lol22:41
popey22:41
* popey installs GPS Status on his ages old HTC Hero22:42
ahayzenpopey, u sound like u would fit right in a marketing department ;)22:42
popeyhehe22:42
ybonpopey: http://i.imgur.com/2hHAVDI.png22:42
popeyooh!22:42
nik90ybon: does it autoupdate?22:42
ybonnik90: humm22:42
* popey hugs ybon 22:42
ybonnik90: yes it autoupdate22:42
nik90ybon: I will stand in the balcony to check22:42
ahayzenybon, thats cool :)22:42
nik90ybon: brb in 15 mins..going to check :)22:42
popeyis that in the store?22:43
ybonpopey: no, I was asking you if it makes sense to release it22:43
* ahayzen wishes there was an app that showed all the info from all the sensors so u could debug them22:43
popeyyeah me too22:43
popeymakes sense to me.22:43
ybonok22:44
ybonlet me do a proper icon, so :p22:44
nik90ok too cold outside to check :(22:45
dakernik90: man it's summer22:45
nik90daker: its raining here as well :)22:46
ybonnik90: near a window should work as well22:46
ahayzenpopey, my old phone had a service menu u could get to after doing certain key presses... wouldn't having an app/multiple apps that show the info of the sensors be useful for this purpose? .. eg the app ybon has made for gps?22:46
popeyyes22:47
popeyI'd love to see all that kind of raw data22:47
popeyhave you seen the compass app on ios recently. It's brilliant22:47
ahayzenpopey, is that something that the OEMs would want? ... as u could send it into a test thing as well22:47
popeyridiculously simple22:47
popeypossibly22:47
* ahayzen doesn't own an iPhone22:47
popeylemme screenshot22:48
popeyhttp://imgur.com/PxWByRW,is2M5wG,o2B4FAM,vQQJSK9#322:49
popey4 images there22:49
popeycompass, and then swipe to get a spirit level thing22:50
nik90ybon: yup worked22:50
nik90ybon: took about 1 min for me22:50
popeythey show me having the phone tilted towards me in one, flat(ish) on the desk and tilted sideways22:50
ahayzenpopey, oh wow nice :)22:50
nik90ybon: I am going to implement this for the clock app.22:50
nik90ybon: thnx for the code22:50
ahayzenpopey, do we even have a compass app?22:51
ybonnik90: :)22:54
popeyno22:55
* popey sleeps22:55
popeyttfn22:55
ahayzenpopey, o/22:55
nik90popey: nite22:56
* ybon itou23:01
ybongood night23:01
ahayzennik90, on the clock reboot in the store the tip for the bottom edge is like half hidden.... a) is that intended...and if it is b) how did u do that? as we want to do tht23:15
nik90ahayzen: intended. It is in the upstream code. I didnt change anything23:15
nik90ahayzen: it has a timer which basically pushes it down using anchoring I think23:16
ahayzennik90, ah i may need to sync my PageWithBottomEdge with upstream then...23:16
nik90+123:16
ahayzennik90, this is where the real fun begins with all the custom stuff23:16
nik90ahayzen: :)23:16
ahayzennik90, where did u get the latest one from?23:16
nik90ahayzen: https://bazaar.launchpad.net/~phablet-team/address-book-app/trunk/view/head:/src/imports/Ubuntu/Contacts/PageWithBottomEdge.qml23:17
ahayzennik90, awesome thanks :)23:17
nik90ahayzen: every component I took is from the folder https://bazaar.launchpad.net/~phablet-team/address-book-app/trunk/files/head:/src/imports/Ubuntu/Contacts/23:17
ahayzennik90, hehe23:17
jdstrandpopey: hey, do you know if labsin frequesnts this channel? the new solitaire-games has a rather annoying bug but I don't know what to file it against23:17
jdstrandpopey: oh, nm, I found it23:17
=== salem_ is now known as _salem

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!