| === Guest84363 is now known as wgrant | ||
| === chriadam|away is now known as chriadam | ||
| === chriadam is now known as chriadam|away | ||
| AskUbuntu | usr/lib/i386-linux-gnu/qt5/bin/qmlscene exited with code 255 error in Ubuntu QT SDK | http://askubuntu.com/q/328768 | 05:05 |
|---|---|---|
| mihir_ | popey: Hey | 06:06 |
| mihir_ | popey: could you please test this merge proposal on device? https://code.launchpad.net/~ravirdv/ubuntu-calculator-app/calculator_label_width_fix/+merge/178497 | 06:06 |
| mihir_ | mhall119: Hey , Good Morning !! | 06:38 |
| mihir_ | mhall119: Do you have device to test? | 06:38 |
| dpm | good morning all | 06:57 |
| mihir_ | Good morning :) | 07:02 |
| oSoMoN | good morning dpm | 07:04 |
| dpm | morning mihir_, morning oSoMoN :) | 07:05 |
| dpm | hey mihir_, last time we spoke before my holidays you were trying to get involved in core apps. Have you had the chance to look more into it? | 07:05 |
| mihir_ | dpm: yeah i have started in Calculator app | 07:12 |
| dpm | excellent! | 07:12 |
| mihir_ | dpm: I am awaiting for one confimration for bug from design team to get it resoleved once it has been confirm I can work on that | 07:13 |
| dpm | mihir_, which bug is it? I can perhaps help | 07:15 |
| mihir_ | Bug :- https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1203267 | 07:15 |
| ubot2` | Ubuntu bug 1203267 in Ubuntu UX "The equal sign is placed under the result" [Undecided,New] | 07:15 |
| dpm | mihir_, thanks. I'll talk to the design team once they're up in about 1h | 07:17 |
| mihir_ | Sure :) and as i had talk with popey . he told me that we should remove (=) equal sign from the result as it is not there in our wireframes :) | 07:18 |
| mihir_ | dpm: do you have device to test ? | 07:21 |
| dpm | mihir_, I do, but I need to update it and I'm not sure I'll be able today with my mobile broadband connection | 07:22 |
| mihir_ | okay no problem :) I'll wait for someone to test one MR , before approving it :) | 07:22 |
| dpm | mihir_, I'd suggest to go ahead and propose the merge request rather than waiting. | 07:24 |
| dpm | this way people can test it and review it already | 07:24 |
| dholbach | good morning | 07:32 |
| WebbyIT | Hi dpm, welcome back :) | 07:36 |
| WebbyIT | dpm, when you have time, can you help me with https://code.launchpad.net/~rpadovani/ubuntu-clock-app/language_support_city_search/+merge/175464 ? | 07:37 |
| zsombi | oSoMoN: ping | 07:47 |
| dpm | hey WebbyIT :) | 07:47 |
| dpm | WebbyIT, done | 07:55 |
| WebbyIT | dpm, thanks! | 07:55 |
| gusch | nerochiaro: can you review? https://code.launchpad.net/~schwann/qtubuntu-camera/qtcamera-flash-reset/+merge/178508 | 07:58 |
| nerochiaro | gusch: yes, flashing my phone to latest then I'll be on it | 07:59 |
| nerochiaro | gusch: looking at the code already | 07:59 |
| gusch | nerochiaro: cool thx | 07:59 |
| oSoMoN | zsombi: pong | 08:00 |
| zsombi | oSoMoN: it seems that the IMA rework will cause some behavioral changes, which will be close to the MouseArea's behavior... | 08:01 |
| oSoMoN | zsombi: what are those changes? | 08:01 |
| zsombi | oSoMoN: like event propagation for instance happens only if you say event.accepted = false in pressed() and clicked() signals | 08:01 |
| oSoMoN | zsombi: sounds good to me | 08:02 |
| zsombi | oSoMoN: ok :) | 08:02 |
| zsombi | oSoMoN: I'm still working on it, I have few test cases broken still, but hopefully today I'll be able to fix those, so we can test it | 08:03 |
| zsombi | oSoMoN: do you have many places where you propagate events? | 08:03 |
| oSoMoN | zsombi: excellent, once you have a MR out I’ll test it | 08:04 |
| oSoMoN | zsombi: only one atm | 08:04 |
| zsombi | oSoMoN: good... however I think we will have few places in SDK where we need to check this thing, like for instance in toolbar handling... and notepad may also have some issues in case they use event propagation... let's see :) | 08:05 |
| oSoMoN | yes | 08:06 |
| tsdgeos | do we have plural handling built into i18n.tr() ? | 08:09 |
| tsdgeos | we do | 08:09 |
| timp | tsdgeos: yes, but apparently you need to add a .arg(..) that is not documented, see https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1184810 | 08:13 |
| ubot2` | Ubuntu bug 1184810 in Ubuntu UI Toolkit "[i18n] Plural form does not work as documented" [Medium,Confirmed] | 08:13 |
| tsdgeos | timp: really? that's bad :D | 08:14 |
| mihir_ | dpm: any ways to get keyboard on machine simulator ? | 08:23 |
| dpm | mihir_, I don't know, sorry :/ | 08:24 |
| nerochiaro | gusch: i'm trying your script to run jenkins test artifacts on the device, but in my case it seems to upload everything to the deveice but never actually run the tests. anything you can suggest ? | 08:27 |
| gusch | nerochiaro: did you run it with the "-s" option? | 08:28 |
| nerochiaro | gusch: yes, i tried that too | 08:29 |
| nerochiaro | gusch: http://pastebin.ubuntu.com/5950429/ | 08:29 |
| gusch | nerochiaro: hmmm - can you run "phablet-test-run -p camera-app-autopilot camera_app"? | 08:29 |
| gusch | nerochiaro: ah - the problem is, that it only works for the apps (and their autopilot tests) | 08:30 |
| gusch | nerochiaro: qtubuntu-camera does not have autopilot test | 08:31 |
| gusch | nerochiaro: you should run the camera-app tests (use my the command above) | 08:31 |
| nerochiaro | gusch: got it | 08:34 |
| gusch | nerochiaro: hmmm - and the script should be smart enough to catch such cases - but it's only a hack ;) | 08:35 |
| nerochiaro | gusch: it's ok, i wasn't thinking enough when i ran it on that branch :) | 08:35 |
| weblife | How to for Ubunto: Node.js / MongoDB setup to program to cloud with Juju: http://www.youtube.com/watch?v=d2GrIUlxOcs&feature=share&list=PLCTh_XwrphiKQHpqSG8mz_74U0_nrGxar | 08:43 |
| === dholbach_ is now known as dholbach | ||
| weblife | lol, ubunto. you know thats a typo I hope | 08:48 |
| nerochiaro | gusch: i am trying your MR for the qtubuntu-camera plugin together with the latest camera-app (from phablet-flash --pending) and i still see the same issue when i enable video recording torch then switch to front camera | 08:53 |
| gusch | nerochiaro: hmmm - maybe I adapted the camer-ui as well last week let me check | 08:54 |
| gusch | nerochiaro: works fine for me -what exact steps are you using to reproduce that issue? | 09:04 |
| nerochiaro | gusch: switch to video recording, activate torch, switch to front camera, (torch will turn off), switch back to photo capture, click on flash button, flash modes will not cycle anymore | 09:05 |
| gusch | nerochiaro: so you are still using the fron camera? | 09:07 |
| gusch | nerochiaro: so you are still using the front camera? | 09:08 |
| JamesTait | Good morning all, happy Traffic Light Day! :-D | 09:10 |
| timp | JamesTait: Good morning. Do you happen to know who came up with a Traffic Light Day, and *why*? :) | 09:13 |
| nik90 | dpm: Hi, you are back :) | 09:17 |
| JamesTait | timp, although there's a reference to a traffic light day in May, I'm going to ignore that and claim credit for celebrating the installation of the world's first electric traffic signal on this day in 1914. :) | 09:17 |
| dpm | hey nik90 :) | 09:17 |
| dpm | I am indeed | 09:17 |
| nik90 | dpm: How was your vacation? | 09:17 |
| dpm | tiring :) | 09:17 |
| nik90 | hehe | 09:18 |
| dpm | not really a vacation, I've moved countries | 09:18 |
| dpm | so I took the time to do all the move | 09:18 |
| dpm | so now I need holidays to recover from my "holidays" ;) | 09:18 |
| nik90 | ah, so where are you currently moved to? | 09:18 |
| dpm | to Germany | 09:18 |
| dpm | Stuttgart | 09:18 |
| nik90 | ooh closer :) | 09:18 |
| timp | JamesTait: :) | 09:19 |
| gusch | nerochiaro: so you are still using the front camera when you try to switch the flash? | 09:23 |
| dpm | nik90, were are you based? Was it somewhere in the Netherlands? | 09:23 |
| nik90 | yeah Delft, Netherlands | 09:23 |
| dpm | cool :) | 09:29 |
| nik90 | dpm: do we need the autopilot ppa on suacy? | 09:39 |
| nik90 | oops meant to ask baloons | 09:40 |
| nik90 | balloons: do we need the autopilot ppa on saucy? | 09:40 |
| dpm | nik90, I'm not sure, best to ask balloons when he's around or perhaps om26er | 09:42 |
| WebbyIT | dpm, do you know that nik90 became an Ubuntu Member? :) | 09:42 |
| nik90 | WebbyIT: hi :) Good Morning | 09:43 |
| dpm | WebbyIT, I wrote a testimonial for him, as he very much deserved it, but I didn't know it had already happened. Congrats nik90!!! | 09:44 |
| nik90 | dpm: thnx. I just got it a few days back. You were on holiday | 09:44 |
| dpm | nice! | 09:44 |
| WebbyIT | hi nik90 :) Following dpm suggestions I improved https://code.launchpad.net/~rpadovani/ubuntu-clock-app/language_support_city_search/+merge/175464 | 09:45 |
| WebbyIT | dpm, absolutely! | 09:45 |
| nik90 | WebbyIT: Does the online API provide good results when used in other languages? | 09:46 |
| nik90 | WebbyIT: Also what happens when the system returns a language code not present in the online API? | 09:46 |
| nik90 | For instance if there is a language used by a user but which is not available in the online api | 09:47 |
| WebbyIT | nik90, I tring only with italian and it's work... if there is no language it uses the toponym Name | 09:48 |
| nik90 | WebbyIT: ah ok. So it does fallback to something useful. That's nice | 09:49 |
| mihir_ | dpm: I am getting this error while pushing MR bzr: ERROR: Invalid url supplied to transport: "bzr+ssh://bazaar.launchpad.net/~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick": no supported schemes | 09:51 |
| mihir_ | any idea? | 09:52 |
| WebbyIT | nik90, but there is a problem: country name is not transable (eg, London, United Kingdom in Italian is Londra, United Kingdom, but has to be Londra, Regno Unito) | 09:53 |
| dpm | mihir_, can you paste the command line you used to push? | 09:54 |
| mihir_ | dpm: bzr push lp:~mihirsoni-123/ubuntu-calculator-app/RemovedOnclick/ | 09:54 |
| nik90 | WebbyIT: ah. let me check the online api | 09:56 |
| WebbyIT | nik90, http://api.geonames.org/search?q=londra&maxRows=5&username=krnekhelesh&style=full no way :-/ | 09:57 |
| dpm | mihir_, are you also getting this message by any chance? -> | 09:57 |
| dpm | You have not informed bzr of your Launchpad ID, and you must do this to | 09:57 |
| dpm | write to Launchpad or access private data. See "bzr help launchpad-login". | 09:57 |
| nik90 | WebbyIT: yeah it seems they only provide translations for the city name | 09:58 |
| nik90 | WebbyIT: it would look really wierd if the country name was in english while the city name is translated | 09:59 |
| nik90 | WebbyIT: that said, in current phones, are the city names translated? | 09:59 |
| mihir_ | dpm: resolved thanks. my mistake forgot to login in bzr launchpad | 10:00 |
| dpm | ok, cool :) | 10:00 |
| mihir_ | I forgot to setup in my new machine | 10:00 |
| mihir_ | can you help me to review that ? | 10:00 |
| dpm | no worries, glad it's sorted :) | 10:00 |
| WebbyIT | nik90, let's me check | 10:00 |
| dpm | mihir_, sure, have you sent the merge proposal? | 10:00 |
| WebbyIT | nik90, yes, on Android world city are in Italian | 10:01 |
| nik90 | WebbyIT: okay. In that case, I will create a forum post in the online API website asking if this can be done or if that feature can be added soon. | 10:02 |
| WebbyIT | nik90, looks good.. I have shared a photo on G+ with you with italian translation, FYI | 10:03 |
| mihir_ | dpm: yes done :) | 10:03 |
| mihir_ | dpm: here is the bug link :- https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1207685 | 10:03 |
| ubot2` | Ubuntu bug 1207685 in Ubuntu Calculator App "Tapping a number pastes it each time you tap" [Undecided,New] | 10:03 |
| nik90 | WebbyIT: okay. It seems we have the same phone :) | 10:04 |
| WebbyIT | nik90, \o/ | 10:05 |
| mihir_ | dpm: let me know if I have misunderstood bug :) | 10:06 |
| AskUbuntu | Glade, quickly and changing an image with code | http://askubuntu.com/q/328836 | 10:07 |
| dpm | mihir_, reviewed and commented, thanks! | 10:08 |
| WebbyIT | mihir_, dpm that's not the way: the result has to be paste only once | 10:08 |
| WebbyIT | Uops, dpm was faster than me :) | 10:09 |
| dpm | WebbyIT, yeah, that's exactly the comment I added in my review. Good to see we're thinking alike :) | 10:09 |
| WebbyIT | dpm, I need your help to understand how we work with design team :) In this bug I asked a clarification, now the bug for UX is mark as fix commited, but I don't understand what I've to do | 10:10 |
| WebbyIT | https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1207679 | 10:10 |
| ubot2` | Ubuntu bug 1207679 in Ubuntu Calculator App "Pencils appear before calculations are made" [Undecided,In progress] | 10:10 |
| * dpm looks | 10:10 | |
| dpm | WebbyIT, let me see if I can get Christina on this channel | 10:12 |
| WebbyIT | thanks dpm | 10:13 |
| * nik90 would like to know how it works as well | 10:14 | |
| nik90 | WebbyIT: Nice question! | 10:14 |
| zsombi1 | oSoMoN: Huston, we have a problem! The event propagation is an internal feature of the MouseArea and it only considers QQuickMouseAreas. This means that we can only handle proper event propagation if we derive from QQuickMouseArea. And that one is not so trivial :( | 10:19 |
| === zsombi1 is now known as zsombi | ||
| oSoMoN | zsombi: why not? QQuickMouseArea inherits from QQuickItem itself, right? | 10:21 |
| zsombi | oSoMoN: yes, and overriding contains() invokable would be enough... it's the composedEventPropagation that causes problems | 10:22 |
| zsombi | oSoMoN: if you check QQuickMouseArea implementation, you will see the propagate() and propagateHelper() methods whiuch actually do the job. No matter what I do, I cannot do proper propagation without that... but the main issue is to extend QQuickMouseArea | 10:23 |
| zsombi | oSoMoN: so far we had IMA derived from QQUickItem overriding the contains() method and filtering app events. But that's not good and implementing IMA just based on QQUickItem's mouse handling does not propagate the composed events properly :( | 10:24 |
| zsombi | oSoMoN: so clicked does not pass through :( | 10:26 |
| oSoMoN | zsombi: and can’t we simply implement IMA with two MAs, on that covers the surface of the view, and one on top that covers the item and swallows mouse events, preventing them from being propagated to the "IMA" ? | 10:27 |
| oSoMoN | as a benefit, that would be a pure QML solution | 10:27 |
| nik90 | timp: ping | 10:28 |
| zsombi | oSoMoN: I was thinking on that too. The one covering the exclusion area (not the sensing) needs to pass through the mouse events, so a MouseArea from that hole shoudl be able to get mouse events... | 10:29 |
| nerochiaro | gusch: sorry, the electricity in the building started being very sketchy, and I found out they were doing some maintenance. should be over now. I might have missed some message from you, last i saw was you asking if I am still using the front camera: basically when i switch to front camera (screen side) the torch turns off but then no matter what i do i the button that controls torch/flash can't be operated anymore | 10:30 |
| nerochiaro | (regardless if i change recording mode or camera) | 10:30 |
| oSoMoN | zsombi: unless we ensure it is always behind the associated contents in the exclusion area | 10:30 |
| zsombi | oSoMoN: there is a QML solution which uses 4 MAs, but that is a bit resource eating too. | 10:30 |
| oSoMoN | zsombi: yeah, I don’t like this solution either | 10:31 |
| zsombi | oSoMoN: the one we have now is nicer as we have one component doing the job | 10:31 |
| gusch | nerochiaro: even after switching back to the back camera? | 10:31 |
| nik90 | dpm: are any of the designers on irc? I am unable to find ckpringle, mehow or christina | 10:32 |
| oSoMoN | zsombi: an implementation with only 2 MAs should be acceptable, don’t you think? | 10:32 |
| dpm | nik90, they're not. I'm trying to get them in here | 10:32 |
| nerochiaro | gusch: ah, you're right, if i switch back it works again. i think the issue is that the screen side camera has no light/flash support and the icon should be greyed out, but isn't. | 10:33 |
| zsombi | oSoMoN: yes, if we can pass the events through the hole properly | 10:33 |
| dpm | nik90, also Calum left Canonical last week to seek new opportunities in another job at a design agency | 10:33 |
| zsombi | oSoMoN: I'll keep trying to get QQuickMouseArea derivate, and if it does the job, then we're pretty much done with it. Unless every touch event is really converted onto a mouse event | 10:33 |
| nik90 | dpm: wat. that's too bad | 10:34 |
| timp | nik90: pong | 10:34 |
| dpm | nik90, yeah, that's what I told him too :/ | 10:34 |
| oSoMoN | zsombi: but how about ensuring that the MA is below the content, you don’t need to pass events through anymore | 10:34 |
| nik90 | timp: I found a bug because of a recent update to the tab headers. | 10:34 |
| nik90 | timp: Let me grab a screenie for you | 10:34 |
| timp | nik90: ok | 10:35 |
| zsombi | oSoMoN: what do you mean? | 10:35 |
| timp | nik90: please also report it on http://bugs.launchpad.net/ubuntu-ui-toolkit | 10:35 |
| nik90 | timp: http://imgur.com/0gn6hy3 | 10:36 |
| nik90 | timp: sure | 10:36 |
| gusch | nerochiaro: yep - so I'd say it works - but the app needs to hande the "no flash"-case https://bugs.launchpad.net/camera-app/+bug/1124572 | 10:36 |
| ubot2` | Ubuntu bug 1124572 in camera-app "[camera] the n7 has no flash but the torch toggle is still enabled" [Medium,Confirmed] | 10:36 |
| nik90 | timp: The tab header is colliding with the pagestack header | 10:37 |
| gusch | nerochiaro: but the hybris and plugin needs some work for that as well (working on that) | 10:37 |
| christina | hi dpm | 10:37 |
| oSoMoN | zsombi: take the case of a popover, you want to ensure that mouse events go to whatever contents is inside the popover, right? well if the MA that blocks events from going to the IMA is below that content, then you don’t have to deal with passing events through anymore, the z-order handles it automatically for you | 10:37 |
| dpm | hey christina, thanks for joining! | 10:38 |
| nerochiaro | gusch: ok, and after the plugin tells me if the camera is supported or not please reassign the bug to me so i'll have the UI handle it properly | 10:38 |
| dpm | christina, WebbyIT, from the calculator team had a question about bug 1207679 - it seems it's been marked as fix committed without further comment | 10:38 |
| ubot2` | Launchpad bug 1207679 in Ubuntu Calculator App "Pencils appear before calculations are made" [Undecided,In progress] https://launchpad.net/bugs/1207679 | 10:38 |
| timp | nik90: how did you structure your code? similar to the pagestack+tabs example on http://developer.ubuntu.com/api/devel/ubuntu-13.10/qml/ui-toolkit/qml-ubuntu-components0-tabs.html ? | 10:39 |
| timp | (it is a new example) | 10:39 |
| WebbyIT | hi christina, thanks for your availability. I don't understand if have I to delete both pencil or only one which refers to calc | 10:39 |
| nik90 | timp: looking at it now to check | 10:39 |
| nik90 | timp: I have structures it as shown in the examples | 10:40 |
| christina | ubot2`, i will take a look now | 10:40 |
| ubot2` | christina: I am only a bot, please don't think I'm intelligent :) | 10:40 |
| nik90 | timp: so mainview -> pagestack -> tabs -> pages | 10:40 |
| zsombi | oSoMoN: I see... yes, in that case it is OK to use that. However this is one use case and in most of the cases the MA covering the hole should not even be used... so in that sense we could simply have one MA covering the sensingArea and one which is "alive" only when needed. But then we're almost there where we have one custom MA, which is what I'm trying to do now :) Let me try this prior we go with this approach | 10:40 |
| christina | dpm, WebbyIT, i will take a look now | 10:41 |
| dpm | great, thanks | 10:42 |
| timp | nik90: the example code works fine for me with qtdeclarative5-ubuntu-ui-toolkit-plugin version 0.1.46+13.10.20130731-0ubuntu1 | 10:42 |
| mihir_ | christina: Need help on https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1203267 | 10:42 |
| ubot2` | Ubuntu bug 1203267 in Ubuntu UX "[Calculator] The equal sign is placed under the result" [Undecided,New] | 10:42 |
| nik90 | timp: yeah I just tried it as well on my comp | 10:42 |
| timp | nik90: can you check your version, and see if the example code works for you? | 10:42 |
| timp | nik90: ok, it works? | 10:42 |
| nik90 | timp: the example works | 10:42 |
| timp | nik90: do you have a simple version of your code where it doesn't work well? | 10:43 |
| nik90 | timp: will try to make it simple and then show you | 10:43 |
| gusch | nerochiaro: I'll do once, hybris and plugin are done | 10:44 |
| gusch | nerochiaro: although - you can already do the UI part (and query for the supported modes) | 10:45 |
| christina | mihir_, yep looking now | 10:45 |
| timp | nik90: great, thanks | 10:45 |
| timp | nik90: when you create a bug report, can you attach the code there? Full code please including imports so I can run it directly | 10:45 |
| oSoMoN | zsombi: ok, please keep me posted | 10:46 |
| zsombi | oSoMoN: will do | 10:46 |
| zsombi | oSoMoN: hehe :) | 10:48 |
| nik90 | timp: yes | 10:48 |
| mihir_ | i am leaving will be back after some time :) thank you :) | 10:48 |
| zsombi | oSoMoN: that's the new interface :) https://pastebin.canonical.com/95423/ The API is not changed as we derive from QQuickMouseArea, which has all the properties we had (+ more of course) | 10:49 |
| mihir_ | dpm: when user clicks on that result , it should be pasted only once correct ? | 10:49 |
| dpm | mihir_, that was my understanding, yes | 10:50 |
| mihir_ | Okay :) | 10:50 |
| gusch | nerochiaro: so are you approving the MR? | 10:50 |
| mihir_ | I am done with that I will send MR once I reach home :) | 10:50 |
| nerochiaro | gusch: yeah, i was checking a few more things but it's a yes so far | 10:51 |
| oSoMoN | zsombi: looks good, where is the IMA going to be positioned (which item will be its parent?)? | 10:52 |
| zsombi | the parent can be the hole, as it was before, or more precisely the hole is the item the IMA is anchored to | 10:52 |
| zsombi | oSoMoN: ^ | 10:52 |
| christina | mihir_, hi, i've made a comment now. hope it helps :) let me knwo | 10:53 |
| zsombi | oSoMoN: the contains() method does and also did the magic. That one decides when to emit the signals | 10:53 |
| oSoMoN | zsombi: right, but effectively, will the IMA still be on top of every other item in the scene? | 10:53 |
| === MacSlow is now known as MacSlow|lunch | ||
| zsombi | oSoMoN: nope, works in the same way as the MA | 10:54 |
| zsombi | oSoMoN: I have your test.qml also included in my tests | 10:54 |
| gusch | nerochiaro: ah ok | 10:55 |
| nik90 | timp: I am unable to reproduce this with the example code even after modifying it to be similar to the clock app. But suprisingly even the rss feeder app has this same issue. | 10:55 |
| nik90 | timp: This is the structure used in the clock app http://paste.ubuntu.com/5950796/ | 10:57 |
| oSoMoN | zsombi: perfect, thanks | 10:57 |
| nik90 | timp: the interesting bit starts at line 103 | 10:57 |
| zsombi | oSoMoN: just need to adjust few tests to get the propagation properly done, then I'll push it to you for testing | 10:57 |
| oSoMoN | cool | 10:58 |
| oSoMoN | gusch: could you please review https://code.launchpad.net/~osomon/webbrowser-app/increase-coverage/+merge/178506 ? | 11:00 |
| gusch | oSoMoN: sure | 11:01 |
| oSoMoN | thanks! | 11:01 |
| nerochiaro | gusch: approved, but it would be nice to have an autopilot test on camera-app to cover this case to prevent regressions | 11:02 |
| gusch | nerochiaro: thx - can you write that test? | 11:03 |
| nerochiaro | gusch: not right now but yes. | 11:04 |
| gusch | nerochiaro: cool thx | 11:04 |
| gusch | oSoMoN: I got that error when running the autopilot test on the desktop https://pastebin.canonical.com/95424/ | 11:36 |
| oSoMoN | gusch: can you reproduce reliably the error? | 11:46 |
| gusch | oSoMoN: yes | 11:49 |
| oSoMoN | gusch: weird, let me test here (note that they pass in jenkins, so you shouldn’t have to worry about running them on your desktop) | 11:55 |
| gusch | oSoMoN: I wasn't able to run them on the device, so I did run the test on the desktop first | 12:00 |
| oSoMoN | gusch: ah, got it, that’s the test for which I had to increase the timeout, it’s racy, sometimes 2 seconds is not enough for the page to fully load and the chrome to hide… | 12:01 |
| oSoMoN | gusch: not related to this MR at all, but let me see if I can make the test more robust | 12:01 |
| === MacSlow|lunch is now known as MacSlow | ||
| oSoMoN | btw, why weren’t you able to run the tests on your device? | 12:20 |
| zsombi | oSoMoN: https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/inversemousearea-refactoring/+merge/178544 | 12:30 |
| zsombi | timp: nerochiaro: hi guys, I need your help on testing/reviewing this MR https://code.launchpad.net/~zsombi/ubuntu-ui-toolkit/inversemousearea-refactoring/+merge/178544 | 12:31 |
| zsombi | timp: nerochiaro: pls test on the device too (my tablet id dead for now, it takes some time for its revival) | 12:32 |
| nerochiaro | zsombi: i can test with notes-app on the phone | 12:32 |
| zsombi | nerochiaro: thx!!! hopefully it handles touch events also properly!!! | 12:33 |
| nerochiaro | zsombi: but looking at the code of that MR ther are various commented out pieces of code in there. i don't think you should leave them in | 12:33 |
| nerochiaro | zsombi: also debug print statements | 12:34 |
| zsombi | nerochiaro: sure not! it's a work in progress MR, if you see it working properly, I'll make it clean! | 12:34 |
| zsombi | nerochiaro: just want to keep it in needs review status to get jenkins testing it | 12:34 |
| oSoMoN | zsombi: I’m going for lunch now, will test thoroughly when I’m back | 12:35 |
| nerochiaro | zsombi: ok, i'll test it on my device when jenkins has built it. it will take me forever if i have to build it in my chroot | 12:35 |
| zsombi | oSoMoN: thx!!! | 12:35 |
| zsombi | nerochiaro: sure! it's not crucial to be tested in the next ~1h :) however if you can test it with the apps the IMA is used, it will be awesome! | 12:36 |
| nerochiaro | zsombi: the only one i know is notes app | 12:37 |
| zsombi | nerochiaro: it's enough :) oSoMoN will check the Browser, timp hopefully can check the Panel, Toolbar, Popovers | 12:38 |
| * zsombi brb | 12:44 | |
| * zsombi actually needs to be away for a bit longer, so bbl | 12:47 | |
| === zsombi is now known as zsombi|afk | ||
| === _salem is now known as salem_ | ||
| zsombi|afk | oSoMoN: nerochiaro: actually one test I touched was failing, so I pushed the cleanup to get jenkins failures fixed, now the MR should be clean for review | 12:52 |
| christina | WebbyIT, hi, just saw your earlier message about the pencil. how do you mean by delete both or only one? there should be a pencil icon per line | 13:09 |
| WebbyIT | hi christina, look at this screen: https://launchpadlibrarian.net/146464308/device-2013-08-02-104242.png | 13:10 |
| WebbyIT | christina, first pencil is for the title of the calculation, second one is a label for the number. IMO (but you are the designer) first has to be visibile without a calculation because I can write the name of the calc (e.g Shopping) before do the calc | 13:11 |
| christina | WebbyIT, ahh yes | 13:11 |
| christina | WebbyIT, i see what you mean now. In our original spec, the pencil icons (for both) would only appear after you have created a calculation | 13:12 |
| christina | WebbyIT, as this is because the focus of the app is to do calculations, n labels are nice to have. so we want users to concentrate on making calculations first, then they can add labels later. | 13:12 |
| christina | WebbyIT, hope this helps | 13:12 |
| WebbyIT | christina, thank you very much, very kind :) | 13:14 |
| christina | WebbyIT, this is the original spec http://design.canonical.com/2013/03/app-patterns-applied-calculator-key-journeys/ | 13:15 |
| christina | WebbyIT, you're more than welcome! let me know if you got other questions | 13:15 |
| WebbyIT | christina, ok, thank | 13:17 |
| gusch | nerochiaro: I'd say this rather a bug for you https://bugs.launchpad.net/camera-app/+bug/1204135 can I re -assign it to your? | 13:57 |
| ubot2` | Ubuntu bug 1204135 in camera-app "n7 preview video is off-center and the aspect ratio is wrong" [Medium,Triaged] | 13:57 |
| gusch | nerochiaro: I'd say this rather a bug for you https://bugs.launchpad.net/camera-app/+bug/1204135 can I re -assign it to you? | 13:57 |
| nerochiaro | gusch: i dont have an n7 though | 14:01 |
| gusch | nerochiaro: me neither | 14:02 |
| gusch | oSoMoN: I flashed my N10 with 20130805, and I get the following, when running my script (trying to instal) https://pastebin.canonical.com/95440/ | 14:16 |
| oSoMoN | gusch: install unity-webapps-qml manually first | 14:17 |
| gusch | oSoMoN: is this a new dependency | 14:20 |
| oSoMoN | gusch: yes | 14:20 |
| gusch | oSoMoN: ah | 14:21 |
| gusch | oSoMoN: approved | 14:34 |
| oSoMoN | zsombi|afk: looks like the unit tests are segfaulting when run inside a chroot | 14:34 |
| oSoMoN | gusch: thanks | 14:34 |
| nerochiaro | zsombi|afk: looks like your IMA merge request is still failing according to jenkins | 14:52 |
| nerochiaro | gusch: how do we deal with bugs that happen on devices we don't have ? | 14:53 |
| gusch | nerochiaro: either find someone with that device to fix it, or someone has to send the device | 15:03 |
| gusch | nerochiaro: but that bug is on the n7, which is not the current focus - so we could wait for a while to fix it | 15:04 |
| nerochiaro | gusch: ok | 15:05 |
| nerochiaro | om26er: can you repro this bug with today's image ? https://bugs.launchpad.net/camera-app/+bug/1179592 | 15:39 |
| ubot2` | Ubuntu bug 1179592 in camera-app "Zoom slider +/- icons missing, causes autopilot test to fail" [Medium,Confirmed] | 15:39 |
| nerochiaro | om26er: on galaxy nexus | 15:39 |
| om26er | nerochiaro, I only have nexus 4 | 15:39 |
| om26er | nerochiaro, but that should be fine I guess | 15:39 |
| nerochiaro | om26er: it works fine here on galaxy nexus | 15:39 |
| nerochiaro | om26er: is nexus 4 one of the devices we're focusing on ? | 15:40 |
| om26er | nerochiaro, yeah its the second priorirt | 15:40 |
| om26er | *priority | 15:40 |
| om26er | nerochiaro, actually when I reported that but the issue was on desktop, and I just checked and it happens there still | 15:40 |
| nerochiaro | om26er: ok, then someone with the nexus 4 need to take a look at fixing it | 15:40 |
| nerochiaro | boiko: ^ | 15:41 |
| om26er | nerochiaro, the issue also does not happen on nexus 4, only on desktop/laptops | 15:41 |
| nerochiaro | om26er: hmm, then the bug is definitely not a priority, right ? | 15:41 |
| om26er | nerochiaro, yeah, I guess | 15:42 |
| nerochiaro | om26er: in any case can you please comment on the bug mentioning that it happens only on desktop ? | 15:42 |
| om26er | nerochiaro, sure | 15:42 |
| abhidoeslinux | hello everyone, is there a privilege to use java in QT creator? | 16:16 |
| === zsombi|afk is now known as zsombi | ||
| mihir | dpm: ping !!! | 16:39 |
| mihir | dpm: you there ? | 16:41 |
| ahayzen | dpm, Thanks for the email | 16:54 |
| dpm | hey ahayzen, no worries, just came back from holiday and while doing catch up I thought I'll forward it to you | 17:11 |
| ahayzen | dpm, thanks :) how was ur holiday? | 17:12 |
| mihir | dpm: we just need result to be copied once in bug https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1207685 ? | 17:20 |
| ubot2` | Ubuntu bug 1207685 in Ubuntu Calculator App "Tapping a number pastes it each time you tap" [Medium,In progress] | 17:20 |
| UbuPhillup | popey: what about this bug https://bugs.launchpad.net/ubuntu-filemanager-app/+bug/1199461 still in progress? | 17:22 |
| ubot2` | Ubuntu bug 1199461 in Ubuntu File Manager App "The app is not translatable" [High,In progress] | 17:22 |
| mihir | WebbyIT: Ping !! you there? | 17:23 |
| WebbyIT | hi mihir :) | 17:24 |
| mihir | Could you help me to review MR ? | 17:24 |
| WebbyIT | mihir, which MR? | 17:25 |
| mihir | WebbyIT: https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1207685 | 17:27 |
| ubot2` | Ubuntu bug 1207685 in Ubuntu Calculator App "Tapping a number pastes it each time you tap" [Medium,In progress] | 17:27 |
| mihir | WebbyIT: Now, it just copy result once :) | 17:27 |
| WebbyIT | mihir, good news! Let's me look | 17:28 |
| WebbyIT | mihir, needs fixing, see my comment | 17:36 |
| mihir | WebbyIT: Okay Let me check this scenario :) | 17:39 |
| mihir | mhall119: for this xx-large fonts will work ?http://ubuntuone.com/3uhvRhJ0LXNgpnlbPUzwcX | 18:27 |
| mihir | WebbyIT: I am unable to reproduce first scenario could you help me ? | 18:31 |
| WebbyIT | mihir, I'm sorry, I have to go: but I checked and yes, I can't reproduce the first scenario, I don't know what I saw :-/ | 18:57 |
| WebbyIT | Have a nice evening! | 18:57 |
| mihir | Okay :) bye :) | 18:57 |
| om26er | so the NewTabBar is now known as TabBar we need to change that in our tests, | 19:21 |
| om26er | timp, hey | 19:24 |
| om26er | timp, when was that change. NewTabBar to TabBar ? | 19:25 |
| om26er | boiko, ping | 19:27 |
| === om26er is now known as om26 | ||
| === om26 is now known as om26er | ||
| boiko | om26er: pong | 19:29 |
| om26er | boiko, please review this https://code.launchpad.net/~om26er/phone-app/adapt_autopilot_test_to_latests_sdk_change/+merge/178625 | 19:29 |
| om26er | boiko, it seems the sdk changed recently | 19:29 |
| boiko | om26er: ok, I'll just wait for CI to run to make sure tests pass, thanks for fixing it | 19:30 |
| timp | om26er: the change was last week | 20:55 |
| om26er | timp, ok, we need to change our app's autopilot suites to adapt :) | 20:56 |
| timp | om26er: support for autopilot is being added to the UITK, so you can call functions from there, and you ddondfdfdon't need to make these changes in the future | 20:56 |
| timp | om26er: elopio_ is the best person to ask for details about that | 20:57 |
| om26er | timp, yeah, I looked at that, that's a bit more work that just renaming NewTabBar in our tests, we'd be doing that for our apps, soonish | 20:57 |
| om26er | I do have a few improvements for the emulators in the UITK in mind, which i'll propose soon | 20:58 |
| timp | ok, good | 20:58 |
| timp | oops | 20:59 |
| timp | "ddondfdfdon't" meant don't (network issues, I didn't see what I was typing) | 20:59 |
| === salem_ is now known as _salem | ||
| mpage | I am trying to hook apt pre-install-pkgs, my script runs but I can't figure out how to get the list of changes | 23:12 |
| mpage | if I try reading from stdin it just waits for user input | 23:12 |
| mpage | google says the info should be on stdin, but that there was a proposal to put it in a pipe | 23:13 |
| mpage | named pipe | 23:13 |
| mpage | any ideas | 23:13 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
