| === Pici` is now known as Pici | ||
| === tsimpson_ is now known as tsimpson | ||
| === dkessel_ is now known as dkessel | ||
| liuxg | is there anyway to restart a scope? My current department url depends on the settings. Once I change the settings, my scope exits. is there any way to resolve this? I got error like: "Error happened: unity::LogicException: SearchReplyImpl::register_departments(): Failed to validate departments:" | 03:05 |
|---|---|---|
| === frecel_ is now known as frecel | ||
| === ubot5` is now known as ubot5 | ||
| === rpadovani|Dota is now known as rpadovani | ||
| === rmescandon is now known as rmescandon-brb | ||
| rpadovani | dpm, hey, are you here? :-) | 07:21 |
| dpm | hey rpadovani, I am :) | 07:23 |
| rpadovani | dpm, I'm thinking about the account selector page - do you prefer two option selector (one for authorized and one for not) or only one option selector with text: name + " - Tap to authorize" for unauthorized accounts? | 07:25 |
| === rmescandon-brb is now known as rmescandon | ||
| JamesTait | Good morning all; happy Ada Lovelace Day! :-D | 08:09 |
| dpm | mzanetti, rpadovani, thanks to popey, the evernote300 account has now a bunch of images to test if the performance with lots of images decreases, and if we need further optimizations | 08:18 |
| rpadovani | dpm, popey thanks :-) | 08:20 |
| mzanetti | dpm: ah nice... how is it performing? | 08:39 |
| dpm | mzanetti, not tried it yet, upgrading phone now and then I will | 08:42 |
| DanChapman | Good Morning all o/ | 08:45 |
| dpm | hey DanChapman | 09:09 |
| dpm | bfiller, oSoMoN, could someone from the apps team review https://code.launchpad.net/~seb128/gallery-app/some-translations-tweaks/+merge/236511 ? | 09:14 |
| seb128 | dpm, thanks for pinging them ;-) | 09:15 |
| seb128 | we really need somebody from the gallery-team to keep up with reviews | 09:15 |
| dpm | np, just saw that MP again on my inbox | 09:15 |
| oSoMoN | seb128, commented on the MR, your branch needs updating | 09:23 |
| seb128 | oSoMoN, thanks | 09:24 |
| oSoMoN | seb128, yw, ping me when done, and I’ll approve | 09:24 |
| dpm | mzanetti, so I think we might need to add the performance optimizations for images as well. After testing it, I noticed a few things: a) at the beginning, the app does not slow down, however, it loads a few images (not those that are in view, though) and then it gets busy downloading all of the rest. Meaning you cannot really view any other notes, as it seems to be busy downloading other things. It does not block the UI, but it's always showing the | 09:25 |
| dpm | busy spinner b) Ultimately the app crashes | 09:25 |
| === caribou_ is now known as caribou | ||
| === lool- is now known as lool | ||
| nik90 | bzoltan: ping | 10:39 |
| bzoltan | nik90: shoot | 10:39 |
| nik90 | bzoltan: when you run the test suite of app while releasing a new sdk, do you run the app's AP test suite or all the test suite that the app provides? | 10:40 |
| nik90 | bzoltan: just curious to know if you also run the clock app's qml test suite in addition to the AP test suite. | 10:40 |
| nik90 | this feels like a deja-vu (asking you this question i.e) | 10:41 |
| bzoltan | nik90: I run the app's AP tests only | 10:41 |
| bzoltan | nik90: do you think I should run some other tests too? We can do it if you tell me what exactly you want to test. | 10:41 |
| nik90 | bzoltan: ok..atm the clock app qml tests coverage is much much higher than the AP test suite. It would be best to run both. | 10:42 |
| nik90 | bzoltan: in the near future, I am planning to discuss with the QA team to remove the clock app AP suite since it is becoming a hassle to maintain | 10:42 |
| nik90 | I have used the qml test suite for unit, integration and feature tests as of today | 10:43 |
| nik90 | taking a page out of the unity8 test suite | 10:43 |
| bzoltan | nik90: how to run those tests on an installed clock app? | 10:46 |
| nik90 | bzoltan: hmm I am not sure if they are installed in the system. Currently I build the clock app and then run "ctest -V" to run the qml tests. These are also run by jenkins as part of every MP. | 10:47 |
| nik90 | If you need them to be installed, I can look into that. | 10:48 |
| nik90 | I just need your requirements | 10:48 |
| bzoltan | nik90: well.. we validate stuff on a clean installation | 10:48 |
| bzoltan | nik90: and we run tests with AP :) | 10:48 |
| nik90 | bzoltan: what does the SDK do? I noticed that it has qml tests as well..do you install them to the system and then run it alongside the App's test suites? | 10:49 |
| bzoltan | nik90: the SDK does nothing... the test script simple runs the AP tests like this `/usr/bin/phablet-test-run -r 0000 -s JW024063 ubuntu_weather_app` | 10:50 |
| bzoltan | nik90: a tool copies the autopilot content of each app's project to the /home/phablet/autopilot and the phablet-test-run just runs those tests | 10:52 |
| nik90 | bzoltan: ok you use phablet-test-run..I will discuss this with elopio and see what can be done about it to make it easier for you to run them. | 10:52 |
| nik90 | bzoltan: copies it from the installled app folder or from the source tree? | 10:52 |
| bzoltan | nik90: yeps | 10:52 |
| nik90 | bzoltan: it was a question with 2 options ;) "yes" is not a valid answer | 10:53 |
| bzoltan | nik90: yes ..."copies it from the installled app folder or from the source tree" :D | 10:54 |
| nik90 | bzoltan: ack :)..I will get back to you when I know more | 10:54 |
| bzoltan | nik90: from the source tree for the click apps and from the autopilot package for other apps | 10:54 |
| diogo_ | hi | 11:10 |
| pindonga | morning | 11:21 |
| === _salem is now known as salem_ | ||
| === rmescandon is now known as rmescandon-lunch | ||
| nik90 | rpadovani: got time to do a quick review of https://code.launchpad.net/~nik90/ubuntu-clock-app/improve-world-clock-startup/+merge/238057 ? | 13:10 |
| ahayzen | popey, could we make another click for remix when you have a moment? | 13:11 |
| rpadovani | nik90, for you I always have time :D | 13:11 |
| nik90 | rpadovani: thnx :D | 13:11 |
| ahayzen | jdstrand, ping | 13:12 |
| rpadovani | nik90, tested, lgtm | 13:19 |
| nik90 | thnx | 13:19 |
| t1mp | ahayzen: hello | 13:19 |
| t1mp | ahayzen: I started on bug https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1341814 | 13:19 |
| ubot5 | Ubuntu bug 1341814 in Ubuntu UI Toolkit "Using search in the header can sometimes have a text field from a different tab" [High,In progress] | 13:19 |
| t1mp | ahayzen: see my comment there | 13:19 |
| ahayzen | t1mp, hey i saw you comments...i'll have to try myself in a bit | 13:20 |
| ahayzen | t1mp, maybe something else fixed it :) | 13:20 |
| t1mp | ahayzen: still it is a bug, it seems like the previous header contents is still there somewhere even though that page is no longer active | 13:20 |
| ahayzen | oh hmm interesting | 13:20 |
| ahayzen | t1mp, do you have any idea what is going on? | 13:20 |
| t1mp | it might be somewhat random in which textfield you are actually entering text. so far I always got the correct one, but I could see the other one | 13:21 |
| t1mp | ahayzen: no, I'll dig into the code a bit and hopefully fix it :) | 13:21 |
| ahayzen | t1mp, thanks :) do you need me to do anything? | 13:21 |
| popey | ahayzen: sure, after the meeting tonight or before? | 13:23 |
| ahayzen | popey, erm probably now as nothing is going to land until this evening anyway | 13:23 |
| ahayzen | popey, have you got that pad i can add the changelog to? | 13:23 |
| popey | kk will do in 30 mins after my next call | 13:24 |
| ahayzen | popey, cool thanks :) | 13:24 |
| t1mp | ahayzen: check if you get the same problem as I have, or still the problem that you reported | 13:24 |
| ahayzen | t1mp, ok will do | 13:24 |
| popey | ahayzen: http://pad.ubuntu.com/MusicAppReadMe | 13:24 |
| t1mp | ahayzen: hmm, I am testing on desktop. Perhaps on device behavior is slightly different | 13:24 |
| ahayzen | t1mp, maybe | 13:24 |
| ahayzen | popey, thanks | 13:24 |
| === dpm_ is now known as dpm | ||
| === Guest24850 is now known as balloons | ||
| popey | ahayzen: done | 13:38 |
| ahayzen | popey, thanks :) | 13:38 |
| popey | np | 13:39 |
| popey | will mail | 13:39 |
| ahayzen | popey, there is no click here http://people.canonical.com/~alan/music_remix/20141014/trunk/com.ubuntu.music_1.3.latest_all.click | 13:41 |
| popey | odd | 13:43 |
| popey | now? | 13:44 |
| popey | dunno what happened there | 13:45 |
| ahayzen | popey, i see a click in http://people.canonical.com/~alan/music_remix/20141014/ ... not inside a folder called trunk | 13:45 |
| popey | oh balls | 13:46 |
| popey | fixed | 13:46 |
| popey | thanks | 13:46 |
| ahayzen | popey, there we go \o/ :) thanks | 13:46 |
| === rmescandon-lunch is now known as rmescandon | ||
| === dpm_ is now known as dpm | ||
| JoeyChan | hello, could anyone pls tell me how to run core-app directly on device vis SDK (￣(エ)￣) | 13:57 |
| ahayzen | t1mp, when running on device i get the same behaviour as the steps state in bug 1341814 | 14:20 |
| ubot5 | bug 1341814 in Ubuntu UI Toolkit "Using search in the header can sometimes have a text field from a different tab" [High,In progress] https://launchpad.net/bugs/1341814 | 14:20 |
| ahayzen | *stated | 14:20 |
| === karni is now known as karni-afk | ||
| t1mp | ahayzen: okay, thanks for verifying | 14:20 |
| ahayzen | t1mp, do you see the same on device? | 14:21 |
| t1mp | ahayzen: I got caught up in some other stuff and didn't test it on the device yet | 14:22 |
| ahayzen | t1mp, cool no worries | 14:22 |
| t1mp | ahayzen: I'll try to fix the issue on the desktop and then see if that fixes it on device as well | 14:22 |
| ahayzen | t1mp, thanks | 14:22 |
| jdstrand | ahayzen: hey | 14:29 |
| ahayzen | jdstrand, Hi, we were looking at landing the confinement last night, but noticed that the new artist cover art isn't coming through from thumbnailer. This is the denial you get.. | 14:30 |
| ahayzen | jdstrand, Oct 14 01:29:14 ubuntu-phablet dbus[1654]: apparmor="DENIED" operation="dbus_method_call" bus="session" path="/com/canonical/Thumbnailer" interface="com.canonical.Thumbnailer" member="GetArtistArt" mask="send" name="com.canonical.Thumbnailer" pid=26358 profile="com.ubuntu.music_music_1.3.657" peer_pid=3667 peer_profile="unconfined" | 14:30 |
| ahayzen | jdstrand, i assume that just needs to be added to the profile as i can see GetAlbumArt in there but not GetArtistArt | 14:31 |
| ahayzen | jdstrand, eg i see this in /var/lib/apparmor/profiles/click_com.ubuntu.music_music_1.3.657 .. http://pastebin.ubuntu.com/8559419/ | 14:33 |
| jdstrand | ahayzen: yes | 14:33 |
| jdstrand | ahayzen: is there a bug for the landing? | 14:33 |
| ahayzen | jdstrand, erm we are using this one really https://bugs.launchpad.net/music-app/+bug/1315386 | 14:34 |
| ubot5 | Ubuntu bug 1315386 in Ubuntu Music App "music-app should run under confinement under click" [Medium,In progress] | 14:34 |
| ahayzen | jdstrand, i can make a specific one to another project for that issue if you want? | 14:34 |
| jdstrand | ahayzen: can you file one against apparmor-easyprof-ubuntu? | 14:35 |
| ahayzen | jdstrand, yep i'll ping you the link when done :) | 14:35 |
| ahayzen | jdstrand, the only other denial/error we have seen while testing is this ... Oct 14 01:29:12 ubuntu-phablet dbus[1654]: apparmor="DENIED" operation="dbus_bind" bus="session" name="org.freedesktop.Application" mask="bind" pid=26358 profile="com.ubuntu.music_music_1.3.657" | 14:38 |
| ahayzen | jdstrand, is ^^ safe to ignore? | 14:38 |
| jdstrand | ahayzen: should be safe to ignore-- that is a known bug that is being fixed in the platform | 14:40 |
| ahayzen | jdstrand, cool thanks just wanted to double check :) | 14:40 |
| === karni-afk is now known as karni | ||
| jdstrand | ahayzen: you'll want to double check that your url handler is working correctly though | 14:40 |
| ahayzen | jdstrand, yeah ... assuming my url handler opens music instead of facebook lol | 14:41 |
| jdstrand | heh | 14:41 |
| ahayzen | jdstrand, is this clear enough? https://bugs.launchpad.net/ubuntu/+source/apparmor-easyprof-ubuntu/+bug/1381102 | 14:41 |
| ubot5 | Ubuntu bug 1381102 in apparmor-easyprof-ubuntu (Ubuntu) "Apparmor stops applications from using GetArtistArt from the thumbnailer" [Undecided,New] | 14:41 |
| ahayzen | urlhandler works we're good :) | 14:42 |
| jdstrand | cool | 14:43 |
| jdstrand | I'll get that fixed | 14:43 |
| ahayzen | jdstrand, thanks i feel we're getting close to having music-app under confinement again :) | 14:44 |
| jdstrand | yay! :) | 14:44 |
| seb128 | popey, nik90, is mihir doing IRC? | 14:56 |
| popey | seb128: yeah, he's about a lot of the time outside his work hours | 14:58 |
| seb128 | popey, ok, thanks, in what tz is he? | 14:58 |
| nik90 | seb128: India so UTC +5 | 15:01 |
| rpadovani | seb128, india, UTC+5:30 | 15:02 |
| seb128 | nik90, rpadovani, thanks | 15:07 |
| seb128 | he wrote on https://code.launchpad.net/~uriboni/ubuntu-calendar-app/fix-translations-with-arg/+merge/232903 that he would fix the i18n part | 15:07 |
| seb128 | I would like to know what's the status of that | 15:08 |
| === justCarakas is now known as firefox | ||
| === firefox is now known as justcarakas | ||
| === gatox is now known as gatox_lunch | ||
| vitimiti | Hi | 15:23 |
| === johnlage is now known as Artemis | ||
| === Artemis is now known as johanleg | ||
| === johanleg is now known as johnlage | ||
| === johnlage is now known as Andromeda-DT | ||
| === Andromeda-DT is now known as johnlage | ||
| nik90 | elopio: hey, I addressed all your comments in the MP https://code.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/utopic-3.0/+activereviews. I think they should be good to go in now. | 15:35 |
| rpadovani | mihir | 15:38 |
| rpadovani | <seb128> he wrote on https://code.launchpad.net/~uriboni/ubuntu-calendar-app/fix-translations-with-arg/+merge/232903 that he would fix the i18n part | 15:38 |
| rpadovani | <seb128> I would like to know what's the status of that | 15:38 |
| seb128 | rpadovani, thanks | 15:38 |
| seb128 | mihir, hey | 15:39 |
| rpadovani | yw | 15:39 |
| mihir | hey rpadovani seb128 | 15:39 |
| mihir | it has been already merge i believe. | 15:39 |
| seb128 | mihir, trunk is still buggy | 15:40 |
| mihir | seb128: we have been waiting for you to merge on that MR. | 15:40 |
| seb128 | which one? | 15:40 |
| seb128 | like trunk has | 15:41 |
| seb128 | DeleteConfirmationDialog.qml: i18n.tr('Delete only this event "'+event.displayLabel+'", or all events in the series?'): | 15:41 |
| seb128 | DeleteConfirmationDialog.qml: i18n.tr('Are you sure you want to delete the event "'+ event.displayLabel +'"?'); | 15:41 |
| seb128 | EditEventConfirmationDialog.qml: text: i18n.tr('Edit only this event "'+event.displayLabel+'", or all events in the series?'); | 15:41 |
| seb128 | 15:41 | |
| seb128 | those 3 are buggy/untranslatable | 15:41 |
| mihir | seb128: okay i got that | 15:41 |
| mihir | seb128: that's our bad communication , sorry about that. | 15:41 |
| === salem_ is now known as _salem | ||
| mihir | seb128: i'll push that changes in few hours , is that okay ? | 15:42 |
| mihir | seb128: i wrote this comment and i forgot to to apply patch on that , https://code.launchpad.net/~uriboni/ubuntu-calendar-app/fix-translations-with-arg/+merge/232903/comments/578453 | 15:42 |
| seb128 | mihir, yeah, no worry, I was about to work on submitting a fix and then I saw your comment on the other mp and decided to check to no duplicate work | 15:42 |
| mihir | seb128: yeah if you have ready MR , just push that , otherwise i'll push that. | 15:42 |
| seb128 | mihir, note that I listed 3 strings, they all need to be fixed the same way, I don't think that mp included the 3 | 15:42 |
| mihir | seb128: okay i'll do that. | 15:43 |
| seb128 | thanks! | 15:43 |
| mihir | seb128: apologize for mis-communication. | 15:43 |
| seb128 | no worry | 15:43 |
| seb128 | well, there was no miscommunication from my side | 15:43 |
| seb128 | I saw your comment like an hour ago | 15:43 |
| seb128 | I was just unsure if you still planned to work on it | 15:44 |
| seb128 | since it has been some time and there was no action since | 15:44 |
| seb128 | in any case thanks for working on it ;-) | 15:44 |
| mihir | seb128: :) | 15:44 |
| seb128 | I like the calendar app | 15:44 |
| mihir | seb128: :) thank you. | 15:44 |
| mihir | rpadovani: thanks for patching us :) | 15:44 |
| seb128 | going to be even better with those strings translated! | 15:44 |
| rpadovani | mihir, yw :-) | 15:45 |
| mihir | seb128: honestly i tried too find you on IRC but couldn't find with nick you mentioned on your profile https://launchpad.net/~uriboni | 15:45 |
| seb128 | mihir, that's not me ;-) | 15:45 |
| mihir | seb128: i believe that's your LP | 15:45 |
| mihir | ahhhh my bad | 15:45 |
| seb128 | mihir, I'm seb128 on launchpad as well | 15:45 |
| seb128 | no | 15:46 |
| mihir | seb128: ahhh okay. | 15:46 |
| seb128 | I just happened to be annoyed by the calendar strings showing in english on my phone | 15:46 |
| seb128 | so I started looked at the issue and find that merge request | 15:46 |
| seb128 | mihir, note that the lp page you point has the IRC nickname info | 15:46 |
| seb128 | mihir, see "IRC:" in the middle section | 15:46 |
| mihir | seb128: yeah i see that. | 15:47 |
| mihir | seb128: i'll make sure this get merged today. | 15:49 |
| seb128 | mihir, thanks, feel free to ping me for review when you have the merge request ready | 15:49 |
| seb128 | I'm happy to test and confirm it works for me | 15:49 |
| mihir | seb128: sure, i'll do that. i am taking my dinner. will do that after that. | 15:50 |
| seb128 | mihir, enjoy dinner! | 15:51 |
| elopio | nik90: now the usage comment of ClockTestCase is not accurate. | 15:55 |
| elopio | on line 82 it says Utils.qml. On line 86 it says UbuntuTestCase. | 15:55 |
| elopio | I'm leaving my approval on that one, but please make the changes to the comments. | 15:56 |
| nik90 | elopio: lol, sure, I always forget some thing | 15:57 |
| elopio | nik90: and approved the other one. Thank you for the nice work! | 15:58 |
| nik90 | elopio: thnx for the review. | 15:59 |
| nik90 | elopio: I am following the qml test tasks at https://trello.com/c/wlrY9p01/286-clock-qml-test-coverage | 15:59 |
| nik90 | in case you are wondering the reason for these MPs now :0 | 15:59 |
| === gatox_lunch is now known as gatox | ||
| nik90 | elopio: btw I can reproduce https://bugs.launchpad.net/autopilot-qt/+bug/1341671 consistently now | 16:05 |
| ubot5 | Ubuntu bug 1341671 in Autopilot Qt Support "Versioned QML classes are not recognized by their public type name" [High,Confirmed] | 16:05 |
| elopio | nik90: when you add properties, qt will report that the qml has a new type, that's the same as the file name | 16:23 |
| elopio | if you extend a type without adding properties, qt will report it to have the same name as the extended type. | 16:23 |
| elopio | those and some other weird qt rules about the type names are the reason for needing a more flexible matching between types and autopilot proxy objects. | 16:24 |
| elopio | I hope next week we discuss about it to have a solution soon. | 16:24 |
| nik90 | elopio: yup this bug is annoying...pls do..test writers hit that quite frequently and spend a lot of time figuring what causes the failures | 16:25 |
| mihir_ | seb128, you around ? | 16:44 |
| seb128 | mihir_, yes | 16:44 |
| mihir_ | seb128, here is the MR , would be great if you can review https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/fix-translation-args/+merge/238325 | 16:44 |
| seb128 | looking | 16:44 |
| seb128 | mihir_, commented | 16:54 |
| mihir_ | seb128, thanks , so you mean "%1" this we should use \"%1\" , have i understood correctly ? | 16:56 |
| seb128 | yes | 16:57 |
| seb128 | strings under ' don't end up in the templates for some reason | 16:58 |
| seb128 | that's not specific to calendar | 16:58 |
| seb128 | so they need to be under " | 16:58 |
| mihir_ | okay got it ,so if i want to display " this in string i should use escape sequ | 16:58 |
| seb128 | or you can't put an un-escaped " in the middle of a string under " | 16:58 |
| seb128 | right | 16:58 |
| mihir_ | so this should take care of quotes , ('Edit only this event \"%1\", or all events in the series?').arg(event.displayLabel) | 16:59 |
| mihir_ | seb128, pushed the changes :) | 17:00 |
| seb128 | mihir_, , you need to replace the single quote ' by " on the start/end of the string | 17:03 |
| mihir_ | seb128, ohh yes , got that..anyway pushed to MR. | 17:06 |
| nik90 | popey, balloons: Can we release a new clock app to the store. It has 2 performance patches related to the clock app startup time and 3 QML Feature tests added. I ran the AP, qml and manual test suites and they all pass as expected. No regressions ;) | 17:09 |
| popey | sure thing! | 17:09 |
| * popey looks at balloons | 17:09 | |
| popey | ʘ‿ಠ | 17:10 |
| nik90 | did he just quit after we both pinged him :P | 17:10 |
| === balloons is now known as Guest18783 | ||
| popey | He's a git ☻ | 17:10 |
| === _salem is now known as salem_ | ||
| nik90 | lol | 17:11 |
| Guest18783 | double ping killed my poor bouncer.. you are so mean! | 17:12 |
| * nik90 replaces Guest18.... standard image with balloon's face to better adjust to the conversation | 17:13 | |
| * popey goes to make curry, back soon. | 17:13 | |
| === Guest18783 is now known as balloons_ | ||
| balloons_ | nik90, rev # please? | 17:17 |
| nik90 | balloons_: 147 | 17:17 |
| === balloons_ is now known as balloons | ||
| jdstrand | beuno_ (or pindonga): can someone do a pull request to the store for the click-reviewers-tools? (r277) | 17:20 |
| balloons | nik90, popey clock rev 147 uploaded | 17:25 |
| nik90 | balloons: woop woop .. that was fast thnx. | 17:26 |
| nik90 | balloons: does it need to approved? | 17:26 |
| balloons | nik90, it's in review yes | 17:28 |
| nik90 | kk | 17:28 |
| mihir_ | nik90, i am trying to use Loader , http://paste.ubuntu.com/8560263/ | 17:31 |
| mihir_ | to show details only in dayview , which is wideType , do you find any issue the way i am using ? | 17:32 |
| nik90 | mihir_: looks good | 17:33 |
| popey | balloons: nik90 approved | 17:33 |
| nik90 | me updates | 17:34 |
| mihir | nik90: JS function is not able to find the object which is inside that component file:///home/mihir/Documents/apps/blankBubbles/ubuntu-calendar-app/EventBubble.qml:38: ReferenceError: timeLabel is not defined | 17:34 |
| mihir | nik90: FYI , i also moved Component.Complete to wideType | 17:34 |
| nik90 | mihir: if you want to reference components inside a loader you need to call it by loadername.item.propertyname | 17:35 |
| nik90 | the official loader docs shows how to do this | 17:35 |
| nik90 | mihir: so yeah the loader and the details component by themselves look good. Just that the way you access them from outside the loader will change a bit | 17:39 |
| pindonga | jdstrand, on it asap | 17:39 |
| nik90 | mihir: I feel though that you need to increase test coverage before using loaders since with loaders it changes the way you access data and thereby something could silently break without you realising about it. | 17:40 |
| mihir | nik90: hmm true | 17:42 |
| mihir_ | nik90, to make myself more clear. | 17:43 |
| mihir_ | nik90, if we use we need to declare properties and assign values to the respective controls | 17:43 |
| mihir_ | nik90, have I understood correctly ? | 17:43 |
| nik90 | mihir_: I totally lost you now :P | 17:43 |
| * balloons updates to click | 17:43 | |
| mihir_ | nik90, lol , so this is scenario , i have label which is inside Component , and that component is being loaded using Loader. | 17:44 |
| mihir_ | nik90, now if i have to access the value using JS function , do i required alias property required on top, or i can do other way. | 17:44 |
| nik90 | mihir_: yes, since that label is inside a component (which is similar to definining it in a separate qml file), you will need to declare public properties (Public API) for that component and change internal variables using them. Hope this makes it more clear | 17:45 |
| mihir_ | nik90, understood :) | 17:46 |
| popey | mihir_: do we want to update calendar in the store? it's r492, and r499 is trunk now... | 17:47 |
| mihir_ | nik90, thanks for making me understand. | 17:47 |
| mihir_ | popey, i am waiting for 2 MRs to get it merged , one is from akiva's one is mine. | 17:47 |
| nik90 | mihir_: np, tbh I am also still learning. Btw did you figure out the Qtcreator analyse tab. I see now how handy it is.. | 17:47 |
| popey | ok | 17:48 |
| mihir_ | popey, can you top approve this ? https://code.launchpad.net/~mihirsoni/ubuntu-calendar-app/fix-translation-args | 17:48 |
| nik90 | hmm I can't believe this...the datetime picker takes soooo much time to load...which is why the EditAlarm page in clock app takes time to load :/ | 17:49 |
| mihir_ | nik90, yes i did but still need to spend more time on that tool , to figure out how exactly it behaves. | 17:49 |
| mihir_ | nik90, yes it does :| | 17:49 |
| mihir_ | popey, that's small i18 change MR :) | 17:51 |
| popey | sure | 17:54 |
| popey | thanks mihir_ | 17:55 |
| mihir_ | popey, thank you :) | 17:56 |
| mihir_ | balloons, is this failing because there are too many events on Dayview , http://91.189.93.70:8080/job/generic-mediumtests-utopic-python3/925/artifact/calendar_app.tests.test_new_event.NewEventTestCase.test_delete_event_must_remove_it_from_day_view.ogv ? | 17:57 |
| mihir_ | popey, as user what would prefer to have calendar choice option ,http://i.imgur.com/bvbyW7C.png | 17:59 |
| mihir_ | on top or bottom as it is there in screenshot. | 18:00 |
| === wendar_ is now known as wendar | ||
| balloons | mihir_, wow, so many events :-) | 18:01 |
| balloons | popey, can you build and run terminal atm? I noticed the depends aren't correct, and the konsole plugin isn't being recognized by the tests | 18:03 |
| jdstrand | pindonga: thanks! | 18:03 |
| mihir_ | balloons, hehehe , i believe it is because of so many events..but tbh not sure . | 18:03 |
| balloons | mihir_, it needs to be able to see the event, but yea.. | 18:11 |
| mihir_ | balloons, lets wait to run it next day | 18:14 |
| mihir_ | heheh | 18:14 |
| mihir_ | nik90, again a question , how do we define multiple alias of a property ? | 18:22 |
| nik90 | just alias of alias.. | 18:23 |
| nik90 | so if you have the first alias like "property alias test: label.text" ...you should be able to do the second alias like property alias test2: test | 18:23 |
| mihir_ | like i need 4 text property alias | 18:23 |
| nik90 | or did that not work? | 18:23 |
| mihir_ | ahh got it | 18:24 |
| mihir_ | i thought it has to be same name | 18:24 |
| nik90 | as in 4 layers? or 4 distinct alias | 18:24 |
| mihir_ | 4 alias for 4 different labels. | 18:25 |
| nik90 | ah ok | 18:25 |
| mihir_ | i guess what you suggested should work | 18:26 |
| mihir_ | nik90, it works now | 18:59 |
| mihir_ | thanks :) | 18:59 |
| nik90 | kk | 19:00 |
| mihir_ | nik90, could you just check if i did it in right way , like not violating any qml rules , | 19:00 |
| mihir_ | nik90, http://paste.ubuntu.com/8560686/ | 19:00 |
| mihir_ | i can put this in diff file though | 19:01 |
| nik90 | I don't think you need property alias detailsItemsHeight : detailsItems.height | 19:01 |
| nik90 | property alias detailItemsY : detailsItems.y | 19:01 |
| nik90 | you can access them using eventDetails.item.y and eventDetails.item.height | 19:01 |
| mihir_ | nik90, then how can I access , item.height ? | 19:01 |
| * mihir_ learns something new everyday in QML :D | 19:02 | |
| mihir_ | nik90, thanks that works :) | 19:04 |
| nik90 | ;) | 19:06 |
| mihir | popey: if you still around , one small input required | 19:15 |
| mihir | mihir_> popey, as user what would prefer to have calendar choice option ,http://i.imgur.com/bvbyW7C.png | 19:15 |
| mihir | 11:30 PM <mihir_> on top or bottom as it is there in screenshot. | 19:15 |
| popey | hmm | 19:15 |
| popey | so my thought process is usually "1) what calendar will I put this in, 2) when is the event, 3) all other details" | 19:16 |
| popey | ahayzen: what do you use to rip CDs to flac? (if you do) | 19:23 |
| ahayzen | popey, rhythmbox | 19:27 |
| mihir | popey: okay so on bottom it is not suggested . | 19:28 |
| popey | ahayzen: ta | 19:41 |
| popey | mihir: well, for me.. it seems odd.. but we might want to think on this a bit. | 19:42 |
| ahayzen | popey, no problem ... u joining my side where music is crisp and clear :) | 19:42 |
| mihir | popey: yes , i thought of same, putting at bottom seems ood | 19:42 |
| popey | ahayzen: well, i need to re-rip now ☻ | 19:42 |
| mihir | popey: because if i just want to choose Calendar and event title , i would feel annoying to go to bottom to select the calendar :| | 19:42 |
| popey | +1 | 19:42 |
| popey | the absolute minimum required things should be at/near the top | 19:43 |
| ahayzen | popey, i redid the whole house's CDs over the summer in flac :) was definitely worth it lol | 19:43 |
| popey | heh | 19:43 |
| mihir | popey: okay got it :) i commented. | 19:43 |
| * mihir really needs to push calendar (waiting for 2 MRs to get merged) :( | 19:43 | |
| mihir | nik90: how do you use QtQuick setting ?to store some meta data ? | 19:44 |
| === dpm is now known as dpm-afk | ||
| mihir_ | Good night guys :) | 20:11 |
| vitimiti | Hi | 20:20 |
| pindonga | jdstrand, r277 on prod | 20:26 |
| === salem_ is now known as _salem | ||
| jdstrand | inawesome, thanks! | 21:36 |
| jdstrand | pindonga: tab complete fail | 21:36 |
| jdstrand | pindonga: awesome! | 21:36 |
| bmatusiak | where would i submit apn settings for a carrier? | 22:45 |
| bmatusiak | or check what settings there are | 22:45 |
| popey | bmatusiak: one for awe | 23:05 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
