| === chriadam|away is now known as chriadam | ||
| === _salem is now known as salem_ | ||
| === salem_ is now known as _salem | ||
| dholbach | good morning | 06:57 |
|---|---|---|
| mzanetti | tmoenicke: hey... how's the progress with the keyboard testing? | 08:20 |
| mzanetti | tmoenicke: we would like to start integrating the shell tests with some OSK usage | 08:23 |
| mzanetti | tmoenicke: do you have some emulators ready we could reause? | 08:23 |
| mzanetti | reuse | 08:23 |
| tmoenicke | mzanetti: lets talk to bill about this in the afternoon | 08:24 |
| JamesTait | Good morning all, happy Autistic Pride Day! :-D | 08:31 |
| seb128 | hum | 08:36 |
| seb128 | is there a "ubuntuorange" color in the toolkit? ;-) | 08:36 |
| dpm | hi seb128, the sdk guys can probably give a better answer, but I haven't found it, I just specify "#DD4814" :) | 08:46 |
| seb128 | dpm, ok, that's cheating :p | 08:46 |
| seb128 | dpm, hey btw, and thanks ;-) | 08:47 |
| dpm | :) | 08:47 |
| seb128 | I was going to do that but I figured out that it would be nice to have a named color for that one | 08:47 |
| seb128 | I guess quite some apps want to use to give an ubuntu look | 08:47 |
| timp | seb128: no, there is no UbuntuOrange yet https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1098209 | 08:50 |
| ubot2 | Ubuntu bug 1098209 in Ubuntu UI Toolkit "define the ubuntu orange color in the SDK" [Wishlist,Confirmed] | 08:50 |
| seb128 | timp, thanks ;-) | 08:57 |
| gusch | om26er: can you review this? https://code.launchpad.net/~schwann/gallery-app/gallery-video-autopilot/+merge/170015 | 09:08 |
| om26er | gusch, looking | 09:08 |
| gusch | om26er: thx | 09:08 |
| dholbach | dpm, seems like the developer.ubuntu.com/packaging link is broken | 09:14 |
| dholbach | dpm, do you know what happened there? | 09:14 |
| dholbach | dpm, FWIW http://developer.ubuntu.com/resources/tools/packaging/ is broken too | 09:23 |
| dpm | dholbach, looking at it now | 09:35 |
| dholbach | dpm, thanks | 09:37 |
| zsombi | guys, do we have a way to detect the running QML application name? (i.e. desktop file name, main QML file)? Arguments SDK component aint seem to provide that info... | 09:39 |
| timp | gusch: I'm a bit stuck with autopilot (I don't have much experience with it). | 10:02 |
| timp | gusch: when I return the "cancel icon", I know that I need tools.back from the toolbar. But is there a way with autopilot to do that besides adding objectNames to the back/cancel button everywhere? | 10:02 |
| gusch | timp: you can use any property | 10:03 |
| timp | gusch: an easy way to do it is by looking for the button with text "Cancel" or "Back", but that will break on systems that use different languages | 10:03 |
| gusch | timp: correct | 10:03 |
| timp | gusch: #toolbar_items = toolbar.select_single("ToolbarItems") #return toolbar_items.back | 10:03 |
| timp | without the # | 10:03 |
| timp | that doesn't work, but you get the intention. I want to get the back-property of toolbar_items | 10:03 |
| gusch | use select_many instead of select_single | 10:04 |
| gusch | timp: ^ that might help | 10:04 |
| timp | why many? there is only one | 10:04 |
| gusch | for sure? | 10:04 |
| timp | the problem is AttributeError: Class 'ToolbarItems' has no attribute 'back'. | 10:05 |
| timp | in qml I have this in ToolbarItems: property Item back | 10:05 |
| timp | I just don't know how to access it from python | 10:06 |
| gusch | "back" is a QML property? | 10:06 |
| timp | yes | 10:06 |
| gusch | which type? | 10:07 |
| timp | Item | 10:07 |
| timp | well, I noticed that now I have several subclasses of ActionItem (including Button, ToolbarButton), and if select all ActionItems it doesn't seem to include Buttons | 10:08 |
| gusch | then return toolbar_items.select_many("QQuickItem")[0] | 10:08 |
| gusch | then "return toolbar_items.select_many("QQuickItem")[0]" | 10:08 |
| gusch | or whatever index the back item has (that's a least not worse than the current test ;) | 10:08 |
| timp | ohh I need to use the cpp classname, not the qml name? | 10:08 |
| gusch | yep | 10:08 |
| gusch | you can run "gallery-app -testability" - and "autopilot vis" | 10:09 |
| gusch | the select the gallery, and you can browse the QML tree nicely | 10:09 |
| timp | ok, thanks | 10:10 |
| timp | maybe I can do it all via the objectName | 10:15 |
| timp | gusch: it is a bit confusing to me. I have ActionItem, and subclasses of that called ToolbarButton and Button (in qml) | 10:23 |
| timp | gusch: autpilot vis shows me a classname "Button" for the classic buttons, and "ActionItem" for the ToolbarButtons | 10:23 |
| gusch | ActionItem for ToolButtons ?!? | 10:24 |
| gusch | that sounds weired | 10:24 |
| timp | gusch: and it seems that the classname for autopilot must match exactly, so I cannot say "ActionItem" and select all instances of (subclasses of) ActionItem | 10:24 |
| gusch | but if you can set a unique objectName, that would be the best for testing for course | 10:24 |
| gusch | timp: yes, the names have to match exactly | 10:25 |
| timp | gusch: yes, I was trying that. but it doesn't work for me because of the classname | 10:25 |
| timp | some times it is ActionItem (for back buttons), some times Button (for cancel buttons) | 10:25 |
| timp | I guess it'll work if I introduce an additional get_toolbar_back_icon function in addition to the get_toolbar_cancel_icon | 10:26 |
| timp | just weird that for ToolbarButton I need to search for ActionItem instead. | 10:27 |
| timp | the full implementation of ToolbarButton is this: https://pastebin.canonical.com/93000/ | 10:27 |
| timp | perhaps it doesn't change the className because I didn't add properties to it | 10:27 |
| timp | that would mean the tests that I am planning will break if I ever add a property to ToolbarButton | 10:28 |
| gusch | timp: damn - but might be | 10:31 |
| om26er | gusch, Hi! gallery-app crashes on 'Auto Enhance' btw | 10:32 |
| timp | gusch: thanks for the help. I'll have a break now and then I give it another try. | 10:32 |
| gusch | om26er: yep https://bugs.launchpad.net/gallery-app/+bug/1180345 | 10:32 |
| ubot2 | Ubuntu bug 1180345 in gallery-app "Auto enhance crashes gallery app and deletes photo" [Critical,Confirmed] | 10:32 |
| gusch | om26er: you could do a quick fix in C++ if you want ... | 10:33 |
| gusch | om26er: in src/photo/photo.cpp comment line 805 (QApplication::processEvents();) | 10:34 |
| om26er | gusch, cool, that works | 10:36 |
| gusch | om26er: well - it blocks the UI - so I need another (proper) fix | 10:39 |
| === oSoMoN is now known as oSoMoN|afk | ||
| om26er | gusch, should I keep your workaround with my test branch ? | 11:24 |
| gusch | om26er: good question | 11:25 |
| gusch | om26er: I'd say yes - and update the bugreport | 11:25 |
| om26er | ok | 11:27 |
| nerochiaro | om26er: just to recap: all the autopilot tests that are in the branch at the momnent for the browser plugin are working, right ? i just tried and they all seem to pass successfully | 11:29 |
| nerochiaro | om26er: didn't you say there were some failing ? | 11:29 |
| om26er | nerochiaro, yes, I have disabled a few so you would not see them failing | 11:29 |
| om26er | look in test_history.py | 11:30 |
| nerochiaro | om26er: ok, thanks | 11:30 |
| nerochiaro | om26er: oh, i thought you had disabled them with the decorator, not with comments | 11:36 |
| nerochiaro | om26er: so it's basically only test_history and test_chromeless, right ? | 11:37 |
| om26er | nerochiaro, yep | 11:37 |
| nerochiaro | oSoMoN|afk: when you're back i have one question for you on a traslations-related issue | 11:38 |
| nerochiaro | oSoMoN|afk: please pingme | 11:38 |
| === MacSlow is now known as MacSlow|lunch | ||
| === greyback is now known as greyback|lunch | ||
| om26er | gusch, I added tests for auto-enhance and 'delete last photo' .. https://code.launchpad.net/~om26er/gallery-app/add_test_for_auto_enhance/+merge/170054 | 12:06 |
| om26er | for the latter case the bug would need to be fixed before the branch goes in. | 12:06 |
| gusch | om26er: cool - looking | 12:06 |
| === oSoMoN|afk is now known as oSoMoN | ||
| oSoMoN | nerochiaro: ping | 12:15 |
| gusch | om26er: looks good - I guess now it's my turn ;) | 12:18 |
| om26er | gusch, yep :D | 12:18 |
| nerochiaro | oSoMoN: hi, i've been trying to figure out why my translation files were being installed under /usr/local/share instead of /usr/share and I'm not really finding any difference with the other packages that use translations. Wonder if you could have a quick look at lp:~amanzi-team/ubuntu-ui-extras/ubuntu-ui-extras-initial and confirm that debuild does the same for you and give any clue | 12:20 |
| oSoMoN | nerochiaro: sure, I’ll look into it shortly and get back to you | 12:22 |
| === greyback|lunch is now known as greyback | ||
| nerochiaro | oSoMoN: ok, i'll go for some food in the meantime | 12:28 |
| === _salem is now known as salem_ | ||
| === chriadam is now known as chriadam|away | ||
| gusch | nerochiaro: can you review the camera? https://code.launchpad.net/~schwann/camera-app/camera-video-orientation/+merge/170053 | 12:43 |
| dholbach | dpm, could you find out anything? | 12:46 |
| oSoMoN | nerochiaro: independently from the issue you asked me to look into, building a package with debuild fails with the following error: | 12:49 |
| oSoMoN | dh_install: qtdeclarative5-ubuntu-ui-extras-plugin-autopilot missing files (usr/lib/python*/dist-packages/ubuntu-ui-extras/*), aborting | 12:49 |
| timp | zsombi: could you review https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/toolbarItems/+merge/170070 ? | 12:49 |
| timp | zsombi: don't happrove yet, when you think the MR is good, I'll first send an e-mail to ubuntu-touch mailing list | 12:50 |
| dpm | dholbach, back online at http://developer.ubuntu.com/resources/tools/packaging/ - it was my fault, I had reshuffled a lot of the content on the resources page and had made the packaging page private by mistake | 12:50 |
| gema | boiko: ping | 12:52 |
| boiko | gema: pong | 12:53 |
| gema | boiko: I'd like to have a quick chat with you as to where you are functionality - wise with the phone-app | 12:53 |
| gema | boiko: we are going to start testing it in the lab and I'd like to know what to expect | 12:54 |
| gema | and how to prioritise the testsing | 12:54 |
| boiko | gema: yep, nice! | 12:54 |
| dholbach | dpm, thanks | 12:55 |
| gema | boiko: so I had an interesting morning today not being able to do some things with the app, it'd fail to hang up, or to send the conference id to a conference call | 12:55 |
| gema | boiko: it'd also not be able to call to the provider on a service number | 12:55 |
| gema | boiko: are these things on your TODO or do you think they are there | 12:56 |
| boiko | gema: so conf call is not supported yet | 12:56 |
| gema | ok | 12:56 |
| boiko | gema: failing to hang up is something new to me | 12:56 |
| boiko | salem_: ^ | 12:56 |
| gema | boiko: I was calling the canonical conference system | 12:57 |
| gema | I had tried to enter the conf id | 12:57 |
| gema | then tried to hang up | 12:57 |
| gema | didn't happen | 12:57 |
| boiko | gema: oh, you mean DTMF then ok | 12:57 |
| salem_ | gema, ah, I have seen this before. I think it only happens if you try to hangup from the dtmf screen. | 12:57 |
| gema | salem_: waht is the dtmf screen? | 12:58 |
| salem_ | gema, the screen to type numbers during an ongoing call. | 12:58 |
| gema | salem_: ah, ok | 12:58 |
| gema | salem_: which screen should I use to hang up? | 12:58 |
| salem_ | gema, well, you should be able to hangup from that screen, but as it seems we have a bug there. I would suggest you to tap back, and then hangup from the main screen | 12:59 |
| gema | salem_: ack, do you want me to raise a bug? | 13:00 |
| boiko | gema: please do | 13:00 |
| gema | ack | 13:00 |
| salem_ | gema, that would be great. thanks | 13:00 |
| boiko | salem_: I'll try dtmf in today's image to see if it works | 13:00 |
| salem_ | boiko, ok. I think this bugs is not 100% reproducible, but I have seen it before | 13:01 |
| boiko | salem_: ok, let's see what I can find here | 13:01 |
| gema | boiko, salem_ : https://bugs.launchpad.net/phone-app/+bug/1192158 | 13:02 |
| ubot2 | Ubuntu bug 1192158 in phone-app "Unable to hang up from dtmf screen" [Undecided,New] | 13:02 |
| gema | I used your nomenclature, feel free to change it :) | 13:02 |
| gema | boiko: on the bright side I was able to gather some energy consumption data with a call that lasted 28 mins | 13:03 |
| gema | (until credit ran out) | 13:03 |
| boiko | gema: oh, and how did it go? | 13:03 |
| gema | boiko: I still need to do the math, I will let you know soon | 13:03 |
| gema | how long I expect the battery to last with an engaged call | 13:04 |
| oSoMoN | nerochiaro: that said, regarding your issue, I’ve added some debugging messages to src/Ubuntu/Components/Extras/Browser/po/CMakeLists.txt, and the value of CMAKE_INSTALL_LOCALEDIR when building a package is share/locale, which looks correct | 13:04 |
| boiko | gema: not sure how long, cause battery usage does not depend exclusively on the phone-app | 13:06 |
| gema | boiko: I know, that's why we are measuring the whole system :) | 13:07 |
| gema | boiko: but I need the phone-app to behave if we are going to measure this kind of thing | 13:07 |
| boiko | gema: yep | 13:07 |
| gema | i.e. being able to make calls, keep them up, hang up | 13:07 |
| gema | boiko: do you have an ETA for the conferencing? | 13:08 |
| === MacSlow|lunch is now known as MacSlow | ||
| nerochiaro | oSoMoN: yes, it looks correct, but then if you look under obj-i686-linux-gnu/ you will see that /usr/local/ is prepended to that path | 13:12 |
| boiko | gema: that's DTMF, conferencing is something different (creating a conf call on the phone) | 13:13 |
| boiko | gema: I'll test that, and I bet salem_ will also give it a try too | 13:13 |
| oSoMoN | nerochiaro: I don’t see anything there, probably because the package fails to build | 13:13 |
| oSoMoN | nerochiaro: have you tried building from a clean copy of your branch? | 13:14 |
| boiko | gema: actually there is a simple test for that: call some other phone, and press the keypad buttons to see if any sound is heard on the other side | 13:14 |
| === dholbach_ is now known as dholbach | ||
| nerochiaro | oSoMoN: i'm always doing that from a clean branch now. i'm using debuild -uc -us -b and the obj-i686-linux-gnu/ dir is there even if the build fails (which i confirm it does with the issue you mentioned, i'm fixing it) | 13:15 |
| gema | boiko: it is not at the moment | 13:16 |
| boiko | gema: so there is indeed a but there, the phone has just finished flashing, let me try it here | 13:17 |
| nerochiaro | oSoMoN: ok, fixed the issue you were noticing before, pushed | 13:17 |
| nerochiaro | oSoMoN: but there are other problems, i'm working on them | 13:17 |
| gema | boiko: ack | 13:17 |
| boiko | gema: just tested using today's image, and DTMF seems to be working here, I'll do some more tests | 13:19 |
| oSoMoN | nerochiaro: next error when building package: | 13:19 |
| oSoMoN | dh_install: tmp/ubuntu-ui-extras-initial/obj-x86_64-linux-gnu/tests/unittests/Ubuntu/Components/Extras/libubuntu-ui-extras-plugin.so exists in debian/tmp but is not installed to anywhere | 13:19 |
| oSoMoN | nerochiaro: does the package actually build successfully on your machine? | 13:19 |
| nerochiaro | oSoMoN: no, it does not, i was trying to approach one issue at a time. and the translation one is the one i could not explain | 13:19 |
| === francisco is now known as Guest7324 | ||
| nerochiaro | oSoMoN: this one you just pointed out to me I know where it comes from | 13:20 |
| oSoMoN | nerochiaro: well how can you tell there’s an issue with the package if the package doesn’t build at all? | 13:20 |
| nerochiaro | oSoMoN: because i see the translation files installed in the wrong dir | 13:20 |
| boiko | gema: are you using today's image? | 13:20 |
| oSoMoN | nerochiaro: fix the package build first, and then see if there are issues, not the other way around | 13:21 |
| nerochiaro | oSoMoN: fair enough | 13:22 |
| gusch | nerochiaro: can you review the camera? https://code.launchpad.net/~schwann/camera-app/camera-video-orientation/+merge/170053 | 13:33 |
| gusch | om26er: here the delete fix https://code.launchpad.net/~schwann/gallery-app/gallery-delete-last-photo/+merge/170077 | 13:33 |
| gusch | boiko: can you review? https://code.launchpad.net/~schwann/gallery-app/gallery-previewmanager-mediacollection/+merge/169998 | 13:33 |
| boiko | gusch: probably after the standup only, is that fine? | 13:34 |
| gusch | boiko: ok - no problem | 13:34 |
| gusch | boiko: that branch sits and waits for about a week already ;) | 13:35 |
| boiko | gusch: hehe, ok :) | 13:35 |
| timp | gusch: are all gallery autopilot tests supposed to pass on my local machine? | 13:39 |
| nerochiaro | gusch: i'm stuck with some build issues to solve ASAP, but i can try to take care of that later | 13:39 |
| gusch | nerochiaro: ok | 13:39 |
| timp | gusch: I fixed all the button tests in my branch, but I have two (seemingly unrelated) fails: | 13:40 |
| gusch | timp: yes | 13:40 |
| timp | FAIL: gallery_app.tests.test_photo_viewer.TestPhotoEditor.test_photo_editor_crop(with mouse) | 13:40 |
| timp | FAIL: gallery_app.tests.test_photo_viewer.TestPhotoViewer.test_double_click_zoom(with mouse) | 13:40 |
| gusch | om26er: ^ | 13:40 |
| timp | in the crop test, I don't see the frame for cropping when the test is run. | 13:40 |
| gusch | timp: but they all run fine locally and even on Jenkins | 13:40 |
| gusch | timp: whaaaat?!? | 13:41 |
| timp | in the double_click_zoom test, the mouse cursor moves to the center of the photo, but I don't see zooming | 13:41 |
| timp | gusch, om26er oh. doubleclick to zoom, and cropping also don't work for me when I run gallery myself | 13:41 |
| timp | ah.. | 13:41 |
| timp | probably I broke that. | 13:42 |
| timp | because I commented out some HUD stuff. | 13:42 |
| timp | ^so that I can run it without having a HUD. But the code is broken because of my chagnes | 13:42 |
| timp | *changes | 13:42 |
| gusch | timp: you shouldn't do that ;) | 13:42 |
| gema | boiko: no, I am using the first saucy image, I will be upgrading soon | 13:43 |
| boiko | gema: ok | 13:43 |
| timp | gusch: I don't have Ubuntu.HUD | 13:43 |
| gema | boiko: for some reason I cannot turn it on right now, need to look into that | 13:43 |
| timp | gusch: last time I installed shell stuff it broke my whole system. (that was months ago) | 13:43 |
| gema | boiko: we are looking at 7.3 hours worth of battery | 13:43 |
| gema | with an active call | 13:44 |
| gema | pmcgowan: ^ | 13:44 |
| gusch | timp: well, then uncooment it, and push hope the best, and jenkins will test it | 13:44 |
| gusch | timp: well, then uncomment it, and push hope the best, and jenkins will test it | 13:44 |
| pmcgowan | gema, good | 13:44 |
| timp | gusch: yes, I will do that. | 13:45 |
| seb128 | hum | 13:47 |
| seb128 | did anyone look adding qtsystems (https://qt.gitorious.org/qt/qtsystems) to the touch image? | 13:47 |
| seb128 | that might be a question for #ubuntu-touch | 13:47 |
| om26er | gusch, timp, ah sorry, unity notifications are not working, looking now | 13:53 |
| gusch | om26er: should be sorted timp broke the tests himself ;) | 13:54 |
| === ckpringle_ is now known as ckpringle | ||
| seb128 | Kaleo, hey | 14:08 |
| seb128 | Kaleo, I've a quick question for you about keyColor if you are around | 14:08 |
| === oSoMoN_ is now known as oSoMoN | ||
| Kaleo | seb128: sure | 14:17 |
| seb128 | Kaleo, is there any way to use that in a ListItems.Standard? | 14:18 |
| seb128 | Kaleo, I was talking with tiheum, the icon he has atm are using a #cccccc grey rather than #888888 | 14:18 |
| seb128 | he recommended using keyColor to change the grey | 14:19 |
| seb128 | but I'm not sure that works with listitems? | 14:19 |
| Kaleo | seb128: where did you get keyColor from? | 14:22 |
| Kaleo | seb128: you know it's part of the Icon class right? http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components0-icon.html | 14:22 |
| seb128 | Kaleo, right ... I guess my question is "can we set the property for an icon: embedded in a ListItem.Standard"? | 14:23 |
| Kaleo | seb128: http://developer.ubuntu.com/api/ubuntu-12.10/qml/mobile/qml-ubuntu-components-listitems0-standard.html#icon-prop | 14:23 |
| seb128 | or do I need new icons with the proper grey? | 14:23 |
| Kaleo | seb128: you are probably confused by that bad API & documentation for ListItem.Standard | 14:23 |
| pmcgowan | gema, kind of interesting comparison here http://www.anandtech.com/show/6440/google-nexus-4-review/2 | 14:23 |
| Kaleo | seb128: ultimately the API should be: | 14:24 |
| Kaleo | seb128: ListItem.iconSource and ListItem.iconName | 14:24 |
| Kaleo | seb128: the source would be any url to any image file | 14:25 |
| Kaleo | seb128: iconName would be the name of the icon in the theme | 14:25 |
| Kaleo | seb128: right now the icon property can take 2 different values | 14:25 |
| Kaleo | seb128: either a URL to an image file | 14:25 |
| gema | pmcgowan: yep, we are pretty close to those figures :) | 14:25 |
| Kaleo | seb128: or any Item derived class | 14:25 |
| pmcgowan | gema, yep, and I like the way they describe the testing | 14:26 |
| seb128 | Kaleo, if I set an iconSource, does it use an Icon {} element to build the widget, and can I set keyColor: for that Icon{} in some way? | 14:26 |
| gema | pmcgowan: I will definitely be reading that in detail | 14:26 |
| Kaleo | seb128: so no it does not use an icon | 14:26 |
| timp | Kaleo: fyi, I'd propose to add iconName to ActionItem (all ListItems, Buttons, ... inherit from that now) and property var icon can be deprecated | 14:27 |
| Kaleo | seb128: since the API does not accept an icon name | 14:27 |
| Kaleo | seb128: it would not work | 14:27 |
| seb128 | Kaleo, ok, I guess that settle it, I will ask for new icons | 14:27 |
| seb128 | Kaleo, thanks | 14:27 |
| Kaleo | seb128: so all you can do right now | 14:27 |
| Kaleo | seb128: is to create an Icon class yourself | 14:27 |
| Kaleo | seb128: ListItem.icon: Icon {} | 14:27 |
| gema | pmcgowan: although I am expecting us on the nexus 4 to be pretty close to their figures (since it is mainly hw usage all those tests) | 14:27 |
| Kaleo | seb128: (and there is no "iconSource" property) | 14:27 |
| pmcgowan | gema, right | 14:28 |
| gema | pmcgowan: but it is comforting to know we are :) | 14:28 |
| timp | Kaleo: there is, from its parent ActionItem | 14:28 |
| pmcgowan | exactly | 14:28 |
| timp | Kaleo: but it is still ignored in the list items | 14:28 |
| Kaleo | timp: don't make it more confusing for seb128 | 14:28 |
| Kaleo | :) | 14:28 |
| Kaleo | timp: so yes, please make sure API includes iconName & iconSource | 14:29 |
| Kaleo | timp: and wherever we are using Item icon, we probably need to keep it for now since people rely on it | 14:29 |
| Kaleo | timp: but we can make it so that it does not accept a URL anymore | 14:29 |
| seb128 | Kaleo, I've the feeling I will run into https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1190751 if I try your workaround ;-) | 14:29 |
| ubot2 | Ubuntu bug 1190751 in Ubuntu UI Toolkit "ListItem.Standard icon's item geometry is incorrect " [Undecided,New] | 14:29 |
| Kaleo | timp: (since iconSource will support that) | 14:29 |
| Kaleo | seb128: yes, that's a dupe btw | 14:30 |
| Kaleo | seb128: we can fix that | 14:30 |
| seb128 | Kaleo, sorry for the dup, I looked for reports bug didn't find it | 14:30 |
| Kaleo | seb128: no worries | 14:31 |
| timp | Kaleo: Item icon was added because iconSource was not flexible enough, but I don't remember the specific use-case | 14:32 |
| timp | brb food | 14:32 |
| seb128 | Kaleo, timp: thanks for the help! | 14:33 |
| timp | Kaleo: hmm. instead of iconName: "name" it is now possible to say icon: Icon { name: "name" } | 14:34 |
| Kaleo | timp: yes that's what I told seb128 | 14:36 |
| seb128 | timp, Kaleo: I tried that with an icon and ran into the scaling issue ... I will keep the color issue instead until that bug is fixed :p | 14:39 |
| seb128 | "pick your bug" ;-) | 14:39 |
| Kaleo | seb128: k | 14:40 |
| gusch | om26er: are you looking for my review? | 14:51 |
| om26er | gusch, hmm, missing context. what review? | 14:52 |
| gusch | om26er: here the delete fix https://code.launchpad.net/~schwann/gallery-app/gallery-delete-last-photo/+merge/170077 | 14:52 |
| om26er | gusch, I am going to do the functional review to make sure if it fixes | 14:55 |
| gusch | om26er: fine | 14:55 |
| === ckpringle_ is now known as ckpringle | ||
| === salem_ is now known as _salem | ||
| boiko | gusch: MR approved | 16:19 |
| gusch | boiko: \o/ | 16:19 |
| gusch | om26er: please merge trunk and push again for https://code.launchpad.net/~om26er/gallery-app/add_test_for_auto_enhance/+merge/170054 | 16:47 |
| om26er | gusch, on it | 16:47 |
| gusch | :) | 16:47 |
| === _salem is now known as salem_ | ||
| renato_ | om26er, ping | 18:03 |
| om26er | renato_, pong | 18:03 |
| renato_ | om26er, I am getting this error on my project when I try to run lcov: https://pastebin.canonical.com/93039/ | 18:05 |
| renato_ | do you have any idea? | 18:05 |
| om26er | renato_, no, no clue :/ | 18:08 |
| === salem_ is now known as _salem | ||
| === _salem is now known as salem_ | ||
| om26er | renato_, there might be some compatibility problem? https://bugs.launchpad.net/ubuntu/+source/lcov/+bug/1163758 | 18:40 |
| ubot2 | Ubuntu bug 1163758 in lcov (Debian) "geninfo: Argument "=====" isn't numeric in numeric gt (>) at /usr/bin/geninfo line 1126." [Unknown,Confirmed] | 18:40 |
| renato_ | om26er, but the real problem is: geninfo: ERROR: /home/renato/Projects/phablet/contact-service/fix-dbus-lost/build/src/CMakeFiles/address-book-service-lib.dir/qindividual.cpp.gcno: reached unexpected end of file | 19:10 |
| renato_ | this crash gcov compilation, and I am not able to see any resuts | 19:10 |
| om26er | renato_, tried split_crc ? http://ltp.sourceforge.net/coverage/lcov/geninfo.1.php | 19:15 |
| om26er | renato_, that error is specified there | 19:15 |
| renato_ | om26er, let me try | 19:18 |
| === francisco is now known as Guest75516 | ||
| renato_ | om26er, I am not using geninfo command | 19:26 |
| renato_ | only lcov | 19:26 |
| === boiko_ is now known as boiko | ||
| === marlinc is now known as marlinc|away | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!