[01:05] Elleo, hey with regard to https://bugs.launchpad.net/music-app/+bug/1379817 [01:05] Ubuntu bug 1379817 in Ubuntu Music App "When importing 7 digital albums music-app sometimes displays "Imported files not found" when all files have been imported" [Undecided,Confirmed] [01:05] Elleo, are you around? [01:05] vthompson: yep? [01:05] vthompson, o/ [01:06] vthompson, have you checked out the ms2 log around the time of first import? [01:06] vthompson, (as i just added to the bug report) [01:06] Elleo, I just read your update about the bug having to do with a fresh install. I wasn't able to reproduce by deleting "things" but I have a suspicion that mediascanner2 might not be watching for the new folder to be created [01:06] ahayzen, I' [01:07] vthompson, i remember having something similar to this when initially creating it [01:07] ahayzen, I've been watching the ms2 log and it quickly detects the new folder and the new contents... I just wonder if the daemon might "go to sleep" and not be so active at watcing for filesystem changes [01:07] vthompson, so if you restart the app does it see the file or not? [01:08] ahayzen, I can do that next. I also went as far as restarting ms2 incase it was still watching the old directory [01:08] vthompson, yeah good shout [01:08] vthompson, what is interesting is that is picks up the second file? but not the first [01:08] oh restart the app? yea, the app running or not does not seem to have an affect [01:09] I thought you meant restart the device... :) [01:09] vthompson, yeah restart the app does it then see the first file? [01:09] yes [01:09] vthompson, i mean run the failed import... restart the app..can you see it in the models [01:09] Why do you think that'd change? [01:09] I can't get a failed import :) [01:09] vthompson, oh [01:10] I can clear a few things and it still picks up the song reliably. I could try different songs, I suppose [01:10] vthompson, have you tried clearing the folders and the ms2 db...then doing the process again [01:10] Elleo, could you get us a copy of your /home/phablet/.cache/upstart/mediascanner-2.0.log file? [01:11] I didn't clear the ms2 db yet [01:11] ahayzen, I don't think that alone would do it.. but it might [01:11] vthompson, it may not remove the watches until a restart or something [01:12] yea, I assumed that'd happen after a restart of the service... but a reboot might be my next test [01:12] vthompson: okay, just a sec [01:13] Elleo, awesome, thanks [01:16] ahayzen, reboot didn't help. I'm going to let the service idle a bit [01:17] vthompson, interesting [01:17] ahayzen, vthompson: attached to the bug report [01:17] Elleo, thanks [01:17] Thanks :) [03:28] hi all [03:40] ahoneybun, havn't heard from you lately; whats new? [06:11] akiva-thinkpad: ping [06:11] akiva-thinkpad: something is wrong with your MP , can you check ? [06:12] mihir, yah sure [06:12] akiva-thinkpad: it is failing tests. [06:12] akiva-thinkpad: i believe it is not able to save event. [06:12] mihir, bah [06:12] it might be because of the bug I found, but I found that in trunk as well [06:15] mihir, it says with default values, but lately I have noticed in trunk, I no longer have any calendars available to choose from [06:15] so I can't add any events either. [06:15] akiva-thinkpad: Jenkins just passed my MP though. [06:15] mihir, do you think jenkins might be suffering from that as well? [06:15] mihir, interesting. [06:16] akiva-thinkpad: see this , it just ran few mins before https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/blankBubbleInWeekView/+merge/237069 [06:16] mihir, can i ask something; does jenkins do all tests when it runs, or just tests related to the code you editted? [06:16] akiva-thinkpad: it runs all test cases :) [06:16] okay thanks [06:16] akiva-thinkpad: try to run testcases locally. [06:17] mihir, how? [06:17] you on 14.10 ? [06:17] that would be really useful to know [06:17] yep [06:18] akiva-thinkpad: in your directory , navigate to /tests/autopilot/ [06:19] okay [06:20] done [06:20] autopilot3 list calendar_app [06:21] akiva-thinkpad: ^^ [06:21] autopilot3: error: argument mode: invalid choice: 'calendar_app/' (choose from 'run', 'list', 'vis', 'launch') [06:21] o/ good morning evening whatever it is at your place all [06:21] I take it is run? [06:21] mihir, [06:22] justCarakas, evening [06:22] justCarakas, did you end up finishing your app? [06:23] nop, I stalled, not everything is clear and I need time to find out how QML works but I'm also making an PHP framework atm to improve my skills and that has been the focus of my free time [06:24] maybe I'll finish it when I'm a weekend at home, but my agenda is kinda full [06:24] and I work a lot while traveling, and than its harder to look up qml stuff [06:26] akiva-thinkpad: yeah it is run, i thought to make you understand [06:26] mihir, curses you! [06:26] :) [06:26] akiva-thinkpad: if you want to run individual test case then you can pick name and do autopilot3 run testcaseName [06:26] okay this is great; thanks for taking the time to walk me through that. [06:26] mihir, ah very cool [06:26] akiva-thinkpad: if you want to run whole test suite just do , autopilot3 run calendar_app/ [06:28] mihir, quickly, merge-source and tree; I keep forgetting which one is my code, and which one is the trunks code that I merged from. [06:29] akiva-thinkpad: i lost you now :) [06:29] mihir, this is for conflicts in merges [06:30] you have >>>>>>>>>>>tree ================== >>>>>>>>>>>>>>>>>>>merge-source [06:39] akiva-thinkpad: hmm , the merge source is yours , there will be 3 files on your directory [06:39] akiva-thinkpad: check and resolve properly [06:48] mihir, hey, could you get a pot update in calendar? [06:49] seb128: hey morning , ahh, could it wait till evening , i am at work place. if that's urgent i can get it done with help of someone. [06:50] mihir, no hurry, it's just for the commit you did with the string fixes, we need an updated pot so they are translatable [06:50] mihir, I pointing it in the merge request review but maybe you didn't notice [06:50] mihir, anyway, tonight is fine, thanks! :-) [06:50] seb128: yeah i should have done that. i missed that. === tvoss__ is now known as tvoss [06:53] good morning [06:53] Morning dholbach :) [06:53] hi mihir [06:53] dholbach: hello :) what's up? [06:54] doing well - how about you? [06:54] doing great :) === wgrant_ is now known as wgrant [06:58] mardy, could you ping me when you have 10 minutes please? :-) I have a core dump and a question for you [06:59] sounds like a start of an interesting conversation :) [06:59] :-) [07:20] * mardy hides [07:25] rpadovani: ok, tell me :-) [07:26] lol mardy, thanks :-) I don't find my core dump paste right now, so before the question :D If I have a list of unauthorized accounts and I want to launch the UI to enable it on click, how I could do that? [07:26] https://bazaar.launchpad.net/~rpadovani/reminders-app/multipleAccounts/view/head:/src/app/qml/ui/AccountSelectorPage.qml#L59 [07:28] rpadovani: unfortunately you cannot choose in advance which unauthorize account you want to prompt the user about [07:28] rpadovani: that selection happens inside Online Accounts [07:29] mardy, gotcha, thanks. So I have to launch it with setup.exec(), as adding a new account? [07:29] rpadovani: that's according to design, just read the first paragraph here: https://wiki.ubuntu.com/OnlineAccounts#App_access [07:29] rpadovani: exactly [07:30] mardy, yeah, documentation, I definitely need to read it more often :D [07:30] dpm, ^^ [07:30] rpadovani: in other words, I think you shouldn't even show the list of unauthorized accounts, just have a button "Add an Evernote account" or something like that [07:32] mardy, thanks. And how can I authorize an account that I've created via System Settings, then? I.e. if I 1) Create Evernote account via u-s-s-o-a 2) Start Reminders 3) If there is only an "Add account" button, how can I ever use my existing account? [07:33] dpm: press the "Add account" button :-) It will allow to select the existing account [07:33] dpm: see the wiki link above, it shows all the possible cases [07:33] dpm, and this was my first MR, list of available accounts, a text if there are unauthorized accounts, and the button [07:37] mardy, ok, thanks! About the core dump, I have it when I try to change account. I click on new account and I assign to accountService.objectHandle accountServiceHandle. It used to work, like 6 months ago, then we disabled the account switching feature so I dunno when it brokes [07:37] mardy, I give you the core asap [07:37] rpadovani: OK [07:38] rpadovani: is it easy to reproduce? [07:41] mardy, yes, just take lp:~rpadovani/reminders-app/multipleAccounts, when you start the app choose an account (you have to have 2 accounts in oa, of course), then after the login clic on the menu in the header and choose 'accounts' voice, and select the other account [07:41] mardy, it freezes, and after some time it crashes. My problem right now is that freezes but doesn't crash :/ [07:41] rpadovani: OK, I cannot look at it right now, but could you please file a bug with these instructions? [07:41] mihir, FailedTestScreenshot (image/png) -- is this saying that there is a screenshot that was taken of the failure? [07:41] mardy, sure [07:44] mardy, rpadovani, ack, just read the wiki. However, the "more than 1" already set up accounts seems to imply that we can read the name of the account to show a prompt from the app, but I understand that we cannot actually do that? [07:45] dpm, mardy if I understand right, we actually can show the name of the unauthorized account - we have a model for that - but we cannot say to OA which account we want to authorize [07:46] ah, gotcha, then what rpadovani said is actually what I meant :) [07:47] dpm, rpadovani: yes, that's correct. [07:49] mardy, then we cannot implement the "more than 1" case in the wiki? Or did we get it wrong? [07:49] akiva-thinkpad: from where you getting this error [07:50] dpm: you can [07:50] dpm: your UI should show a list of the authorized accounts, [07:50] dpm: and then a button "Add another" [07:50] dpm: that's it [07:52] mardy, and what does "Add another" do? I.e. I can see the workflow in the diagram, but I don't quite understand how after clicking "Add another" the app should authorize (or defer authorization to u-s-s-o-a) existing but unauthorized accounts [07:53] dpm, you can try it adding a google account and doing login in gmail webapp [07:54] mihir, nevermind. Anyways I found the test that my app is failing, well the remix, and when I watch it... It doesnt make sense what its trying to accomplish. Is there a way I can check what this specific test is trying to achieve? [07:54] dpm, mardy actually, the app doesn't crash anymore, at least on this computer. It only freezes and takes ages to switch account, but this maybe is fail of reminders backend. I'll ask to mzanetti to take a look to it next week [07:55] akiva-thinkpad: you can try to run that individual test case look at the errors , and identify where it is failing . [07:55] brb [07:55] yah its here [07:55] sec [07:56] dpm: the only thing the app has to do is call Setup::exec(), then all the flow is implemented in OA [07:56] mardy, ok, thanks [07:56] dpm: OA will check how many unauthorized accounts there are for the app, and decide what to show to the user [07:56] mihir, http://imgur.com/lVfJaY1 [07:57] 00:52:54.894 INFO _launcher:544 - waiting for process to exit. [07:57] akiva-thinkpad: is it music app ? [07:57] mihir, yep [07:57] rpadovani, I think that answers all my questions - do you have any more questions related to the multiple accounts UI, or are you good to go? [07:57] akiva-thinkpad: i have no idea of music app , thought you talking regarding calendar app :| [07:57] ah [07:58] dpm, good to go, at least when I understand why it freezes. Thanks mardy for your time! [07:58] mihir, ah but in this case; is it obvious to you what the issue is? [07:58] dpm, so, how do you want to implement that? [07:58] or do I have to bring it up with vic and ahayzen? [07:59] rpadovani, I guess we can just go with the sketches on the wiki page? I.e. a dialog instead of a page [07:59] akiva-thinkpad: may be try adding more logs on the functions mentioned in that traceback [08:01] dpm: if you are referring to this wiki page (https://wiki.ubuntu.com/OnlineAccounts#App_access), you don't have to implement any dialogs: all of the right-hand side of the diagram is already implemented in OA [08:02] mihir, so debugs? do console.log work? [08:02] akiva-thinkpad: there is different way doing in python, you can take reference from previous logging. [08:03] mardy, so I could easily do (if accounts.count = 1) autologin() (else accounts.count = 0 || >1) Setup.exec() [08:03] mardy, and switchAccount() = Setup.exec() ? [08:04] mihir, oh I think I found the issue... its something that is depricated. [08:05] akiva-thinkpad: hmm , were you able to find a root cause in calendar ? [08:05] rpadovani: not exactly; you should call Setup.exec() only when you want to add a new account [08:05] rpadovani: my suggestion is: [08:06] 1a) on statup, if there's only one enabled account, autologin [08:06] mihir, i'll go do that after. Sorry :S -- btw; I really want to thank you for being so good with merge requests and walking me through this. It honestly means a lot to me. [08:06] 1b) on startup, if there are several enabled account, present a list of them (or maybe remember the one used last time?) [08:07] 2) on the account selection page, show a list of the enabled accounts, and one "Add another evernote account" button which triggers Setup.exec() [08:07] akiva-thinkpad: no issue :) thank you for Merge request and your contribution. [08:08] mardy, ok, thanks. dpm sounds good to you? [08:09] mihir, so I am fairly certain that the test is depricated; should I delete the test, or edit it, or not touch it and speak with other maintainers? [08:09] akiva-thinkpad: i can't comment on Music app , as i am not aware about that , so it is better to talk to music app developers. [08:09] okay [08:10] akiva-thinkpad: that might be the case , if you have changed the functionality , you might have to remove/update your test case accordingly. [08:13] rpadovani, sounds great [08:14] Good Morning all :-) [08:17] mardy, how can I prompt the dialog to authorize an account? Setup.exec adds a new account, but if user already added it in OA? [08:18] rpadovani: no changes, always use Setup.exec() [08:18] mardy, gotcha thanks [08:20] dpm, do you want also a text that says "We found unauthorized accounts, please click on the button below to authorize it" or we leave only the "Add account" button? [08:21] rpadovani, let's keep it simple for now and use "Add another..." [08:22] dpm, ack. So, the code is ready, it's the first revisios I did, I have only to delete the text. I'm investigating on the freeze, so I don't push right now [08:22] thanks rpadovani! [08:39] zsombi: hey, I did some preliminary testing of your alarm-data-adapation branch and it did not refresh the listview on adding/deleting an alarm which is awesome. However it broke the alarm switch. I wasn't able to enable/disable alarms in the clock app. [08:39] zsombi: can you verify that on your end as well? [08:49] Good morning all; happy Information Overload Day! :-D [08:53] time to send spam :D [08:53] JamesTait: does that mean I can send you wikipedia pages as emails? :D [08:53] JamesTait, that's my every day! [08:56] justCarakas, you and everyone else. :-P [08:56] dpm, amen to that. [09:00] morning ogra_! What's the status of the removal of evernote-plugin-account from the seeds? Do you think it will make it to the next promoted image? [09:00] on rtm, I mean [09:02] dpm, it has to :) i'll ping you after the lanbding meeting so we can coordinate [09:04] * dpm hugs ogra_ [09:07] Hi [09:14] akiva-thinkpad: not sure if you tried, but you can now run apps on the emulator directly through qtc instead of the hack we did due to a bug at the time. [09:21] ahayzen, with the music remix app, is there a way to add songs to a playlist? I could only create playlists, but not populate them [09:21] dpm, drag a listitem to the left and click on the second icon [09:22] dpm, i think we're looking to add header actions to the album/artist itself so you could add the whole album/artist to a playlist as well [09:23] ahayzen, slick! I'd not noticed the swipe elements other than "Delete" :) [09:23] thanks! [09:23] dpm, haha no problem :) [09:23] dpm, yeah listitem actions are very discoverable at the moment [09:24] dpm, i've been working on multiselect support as well...so you would press'n'hold then tick the ones you want and click 'Add to playlist' in the header [09:33] nik90, hey, do you have an opinion on https://bugs.launchpad.net/ubuntu-clock-app/+bug/1381429? [09:33] Ubuntu bug 1381429 in Ubuntu Clock App "Shouldn't code button's geometry, but rather let it adapt to the label" [Undecided,New] [09:33] * nik90 takes a look [09:35] seb128: The reason why I set it to 17 grid units was because the visual spec I was given had that dimension. But I agree it makes sense to make the width depend on the label + some margin. [09:36] nik90, why a margin? buttons should be consistent in look, if they need a marging between text and borders the toolkit should do that for you [09:36] seb128: although I will ensure in that case that the button never touches the edges of the app by using Math.min(label.width, page.width) [09:37] seb128: true [09:37] nik90, I doubt you are going to hit translations where that string go over the screen, but your call ;-) [09:38] seb128: do you have a patch for this or should I fix this? [09:38] seb128: true, I just take precautions here and there sometimes ;) [09:38] nik90, I don't have a patch yet, I can do one if you want [09:38] nik90, I just wanted to discuss the change in principle first [09:39] seb128: whatever suits you, since it should just be a 2 liner code change at most. [09:39] seb128: I will take care of this [09:39] nik90, thanks [09:39] let me know if you want a review/tester [09:39] seb128: will do. [09:39] I'm happy to adb push the qml to my device and test on french locale :-) [09:40] :) [09:51] ahayzen, wow, looking forward to multiselect, then! [09:51] dpm, haha yeah not sure when we'll land that it is fun trying to integrate with the custom reorder code...will probably be in the sprint that it gets done [09:52] cool === rmescandon is now known as rmescandon-afk [09:57] seb128: https://code.launchpad.net/~nik90/ubuntu-clock-app/dynamic-delete-alarm-size/+merge/238407 [09:58] seb128: I tested the code with a really long & short string and it worked as expected. [09:58] akiva-thinkpad: sparingly? [10:07] nik90, yah; the music app for example last time I tried, opened, then just crashed. [10:07] mihir, ping -- I tried the test on trunk [10:07] hmm that's strange [10:07] it failed there as well. [10:07] akiva-thinkpad, crashed on what? [10:07] ahayzen, oh hey [10:08] ahayzen, just running the emulator. [10:08] akiva-thinkpad: oh btw there is one limitation [10:08] akiva-thinkpad, ah emulator right [10:08] i didnt bother to debug [10:08] akiva-thinkpad: you can run the app on the emulator for about 15-20 times before you need to reboot the emulator [10:08] nik90, hmm? [10:08] nik90, hmmm? interesting [10:08] okay [10:08] akiva-thinkpad: is is related to the app not releasing some EGL surfaces when an quits in the emulator. [10:08] it is a known bug but no time to fix it yet [10:09] nik90, respectfully [10:09] cool thanks for explaining that [10:09] yw [10:10] nik90: I'll do that, good point! [10:11] zsombi: hey, one more thing if you got time. [10:11] nik90: shoot [10:11] zsombi: I have been using the qtc profiler to see bottlenecks in the clock app and noticed that the edit alarm page takes a while to load because of the date time picker. Is this known? [10:12] zsombi: the datetime picker took about 150-250 ms to load.. (cant remember exactly) [10:13] nik90: yes, partly because of the slowness of the ListItems partly because of the PathView instantiating every single item from the model (not like ListView) and partly due to the date model handling :( [10:14] zsombi: ah..I am trying to circumvent the issue by showing the edit alarm page and then loading the datetime picker. [10:15] so it might seem a bit more snappy [10:15] nik90: yep... sorry for that, we will reach those as well some time the new ListItem is ready... === rmescandon-afk is now known as rmescandon [10:16] zsombi: cool, no worries [10:17] nik90: so, returning to the disabling issue, what have you experienced? no refresh on teh item? [10:17] nik90: or that the disabling ain't got completed? [10:17] zsombi: no it doesn't disable it at all..so I press the switch, but it doesn't change. [10:17] zsombi: tbh I didnt further debug and console output the alarm status due to lack of time. [10:17] nik90: I see. the tests show that disabling goes thru, so we might need to adjust the switch handling then... [10:18] ah ok [10:18] nik90: np, I'll take the clock app branch and test it on desktop [10:18] ok [10:18] nik90: you've tested on desktop, right? [10:18] zsombi: yup only on utopic desktop vm. [10:19] nik90: thx [10:21] akiva-thinkpad, you shouldn't really be removing the seek_to() from the tests [10:30] akiva-thinkpad: you talking about calendar ? [10:42] zsombi: can you give your insight on bug 1381432. I already commented on it, but it needs to be validated by someone like you. [10:42] bug 1381432 in Ubuntu Clock App "Display "no active alarm" while checking" [Medium,Confirmed] https://launchpad.net/bugs/1381432 [10:44] nik90: after the alarm-data-adaptation MR you won't need to load AlarmModel with a Loader anymore [10:45] zsombi: really? [10:45] nik90: yep :) [10:45] zsombi: wow pretty cool. [10:45] zsombi: is there a way to still know when the alarm model is ready (after fetching data)? [10:46] nik90: and you will receive a QAbstractItemModel signal when data comes available [10:46] zsombi: i can disable the bottom edge hint until it is ready to avoid showing false status to the user [10:46] sweet [10:47] I have to ask, what is the core change of the alarm-data-abstraction MR in a few words? [10:47] How does it seem to fix a couple of bugs in one go? [10:47] nik90: the signal is modelReset() [10:48] nik90: well, the main change is that there is no alam cache duplication per model [10:48] mihir, bah my notifcation system... darn hexchat :/ Sorry [10:48] mihir, and yes, talking about the calendar [10:48] the trunk fails the tests [10:49] nik90: then there is no alarm request handling generalized, all that has been moved to AlarmDataAdapter and AlarmsAdapter [10:49] ahayzen, the issue is I don't know how to fix the test. That test relies upon the old version of the toolbar [10:49] not toolbar, slider* [10:49] akiva-thinkpad, yeah the test will need to be rewritten to use the new slider [10:49] nik90: then, alarm cache is one per app instance, all models connect to this cache and reflect all changes immediately [10:50] ahayzen, If you can help me through rewriting it, I would be appreciative. I don't know how to set the square as an object with the slider. [10:50] nik90: no more fetch on alarm data change/add/remove, each of these are handled as separate model updates [10:50] zsombi: I am using the loader atm to load the model *after* the startup animation is done since otherwise it causes the startup animation to be laggy. [10:50] akiva-thinkpad, i'll have a look when i get back, i'm at uni at the moment [10:50] zsombi: ah that's cool [10:50] ahayzen, thanks [10:51] nik90: the fetch will only happen when the alarm model is created, more precisely when the QML model is completed [10:51] nik90: that results in dataChanged() emision, which btw happens even now [10:52] nik90: sorry, modelReset()!!!!!!! [10:52] zsombi: ah, so I can use onModelReset() to know when to show the bottom edge hint to the user then. [10:52] probably better here [10:52] nik90: alarm data change may result in either dataChanged() if the time is not changed or if the time change does not require the alarm cache index change [10:52] nik90: yes [10:53] popey, dpm ping [10:53] popey, dpm could someone raise priority of https://bugs.launchpad.net/osmtouch/+bug/1381463 [10:53] Ubuntu bug 1381463 in OSMTouch "PositionSource.update() cannot be used synchronously" [Undecided,New] [10:53] nik90: alarm addition will cause rowsAboutToBeInserted() + rowsInserted() signals [10:53] nik90: alarm removal will cause rowsAboutToBeRemoved() + rowsRemoved() [10:54] nik90: ah, sorry, one more mistake :D [10:54] zsombi: ah nice..individual signals for each action..more control cool [10:54] nik90: alarm data change may cause dataChanged() or rowsToBeMoved() + rowsMoved() signals [10:54] nik90: depending on what data is changed [10:54] ok [10:54] tvoss, that's an app created by a community member, not part of core apps. We don't have any permissions for setting bug priorities [10:55] nik90: like disabling an alarm, description, soundChange results in dataChanged() [10:55] dpm, could you reach out to the developer please? [10:55] nik90: but date change may result in move [10:55] zsombi: ack. [10:55] nik90: as the date may become later/earlier than the other ones are, so the list will be rearranged [10:56] tvoss, sure, we can do that [10:56] akiva-thinkpad: Jenkins passed other MR [10:56] so that shouldn't be problem i believe [10:56] zsombi: yeah at the moment the list is not rearranged when changing dates. thereby forcing me to go through every alarm to figure out the next closest active alarm. [10:56] nice to know though that this will be fixed [10:57] mihir, but I just tried it with the trunk; it failed o_o [10:57] nik90: you will get the model always showing the alarms in the cronological order, but disabling a passed alarm will still be as first in teh model [10:57] like I didnt edit the trunk at all [10:57] * akiva-thinkpad double tries it [10:58] zsombi: ah..ok .. I can do the search in the clock app myself [10:58] nik90: so we may still need a filter on teh model to exclude disabled alarms? [10:58] zsombi: that's your call since for my use case, a user might have at most 5-6 alarms which is not that hard to go through and find the closest active alarm. [11:00] nik90: well, then we keep it as is now, because that would require some index mirroring in the model [11:00] zsombi: fine by me [11:03] nik90: so, all kewl features on that MR :) [11:03] zsombi: :D [11:03] nik90: and some code simplification as well :) [11:04] nik90: but the model changes were requiring heavy changes in the cache [11:05] ah [11:09] zsombi, hey :-) Does header is supposed to hide automagically if I have listview inside a page inside a pagestack? I know it is in a listview inside a page, but if I have also a pagestack? Because in reminders app it doesn't hide, and I don't understand why [11:09] t1mp: ^ [11:10] rpadovani: it supposed to, but it depends what are you doing with your ListView... [11:11] rpadovani: like how you anchor, what else you have in the page, etc [11:11] zsombi, som I'll have to do a looong work of commenting out everything to understand what's going on :-) [11:11] thanks [11:12] rpadovani: you better start a new Page with a ListView as content and start add more stuff to see where it breaks :) [11:12] rpadovani: this would be a better approach :D [11:13] ok, I'll try this one then :-) Thanks! [11:24] rpadovani: it should work if the ListView is anchored to the top of the page or fills the page [11:25] t1mp, thanks, this could be a good suggestion! [11:27] rpadovani: that's for the auto-detection, but you don't need that if you set Page.flickable to your listview [11:28] i'll need to test then when i go back home. [11:28] akiva-thinkpad: ^^ === simion314_ is now known as simion314 [12:04] * dpm lunches === _salem is now known as salem_ [12:25] nik90, thanks, commented, looks good to me (though the pot update doesn't seem useful) [12:26] seb128: it seems that every time i test on a device or build a click, the pot file is automatically updated even with no string change [12:26] seb128: and I accidentally committed it the first time [12:26] seb128: shall I revert it [12:26] nik90, your call, it doesn't hurty [12:34] nik90, Hi! do you know where does the calculator app saves its history ? [12:35] om26er: tbh I am not sure, it should most likely be in .local/share/com.ubuntu.calculator ? [12:35] rpadovani: do you know ^^ ? [12:35] speaking of which, wasn't the history notes supposed to be cleared out? [12:36] nik90, that seems like it. [12:36] om26er: I just checked as well on my emulator. that's the location. [12:37] seb128: I heard that...not sure who is working on that [12:37] afaik all calc devs were moved to other projects [12:38] nik90, well, there is https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/1365564/+merge/233400 pending for a while [12:38] popey, ^ do you know what's the status there? [12:39] hmm looks like it is waiting on design review [12:40] dpm: hi! Have you found time to have a look at the weather app and the CMAKE rules? [12:41] popey, http://i.imgur.com/NilzBui.png :) I got "Up" working nicely, as well as the toolbar in the header. Now I'm just working on getting the other components in. [12:51] seb128, yes, popey said we need a design review on that package. Anyway, calc was set as 'stable', so I moved to other projects [12:51] seb128: btw do you need the bug 1381429 fix to be pushed to the store today or can this wait? We just released a new clock yesterday and since then this is the only new commit that landed,. [12:52] bug 1381429 in Ubuntu Clock App "Shouldn't code button's geometry, but rather let it adapt to the label" [Low,Fix committed] https://launchpad.net/bugs/1381429 [12:52] nik90, that can wait, it's looking slightly odd but not blocking anything [12:52] the button works, the label is just truncated [12:52] ah ok [12:52] nik90, but feel free to fix some other bugs and do another upload ;-) [12:53] ;) [12:53] hi m-b-o1 o/, unfortunately, I haven't. Perhaps this evening. But I want to have a look at your MP fixing the mobile links first, which I've not had the chance to look at in detail yet, but I'd still want to try to get in by tomorrow [12:54] dpm ok! [12:54] m-b-o1, quick question: I see you've got a few if/elses depending on the language code. What about the other languages that are not covered by the ifs? [12:54] I mean on the mobile links merge proposal [12:55] dpm: there was an excel sheet (or csv) from TWC, with all the links [12:55] and the domains [12:56] dpm: so only the domains from the excel sheet were implemented. Some urls-schemes in some domains are different on desktop/mobile. I've checked all links in the script [12:56] and corected them for mobile if neccessary === rmescandon is now known as rmescandon-lunch [13:01] m-b-o1, hm, I can't recall that excel sheet, do you have it handy? [13:03] mihir, does calendar work for you on the latest ubuntu-rtm/devel-proposed image? It crashes for me on startup [13:03] It seems related to qmlscene, by looking at the logs [13:04] actually, it doesn't crash on startup, but after having started and changing to day view [13:04] dpm: you'll have a mail in a few seconds [13:04] awesome, thanks m-b-o1! [13:09] popey, gventuri: Hey, does this splash screen for clock look okay https://imgur.com/YhMeHKv ? [13:09] popey, gventuri: I didn't add a header since clock doesn't have one, so I changed the background color for now. [13:10] it makes the startup transition less jarring [13:10] m-b-o1, thanks for the e-mail, I had completely forgotten that spreadsheet, I should add it to a blueprint or to the wiki [13:12] m-b-o1, do you think perhaps putting the per-language urls in a JSON file or an array and using that as a sort of lookup table might make more sense than using if/elses? [13:20] dpm: yes, probably. on the other hand the code wasn't modified a lot since we added it [13:21] dpm: but yes, a data structure would be more efficient [13:22] m-b-o1, what would you suggest, to approve as it is to minimize changes and then look at using a data structure later? [13:23] dpm: yes, I would suggest that. Redesign still looming [13:23] m-b-o1, ok, cool, thanks a lot for the discussion [13:23] dpm: looming perhaps the wrong word ;) [13:23] hahaha [13:23] brace yourselves... [13:24] dpm :)) === rmescandon-lunch is now known as rmescandon === karni is now known as karni-afk [13:38] Yes \o/ I got it working the way I want :DDDDD [13:39] * akiva-thinkpad celebrates programming victory. Its okay if you don't understand === karni-afk is now known as karni [14:15] nik90: it works for me [14:22] hey anyone know if head.actions can do popovers? I am getting a qquickitem not being a qobject error [14:31] dpm: meeting? === gatox is now known as gatox_lunch [14:54] dpm: that's what I call a productive meeting :) [14:54] mzanetti, indeed, Argentinian team ftw! [14:55] :-) [15:14] Is there a way to try out Dekko on the desktop? It's not in the desktop PPA AFAIK === rmescandon is now known as rmescandon-brb === gatox_lunch is now known as gatox [15:54] akiva-thinkpad: ping [15:54] akiva-thinkpad: were you able to resolve the issues? [15:54] mihir, pong [15:54] mihir, on the calendar? [15:54] akiva-thinkpad: yup [15:55] mihir, didnt try much. As said; I ran the tests on trunk, and they failed [15:55] so I don't know why they are passing for you. [15:55] akiva-thinkpad: hmmm..:| [15:55] mihir, yep; thats been a lot of what I have been doing [15:55] :P [15:56] akiva-thinkpad: i'll try and update you. [15:56] mihir, atm I am working on the filemanager app. I'm pretty proud with what Ive accomplished thus far. === rmescandon-brb is now known as rmescandon [16:03] mihir, https://code.launchpad.net/~akiva/ubuntu-filemanager-app/1380575+1373346-toolbar-to-header/+merge/238470 :) [16:04] akiva-thinkpad: great :) [16:04] popey, dpm -- would you mind taking a look at the design here, and letting me know what needs to be tweaked? This addresses the two high importance bugs you touched upon the other day. [16:04] https://code.launchpad.net/~akiva/ubuntu-filemanager-app/1380575+1373346-toolbar-to-header/+merge/238470 [16:05] akiva-thinkpad, sure, looking [16:05] Considering that I took out the bottom toolbar to address these, the jenkins tests will break [16:05] thanks! [16:05] ahayzen, o/ [16:06] akiva-thinkpad, do you think you could create a couple of screenshots and add the links to the description? It might be easier for them if we manage to find someone from design to have a look at them [16:07] yah [16:07] will do that right now [16:07] dpm, can a short video suffice? [16:07] akiva-thinkpad, wow, suffice? that would be even better! :-) [16:07] lol :) [16:08] dpm, I havnt tested it on a device; I am having library problems with libtag [16:09] akiva-thinkpad, from the emulator would work equally well. What's the problem with libtag on a device? have you installed libtag-dev:armhf on the phone click chroot? [16:22] dpm, I tried going into phablet shell, and sudo apt-getting it [16:22] although i didn't append :armhf [16:22] i'll try that I suppose [16:22] akiva-thinkpad, you don't need it on the device [16:23] akiva-thinkpad, you only need to install it on the click chroot at build time [16:23] akiva-thinkpad, but yeah, remember to append the :armhf [16:23] dpm, ah so I need to go into Projects to do that [16:23] got it [16:23] akiva-thinkpad, yes, how did you build the app? Did you use Qt Creator? [16:23] dpm, yah; [16:24] i'll toss it in as an instruction [16:24] akiva-thinkpad, there's a README file in the file manager sources that explains everything in detail [16:24] * akiva-thinkpad is a bit new at modding that. [16:24] dpm okay I'll read it now then [16:24] dpm, did you try the branch btw? :) [16:25] the one thing I didnt put in there that I wanted, was memory of child directories [16:25] akiva-thinkpad, I've not had the chance yet, but I'm really looking forward to this change :) [16:25] I'll see if I can do it this evening [16:25] akiva-thinkpad, http://bazaar.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/trunk/view/head:/README [16:25] dpm, okay I'll show you a video [16:25] :P [16:25] sec [16:25] :-) [16:32] dpm, popey mihir http://filebin.ca/1dpmqZc83f51/Cast-_00006.mp4 === randomcpp is now known as gcollura [16:43] akiva-thinkpad, that looks really awesome! I've done a quick, non-code review of the UI and added some comments [16:44] dpm, thanks [16:44] ahayzen, bah, vic rejected my proposal >:/ [16:44] akiva-thinkpad, :/ [16:45] akiva-thinkpad, probably best to come back to it when we start focussing on tablet/desktop [16:45] huh... The only issue I am trying to fix are the bloody deprecated tests [16:48] ahayzen, meh; I think I just left a bad impression on vic or something. Oh well~ [16:49] akiva-thinkpad, hah i'll see if i can figure out the tests later :/ the problem is we have a tight deadline for getting the mobile app done so we are pretty focused on that [16:51] ahayzen, my complaint that is his code reviews were shoddy. For example; He told me the text for the times was too large, but when I showed him the spec, it clearly showed that the text was not "small". [16:51] and when I brought it up, he didnt bother responding. [16:51] akiva-thinkpad, yeah sorry about that, that spec in particular was tricky to interpret as the height of it was larger than a device [16:54] blah; okay well; i'm gonna be bugged about it. No avoiding that. Okay I'm just going to drop out of it. I'll cool down eventually, and I understand that he has good intentions... But I'm sure you can understand how I feel. [16:54] ahayzen, anyways, good luck with the deadline. Wish I could have been of more help. [16:55] akiva-thinkpad, yeah i know how you feel, its difficult to get reviews past him sometimes ;) Once we get onto the tablet/desktop designs (hopefully soon) it should all calm down a bit [17:00] seb128, you around ? [17:01] ahayzen, Do you think if I can get jenkins to approve my branch, he'll come around? [17:01] or should I not bother? [17:01] akiva-thinkpad, maybe i don't know [17:01] okay fair enough. [17:02] dpm, popey anyone can do review for this MR - > https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/updatePotFiles/+merge/238479 [17:02] it is just pot file update. [17:02] akiva-thinkpad, i mean it should just be a matter of putting the objectName on the slider or something? [17:02] ahayzen, slider does not have the objectname property [17:03] akiva-thinkpad, yeah but it...did? [17:03] * ahayzen looks at diff [17:03] akiva-thinkpad, yeah before it had objectName: "progressSliderShape" [17:03] ahayzen, but that was when the slider was manually built [17:03] akiva-thinkpad, try putting the test code back for seek_to and adding objectName: "progressSliderShape" to your new slider object [17:03] I tried changing it to that, but no potato. [17:03] i'll try again I suppose. [17:04] akiva-thinkpad, which revision did you try that at? [17:04] a few back, before I took out the code. It was today though. [17:04] progressSliderShape refers to the square that slides along the slider [17:05] ahayzen, so I need to reference that specific part of the slider, which I have no idea how to do now that it is a self contained component. [17:05] before it was a square rect sitting on a bunch of other rects, and they all had object names. [17:06] akiva-thinkpad, but trunk uses a Slider {} now? [17:06] akiva-thinkpad, you should be able to add objectName: "progressSliderShape" ... to the object id: progressSliderMusic...and then restore the tests [17:06] ahayzen, yah you are on to something. I'll have to check again. [17:11] ahayzen, well anyways I'll look at it tomorrow. [17:12] akiva-thinkpad, cool sorry for this being such a mess :( [17:12] ahayzen, is there a set date for the rtm yet? [17:12] akiva-thinkpad, idk 'soon' aha [17:12] ahayzen, its not your fault. I only really learned how to deal with tests today, so there is a lot of merit to what he is saying. [17:15] mihir_, reviewed, thanks! === randomcpp is now known as gcollura [17:31] akiva-thinkpad: trunk should be an issue, it just passed mine though [17:32] * mihir looks at dpm comment [17:32] mihir, okay well now that I have a bit better grasp on tests, I'll take a look at this tomorrow. [17:32] akiva-thinkpad: excellent [17:32] thanks again for walking me through it. [17:33] dpm: I'll propose new MP for that changes , is that fine ? [17:34] mihir, I'd do at least the one for plural forms and the color key (first and last inline comments) in the same MP, as we shouldn't update the .pot file without those fixed [17:34] the rest can go in a separate MP if you prefer [17:38] dpm, pushed pot file changes. i am about to push MP for translators comment. [17:38] dpm, thanks for reviewing. [17:42] dpm, regarding plural i am looking into it, i guess we got code change that's why. [17:48] thanks mihir_ [17:48] dpm, i am planning to add Translator comments in same MP , is that fine ? [17:49] ogra_, seb128, what's required to do to add a package to the rtm archive? I.e. I'm trying to add the Catalan keyboard layout, which depends on myspell-ca, but that package is only on the ubuntu archive, but not on rtm [17:50] mihir_ let me re-look at the MP in a minute [17:50] dpm, we usually channel sync requests through cjwatson for such stuff [17:50] (but he is gone ... not sure if for the day or if he'll return) [17:51] ogra_, thanks, I'll ping him tomorrow morning [17:51] dpm, that wont make the milestone image then though [17:52] (but not seeing your keyboard change in a silo i assume the keyboard wont either i guess) [17:52] ogra_, yeah, I know, it's too tight for the keyboard layout to make it in anyway [17:52] k [17:53] but I just want to get it done. I've been working on it on my ample spare time, and I didn't manage to finish it until last night :) [17:55] dpm: hmmm no issues :) [18:02] popey, you around ? [18:28] popey: can you review and approve https://code.launchpad.net/~nik90/ubuntu-clock-app/customize-splash/+merge/238484. Seems like you proposed custom splash screen for quite a number of apps ;) [18:29] popey: it has been design approved [18:31] \o/ [18:32] done [18:37] thnx :) === tvoss is now known as tvoss|test [18:44] popey, most of the critical / High bugs are fixed on Calendar :D === irv_ is now known as irv [18:56] mihir_: \o/ [19:01] Is it worth to call a reduction of 3.5ms of an item used as a delegate throughout the clock app a performance improvement :P ? [19:04] popey: could you review trivial MR , https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/MissingTranslators/+merge/238488 ? [19:04] popey: have added translator comments . [19:05] done [19:48] mihir, you around [19:53] nik90, hey, where you think I should report a bug about ListItemWithActions? [19:53] rpadovani: well its a custom component written by renatu, so he would be the best person to talk to [19:53] and so renatu, this is a ping :-) [19:53] thanks nik90! [19:54] yw [19:54] rpadovani, you can report against the address-book-app [19:54] renatu, thanks [19:54] rpadovani: btw I presume you took the ListItemWithActions from addressbook and not clock? [19:54] * nik90 hopes the former [19:54] nik90, sure [19:54] cool [19:57] renatu, bug 1381723 [19:57] bug 1381723 in address-book-app "[ListItemWithActions]After the swipe of an element I expect to press anywhere to dismiss" [Undecided,New] https://launchpad.net/bugs/1381723 === randomcpp is now known as gcollura [20:40] rpadovani, I can not reproduce this bug [20:42] rpadovani, I tried on address-book, dialer, messaging and clock app [20:42] all apps is working fine [20:42] renatu, I'm able to reproduce on them all, rtm on mako [20:43] rpadovani, ok let me flash my mako [20:44] rpadovani, ok now I understand [20:44] rpadovani, you want to tap anywhere in the screen [20:44] yes this is not implemented [20:44] renatu, yap, that is. Sorry for bad english :/ [20:44] and using a inverse mouse are can be dangerous. [20:45] and I am not sure if the designers want that [21:11] rpadovani: hey, I got a really easy MP for you -> https://code.launchpad.net/~nik90/ubuntu-clock-app/replace-base-list-items/+merge/238502 [21:11] rpadovani: basically replaces ListItem.Base with ListItem.Empty [21:36] Hi [21:47] nik90, approved but... you definitely need to take a look to http://qt-project.org/doc/qt-5/qml-qtquick-layouts-layout.html to drop all that anchors, we use it in reminders delegates and it's awesome (thanks mzanetti :D) [21:50] nik90, https://bazaar.launchpad.net/~reminders-app-dev/reminders-app/trunk/view/head:/src/app/qml/components/NotebooksDelegate.qml [22:11] rpadovani: ah yes, I know about it. I will use it in the next MP [22:12] \o/ === salem_ is now known as _salem [22:51] * vitimiti Bye