| Mirv | morning | 04:07 |
|---|---|---|
| Mirv | could someone check/approve https://code.launchpad.net/~timo-jyrinki/ubuntu-ui-toolkit/qmlrunner_use_dpkg_architecture/+merge/166172 ? sergiusens reported a problem and it was indeed there. | 05:04 |
| === chriadam is now known as chriadam|away | ||
| mzanetti | http://wstaw.org/m/2013/05/29/plasma-desktopSR2564.png | 08:40 |
| mzanetti | http://wstaw.org/m/2013/05/29/plasma-desktopMy2564.png | 08:40 |
| mzanetti | first draft ^ | 08:40 |
| mzanetti | The Dialog would need to remove the space for the "text" if it is empty... | 08:41 |
| mzanetti | should I report a bug for that? | 08:41 |
| t1mp | mzanetti: which app is that? | 08:49 |
| mzanetti | t1mp: fahrplan. lets you find trains/busses. | 08:50 |
| t1mp | mzanetti: we don't have a date/time picker yet in the UI toolkit so when we work on that we can re-use yours perhaps :) | 08:50 |
| mzanetti | t1mp: I'm porting it from MeeGo | 08:50 |
| t1mp | mzanetti: ah cool. for which countries does it work? | 08:50 |
| gusch | still pending for review: https://code.launchpad.net/~schwann/gallery-app/gallery-modularity/+merge/166054 | 08:50 |
| mzanetti | t1mp: yeah, yesterday I had a discussion with zsombi about this. right now I'm talking with design and once they're happy with it I'll try to upstream it to the SDK | 08:51 |
| mzanetti | t1mp: http://wstaw.org/m/2013/05/29/plasma-desktoptX2564.png | 08:51 |
| DanChapman | Hi folks, does anyone know how to delete/remove a tarball in python. I'm doing an autopilot tests for fileroller and need to delete the created archive in teardown, but os.remove claims no such file looked through the tar modules and cant find a way to remove an archive just to create/extract/addfiles etc. | 08:52 |
| t1mp | mzanetti: ns.nl would be useful for me :) | 08:52 |
| t1mp | mzanetti: currently the dialog always assumes that you have text and one or more buttons, so if you need something else (check with design), then file a bug/feature request on bugs.launchpad.net/ubuntu-ui-toolkit please | 08:53 |
| mzanetti | t1mp: hehe... shouldn't be too hard... there seems to be a standard called HafasXml. If your provider uses that adding support would be quite straigt forward | 08:53 |
| t1mp | I wasn't aware that different companies in different countries are using a standard | 08:56 |
| t1mp | that is surprising actually, especially since they are big companies | 08:56 |
| mzanetti | t1mp: dunno... just looking at the implementation, half of the sites use the same parser (called ParserHafasXml) while the other half has custom parsers. | 08:56 |
| netcurli | http://www.hacon.de/unternehmen/referenzen/hafas/hafas-referenzen | 08:57 |
| mzanetti | netcurli: awesome :) | 08:58 |
| mzanetti | trenitalia suports it... interesting | 08:58 |
| mzanetti | t1mp: seems you're lucky... ns.nl seems to support it | 09:00 |
| === gusch is now known as gusch|away | ||
| t1mp | mzanetti: yes, its on that hafas-referenzen list :) | 09:03 |
| mzanetti | t1mp: so in theory it should be just a matter of adding the website to some list. I'll check it out when I work on it next time. (spare time project) | 09:04 |
| t1mp | cool, thanks | 09:06 |
| dpm | mzanetti, that's pretty awesome. When you think the app is in a working state, please add it to https://wiki.ubuntu.com/Touch/Collection - looks like a cool app we'd like to showcase :) | 09:07 |
| t1mp | I'm planning to switch to using ubuntu on a phone that I really use (not just development) | 09:07 |
| mzanetti | dpm: its already in polishing phase :) I expect it to be done by the end of the week | 09:07 |
| dpm | excellent! | 09:07 |
| t1mp | indeed, excellent :) | 09:08 |
| mzanetti | t1mp: +1. thats why I started all that porting mania. can't use the phone without the 5 apps I use regualry | 09:08 |
| t1mp | mzanetti: at the moment I am abroad and I'm not sure whether I can reliably disable data (except wifi) on the phone. if that doesn't work well it could cost me some unneccesary money | 09:09 |
| mzanetti | oh... that's indeed a reason to be careful... | 09:09 |
| mzanetti | t1mp: but... can you ENable data already? | 09:10 |
| mzanetti | :D | 09:10 |
| * mzanetti still runs an image from end of last week | 09:10 | |
| t1mp | mzanetti: I'm not sure. So far I only used ubuntu in a phone without a sim-card | 09:10 |
| dpm | t1mp, I've got a question on C++ plugins for QML if you've got a minute: I'm working on a personal project to decode QR codes, and I decided to use a C++ plugin for decoding. There is some existing library for Qt 4.x and I decided to port it to 5.0 and package it up as a plugin, | 09:11 |
| t1mp | dpm: yes I have a minute :) | 09:11 |
| dpm | great, so here's the question :) | 09:11 |
| dpm | t1mp, I've done the port and the packaging, but my C++ is very rusty, not to talk of my Qt skills. Does the change in http://bazaar.launchpad.net/~dpm/+junk/qzxing-plugin/revision/2 look sensible to you? | 09:12 |
| dpm | for the .c and .h files | 09:12 |
| dpm | those were the main changes I did, but as my Qt experience is limited, it felt a bit like stabbing in the dark at times :) | 09:13 |
| mzanetti | dpm: you should keep the QObject *parent in the ctor | 09:14 |
| t1mp | dpm: I'll have a look | 09:14 |
| t1mp | QT += widgets in your pro file is needed if you use the "old" qt widgets if I'm not mistaken | 09:14 |
| t1mp | so if your gui will be qtquick you don't need that | 09:14 |
| dpm | thanks mzanetti. I think that gave me a warning IIRC, but I can try again | 09:15 |
| mzanetti | rest looks sensible to me | 09:15 |
| dpm | great, thanks mzanetti | 09:15 |
| t1mp | I don't see a reason not to make the constructor call QObject constructor | 09:16 |
| dpm | t1mp, good point, I'm not using widgets, so I'll remove that include. | 09:17 |
| mzanetti | dpm: for reference, I did exactly the same yesterday for this: https://gitorious.org/qml-process-element/qml-process-element/merge_requests/1 | 09:17 |
| dpm | ah, interesting :) | 09:18 |
| dpm | t1mp, mzanetti so on line 13 I should basically do + QZXing::QZXing(QObject *parent) right? | 09:19 |
| mzanetti | dpm: QZXing::QZXing(QObject *parent) : QObject(parent) | 09:19 |
| dpm | ah, so keep it as it was before. Let me try | 09:20 |
| mzanetti | dpm: oh... you changed it to inherit QQmlExtensionPlugin... | 09:20 |
| mzanetti | dpm: that seems hackish... | 09:20 |
| t1mp | I just noticed that too | 09:21 |
| mzanetti | dpm: better have another file/class plugin.h that does only the registering and the QZXing then only inheriting from QObject and doing the logic | 09:21 |
| dpm | I warned I wasn't experienced in QT ;) | 09:21 |
| mzanetti | dpm: not saying that there isn't a way to make it work this way, but gets weird... especially once you want to register an additional class... | 09:22 |
| t1mp | dpm: if you have a look at the UITK code, we have there classes like UbuntuI18n in i18n.h/cpp, and we register all the classes that we need exposed to qtquick in plugin.cpp | 09:22 |
| dpm | mzanetti, I agree, I prefer doing it the right way. Just I didn't know what the right way was. This feedback is really useful | 09:23 |
| dpm | ok, let me see if I can create those plugin cpp/h files now... | 09:45 |
| nerochiaro | grr, hate it when i start working in the morning and forgot to turn on irc | 09:48 |
| dpm | mzanetti, t1mp, does this look more sensible now? -> http://bazaar.launchpad.net/~dpm/+junk/qzxing-plugin/revision/5 | 10:12 |
| mzanetti | dpm: purrfect | 10:13 |
| dpm | excellent, thanks for the help! | 10:13 |
| nerochiaro | tmoenicke: some updates: i'm using your latest library (the one with set Property) and the simple test we used yesterday, but with a TextArea doesn't cause the keyboard to bounce. so it's something in the notes-app, and i'm trying to figure out what it is | 10:33 |
| tmoenicke | nerochiaro: ok cool | 10:34 |
| nerochiaro | tmoenicke: in the notes app, when i hide the keyboard manually i see this debug print: QMaliitPlatformInputContext::imInitiatedHide() step 2, setProperty("focus", true) ... << why is it set to true if i'm hiding the keyboard ? | 10:34 |
| tmoenicke | nerochiaro: right, recompiling | 10:35 |
| nerochiaro | tmoenicke: that's weird that even with that mistake it was working in the simple case. wtf ? | 10:36 |
| tmoenicke | indeed | 10:36 |
| nerochiaro | tmoenicke: hmm, actually no, in the simple case i just checked and the focus wasn't being removed anymore. yesterday i think i had just time to verify that it wasn't bouncing back and that it programmatically setting focus to false and then true was doing the right thing regarding focus on the parents | 10:37 |
| nerochiaro | tmoenicke: so it's all right. let's check with this new version you're recompiling that sets focus to false on imInitiateHide and let's hope it's the last thing we need :) | 10:38 |
| tmoenicke | nerochiaro: I just sent you an updated version | 10:39 |
| tmoenicke | hope that works now | 10:40 |
| nerochiaro | tmoenicke: unfortunately no. if i manually hide the keyboard the blinking cursor is removed but the textarea still has focus. but let me try with a textinput and also if i can simply work around this in the notes app | 10:54 |
| nerochiaro | tmoenicke: though knowing why focus isn't removed is probably important | 10:54 |
| === MacSlow is now known as MacSlow|lunch | ||
| nerochiaro | tmoenicke: ok, we have something. TextInput gets the focus removed correctly. TextArea doesn't | 11:01 |
| nerochiaro | tmoenicke: when hiding the keyboard manually | 11:01 |
| nerochiaro | tmoenicke: let's talk to zsombi ? | 11:01 |
| kalikiana | Kaleo, standup | 11:02 |
| tmoenicke | nerochiaro: yeah, after lunch? | 11:02 |
| Kaleo | sorry, conputer broke; hd total faikurr | 11:04 |
| nerochiaro | tmoenicke: ideal | 11:04 |
| Kaleo | going to look if cn buy one now | 11:04 |
| nerochiaro | tmoenicke: i'll see if i can give him a peek now anyway, i'm short on time | 11:04 |
| kalikiana | Kaleo, ouch, I hope you didn't lose much | 11:04 |
| nerochiaro | zsombi: can you please poke me when you have some minutes ? | 11:04 |
| t1mp | Kaleo: good luck :s | 11:04 |
| tmoenicke | ok | 11:09 |
| === gusch|away is now known as gusch | ||
| om26er | nerochiaro, removed the nexus 7 stuff as you said: https://code.launchpad.net/~om26er/camera-app/ap_tests_improvements/+merge/165733 | 11:13 |
| nerochiaro | om26er: added a few comments | 11:25 |
| nerochiaro | om26er: then i will need to test it and if htey all pass approve | 11:25 |
| nerochiaro | om26er: (on desktop) | 11:25 |
| gusch | still pending for review: https://code.launchpad.net/~schwann/gallery-app/gallery-modularity/+merge/166054 | 11:25 |
| om26er | nerochiaro, thanks fixing | 11:26 |
| om26er | nerochiaro, the python docstring convention says that if a doctring is multi-line it should have an empty line before ending it. | 11:26 |
| nerochiaro | zsombi: the thing me and tmoenicke would like to discuss with you is https://bugs.launchpad.net/notes-app/+bug/1182108 and specifically the last few comments about focus in TextArea | 11:26 |
| ubot2 | Ubuntu bug 1182108 in notes-app "keyboard re-appears after swiping it away" [Critical,In progress] | 11:26 |
| nerochiaro | om26er: there are a bunch of other multiline docstrings in that code that don't have that. either fix them all if it's required by python or leave them all more compact. I honestly prefer them more compact, the extra lines are a waste of screen space and make readability worse | 11:28 |
| nerochiaro | om26er: for example | 11:28 |
| nerochiaro | 159+ """Test that the shoot button gets disabled for a while then re-enabled | 11:28 |
| nerochiaro | 160+ after shooting""" | 11:28 |
| nerochiaro | om26er: i think this is good and readable | 11:28 |
| om26er | nerochiaro, ok then i'll change that back | 11:29 |
| oSoMoN | zsombi: hey, it looks like the Panel is broken in today’s image, tapping outside it doesn’t hide it anymore, can you confirm? | 11:30 |
| nerochiaro | om26er: thanks | 11:30 |
| oSoMoN | tmoenicke: hey, regarding inputMethodHints, do you know if there’s already a bug report to track its addition to the TextField component in the SDK? | 11:38 |
| nerochiaro | tmoenicke: zsombi: I think i know what's happening. the OSK is removing the focus from the inner TextInput but the SDK TextArea doesn't track that and remove focus on the entire component | 11:39 |
| nerochiaro | tmoenicke: zsombi: if we fix that I think we can solve our problems | 11:40 |
| oSoMoN | tmoenicke: nevermind, I filed bug #1185392 | 11:42 |
| ubot2 | Launchpad bug 1185392 in Ubuntu UI Toolkit "Need to expose 'inputMethodHints' in the TextField component" [Undecided,New] https://launchpad.net/bugs/1185392 | 11:42 |
| oSoMoN | t1mp: hey, it looks like the Panel is broken in today’s image, tapping outside it doesn’t hide it anymore, can you confirm? | 11:49 |
| dpm | t1mp, is it ok to reopen bug 1182577? I've just found what I think is a valid use case | 11:51 |
| ubot2 | Launchpad bug 1182577 in Ubuntu UI Toolkit "Cannot use the i18n plugin in C++" [Wishlist,Won't fix] https://launchpad.net/bugs/1182577 | 11:51 |
| t1mp | oSoMoN: ok. I'll install today's image to test | 11:53 |
| oSoMoN | t1mp: thx | 11:54 |
| oSoMoN | t1mp: I can observe that in all applications that have a toolbar | 11:56 |
| t1mp | dpm: okay. I re-opened it. | 11:57 |
| dpm | thanks t1mp | 11:57 |
| === t1mp is now known as timp | ||
| tmoenicke | oSoMoN: no i dont think so, thx | 11:57 |
| tmoenicke | nerochiaro: cool | 11:58 |
| nerochiaro | tmoenicke: let's see what zsombi says when he comes back | 12:03 |
| tmoenicke | ok | 12:03 |
| tmoenicke | nerochiaro: also, using the property does not add the dependency to declarative, so the patch should be fine | 12:05 |
| tmoenicke | nerochiaro: though, should be done for widget-based code too at some point | 12:06 |
| nerochiaro | tmoenicke: sweet, let's worry about what we use now ATM and put a note there in the code about widgets | 12:07 |
| timp | oSoMoN: I confirmed. Did you create/re-open a bug for that? | 12:08 |
| om26er | gusch, please approve this one, its just some style changes https://code.launchpad.net/~om26er/gallery-app/reduce_pep8_complaints/+merge/165854 | 12:08 |
| oSoMoN | timp: I’m filing a bug right now | 12:08 |
| gusch | om26er: ok | 12:09 |
| om26er | renato_, Hey! this one is now ready to go https://code.launchpad.net/~om26er/mediaplayer-app/skip_if_phone/+merge/165880 | 12:10 |
| gusch | om26er: approved | 12:11 |
| oSoMoN | timp: bug #1185397 | 12:12 |
| ubot2 | Launchpad bug 1185397 in Ubuntu UI Toolkit "[regression] The panel doesn’t hide when tapping outside it" [Undecided,New] https://launchpad.net/bugs/1185397 | 12:12 |
| om26er | gusch, thanks | 12:12 |
| oSoMoN | timp: can you please confirm the bug, and can we please revert the faulty revision asap? (I’m suspecting rev 517, needs confirmation) | 12:13 |
| om26er | oSoMoN, when you get some time: calendar branch is now ready. | 12:15 |
| timp | zsombi: png | 12:16 |
| timp | +i | 12:16 |
| renato_ | om26er, done | 12:30 |
| om26er | renato_, thanks | 12:30 |
| gusch | still pending for review: https://code.launchpad.net/~schwann/gallery-app/gallery-modularity/+merge/166054 | 12:31 |
| gusch | nerochiaro oSoMoN renato_ tmoenicke ^ | 12:31 |
| zsombi | timp pong | 12:32 |
| nerochiaro | zsombi: ping | 12:44 |
| zsombi | nerochiaro: pong | 12:44 |
| renato_ | gusch, I will take it | 12:44 |
| kalikiana | if anyone's up for a review of documentation/ui toolkit https://code.launchpad.net/~kalikiana/ubuntu-ui-toolkit/fataldocs/+merge/166240 | 12:44 |
| nerochiaro | zsombi: can you please check the messages that i directed to you in the scrollback, or I can repeat if you like | 12:45 |
| zsombi | (02:01:31 PM) nerochiaro: tmoenicke: ok, we have something. TextInput gets the focus removed correctly. TextArea doesn't | 12:48 |
| zsombi | nerochiaro: this is the issue, right? | 12:48 |
| zsombi | nerochiaro: when hiding the keyboard manually | 12:48 |
| zsombi | nerochiaro: manually means by swiping down the OSK, right? | 12:49 |
| nerochiaro | zsombi: yes | 12:50 |
| zsombi | nerochiaro: is this happening with some special build? | 12:50 |
| zsombi | or is it in the latest image? | 12:50 |
| gusch | renato_: cool thx | 12:50 |
| nerochiaro | zsombi: let's hop on mumble when you have a minute and i'll explain to you | 12:53 |
| timp | zsombi: did you read up about the panel regression? | 12:54 |
| timp | zsombi: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1185397 do you think it could be related to r517 with IMA changes? | 12:55 |
| ubot2 | Ubuntu bug 1185397 in Ubuntu UI Toolkit "[regression] The panel doesn’t hide when tapping outside it" [Undecided,Confirmed] | 12:55 |
| nerochiaro | zsombi: it's happening with yesterday image in any case, plus a library from tmoenicke | 12:56 |
| === MacSlow|lunch is now known as MacSlow | ||
| nerochiaro | zsombi: it's explained in the bug. but let's mumble so i can give you more context and we can talk about what it can be done to fix it | 13:01 |
| renato_ | gusch, I added some comments | 13:01 |
| gusch | renato_: thx - I pushed the fixes ;) | 13:03 |
| zsombi | nerochiaro: I am supposed to have a meeting with ckpringle now, so eventually after that | 13:07 |
| zsombi | timp: weird... perhaps... I tested the apps back then and haven't noticed such an issue... | 13:09 |
| zsombi | timp: easy to find out: print the IMA's sensingArea in Panel to see whether it has a valid value | 13:10 |
| timp | oSoMoN: ^ will you have a look? I'm working on something else now | 13:11 |
| oSoMoN | zsombi, timp: will do | 13:13 |
| zsombi | nerochiaro: what is the bug id? | 13:48 |
| nerochiaro | zsombi: https://bugs.launchpad.net/notes-app/+bug/1182108 | 13:50 |
| ubot2 | Ubuntu bug 1182108 in notes-app "keyboard re-appears after swiping it away" [Critical,In progress] | 13:50 |
| nerochiaro | zsombi: but i really need to explain to you what's going on | 13:50 |
| nerochiaro | zsombi: on voice | 13:50 |
| nerochiaro | zsombi: i have a standup in 10 minutes | 13:51 |
| zsombi | nerochiaro: ehh :) | 13:51 |
| zsombi | nerochiaro: I just finished my meeting | 13:51 |
| nerochiaro | zsombi: let's hop on mumble now then | 13:51 |
| zsombi | connecting... | 13:52 |
| nerochiaro | zsombi: i'm in the PS apps channel | 13:52 |
| === mzanetti is now known as mzanetti|food | ||
| zsombi | connecting takes ages... | 13:52 |
| nerochiaro | zsombi: confirm that it works fine with TextArea | 14:08 |
| nerochiaro | zsombi: TextEdit, sorry | 14:08 |
| zsombi | nerochiaro: ok | 14:08 |
| zsombi | thx | 14:08 |
| === mzanetti|food is now known as mzanetti | ||
| * kalikiana wonders if jppiiroinen would like to claim https://code.launchpad.net/~kalikiana/ubuntu-ui-toolkit/fataldocs/+merge/166240 | 14:19 | |
| Mirv | dpm: alert, alert, the 'gomobile' page no redirects to http://developer.ubuntu.com/get-started/ which has completely bollocks instructions, like an antique version with wrong package names / copy-paste errors updated with some Ubuntu version numbers.. | 14:25 |
| dpm | Mirv, on it, thanks for the heads up | 14:25 |
| Mirv | dpm: thanks. it refers to a PPA from half a year back et cetera, just received an e-mail from a precise user, noticed the redirect and then noticed the instructions | 14:27 |
| oSoMoN | om26er: re- the calendar app and objectName, are you using autopilot vis to visualize the hierarchy of items? | 14:31 |
| dpm | Mirv, phew, well spotted. This was due to the replacement of the old get-started page (i.e. Quickly) by the new gomobile page, which unnoticedly set the fields with the PPA instructions to defaults (i.e. as they were at launch date) - the deployment happened a couple of hours ago, so hopefully the confusion didn't last for long. You can now refresh http://developer.ubuntu.com/get-started/ and see the proper instructions | 14:32 |
| Mirv | dpm: much better now! :) it's good that it was caught early. | 14:33 |
| dpm | nicely spotted :) | 14:34 |
| om26er | oSoMoN, yes | 14:35 |
| oSoMoN | om26er: then it must be a problem in autopilot… | 14:35 |
| om26er | oSoMoN, but I also tried a different technique with select_many() and gave only the objectName but there are no objects found | 14:36 |
| oSoMoN | om26er: how do you use autopilot vis on an app started with qmlscene? I’m not getting the hierarchy of items, only the root one | 14:38 |
| om26er | oSoMoN, qmlscene calendar.qml -testability | 14:39 |
| om26er | oSoMoN, if that's not working you might have an old version of libautopilot-qt | 14:39 |
| oSoMoN | om26er: that’s what I’m doing, and I’m only getting the QQuickView root object | 14:39 |
| om26er | oSoMoN, that was a bug which have been fixed in autopilot-qt | 14:39 |
| om26er | oSoMoN, make sure to have libautopilot-qt 1.3daily13.05.24ubuntu.unity.next-0ubuntu1 | 14:40 |
| oSoMoN | om26er: where do I get this one from? I have 1.3daily13.05.09ubuntu.unity.nextbzr58raring1 from the autopilot PPA | 14:42 |
| om26er | oSoMoN, ppa:ubuntu-unity/daily-build-next | 14:43 |
| oSoMoN | ok, thanks | 14:46 |
| oSoMoN | om26er: ok, I understand the issue, you’re setting the objectName on the Action items, right? | 14:50 |
| om26er | oSoMoN, yes | 14:50 |
| oSoMoN | om26er: so in fact it’s not a bug but a missing feature in the UITK, the objectName of the action should be transferred to the corresponding button | 14:52 |
| om26er | oSoMoN, right, makes sense | 14:53 |
| om26er | oSoMoN, so currently what we are doing is fine ? | 14:53 |
| oSoMoN | om26er: I wouldn’t say that, let’s try and find a cleaner way of doing it | 14:55 |
| oSoMoN | om26er: I’m pretty sure the same problem was already solved by other apps’s autopilot tests, have a look at how it’s done in the gallery for example | 14:55 |
| om26er | oSoMoN, will look there now. | 14:56 |
| om26er | oSoMoN, about the issue of my test not saving the event. I have not found a way to delete the event in the app so running the test each time will add a new event | 14:57 |
| zsombi | nerochiaro: so this happens in every TextArea, not just in Notes? | 14:58 |
| oSoMoN | om26er: the application is using the localStorage API, so it’s just a matter of deleting the DB before starting the app, the notes-app does something similar in its tests | 14:59 |
| oSoMoN | om26er: but in a first step I wouldn’t bother with such a complex set up, I’d just test that clicking the timeline button switches to the timeline view, and then back to the main view, for example | 15:00 |
| oSoMoN | om26er: a test for creating a new event can be written later on, the point of this MR is to bootstrap the tests with something simple and illustrative | 15:00 |
| zsombi | jppiiroinen: weird thing: I had enabled the developer mode on my tablet yesterday, and now after I connected it, it sais I should enable the dev mode again... | 15:01 |
| zsombi | jppiiroinen: In QtC... | 15:01 |
| om26er | oSoMoN, ok I can do that | 15:01 |
| om26er | oSoMoN, about the gallery-app, its also using the index number technique it seems | 15:01 |
| oSoMoN | om26er: the technique used by the gallery is slightly safer though (see get_toolbar_button(…)), because it ensures that it gets buttons only inside the panel, not every single button in the application | 15:04 |
| om26er | oSoMoN, yeah, I am going with that. Infact that's what I was using back in the days before I was told to use select_single() | 15:06 |
| om26er | oSoMoN, as now I am only targeting a single test to show/hide timeline, what happens to my already written emulators? shall I just keep them or remove them? They are definitely going to be used in future. | 15:25 |
| oSoMoN | om26er: keep them around | 15:26 |
| zsombi | Kaleo: greyback: new stuff needed in layouting: http://studio.sketchpad.cc/MnCl8Uj9Qa | 15:27 |
| zsombi | Kaleo: greyback: please comment | 15:27 |
| greyback | zsombi: use-case please | 15:28 |
| greyback | because right now I totally don't understand it | 15:28 |
| greyback | zsombi: did you see the other portion of my Layout API proposal: "LayoutItem" | 15:29 |
| zsombi | greyback: as said, I saw it, and it may make sense to use that, however in there you need a LayoutItem per each re-parented item, right? This would give the parameters on a specific fragment | 15:35 |
| oSoMoN | nerochiaro: would you be available for a review today? | 15:37 |
| zsombi | greyback: I'll come back with an example tomorrow... I should really step off now... | 15:39 |
| greyback | zsombi: ok, have a good evening | 15:43 |
| nerochiaro | oSoMoN: to be honest no, i am behind with a task ATM | 15:56 |
| oSoMoN | nerochiaro: ok, nm | 16:05 |
| gusch | renato_: can you please have a look at this again? https://code.launchpad.net/~schwann/gallery-app/gallery-modularity/+merge/166054 | 16:09 |
| nerochiaro | oSoMoN: om26er: do you guys know why this wasn't reviewed yet ? I think it's pending from last week, for some reason i had assumed i gave it to one of you for review | 16:22 |
| oSoMoN | nerochiaro: link? | 16:22 |
| om26er | o_O | 16:26 |
| oSoMoN | om26er: I approved your MR, is there anything specific to do on jenkins to activate autopilot tests for the calendar app? | 16:55 |
| om26er | oSoMoN, I am not sure about that, will have to ask about that from mmrazik | 16:55 |
| om26er | oSoMoN, he was looking into that a few days ago | 16:56 |
| oSoMoN | om26er: ok, anyway I’ll let you handle it | 16:56 |
| === chriadam|away is now known as chriadam | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
