=== salem_ is now known as _salem
ameerHey.... any one there03:40
ameerHow to detect filemanager data in qml03:42
mihir_Good Morning guys :)07:07
dpmgood morning07:38
mihir_dpm:  Good morning :)07:41
mihir_Anyone with device can test this MR on device ?  https://code.launchpad.net/~ravirdv/ubuntu-calculator-app/calculator_label_width_fix07:41
dholbachgood morning07:55
dpmmorning dholbach07:58
dholbachhey dpm07:59
=== seb128_ is now known as seb128
seb128hey dpm dholbach07:59
dpmmorning seb128, how's it going today?08:00
dpmgood work with the system settings summary e-mails btw, they're very useful to see what's going on08:01
seb128dpm, thanks ;-)08:01
mihir_dpm:  hey , how are you doing ?08:36
dpmhi mihir_, very well, thanks, and you?08:41
mihir_yes doing good :) thank you :)08:41
mihir_dpm:  I have translated calculator app in my native lanungage..the application will automatically take the translation ?08:42
dpmmihir_, if you translated it in Launchpad, yes. Translations are picked up from there and automatically committed to trunk daily :)08:43
mihir_dpm:  okay thank you :)08:43
oSoMoNgusch: good morning08:48
guschoSoMoN: hi08:48
oSoMoNgusch: first MR of the day: https://code.launchpad.net/~osomon/webbrowser-app/lastVisit/+merge/17879008:48
guschoSoMoN: you start early ;) - I'm on it ...08:48
oSoMoNgusch: I’m cheating, I submitted it yesterday :)08:54
guschoSoMoN: is there something to test in the UI?09:01
oSoMoNgusch: it’s not 100% reproducible, but I’ve observed that sometimes the list of domains doesn’t remain sorted chronologically, so if you open a number of different websites while keeping the activity view open, eventually you will see that some domains move to where they shouldn’t be, and this MR fixes it09:02
oSoMoNgusch: basically, iterating over the list of entries outside of the model itself was unreliable, as there was no guarantee the indexes had been fully updated (most of the time they were half updated, which caused weird issues)09:03
oSoMoNgusch: doing it inside the model guarantees that when m_lastVisit is assigned, the indexes have been fully updated, and so the first entry is guaranteed to be the one we’re interested in09:04
JamesTaitGood morning all, happy Lighthouse Day! :-D09:14
guschoSoMoN: approved09:15
oSoMoNgusch: thanks!09:15
=== chriadam is now known as chriadam|away
mihir_dpm:  could you please help me to review this https://code.launchpad.net/~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick-1207685/+merge/178526 if you get time :)09:57
dpmmihir_, sure, looking at it now...10:01
dpmmihir_, done, thanks for the branch!10:09
mihir_dpm:  Thank you :) I'll look into it :)10:11
mihir_dpm: One question I found another scenario ,10:18
mihir_2+2 = 4 now what if user tap on the previous result ?? what should be the exepected result ??10:18
nerochiarooSoMoN: zsombi: Kaleo: I have been tasked to transform the share-app in a component. do you think it should live inside ubuntu-ui-extras or in a new separate repo ?10:20
nerochiarooSoMoN: zsombi: Kaleo: or maybe split share-app in app and component and leave everything in there, and the app will be used only as example or for tests ?10:20
dpmmihir_, let me get christina in here to ask her10:25
mihir_dpm:  sure :)10:25
mihir_dpm: I am unable to review translation done by someother guy in calendar application https://translations.launchpad.net/ubuntu-calendar-app/trunk/+pots/calendar-app/gu/+translate?memo=10&start=1010:26
guschnerochiaro: can you do that? https://code.launchpad.net/~schwann/qtubuntu-camera/qtcamera-vp-flash/+merge/17878410:27
nerochiarogusch: are you sure that there is no possibility of phones haveing a screen-facing camera and a light ?10:30
dpmhi christina, thanks for joining! mihir_ had a question on what to do when tapping on the result from a previous calculation. mihir_ could you expand on that?10:30
christinahi dpm, hi mihir_ - i am just going into a meeting now, but if you leave me some notes i can answer when im back :)10:31
zsombinerochiaro :Kaleo: my opinion is to be placed in extras10:32
mihir_christina: Hi Christina thank you for joining I am working on bug https://code.launchpad.net/~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick-1207685/+merge/17852610:32
oSoMoNnerochiaro: re-the split of share app, I think it would be a good candidate for ubuntu-ui-extras, but the other option you’re proposing sounds good too10:32
mihir_It copy result from previous result10:33
mihir_In following scenario , what should be the exepcted result10:33
mihir_2+2 = 4 now what if user tap on the previous result (Without starting new converstation) ?? what should be the exepected result ??10:33
nerochiarooSoMoN: zsombi: Kaleo: i'll go with extras then. Will ask to remove the share-app repo later when it's done10:33
mihir_christina: No issues :) this is a scenario i came across let me know the expected result of this scenario :)10:34
dpmmihir_, my understanding is that nothing should happen as I described on the merge proposal, but let's see if design has got any other view10:34
mihir_dpm: so that means untill user start new converstation it shouldn't react right once equal is tapped?10:35
dpmmihir_, that's what I understand, yeah. Until the user tears off the calculation, tapping on operands or on the result of the current calculation shouldn't do anything10:36
dpmbasically, the same behaviour that the app has at the moment10:37
dpm(except for the bug)10:37
mihir_dpm: Okay :)  let me try to implement that :)10:38
dpmcool, thanks!10:38
dpmnerochiaro, oSoMoN, gusch, if you're on the latest build and have got a device with you, could you help me and christina determine if the weather app toolbar works as expected? It's basically about loading the weather app on the device and pull the toolbar to see if it's empty or if it contains the toolbar actions as expected10:39
dpmit shouldn't take more than a few seconds, and would be really helpful :)10:39
oSoMoNdpm: sure, will do that in a moment once the tests that I’m running on the device are done10:40
dpmgreat, thanks!10:40
guschnerochiaro: thi si not the to properly detect if the light is available (from android)10:46
oSoMoNgusch: next MR of the day: https://code.launchpad.net/~osomon/webbrowser-app/fix-crash-collapsing-domain/+merge/17891910:46
nerochiarodpm: i need to reflash to the latest. do you want me to test on a regular update or with --pending ?10:46
nerochiarogusch: sorry, didn't undersand your last comment10:47
dpmnerochiaro, a regular update would do10:47
oSoMoNdpm: I’m not seeing any toolbar in the weather app, when I swipe from the bottom edge I’m getting the HUD icon only10:47
nerochiarodpm: ok, flashing, will test in a bit10:47
oSoMoNdpm: I’ve got ubuntu-weather-app 0.2bzr84saucy010:47
dpmoSoMoN, thanks. And you had a location defined in the app, too?10:48
guschnerochiaro: about your question "are you sure that there is no possibility of phones haveing a screen-facing camera and a light ?"10:48
guschoSoMoN: looking at it10:48
nerochiarogusch: ok, but then i'm not sure what the MR is about. you basically remove all the code for reporting the flash/torch modes and replace it with hardcoded results10:49
oSoMoNdpm: it says "Current location | Lookup location (Dummy)"10:49
oSoMoNdpm: wait, I was on the "add location" screen which showed up by default when I started the app10:50
guschnerochiaro: well - most important here is, that photo and video share the same flash modes10:50
dpmoSoMoN, ah, you're on the initial screen with the location settings. Could you try to add a location and press confirm, and then see if you can pull the toolbar on the next screen?10:50
guschnerochiaro: that's needed for flash10:50
guschnerochiaro: that's needed for torch10:50
oSoMoNdpm: now that I chose a city, I’m seeing the toolbar work, it has three actions "refresh", "edit", "settings"10:50
dpmoSoMoN, perfect, so it seems to work as expected. Thanks!10:50
oSoMoNdpm: the app looks pretty neat, first time I’m running it :)10:51
dpmnice :)10:51
dpmoSoMoN, if you notice any bugs, feel free to report them on https://bugs.launchpad.net/ubuntu-weather-app/ :)10:52
=== MacSlow is now known as MacSlow|lunch
nerochiarogusch: so basically it should be possible to enable also torch while in picture mode ?10:56
oSoMoNdpm: sure, will do10:56
oSoMoNdpm: would "the weather is weird around here today" be considered a valid bug? ;)10:57
guschnerochiaro: yes, not in the camera, but for the Qt QML Torch component10:57
dpmoSoMoN, perhaps, but don't expect the developers to fix the weather :-)10:57
* oSoMoN knew it…10:58
nerochiarogusch: well, i'm gonna approve it because it doesn't seem to have any real effect on camera.10:58
nerochiarogusch: but i'm still not sure i see the point10:58
guschnerochiaro: hmmm - we don't have a torch app, to see that it currently is broken ...10:59
guschnerochiaro: but thx10:59
=== marcoceppi_ is now known as marcoceppi
guschoSoMoN: approved the crash11:26
oSoMoNgusch: thanks11:27
nerochiarodpm: i got no toolbar on the weather app as well12:02
nerochiarogreyback: on the latest image have you noticed how the shell isn't showing icons for some apps ?12:03
nerochiarogreyback: and the converter app appears twice12:03
=== MacSlow|lunch is now known as MacSlow
greybacknerochiaro: this in apps lens, or launcher?12:05
nerochiarogreyback: app lens12:05
nerochiarogreyback: reflashing with --pending as i need to test something else, will let you know if it's still happening12:06
greybacknerochiaro: that data is coming from an actual apps lens, it's not faked any more. Lens got confused??12:06
nerochiarogreyback: i guess. let's see if it's still there, if it is we can try to figure out what's happening12:07
greybacknerochiaro: agreed12:07
dpmnerochiaro, thanks for testing. And this is after you've added a location?12:09
nerochiarodpm: no, didn't try that12:09
dpmnerochiaro, ah, you'll need to add a location first. The toolbar is only shown on the weather screen12:09
nerochiarodpm: i find it confusing that you can't add a location from the toolbar12:10
nerochiarodpm: i'm currently reflashing the phone again, will try adding a location when it's finished and let you know12:10
dpmnerochiaro, that's probably a question for design, but essentially, the "new location" screen is shown if you don't have any locations defined upon start, so there is no need for a toolbar there. Once you've added a location, the weather screen is shown and from there you can manage your locations from the toolbar12:11
nerochiaroKaleo: zsombi: timp: oSoMoN: ubuntu-ui-extras is still pending the initial commit, can anyone take care of approving it now that Jenkins has all test passing on the MR: https://code.launchpad.net/~amanzi-team/ubuntu-ui-extras/ubuntu-ui-extras-initial/+merge/17108612:11
nerochiarodpm: ok, if that's the way it's been designed let's roll with it12:12
zsombinerochiaro: uhh... that's not that fast to review... hasn't this been reviewed by someone of us months ago?12:15
nerochiarozsombi: that's fast because after the last commit there's only a skeleton example left in there12:15
nerochiarozsombi: and yeah, you guys started reviewing but never finished12:16
zsombinerochiaro: so a proper review has never been done then...12:16
nerochiarozsombi: not a full one I guess12:17
nerochiarozsombi: i'm also not sure who should be in charge of that12:19
zsombinerochiaro: :) "you are not alone..."12:19
om26ergusch, ping12:23
guschom26er: pong12:23
om26erhey, this test is specifically failing on mako camera_app.tests.test_focus.TestFocus.test_focus_valid_and_disappear12:23
om26ergusch, i have seen in a video that the camera goes blank12:24
om26ergusch, it does not happen always but in most cases (given you have the very latest image) running the whole suite will end up with a failure of that test12:24
guschom26er: hmmm - might be https://bugs.launchpad.net/camera-app/+bug/120426012:25
ubot2`Ubuntu bug 1204260 in touch-preview-images "[camera-app][mako] switching cameras crashes mm-qcamera-daemon" [High,Confirmed]12:25
guschom26er: but I don't have mako, so I can't check it ...12:25
om26ergusch, its probable, the issue happened while it was switching cameras12:25
om26ergusch, you should get a nexus 4 :)12:26
guschom26er: and a 7, and ... ;)12:27
nerochiarozsombi: so any idea what should I do about it ?12:39
zsombinerochiaro: checking the MR...12:39
zsombinerochiaro: I guess it should be us who should own it, but discuss this with bzoltan12:41
nerochiarobzoltan: zsombi: Kaleo was of the idea that the sdk team should own it too, if i recall correctly12:45
nerochiarobzoltan: talking about lp:ubuntu-ui-extras12:45
nerochiarogreyback: still broken (missing icons) in latest image with --pending12:46
bzoltannerochiaro: let me see it12:46
nerochiarobzoltan: the main branch is empty, there's a branch pending merge into it that contains the current initial code12:47
nerochiarobzoltan: it's only a skeleton example for adding a new component in there, at the moment12:47
zsombibzoltan: nerochiaro: if we get it under SDK, then my only question is whether we should have a separate project for it, or should we put under the same code base as the current SDK is.12:47
zsombibzoltan: nerochiaro: the current branch (the initial one) is based on cmake and our one is on qmake...12:48
nerochiarobzoltan: zsombi: separate project, as it doesn't have to match the same standards as the sdk (as evidenced by the description that Kaleo put on the lp project)12:48
nerochiarozsombi: i'm fine with either, i just used cmake as it was what i was used to from other projects12:48
zsombinerochiaro: then it can live in its own way, with whatever build system it has now12:49
nerochiarodpm: with latest image and --pending the weather app shows the toolbar after adding a location. toolbar has refresh, edit and settings buttons12:49
nerochiarozsombi: great12:49
dpmperfect, thanks a lot nerochiaro for testing!12:49
zsombinerochiaro: I have nothing else to comment on the MR then :)12:49
bzoltannerochiaro:  let me talk with Kaleo about it.12:49
nerochiarodpm: you're welcome12:50
nerochiarozsombi: then give your comment-approval at least12:50
nerochiarobzoltan: ok, thank you12:50
zsombinerochiaro: you have it!12:51
bzoltannerochiaro:  My inner sense says that if something belongs to the SDK team than the same quality standard applies as to any other project... If that project would be a home for components and APIs we do not officially support or do not consider mature enough then I would suggest to create a Ubuntu.Components-Extras under the UI TK project.12:52
=== _salem is now known as salem_
zsombibzoltan: +112:56
nerochiarobzoltan: the current organization and repo was discussed with Kaleo and AFAIK approved by him12:59
bzoltannerochiaro: Is there a mail, irc log or anything about it?12:59
bzoltannerochiaro: I would like to hear Kaleo's idea in details12:59
Kaleonerochiaro: do you have the link to the google doc?13:00
nerochiaroKaleo: bzoltan: looking for the doc13:00
nerochiaroKaleo: bzoltan: https://docs.google.com/a/canonical.com/document/d/1xGmQd1qRMFIHIupybpzFmIZHRGUmRhSJanX62ElG5EA/edit13:01
=== greyback is now known as greyback|lunch
bzoltanGuys that document starts with perfect explanation why I would never touch that project :)13:02
bzoltanQML components that either: - won’t necessarily see much reuse - does not have the necessary level of quality ...13:02
bzoltannerochiaro: Kaleo:  Why to move our finger if a component (1)is useless crap (2)13:03
nerochiarobzoltan: Kaleo: i don't necessarily agree with that definition. I just want to have a place where to put some components that don't have any other place to go. for example now i need to put the sharing component somewhere, and that seems a pretty good place13:05
Kaleonerochiaro: go for it13:05
Kaleonerochiaro: however, review wise I would suggest to do what you would do for your app13:06
bzoltan nerochiaro:  we need a sharing component. It is on the list....13:06
nerochiaroKaleo: yes, but before that I would like to have the pending merge request merged into the trunk of that repo. i'll then have whoever would be responsible to review the share-app review the share component when i propose it for review into the ubuntu-ui-tookit repo13:07
nerochiarobzoltan: i was tasked by bfiller to transform share-app into a component. i need a place to put it. that's all i know. if you guys are planning to make a different one, we need to talk about it13:07
bzoltannerochiaro:  let's do it the right way than... but I would not suggest to create a shadow UITK where different rules apply than in the real one....I agree that if the community needs components and want to create projects-packages they are free to do.13:08
nerochiarobzoltan: i'm not sure what you're suggesting I do13:11
Kaleonerochiaro: I see the MR is approved13:12
nerochiaroKaleo: thank you13:12
Kaleonerochiaro: 20 minutes ago :)13:12
nerochiaroKaleo: yeah :)13:12
om26ernik90, when you get the time https://code.launchpad.net/~om26er/ubuntu-clock-app/fix_autopilot_on_touch/+merge/17844313:30
=== greyback|lunch is now known as greyback
nik90om26er: definitely13:31
bzoltannerochiaro:  I suggest that if you plan to work on a component what is on the roadmap of the UITK project then just contribute it to the UITK13:32
nerochiarobzoltan: i had no idea it was on the roadmap, and i suppose that i can start working on it there and then if you guys think it's ok we can move it to the main UITK ?13:37
nik90om26er: 3 tests are failing. You can find the output at http://paste.ubuntu.com/5958920/13:38
nerochiarozsombi: are the changes in IMA we were discussing yesterday already in the SDK that goes into today's image smoke tests ?13:38
nik90om26er: you used timer_page and then returned time_page instead13:38
om26ernik90, fixing13:39
nik90om26er: :)13:39
zsombinerochiaro: haven't checked, however it got merged into the SDK trunk...13:39
om26ernik90, done13:40
bzoltannerochiaro:  Good that it came out then :) Yes, let's shoot for normal and full integration to the UI TK13:40
zsombinerochiaro: this one also sais it's in: https://code.launchpad.net/~ps-jenkins/ubuntu-ui-toolkit/latestsnapshot-0.1.46+13.10.20130807-0ubuntu1/+merge/17892013:40
nerochiarozsombi: it seems that we have one test failing and i suspect this to the cause13:42
guschoSoMoN: to make gallery crop test work with the latest SDK https://code.launchpad.net/~schwann/gallery-app/gallery-close-toolbar-on-edit/+merge/17896113:45
zsombinerochiaro: which one?13:45
oSoMoNnerochiaro: in the tests where you wait for the note to be expanded, are you also verifying that the 'state' property of the note is 'expanded' (or something like that)?13:47
nerochiarozsombi: https://jenkins.qa.ubuntu.com/job/saucy-touch-mako-smoke-notes-app-autopilot/47/consoleText13:48
nerochiarooSoMoN: i'm not in this specific case, but the property change is what actually triggers the height change, so i thought there was no need to do it13:49
oSoMoNnerochiaro: agreed, in theory there shouldn’t be any need, but in practice having more checks allows easier debugging of failures, which is why I was asking13:51
nerochiarooSoMoN: makes sense,13:51
guschoSoMoN: to make gallery crop test work with the latest SDK https://code.launchpad.net/~schwann/gallery-app/gallery-close-toolbar-on-edit/+merge/17896114:08
oSoMoNgusch: is there a bug associated to your MR?14:18
guschoSoMoN: no - I only came across that issue, because I wanted to get this merged... https://code.launchpad.net/~schwann/gallery-app/gallery-uitk-emulators/+merge/17833014:18
oSoMoNgusch: ok, thanks14:19
=== mpt_ is now known as mpt
nerochiarooSoMoN: i'm not really sure what to do to fix that failing test, i doubt adding the extra check or isExpanded will solve the problem, but I can certainl try14:43
oSoMoNnerochiaro: can you point me to the code of the failing test? I’ll have a look too14:44
oSoMoNgusch: approved, please see my comment14:49
nerochiarooSoMoN: sure, hold on14:49
nerochiarooSoMoN: http://bazaar.launchpad.net/~phablet-team/notes-app/trunk/view/head:/tests/autopilot/notes_app/tests/test_parts.py#L55 this is the function that fails in that specific case, but as you can see  it is used other cases that don't fail14:51
guschoSoMoN: thx14:56
oSoMoNnerochiaro: what is the height of a collapsed note?15:00
nerochiarooSoMoN: 11 gu15:01
oSoMoNnerochiaro: ok, so in the failing test the note wasn’t expanded at all, it remained collapsed (as opposed to being expanded halfway or something like that)15:02
nerochiarooSoMoN: ok, that means that the click on the note wasn't registered for some reson. but it's a standard click so i'm not sure what can go wrong. self.pointing_device.click_object(note)15:03
mhall119a new contest for Ubuntu Touch app developers!15:34
WebbyITmhall119, \o/15:34
nik90mhall119: awesome!15:39
WebbyITdpm, about bug 1204702 you saw that the music app team as wrote some code, but I'm not able to find it, can you help me?15:45
ubot2`Launchpad bug 1204702 in Ubuntu Calculator App "ubuntu-calculator-app does no db upgrade to 0.1.1 on first start" [High,In progress] https://launchpad.net/bugs/120470215:45
WebbyITdpm, if you have time, I'll say you what problem I'm finding to fix it15:45
dpmWebbyIT, let me see15:46
WebbyITdpm, I'm searching a way to have more upgrade, one after the other, so I think to a recursive function but...15:47
WebbyIT... dpm, db.changeversion doesn't work as I expected, see http://askubuntu.com/questions/327594/db-changeversion-doesnt-work-as-expected15:48
WebbyITdpm, I thinking something like http://paste.ubuntu.com/5959364/15:49
WebbyITdpm, I can do more cases to avoid intermediate version of db, but in case of future upgrade will be a mess15:50
dpmWebbyIT, I've just found how they did it at the bottom of the diff for https://code.launchpad.net/~danielholm/music-app/show-tracks-in-playlists/+merge/17533315:53
dpmhowever, I'm not sure it's the best way15:53
WebbyITdpm, that is a wrong way, because at the first start the db will be create but NOT update to v1.115:54
dpmyeah, I didn't look into it in detail, but somehow it didn't look good to me15:55
WebbyITdpm, is the same code we are in calc, we have to find a way to have something that can allow to have more than one upgrade15:56
WebbyITdpm, I also tried http://stackoverflow.com/questions/11834739/phonegap-android-database-changeversion-not-functioning dut unfortunately doesn't work :(16:00
=== greyback__ is now known as greyback|food
=== greyback|food is now known as greyback
boikoom26er: ping16:47
om26erboiko, pong16:47
boikoom26er: hi, do you know why on this MR jenkins didn't post the URL for the deb packages? https://code.launchpad.net/~tiagosh/telepathy-ofono/audioflinger-speakermode/+merge/17887316:47
om26erboiko, the job needs to be changed to post debs16:49
om26erboiko, I can enable that if you want16:49
boikoom26er: that would be awesome16:50
boikoom26er: thanks16:55
om26erboiko, its good to go, you can re-build to have the debs now17:06
boikoom26er: thanks!17:07
om26erboiko, np :)17:08
om26eroSoMoN, ping17:09
oSoMoNom26er: pong17:10
om26eroSoMoN, what was it about in #phablet ? I mean no one even knew the bug number ?17:10
oSoMoNom26er: looks like there is no bug report, there should be one but I can’t find it17:10
om26eroSoMoN, so probably we cannot be blamed ;)17:10
oSoMoNno indeed17:11
nik90m-b-o: hey the new flip animation looks cool! Nice one!17:32
m-b-onik90: yes, neokore did some really nice work17:38
m-b-onik90, perhaps the visual design will land this week, so more to come :)17:39
neokorethank you nik90 m-b-o17:39
nik90m-b-o: oh wow..looking forward to them17:39
nik90neokore: nice work :)17:39
iBelievenik90, ping19:20
boikoseb128: is the fix for libgsettings-qt-dev already merged? (if CI is failing, autolanding is going to fail too for the sound settings MR)19:27
seb128boiko, yes, that landed on friday19:28
boikoseb128: ah nice, let me trigger CI again then, thanks19:29
seb128boiko, thanks ;-)19:29
n00bHow do I run just the ubuntu touch os in the sdk19:40
=== n00b is now known as Guest63986
Guest63986How do I run just the ubuntu touch os in the sdk?19:41
gatoxhi, i have a question regarding the u1db-qt plugin.... y want to create a db for my app... but i can't use the: "~/.config/db" notation in path.... which would be the proper place to create the db?? i assuming that i don't have write permissions where my app was installed20:08
DavidedduIs it allowed to use a different library for a Showdown application? I want to write my app in Python and Kivy, which is very good for mobile/touchscreen apps.21:14
=== salem_ is now known as _salem

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