| === chriadam|away is now known as chriadam | ||
| === Guest99228 is now known as amithkk | ||
| dholbach | good morning | 07:13 |
|---|---|---|
| mihir | good Morning :) | 07:33 |
| WebbyIT | ehi popey :) How are you? | 08:28 |
| JamesTait | Good morning all; happy Bald And Free Day! :-D | 08:34 |
| popey | WebbyIT: hey | 08:40 |
| WebbyIT | popey, I find a little bug on Calc, can you confirm it please? Do a calc, tear off, edit a label, press power off button without exit label edit, press again power button: in the label there are strages symbol | 08:42 |
| popey | WebbyIT: known bug where spurious characters get inserted | 08:42 |
| popey | happens in other apps too | 08:43 |
| popey | e.g. press power button with terminal open | 08:43 |
| WebbyIT | popey, ohh, you are right, thanks! | 08:43 |
| popey | daker: x-type is now showing me something like a desktop browser | 08:45 |
| popey | daker: http://popey.com/~alan/device-2013-10-14-094607.png | 08:46 |
| om26er | nik90, hey | 08:47 |
| om26er | nik90, btw clock app looks "drunk" :) its leaning towards the right.. http://imagebin.org/index.php?mode=image&id=273642 | 08:48 |
| om26er | also I can't reveal the toolbar | 08:48 |
| popey | om26er: bug 1239088 | 08:51 |
| ubot2 | Launchpad bug 1239088 in Ubuntu Clock App "Display of clock app is too wide and offset to the right" [Undecided,Confirmed] https://launchpad.net/bugs/1239088 | 08:51 |
| om26er | popey, also toolbar does not come up | 08:52 |
| nik90 | om26er, popey: I have no idea how to fix it | 08:58 |
| nik90 | om26er, popey: I have used the proper anchoring methods already | 08:59 |
| om26er | SDK changed probably ? | 08:59 |
| popey | possibly | 08:59 |
| popey | which image did it work fine in.. when did it break? | 08:59 |
| * nik90 checks the sdk commit log | 08:59 | |
| nik90 | popey: it broke only in image 94 | 08:59 |
| nik90 | in image 93, clock did not open (black screen) due to faulty desktop file | 08:59 |
| popey | lots changed in 94 http://people.canonical.com/~j-lallement/touch/changes/20131012.html | 09:00 |
| popey | including switching to click | 09:01 |
| * popey tries removing the click package and installing the deb from trunk | 09:01 | |
| nik90 | om26er: btw nice comparison :_ | 09:03 |
| nik90 | :) | 09:03 |
| om26er | heh | 09:04 |
| popey | clock deb built from upstream also fails in the same way | 09:06 |
| popey | there's a load of sdk changes in there http://people.canonical.com/~ogra/touch-image-stats/20131012.changes | 09:06 |
| nik90 | popey: yes but why is this localised to clock app alone? | 09:07 |
| popey | that i don't know | 09:07 |
| nik90 | popey: It is hard to test this with qtcreator broken | 09:07 |
| popey | nik90: which qtcreator you using? | 09:07 |
| nik90 | 2.8.1 | 09:07 |
| popey | ok, same here | 09:08 |
| nik90 | I added the sdk ppa as recommended | 09:08 |
| popey | nik90: run on device works here | 09:10 |
| * nik90 tries | 09:10 | |
| popey | I made mine read-write with the usual adb shell touch /userdata/.writable_image && adb reboot | 09:10 |
| popey | using clock from trunk | 09:10 |
| nik90 | popey: how do you check which image you got? I am not sure if I am runningn 93 or 94 | 09:12 |
| popey | system-image-cli -i | 09:12 |
| popey | current build number: 96 | 09:12 |
| popey | brb, tea calls | 09:12 |
| nik90 | om26er: which image did you notice the bug on? | 09:13 |
| nik90 | om26er: I am runing image 93 (devel channel) and the clock looks fine | 09:13 |
| nik90 | om26er: (latest trunk) | 09:14 |
| nik90 | popey ^^ | 09:14 |
| om26er | nik90, yesterday I noticed that first | 09:14 |
| om26er | nik90, btw devel channel have 96 now | 09:14 |
| dpm | morning popey, om26er. Do you know the status of Jenkins for the calendar app? fginther mentioned he was working on re-enabling the merger job for it, but I've not talked to him since Friday. We've got the EDS branch to save events blocking on that | 09:16 |
| om26er | Not really sure about that | 09:16 |
| dpm | om26er, and now that I've got you here :) would you mind reviewing this branch that adds tests for the music app? -> https://code.launchpad.net/~vthompson/music-app/add-ap-tests-previous-and-now-playing-play-pause/+merge/190856 | 09:18 |
| om26er | dpm, sure. on it. | 09:19 |
| dpm | awesome, thanks! | 09:19 |
| popey | nik90: might be worth (as you're on 93) doing a "sudo apt-get update" on the device, then "sudo apt-get install ..." the sdk packages like sudo apt-get install ubuntu-ui-toolkit - which will pull in new stuff, reboot and see if that broke it | 09:24 |
| popey | nik90: that way you limit the changes on your device to just the sdk | 09:24 |
| nik90 | popey: i just reflashed :( | 09:26 |
| popey | bum | 09:26 |
| nik90 | popey: hmm, I did phablet-flash ubuntu-system --channel devel and still have image 93 | 09:27 |
| popey | yes, thats right | 09:27 |
| popey | thats good ☻ | 09:28 |
| popey | so can you test above? | 09:28 |
| nik90 | but om26er said that devel has image 96 | 09:28 |
| popey | no, devel-proposed has 96 | 09:28 |
| nik90 | ah | 09:28 |
| popey | we haven't released 96 | 09:28 |
| om26er | devel-proposed yes. | 09:28 |
| nik90 | popey: testing | 09:29 |
| popey | thanks | 09:29 |
| om26er | nik90, is alarm working now ? | 09:29 |
| nik90 | om26er: not sure, I am testing that as well | 09:29 |
| nik90 | popey: so in your changelog link, ui-toolkit-plugin is at 0.1.46+13.10.20131011.2-0ubuntu1 and on sudo apt-get update, I have that same version as well on the phone | 09:34 |
| nik90 | popey: on running clock through qtcreator on phone, it appears normal, | 09:35 |
| nik90 | working properly | 09:35 |
| nik90 | popey: should I install one by one every package in the changelog to see which breaks it? | 09:35 |
| popey | nik90: what gets offered if you "apt-get -s dist-upgrade" (simulate) | 09:35 |
| popey | i dont think one-by-one is possible | 09:36 |
| popey | it will pull in dependencies | 09:36 |
| nik90 | popey: http://paste.ubuntu.com/6235180/ | 09:37 |
| nik90 | popey: ui-toolkit-plugin isnt listed since I already upgrade that one | 09:38 |
| nik90 | upgraded* | 09:38 |
| popey | ah | 09:39 |
| popey | qmlscene perhaps? | 09:39 |
| nik90 | shall i try installing that? | 09:39 |
| popey | I'd be picking off a few packages and installing them one by one | 09:39 |
| popey | yeah | 09:40 |
| popey | but only sdk / qt5 related ones | 09:40 |
| popey | until it breaks. | 09:40 |
| nik90 | okay | 09:41 |
| popey | sorry ☹ | 09:44 |
| nik90 | popey: http://paste.ubuntu.com/6235230/ | 09:55 |
| nik90 | popey: i tested against those packages | 09:55 |
| nik90 | still works | 09:55 |
| nik90 | I do not think the others are revelant | 09:56 |
| nik90 | can I do a full dist-upgrade and try again? | 09:56 |
| popey | feel free | 10:03 |
| popey | you can always downgrade a package later | 10:03 |
| popey | Time for more tea! | 10:04 |
| nik90 | popey: I upgraded all packages and clock appears fine..no dislocation :) | 10:16 |
| nik90 | popey: on the other hand, I am not sure why you see the bug | 10:19 |
| popey | nik90: I have not been drinking, if that's what you're implying! ☻ | 10:22 |
| nik90 | popey: hehe | 10:22 |
| dpm | fginther, just read your e-mail about the status of the automatic upstream merger. Could we get it enabled for ubuntu-calendar-app? We were blocking on some qtorganizer-eds fixes which landed on image 94, and it'd be good if we could land the calendar branch that needed those fixes | 10:23 |
| nik90 | popey: I could try reflashing to image 96 and then try qtcreator run on it | 10:23 |
| nik90 | popey: that's the last thing I can do since this issue is definitely not in the clock code | 10:24 |
| popey | nik90: I have 96 here too, feel free to try it | 10:28 |
| nik90 | popey: so I reflashed to img 96..opening the clock app, I see it is dislocated..however running it from qtcreator, it appears fine | 10:39 |
| popey | well that's even more odd | 10:40 |
| nik90 | popey: actually it is not | 10:40 |
| nik90 | popey: but I fixed it | 10:40 |
| nik90 | :_) | 10:40 |
| nik90 | popey: https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-dislocated-clock/+merge/190910 | 10:46 |
| nik90 | popey: please test and approve | 10:46 |
| popey | ok | 10:46 |
| popey | nik90: works, approved, nice one! | 10:49 |
| nik90 | popey: is the freeze today? | 10:50 |
| popey | nope | 10:50 |
| nik90 | balloons: is this bug still valid -> https://bugs.launchpad.net/ubuntu-clock-app/+bug/1238703 | 10:51 |
| ubot2 | Launchpad bug 1238703 in Ubuntu Clock App "All autopilot tests fail" [Critical,Confirmed] | 10:51 |
| === chriadam is now known as chriadam|away | ||
| === MacSlow is now known as MacSlow|lunch | ||
| === gusch is now known as gusch|lunch | ||
| om26er | nerochiaro, hey! can you review this https://code.launchpad.net/~om26er/mediaplayer-app/disable_settings_button/+merge/190904 | 11:39 |
| om26er | since the mediaplayer does not have any settings, I have dimmed the settings icon so that its clear there are no settings | 11:40 |
| daker | hey oSoMoN i just want to know if the two fixes were released with the new image or not ? | 11:43 |
| oSoMoN | daker: let me check | 11:43 |
| daker | oSoMoN: i pushed a new version(0.4) to the store with the UbuntuWebView | 11:44 |
| oSoMoN | daker: cool, I’ll check it out now | 11:44 |
| oSoMoN | daker: I confirm the two fixes have been released with the latest image (#96) | 11:45 |
| oSoMoN | daker: I’m not seeing the app in the click lens | 11:47 |
| daker | hm, oSoMoN try searching "X-Type" | 11:47 |
| oSoMoN | daker: no results | 11:48 |
| daker | it does appear here http://search.apps.ubuntu.com/api/v1/search | 11:48 |
| daker | maybe it's a bug or something... | 11:48 |
| daker | oSoMoN: ^ | 11:50 |
| nerochiaro | om26er: haveig a look | 11:50 |
| nerochiaro | om26er: the code is simple enough to just approve as it is, but jenkins is not happy with it for some reason | 11:51 |
| davmor2 | popey: is it me or have both the card apps disappeared from the store when you look at the list of apps on the phone? | 11:52 |
| om26er | nerochiaro, jenkins is problematic. probably some problem with CI infrastructure | 11:52 |
| om26er | nerochiaro, try top approve it might just land ;) | 11:52 |
| oSoMoN | daker: maybe… | 11:52 |
| nerochiaro | om26er: done | 11:52 |
| om26er | nerochiaro, thanks | 11:56 |
| popey | davmor2: mzanetti also mentioned apps disappearing. can you file a bug ? | 11:58 |
| daker | oSoMoN: here is the code if you want to test it on device directly https://bazaar.launchpad.net/~daker/+junk/x-type/files | 11:58 |
| davmor2 | popey: will do I'm going to chase things our end first though so I know what I'm filing and why :) | 11:58 |
| oSoMoN | daker: thanks | 12:01 |
| === MacSlow|lunch is now known as MacSlow | ||
| oSoMoN | daker: I can confirm it works nicely, although it’s rather slow here on my galaxy nexus (but then again everything is slow on galaxy nexus with mir) | 12:07 |
| daker | oSoMoN: ok thanks :) | 12:07 |
| davmor2 | popey: the architecture is set to all, it's a known bug that has a high priority | 12:13 |
| === _salem is now known as salem_ | ||
| daker | oSoMoN: did you saw popey comment ? | 12:14 |
| oSoMoN | daker: no | 12:14 |
| oSoMoN | ah, just saw it | 12:14 |
| oSoMoN | so it’s a known issue | 12:15 |
| oSoMoN | apparently | 12:15 |
| daker | davmor2: oSoMoN says that he can't find my app "X-Type" in the click lens too | 12:16 |
| === gusch|lunch is now known as gusch | ||
| davmor2 | daker: oSoMoN: popey: https://bugs.launchpad.net/click-package-index/+bug/1239662 | 12:21 |
| ubot2 | Launchpad bug 1239662 in Click Package Index "Apps with Architecture All are not visible on devices" [Critical,In progress] | 12:22 |
| daker | here we go :) | 12:22 |
| mihir | oSoMoN: still EDS is not merged?? | 12:25 |
| SpotAri | trying to install my app .deb to nexus 4 with todays image 96. not showing in applications anymore (worked fine on Friday).. what's changed? | 12:26 |
| steoria | :-1: error: cannot find -lsqlite3 | 12:31 |
| === gatox is now known as gatox_brb | ||
| fginther | dpm, which branch is that, we may be able to merge only that one | 13:05 |
| dpm | fginther, https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EDS/+merge/188188 - but there are a bunch of other branches that are depending/blocking on this one too. | 13:06 |
| dpm | hi ahayzen, I see you guys were busy over the weekend, the music app is looking better and better :) | 13:07 |
| ahayzen | dpm, thanks | 13:07 |
| ahayzen | dpm, just got 50 emails in the past 10hrs from you guys reporting bugs though :) | 13:08 |
| popey | hah | 13:08 |
| dpm | hahaha, that's the excitement about getting to run and test the latest :) | 13:08 |
| ahayzen | :) | 13:08 |
| ahayzen | popey, can u get lisettte to look at https://code.launchpad.net/~andrew-hayzen/music-app/album-art-toolbar-when-no-back-button/+merge/190823 | 13:09 |
| dpm | ahayzen, I've asked her already this morning, let me ping her again | 13:10 |
| ahayzen | dpm, thanks | 13:10 |
| dpm | she's going though the bugs too | 13:11 |
| === gatox_brb is now known as gatox | ||
| ahayzen | cool | 13:11 |
| ahayzen | dpm, i don't see why Victor won't approve https://code.launchpad.net/~andrew-hayzen/music-app/expander-now-playing/+merge/190820 as it makes the behaviour consistent across the app and then meets the design spec :/ | 13:12 |
| === hatchetation_ is now known as hatchetation | ||
| dpm | let me have a look | 13:12 |
| dpm | ahayzen, actually, I was testing it this morning and got sidetracked. This is implementing the chevron to expand the items in the play queue, instead of the plus sign, right? | 13:13 |
| ahayzen | dpm, yep it implements the expander like in the artists/album/playlist | 13:14 |
| ahayzen | dpm, we didn't implement it before because of the reordering/swipeDelete and the fact the rows have different heights due to states...but i figured out how to do it so pushed it up | 13:15 |
| oSoMoN | mihir: I haven’t followed the latest developments on this branch, let me check | 13:16 |
| dpm | ahayzen, makes sense. I can see Victor's point, because in the current implementation the two chevrons on top are really close to each other. But on https://docs.google.com/a/canonical.com/presentation/d/1nc7RTAD5ViEAdc3VsZyN-lkmG-AnO664uV-5wXZoZZg/edit#slide=id.ge8e1d63f_040 the currently playing item has the chevron below the cover art, which removes the confusion imo | 13:16 |
| ahayzen | dpm, i could try moving the meta data below the album art if tht helps? | 13:17 |
| dpm | ahayzen, I think so, yes, but if it's not a trivial change it might be worth doing it in a separate branch. Let me have a quick look and see if we can approve this one. | 13:18 |
| ahayzen | dpm, ok good plan :) | 13:19 |
| oSoMoN | dpm: hey, could you confirm that https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EDS/+merge/188188 is now good to merge? | 13:24 |
| oSoMoN | dpm: ah, nevermind, just saw the global status | 13:24 |
| oSoMoN | mihir: so it appears the EDS branch is ready to merge, but hasn’t been merged yet because the upstream merger is deactivated for the calendar-app due to failing autopilot tests under MIR | 13:25 |
| dpm | oSoMoN, yeah, what was blocking it were the eds backend fixes, which landed on image #94 | 13:25 |
| oSoMoN | dpm: great, thanks | 13:25 |
| dpm | oSoMoN, fginther is looking into that ^^ | 13:25 |
| oSoMoN | excellent | 13:25 |
| mihir | oSoMoN: okay thank you for updates.. | 13:26 |
| fginther | dpm, oSoMoN I should get back to you on the state of that shortly. It may still be blocked by the mir transition, stay tuned | 13:26 |
| dpm | thanks | 13:27 |
| ahayzen | dpm, could u let me know if ur gonna approve tht branch cause tht depicts whether i do this work in a separate branch | 13:28 |
| dpm | ahayzen, yeah, on it, just taking some screenshots about an issue I noticed | 13:29 |
| ahayzen | dpm, cool | 13:30 |
| mihir | dpm: thank you :) | 13:33 |
| lisettte | ahayzen: i was looking at https://code.launchpad.net/~andrew-hayzen/music-app/album-art-toolbar-when-no-back-button/+merge/190823 | 13:38 |
| ahayzen | lisettte, o/ | 13:38 |
| lisettte | ahayzen: I don't think this is a good idea, because users will tap in the same area without looking and might have an unexpected result if in some cases they go back and in others to the player | 13:39 |
| lisettte | ahayzen: i do wonder whether we have space for album art | 13:39 |
| lisettte | ahayzen: and i want to ask Michal about that | 13:39 |
| ahayzen | lisettte, i don't wanna squash the meta data in the centre anymore though | 13:39 |
| lisettte | ahayzen: ? | 13:39 |
| ahayzen | lisettte, if we were to add album art in as well as the back button and play button | 13:40 |
| lisettte | ahayzen: the question that really matters here is, are single genre and playlist going to be the same as single album? | 13:40 |
| lisettte | ahayzen: because if we are stuck with that we will never actually have a back button | 13:42 |
| ahayzen | lisettte, what do u mean by 'going to be the same'? as in are they gonna use sheets with the close button? | 13:43 |
| lisettte | ahayzen: in the design single album, single playlist, single artist are all outside of the tab nav, and to return the user needs to press back | 13:44 |
| lisettte | ahayzen: in case of the single album it was technically impossible to do this | 13:44 |
| ahayzen | yep | 13:44 |
| lisettte | ahayzen: but the current sheet makes play controls inaccessible, which is weird | 13:45 |
| ahayzen | lisettte, yeah thts the issue with the sheet | 13:45 |
| lisettte | ahayzen: anyway, re the back button, if we have a back button in some places (and it sounds like we do) we should not fill that space with album art | 13:46 |
| lisettte | ahayzen: that part of the toolbar should be reserved for other toolbar actions | 13:47 |
| ahayzen | lisettte, ok please add a comment to the merge and disprove | 13:47 |
| lisettte | ahayzen: for instance, a grid / list view toggle | 13:47 |
| ahayzen | yep | 13:47 |
| lisettte | ahayzen: i agree it is not ideal at the moment though | 13:48 |
| mihir | dpm: now i need to pull & push new MR for Edit event correct? | 13:59 |
| oSoMoN | mihir: you don’t need to push a new MR, just push new revisions to fix conflicts to your branch, and the existing MR will pick up those new revisions | 14:00 |
| dpm | mihir, exactly | 14:00 |
| dpm | mihir, yeah, but more correctly, what oSoMoN says | 14:01 |
| mihir | oSoMoN: complicated i always mess with this confilicts , anyways I should try that once I reach home :) | 14:01 |
| dpm | mihir, essentially: | 14:01 |
| mihir | these* | 14:01 |
| dpm | bzr merge lp:ubuntu-calendar-app | 14:01 |
| dpm | then maually fix any conflicts (if any) | 14:01 |
| dpm | then bzr commit | 14:02 |
| dpm | then bzr push to the same branch as the existing merge proposal | 14:02 |
| mihir | dpm: thank you :) I'll do that | 14:05 |
| ahayzen | dpm, i've moved the labels below the album art :) could you rereview the merges? https://code.launchpad.net/~andrew-hayzen/music-app/move-label-below-current-album-art/+merge/190968 | 14:09 |
| mihir | dpm: i am facing weird problem on QTCreator, each time it updates manifest.json automatiicaly ? any idea ? | 14:13 |
| ahayzen | mihir, i'm having the same issue just have to revert the changes for tht file each time :( | 14:14 |
| mihir | ahayzen: : yeah , i need to do that each time before I commit | 14:14 |
| ahayzen | mihir, yep same only takes like 10 seconds though so not too bad just irritating | 14:15 |
| dpm | mihir, I agree it's annoying. bzr revert manifest.json should make things a bit easier for the time being | 14:20 |
| mihir | dpm: i am doing that only, i thought there is some problem with my machine , but got it it's for everybody :) | 14:26 |
| mihir | dpm: can we merge this MR now ?https://code.launchpad.net/~mihirsoni-123/ubuntu-calendar-app/NewEventdetails/+merge/187489 | 14:26 |
| dpm | mihir, looking at it now... | 14:30 |
| ahayzen | lisettte, ping | 14:31 |
| lisettte | ahayzen: pong :) | 14:32 |
| ahayzen | lisettte, i'm just doing bug 1239715 | 14:33 |
| ubot2 | Launchpad bug 1239715 in Ubuntu Music App "[Music app] toolbar should hide when scrolling the play queue, or when in library mode" [Medium,In progress] https://launchpad.net/bugs/1239715 | 14:33 |
| dpm | mihir, you'll need to merge https://code.launchpad.net/~mihirsoni-123/ubuntu-calendar-app/NewEventdetails/+merge/187489 from trunk as discussed earlier before we can land it | 14:33 |
| lisettte | ahayzen: yep | 14:33 |
| ahayzen | lisettte, should when you select 'Back' after say selecting an artist/playlist should it hide the toolbar as well? | 14:33 |
| ahayzen | lisettte, as in the back button in the toolbar | 14:33 |
| lisettte | ahayzen: yes, unless the toolbar behaves differently everywhere else | 14:34 |
| * lisettte checking | 14:34 | |
| mihir | dpm: so I should first merge then again push the changes correct ? | 14:34 |
| ahayzen | lisettte, i've made it so when you select back from the player mode (now playing page) it hides the toolbar and when you scroll on tht page it hides it | 14:35 |
| dpm | mihir, yes | 14:35 |
| mihir | dpm: okay I'll do it i am commuting to home now , once I reach I'll do it :) thanks for your help !! | 14:35 |
| dpm | thank you mihir! | 14:35 |
| lisettte | ahayzen: that sounds good, but are we changing anything in the default toolbar behaviour? | 14:35 |
| lisettte | ahayzen: the design was done assuming that is the default toolbar behaviour | 14:36 |
| ahayzen | lisettte, we are using a custom component | 14:36 |
| lisettte | ahayzen: right, so let me triple check | 14:36 |
| ahayzen | lisettte, so this is probably making match the 'normal' behaviour as we have to now tell it when to do things | 14:36 |
| lisettte | ahayzen: great | 14:36 |
| lisettte | ahayzen: this part is something we really need to see in action before we know if it works or not | 14:37 |
| ahayzen | lisettte, i'm not so sure if it should because say if u have the toolbar expanded when at the playlists page then u select a playlist then select back if it collapses it then you aren't left how you were before | 14:37 |
| lisettte | ahayzen: if you are in playlists and select a playlist, you are interacting with the library, not the player | 14:39 |
| ahayzen | lisettte, yeah thts the back button i mean | 14:39 |
| ahayzen | lisettte, when u go back up a pageStack | 14:39 |
| dpm | fginther, the upstream merger only lands branches directly into trunk, right? I.e. it does not try to do a merge first and if there are no conflicts then land? | 14:39 |
| lisettte | ahayzen: in library mode, as per the ux design, it should be hidden by default apart from the progress keyline | 14:40 |
| lisettte | ahayzen: but you think this causes a problem when you are inside a single playlist? | 14:41 |
| ahayzen | lisettte, so when i hit the back button on this toolbar to return to the list of playlists it should minimise the toolbar http://ubuntuone.com/3FGUN57Hj74rWL1D5NzrPB | 14:41 |
| lisettte | ahayzen: yes, because we think the user will want to look at other library content, not play controls | 14:42 |
| ahayzen | lisettte, ok do the same in other places where that appears? | 14:42 |
| lisettte | ahayzen: the user might want to select another playlist, so we should give him as much screen real estate as possible | 14:42 |
| ahayzen | lisettte, so basically if goBack() is run then minimise the toolbar | 14:43 |
| lisettte | ahayzen: yes | 14:43 |
| ahayzen | lisettte, cool tht simplifies things... sorry for not explaining ver well | 14:43 |
| lisettte | ahayzen: i am loving the snapping back to the current item in the player btw | 14:43 |
| ahayzen | :) | 14:43 |
| lisettte | ahayzen: no worries! this is very complex, and still handwavy in places anyway | 14:43 |
| lisettte | ahayzen: apologies for not making it clearer from the ux spec! | 14:44 |
| ahayzen | lisettte, it is pretty clear just the really fine details like this :) | 14:44 |
| lisettte | ahayzen: cool cool | 14:45 |
| === davidcalle_ is now known as davidcalle | ||
| fginther | dpm, one moment, otp | 14:48 |
| dpm | np | 14:48 |
| ahayzen | dpm, think you need to approve https://code.launchpad.net/~andrew-hayzen/music-app/expander-now-playing/+merge/190820 to land the other branch? | 14:54 |
| dpm | ahayzen, yes, that's correct. The other branch was relatively easy to review, but I think I'd prefer Victor or Daniel to top-approve the pre-requisite branch. The changes are not trivial, and I'd prefer one of the developers to ack it, as you guys are much more into the logic of the app, and I wouldn't do a good job at reviewing the code. Let me add a comment. | 14:56 |
| ahayzen | dpm, Victor has approved it | 14:56 |
| dpm | ahayzen, in that case, done :) | 14:57 |
| ahayzen | :) | 14:57 |
| ahayzen | dpm, i'll put swapping the orientation of the icon on my list of things to investigate | 14:58 |
| dpm | ahayzen, yeah, it might be worth pinging mehow_ about it if he's online. I'd leave it as a low priority action for now | 15:00 |
| ahayzen | dpm, thanks i'll have a look later once i've got this bunch of work done | 15:00 |
| dpm | cool, thanks a lot ahayzen! | 15:00 |
| dpm | lisettte, mehow_ to fix https://bugs.launchpad.net/music-app/+bug/1239699 could we get some icons for the controls? For instance, we're using a play button that's smaller than the one in the spec, and it'd be nice to have the same icons as in the visuals | 15:02 |
| ubot2 | Launchpad bug 1239699 in Ubuntu Music App "[Music app] play control icons should not be inside button shapes" [Medium,Triaged] | 15:02 |
| lisettte | dpm: the icons should be in the icon theme, and they should be scaleable? | 15:03 |
| zohan | night | 15:04 |
| zohan | all | 15:04 |
| zohan | :D | 15:04 |
| dpm | lisettte, it seems the icon theme only provides media-playback-pause, *-start, *-skip-backward and *skip-forward. For the rest, we can use the icons we already have, but it'd be good to know the display sizes, especially that of the play icon, which is bigger than the rest | 15:09 |
| fginther | dpm, upstream merger does a merge with trunk first, before doing a build or running tests | 15:09 |
| dpm | mehow_, ^ | 15:09 |
| lisettte | dpm: let me count the grid units in the spec | 15:09 |
| lisettte | dpm: play / pause is 4 units high | 15:10 |
| lisettte | dpm: the rest 2 | 15:10 |
| lisettte | dpm: should i add this information to a bug? | 15:10 |
| dpm | fginther, ok, so if a branch does not have conflicts but is behind trunk, then upstream merger takes care of rebasing to trunk without the developers having to do it right? | 15:11 |
| fginther | dpm, yes | 15:11 |
| fginther | dpm, the only time a dev should need to rebase to trunk is when there are actual merge conflicts | 15:12 |
| lisettte | dpm: or should i file a new bug for this? | 15:13 |
| dpm | lisettte, we've got the existing bug for that, let me add the info, including where to find the icons, to it, thanks! | 15:14 |
| dpm | fginther, awesome, thanks. For this branch https://code.launchpad.net/~mihirsoni-123/ubuntu-calendar-app/NewEventdetails/+merge/187489 - could we get the upstream merger retriggered? | 15:15 |
| fginther | dpm, http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/144/ | 15:16 |
| fginther | dpm, does that need to be merged to trunk as well? | 15:16 |
| dpm | fginther, yes. Essentially, the EDS branch was blocking a bunch (well, I think 2) of other branches which can now land | 15:17 |
| fginther | dpm, merge conflict now :-( | 15:19 |
| fginther | http://91.189.93.70:8080/job/ubuntu-calendar-app-saucy-amd64-ci/144/console | 15:19 |
| dpm | fginther, ah, bummer. The developer is commuting home now, but he said he'd look into it when he'd be back. I thought this one would merge cleanly. | 15:20 |
| fginther | dpm, it's a non-trivial conflict, otherwise I might have tried :-( | 15:25 |
| dpm | np, thanks fginther | 15:25 |
| lisettte | ahayzen, danielholm: mehow_ has a default album cover. How big should the image be? | 15:30 |
| ahayzen | lisettte, at the moment we have one version which is 500x500 and another which is 48x48 | 15:32 |
| lisettte | ahayzen: cool! mehow_ ^ | 15:32 |
| dpm | lisettte, I was not sure what to comment on bug 1239513. Would you mind commenting it? | 15:34 |
| ubot2 | Launchpad bug 1239513 in Ubuntu Music App "search bar does not work for music app" [Undecided,New] https://launchpad.net/bugs/1239513 | 15:34 |
| lisettte | dpm: ok | 15:39 |
| janimo` | I see automaticOrientation in MainView that allows rotating the view based on the device orientation. Is there a way to force a certain orientation regardless of sensor data? | 15:41 |
| janimo` | for example landscape mode on the phone | 15:42 |
| lisettte | mehow_: i don't see your question here?? | 15:45 |
| lisettte | ahayzen: mehow_ wonders about the sizes; shouldn't it be 10GU x 10GU is 270px x 270px ? | 15:46 |
| lisettte | ahayzen: but we can give you the images in the sizes you need to fix it today of course :) | 15:46 |
| mehow_ | hey | 15:48 |
| mehow_ | could you confirm that you need that missing album cover artwork 500x500 and 48x48 ? | 15:48 |
| mehow_ | ahayzen: hey | 15:49 |
| mehow_ | could you confirm that you need that missing album cover artwork 500x500 and 48x48 ? | 15:49 |
| ahayzen | mehow_, o/ | 16:13 |
| === charles_ is now known as charles | ||
| ahayzen | mehow_, yep if u have it available :) | 16:14 |
| ahayzen | lisettte, it will scale to the size of the item | 16:16 |
| === dpm is now known as dpm-afk | ||
| lisettte | ahayzen: artwork is now attached to https://bugs.launchpad.net/music-app/+bug/1239095 | 16:18 |
| ubot2 | Launchpad bug 1239095 in Ubuntu Music App "[Music app] Remove "Scrobble cover to last.fm" until the function is implemented" [Medium,Triaged] | 16:18 |
| lisettte | ahayzen: please note that we added all the sizes.. | 16:18 |
| lisettte | ahayzen: so the sizes you mentioned and the sizes we thought it would be calculating from grid units | 16:18 |
| ahayzen | lisettte, thanks | 16:19 |
| ahayzen | lisettte, how did u calculate the 971x971 ? | 16:19 |
| lisettte | ahayzen: from the grid units i guess.. mehow_ probably knows | 16:19 |
| ahayzen | lisettte, just wondering as tht would mean the current would look pixelated somewhere if tht is correct | 16:20 |
| ahayzen | popey, ping | 16:21 |
| popey | ahayzen: hey | 16:21 |
| ahayzen | popey, do we have an ETA on the builds for today? | 16:21 |
| ahayzen | popey, just so i know the deadline | 16:21 |
| popey | image builds? | 16:21 |
| ahayzen | popey, yh | 16:21 |
| lisettte | ahayzen: haven't noticed that myself, but i am not sure if i would | 16:22 |
| ahayzen | lisettte, same thts why i was wondering how u got to that figure | 16:22 |
| ahayzen | lisettte, cause my screen only has 768 horizontal lines :) | 16:23 |
| popey | ahayzen: "in some hours" - will let you know once we have a better idea | 16:23 |
| ahayzen | popey, heh | 16:23 |
| ahayzen | lisettte, i'll put like for like ones in for now | 16:24 |
| lisettte | ahayzen: all i know is that the visual designers are designing for 1 grid unit = 27 x 27 px. i guess just use the ones that work for now | 16:30 |
| ahayzen | lisettte, i thought it was much smaller than tht in real life... http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/resolution-independence.html | 16:31 |
| ahayzen | 'Most laptops 1 gu = 8 px' and 'Phone with 4 inch screen at HD resolution (around 720x1,280 pixels) 1 gu = 18 px' | 16:31 |
| lisettte | ahayzen: interesting… mehow_ ^ | 16:32 |
| ahayzen | lisettte, just a thought do u have the images in SVG? | 16:36 |
| lisettte | ahayzen: i doubt it; mehow_ ? | 16:37 |
| ahayzen | lisettte, we do have a few images in SVG now which has the advantage of being resolution independent, no worries if u don't | 16:38 |
| lisettte | ahayzen: no, it is a pixel image for now; we could change it in the future though? | 16:40 |
| ahayzen | lisettte, something for V2 to go through most of the images and see if we can change them | 16:40 |
| ahayzen | lisettte, right tht is proposed now waiting for someone to approve | 16:41 |
| lisettte | ahayzen: awesome! | 16:41 |
| ahayzen | popey, dpm-afk, do u mind looking at this https://code.launchpad.net/~andrew-hayzen/music-app/hide-toolbar-on-go-back/+merge/190974 | 16:53 |
| harisha | Hello all, I am submitting an application | 17:05 |
| harisha | But i did not get what is package name | 17:05 |
| harisha | what has to be given for package name? | 17:06 |
| popey | ahayzen: top approved | 17:13 |
| ahayzen | cool | 17:13 |
| ahayzen | popey, and https://code.launchpad.net/~andrew-hayzen/music-app/increase-queued-normal-height/+merge/191010 | 17:13 |
| ahayzen | popey, i'm just about to go for a run so will be back in like 30mins cyas | 17:14 |
| popey | approved | 17:15 |
| === jo-erlend__ is now known as jo-erlend | ||
| ahayzen | popey, thanks | 18:04 |
| popey | np | 18:06 |
| ahayzen | popey, looks like dpm wanted a slight adjustment i'll put it in the fix for bug 1239785 | 18:09 |
| ubot2 | Launchpad bug 1239785 in Ubuntu Music App "Metadata display improvements in the play queue view" [Medium,In progress] https://launchpad.net/bugs/1239785 | 18:09 |
| popey | ah ok | 18:09 |
| ahayzen | popey, it wasn't anything introduced in tht branch so not really and issue with the merge | 18:10 |
| wellsb | mhall119, dpm-afk: Any ETA for when we can expect to receive feedback about showdown apps? | 18:26 |
| popey | wellsb: mhall119 is on vacation today | 18:26 |
| ahayzen | popey, is dpm still around? | 18:34 |
| AmandaC | Is this the place to ask for stuff like libmessaging-menu questions ( from a developer's side. ) | 18:36 |
| AmandaC | I ask because I'm toying with an idea in Vala using the messaging menu, and when I select a source, it disappears from the menu. Is that supposed to happen, or am I messing up somewhere along the way? | 18:38 |
| popey | ahayzen: dunno, wassup? | 18:46 |
| ahayzen | popey, just trying to land more stuff aha | 18:46 |
| popey | anything i can help with ahayzen ? | 18:46 |
| ahayzen | popey, this should resolve bug 1239785 and the comment he put in the other merge https://code.launchpad.net/~andrew-hayzen/music-app/split-album-artist-in-now-playing/+merge/191022 | 18:47 |
| ubot2 | Launchpad bug 1239785 in Ubuntu Music App "Metadata display improvements in the play queue view" [Medium,In progress] https://launchpad.net/bugs/1239785 | 18:47 |
| ahayzen | popey, the music app is starting to look really awesome now :) | 18:49 |
| popey | looks good ahayzen | 18:55 |
| ahayzen | \o/ | 18:55 |
| daker | popey: can you retry "X-type" now, oSomon confirmed it's working, and the image #96 has landed so i am sure you'll get the latest browser plugin ? | 19:24 |
| popey | ok | 19:24 |
| popey | hm | 19:24 |
| daker | thanks | 19:24 |
| popey | http://popey.com/~alan/device-2013-10-14-202543.png | 19:26 |
| popey | feels really slow | 19:26 |
| popey | gonna reboot and try again | 19:26 |
| daker | ya i think that's because of...mir :) | 19:26 |
| popey | i can test without mir too | 19:28 |
| daker | popey: one last thing, can you take 2 other screenshots ? i want to update the store | 19:28 |
| popey | i took 3 | 19:28 |
| popey | one mo | 19:28 |
| daker | if you can switch to SF and retest, so we can see if it's only slow under mir | 19:30 |
| popey | ya, will do | 19:31 |
| ahayzen | popey, suppose we should top approve https://code.launchpad.net/~andrew-hayzen/music-app/split-album-artist-in-now-playing/+merge/191022 ? | 19:38 |
| popey | daker: its not that much slower under sf tbh | 19:42 |
| popey | not sure I can tell the difference | 19:42 |
| popey | ahayzen: done | 19:42 |
| ahayzen | popey, thanks :) | 19:42 |
| daker | popey: ok | 19:42 |
| popey | beuno / JamesTait I'm getting 503's from the store when installing apps | 19:56 |
| popey | and lots of images not loading | 19:56 |
| beuno | popey, yeah, the service is flaky, been trying to get a hold on a webops | 19:57 |
| popey | ij | 19:57 |
| popey | er ok | 19:57 |
| ahayzen | popey, has the llast | 21:59 |
| ahayzen | bah | 21:59 |
| ahayzen | popey, has the last build been done yet? | 21:59 |
| renato | hey guys I am trying to use the qtcreator to run a example app on my device and I am getting this error | 22:00 |
| renato | bash: cannot set terminal process group (-1): Invalid argument | 22:01 |
| renato | bash: no job control in this shell | 22:01 |
| renato | any Idea how to fix that? | 22:01 |
| ahayzen | renato, you have to run without Mir | 22:01 |
| ahayzen | renato, or on the build i'm on you did | 22:02 |
| renato | ahayzen, any way to remove Mir? | 22:03 |
| renato | ahayzen, I flashed the device using: phablet-flash cdimage-touch --pending | 22:03 |
| ahayzen | renato, delete the .display_mir file in /home/phablet then restart should then use surfaceflinger ... don't hold me responsible if it breaks :) | 22:04 |
| renato | ahayzen, ok thanks | 22:04 |
| daker | popey: ping | 22:16 |
| daker | another game waiting for you :) | 22:17 |
| nik90 | renato: it works on mir if you use the system-image with devel-proposed | 22:19 |
| nik90 | renato: with qtcreator 2.8.1 (from ppa on 13.10) | 22:19 |
| nik90 | renato: I had the same issue few days back, but was resolved after I added the ppa | 22:19 |
| renato | nik90, ok thanks | 22:19 |
| JamesTait | beuno/popey, is the 503 coming from the index, or the download? | 22:22 |
| === salem_ is now known as _salem | ||
| nik90 | mhall119, popey: https://plus.google.com/113051860352311525753/posts/SzinycjRtLP | 22:39 |
| beuno | JamesTait, same as always, ran out of PG connections, had it restarted | 22:40 |
| JamesTait | beuno, right, thanks. | 22:41 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
