=== daek_ is now known as Daekdroom === jhodapp|afk is now known as jhodapp === jhodapp is now known as jhodapp|afk === salem_ is now known as _salem === deegee__ is now known as drussell [08:24] rpadovani: hey, you around? [08:33] so quiet here [08:33] scary [08:39] ;_ [08:39] hmm [08:40] smiley fail [08:53] lalaaaaaaaaaaaaaaa [08:53] mhr3, only people in uk are working :D [08:54] Cimi, yea, we're the ones keeping the wheels turning :) [08:54] except for the ones that don't have nothing better to do ;P [08:55] Saviq, go away, we won't acknowledge you today! :P [08:55] mhr3, I'm working today ;) [08:56] so is mzanetti [08:56] it's Labour Day, what is it without Labour! [08:56] Saviq, looks to me there's a new module for the infographics, right? [08:56] hah [08:56] Cimi, there shouldn't be, no [08:57] Cimi, there was one already [08:57] Saviq, hr wouldn't be happy about you [08:57] mhr3, I'll get it back the week after next :P [08:59] anyone running utopic yet? [08:59] dednick, yes [08:59] Saviq: you having any issues with nameservers? [09:00] dednick, not really, although there was a bug mentioned yesterday on the ML, me looks [09:00] every time i boot, dnsmasq process is thrashing and my nameservers are reset [09:00] bug #1307981 maybe? [09:00] have to manually edit /etc/resolv.conf and add the ns [09:00] bug 1307981 in network-manager (Ubuntu) "[touch] randomly messed up routing with recent trusty images" [High,Confirmed] https://launchpad.net/bugs/1307981 [09:00] well, then that's not it... [09:01] Saviq, there's this src/modules/Infographics/ [09:02] Cimi, yeah, but that was there all along no? [09:02] Saviq, thought it was added here http://bazaar.launchpad.net/~unity-team/libusermetrics/file-based-infographics/revision/154 [09:02] unless this is trunk [09:03] * Saviq looks for the previous moduel [09:04] Cimi, ah it looks that this was in lp:unity8 until now [09:04] ah ok [09:04] Cimi, or actually, qlightdm [09:04] yes [09:04] so it has to change indeed [09:13] hm. weird. removing /etc/NetworkManager/NetworkManager.conf seems to have fixed my problem [09:14] although it sounds a bit dubious [09:15] Cimi, yeah, and it needs to be cleaned up in our mocks [09:25] Cimi, there's something weird with the rating input widget [09:25] Cimi, i can't focus it 80% of the time [09:29] mhr3, what? [09:29] Cimi, grab https://launchpad.net/~ci-train-ppa-service/+archive/landing-014/+build/5966633/+files/unity-scope-click_0.1%2B14.10.20140501-0ubuntu1_armhf.deb install on phone and try to review an installed app [09:32] mhr3, Cimi, confirmed [09:32] mhr3, trying [09:32] already rebooting [09:33] Cimi, mhr3, looks like something's wrong with "scroll and focus" or so [09:33] it just scrolls a 1gu on tap [09:34] something seems to be on top of the text entry [09:34] Saviq, can you send reviews btw? [09:34] mhr3, trying [09:34] cause i just get [09:34] 2014-05-01 10:21:51,622 - CRITICAL - ../../../../lib/SignOn/connection-manager.cpp 106 setupSocketConnection p2p error: QDBusError("org.freedesktop.DBus.Error.FileNotFound", "Failed to connect to socket /run/user/32011/signond/socket: No such file or directory") 1 [09:35] mhr3, in registry log? [09:35] yea [09:35] mhr3, same [09:35] "cool" [09:35] mhr3, signond isn't running here, isn't it meant to be DBus-triggered or something? [09:36] no idea [09:37] OTOH it's probably wrapped in a library anyway, so there's something bad there [09:50] Cimi, while you're looking at the rating, there's some huge margin between the text and review input widgets, have a look please [09:50] I wonder if we should have a HR before the review input widget, too [09:50] obviously no visuals :| [09:57] Saviq, the apps scope is being redesigned [09:57] will look quite different soon [09:57] mhr3, I know, it's meant to be "different" :P [09:58] mhr3, but it's not just about apps scope, any preview that has description + review will get that [09:58] Saviq, might be minimum height of the review [09:59] Saviq, even the description will look different :) [09:59] mhr3, do you have something to show me? [09:59] Saviq, had a peek yesterday, there was a tabbed view [09:59] contentHeight / lineCount * (maximumCollapsedLineCount - 2) [09:59] so yey! :) [09:59] might be this [10:00] for the padding [10:00] mhr3, heh... right on time for RTM then [10:00] this google drive is a mess [10:00] Saviq, how do you browse between visual designs? [10:02] Cimi, https://drive.google.com/a/canonical.com/?usp=folder#folders/0B-a_7E3tDxOgWDlCSWxkUGZlcnM [10:03] Saviq, much better, thx [10:03] starred [10:04] dash looks better than apps [10:04] https://drive.google.com/a/canonical.com/?usp=folder#folders/0B-a_7E3tDxOgTTBybG1TZG9GWnM [10:04] this is much better than weather app [10:04] can I run the preview instead the weather app ? :D [10:05] mhr3, see! devel branch is evil! [10:05] also, this previews contains a sunshine, while my weather app currently shows clouds :D :D [10:05] Saviq, why, what happened? [10:06] mhr3, https://code.launchpad.net/~michihenning/unity-scopes-api/test-timeout-changes/+merge/217872/comments/518672 [10:06] ah [10:06] minor issue :P [10:07] also, stop peeking where you shouldn't :) [10:07] * mhr3 is afraid Saviq will discover the terrible secrets [10:07] mhr3, stop notifying me about all that! [10:09] Saviq, I think is seeMore being not visible but still counting for the childrenRect [10:10] Saviq, the padding between the rating and text [10:10] Saviq: spurious tags. is that anything with a ? [10:10] Saviq, am I correct that childrenRect doesn't care about visible or not? [10:11] dednick, yeah [10:11] Saviq: there's one in trunk [10:11] 7.85+14.10.20140428.2-0ubuntu1 ? [10:11] dednick, oh interesting [10:11] dednick, that sounds like a bug in train [10:11] dednick, but that one I won't fight [10:11] dednick, not worth it [10:12] mzanetti, hey [10:12] mzanetti, I have ~20 minutes now, otherwise I'll ping you this evening [10:12] hey man. [10:12] dednick, we had over 300 before, of which 280 were ? from lp:unity8.... [10:12] rpadovani: just wanted you to show this: https://code.launchpad.net/~mzanetti/reminders-app/color/+merge/217850 [10:13] rpadovani: and suggest that we create a shared branch on lp:~reminders-app-devs for the new design where we work together [10:13] Saviq: ok [10:13] mzanetti, sounds good to me [10:13] rpadovani: because we probably want the transition to the new design in one big merge [10:14] mzanetti, I saw your branch, seems good, but I didn't test it yet [10:14] mzanetti, yes, perfect, do you do it? (the common branch) [10:14] rpadovani: so lets do small branches but instead of merging them into trunk, lets merge them to the common one. so we still have small code reviews while getting the one big transition in one block [10:14] rpadovani: yeah, I'll set it up [10:16] mzanetti, awesome, thanks! [10:17] rpadovani: ok... talk to you later then... we can set that up tonight [10:17] have to do some unity business now [10:18] mzanetti, ok, see you tonight then, have a nice day! [10:25] greyback, don't touch root CMakeLists.txt in unity-api, increment just the one in include/shell/… [10:25] greyback, also, bump changelog for unity-mir to be able to depend on the change [10:26] Saviq: none of this is is for promoting yet. But point 1 is good [10:26] greyback, kk [10:26] greyback, mark WiP then? [10:26] Saviq: I'm not merging into lp:unity-api but I want a review from maznetti [10:27] ah shoot [10:27] greyback, you can get a review on a WiP branch :) [10:30] Saviq: it's not WiP, I want to iterate on a separate branch for qtcompositor support, so want to go through usual review process [10:30] greyback, ok :D [10:30] Saviq, fix for padding https://code.launchpad.net/~cimi/unity8/preview-text-summary-bottom-padding/+merge/217903 [10:36] Cimi, tx [10:36] Saviq, how do I debug where my touches go? [10:36] Saviq, on the rating input widget issue [10:37] Saviq, looks like indeed scrolling does not move mousearea or so [10:37] Cimi, can't tell you much more than to put MouseAreas here and there [10:38] Cimi, and see which one reacts [10:38] Saviq, nope is not mouse areas moving [10:39] Saviq, it's the scrolling [10:39] Saviq, if you quickly tap the tap is interpreted as scrolling [10:39] Cimi, re: text widget, let's use Column there, childrenRect is unreliable when the height changes [10:39] Saviq, try holding the finger on the text review widget and you see that when you lift the finger it gets input [10:39] Cimi, and implicitHeight will be easier [10:39] Cimi, and anchors simpler [10:40] Saviq, I'll try [10:40] Saviq, let's go back to this input thing [10:41] Saviq, would it be possible that lvwph is stealing inputs? [10:41] Cimi, there's no lvwph in there [10:41] Cimi, just a flickable [10:41] whatever then [10:41] so if you ddon't scroll [10:41] when you open the preview [10:41] textarea immediately gets input [10:42] otherwise, you have to hold [10:42] Cimi, interesting... [10:42] Cimi, btw, it should scroll up on focus [10:42] Cimi, otherwise it ends up behind the keyboard [10:43] but yeah, back to the issue at hand [10:43] I agree with that (how do we do this?) [10:43] I hope we have auto scrolling from the sdk/osk [10:43] Cimi, the expandables in the SDK should help [10:44] Cimi, try and reproduce the issue with a simple TextArea in a Flickable === shiznix_ is now known as shiznix [10:45] Saviq, I think is a focus issue [10:45] buttons seems to behave ok [10:47] Saviq, ok, another finding [10:47] Saviq, scroll, quickly tap on text area, wait 300ms, tap -> focus [10:48] Saviq, seems like the flickable keeps focus or something [10:48] so you either tap and hold after scroll or you need a second tap [10:49] Cimi, flickable can't "keep focus", tapping on a text area forces the focus on it, assuming it actually gets the touch input [10:49] Cimi, and touch input does not care about focus [11:00] Saviq, ok if we land the click scope changes even though the widget is odd? [11:00] Saviq, shall I use anchors left and right inside column items? [11:00] Cimi, yes [11:00] Saviq, or width? [11:00] mhr3, you tell me, it's not working ;) [11:00] Cimi, anchors [11:00] agree, I prefer [11:00] pushing [11:01] Saviq, but theoretically it doesn't introduce a regression :) [11:01] mhr3, back to unity times, eh? :P [11:01] mhr3, let's find a fix, make it work, then land? [11:01] mhr3, unless you're pressed for landing? [11:02] damn apport-bug [11:02] Saviq, nah, i'm ok with waiting [11:02] will talk to dobey about the signon issue [11:03] Saviq, ok https://code.launchpad.net/~cimi/unity8/preview-text-summary-bottom-padding/+merge/217903 [11:03] i do wonder how do they test their stuff if they didn't notice this weirdness [11:03] shit, test fails [11:04] ;) [11:06] Saviq, weird, needs time with column [11:06] * Cimi replaces compare with trycompare [11:06] Cimi, sure, column needs to wait for its children to lay them out [11:07] Cimi, and it's not really about time, but about letting the event loop spin [11:08] Saviq, thx [11:08] test fixed === jhodapp|afk is now known as jhodapp [11:50] Cimi, you can't set topMargin in a column === alan_g is now known as alan_g|lunch === _salem is now known as salem_ [12:07] Saviq, ok [12:07] Saviq, I will add it to height of the text item [12:07] Cimi, you'd know if you looked at the debug output ;) [12:07] Saviq, no debug [12:07] Cimi, sure debug [12:07] well, no debug [12:07] just log output [12:08] Cimi, with try*, you get output after you close the thing [12:08] cimi@draco:~/Development/unity8/card-touchpress/builddir$ make tryPreviewTextSummary [12:08] QML debugging is enabled. Only use this in a safe environment. [12:08] file:///usr/lib/x86_64-linux-gnu/qt5/qml/QtTest/TestCase.qml:345:32: Unable to assign [undefined] to bool [12:08] unless this is the famous debug [12:09] Cimi, maybe that's because you don't have a top anchor, so it doesn't care at all [12:09] Cimi, something's wrong, btw, with your change the expand / contract is not smooth [12:12] Saviq, here is smooth [12:12] Cimi, well, here it isn't ;P [12:12] Saviq, column maybe [12:12] Cimi, column deals just fine with growing items [12:14] Saviq, I fixed and pushed the anchors [12:14] Saviq, no idea for the issue with the animated expansion [12:14] Saviq, there's a Behaviour on height [12:15] you sure your computer is not just skipping frames? [12:15] Cimi, you're not the only one with a i7 quadcore :P [12:15] Cimi, but imagine that I compared the two! [12:16] is smooth before, not smooth after [12:17] crap, if only we could have slow animations... [12:17] Saviq, ah you mean it skips frames? [12:17] so you see the animation but not smooth? [12:18] it must be the column [12:18] btw there were some designs with seemore/less [12:18] cannot find them anymore [12:18] Cimi, yeah, were for tablet, gone atm [12:23] Cimi, I'd be really worried if it was indeed Column that caused the slowdown [12:25] Cimi, but it does seem like it... [12:25] Saviq, cannot reproduce with my i7 4770 [12:25] 8-) [12:28] Cimi, http://pastebin.ubuntu.com/7372630/ [12:28] Cimi, there's less height changes with a Column :| [12:28] wth [12:28] yeah [12:28] btw [12:29] this is more correct probably [12:29] onImplicitHeightChanged: console.log(implicitHeight) [12:29] no it's not, if I can see the difference [12:29] http://paste.ubuntu.com/7372635/ [12:29] in a sense it is [12:32] Saviq, shall I go back with anchors? [12:32] Saviq, also, looks simpler with anchors to me :D [12:33] only thing I am not a fan of is the implicitheight calculation [12:33] Cimi, how can it look simpler than just having a few items in a Column? ;) [12:33] Saviq, look at the last patch [12:34] Cimi, ETOOMANY childrenRect.height [12:34] that's definitely not simpler [12:34] Saviq, I can use the direct id [12:34] same story [12:35] yeah, not simpler IMO [12:35] I want to add padding at the top [12:35] let me try on the device [12:35] with the column [12:35] only way to do it is with an empty item [12:35] sure, yeah, having padding in columns is something I miss dearly [12:35] the old tablet designs had 1 gu spacing, that's why I added [12:36] might be better without [12:36] in my opinion [12:36] but we don't have designs [12:37] Cimi, so yeah, we need ExpandableColumn in there, expanding text should come on screen too [12:37] Cimi, but I can't see the Column issue on the device [12:38] looks plenty smooth (but it's too fast, too, should be velocity based, not duration based) [12:40] Saviq, which preview has seemore/less? [12:40] didn't find long description so far [12:40] Cimi, xbmcremote for example [12:40] cool [12:41] Cimi, but yeah, let's go with Column, it really is simpler - and we want the margin under text as well as under the see more / less widget [12:41] Cimi, but I think we need a HR on top of review input, otherwise it's bad with text [12:41] Saviq, with last patch looks awful with column [12:41] not be able to add topmargin [12:43] Cimi, but you added 1gu in text height, that should be fine [12:43] Cimi, basically what I'm saying, your current branch looks good [12:43] Saviq, no [12:43] but does need a HR [12:43] Saviq, if you add 1px to text height [12:43] Saviq, you add one gu of text clipped [12:44] while we want 1 px of padding [12:44] so the text must not be 1gu taller [12:44] while we need an item to add spacing [12:45] Cimi, yeah, either add a 1GU-high empty item, or wrap the label in an item + 1GU high [12:46] that's what I did [12:46] http://paste.ubuntu.com/7372635/ [12:46] looks bad [12:46] I prefer with anchors [12:47] Cimi, just add a Item { height: units.gu(1) } between them [12:48] Cimi, we want the 1gu padding under text itself, too [12:51] Saviq, http://paste.ubuntu.com/7372703/ vs http://paste.ubuntu.com/7372718/ [12:51] I have no preferences [12:52] Cimi, we *want* it even if there's no seeMore [12:52] nope [12:52] all other preview widgets have no extra padding [12:53] ok, spacing is done in the listview [12:54] Cimi, go for the wrapping one, but drop the id and objectName from seeMore (only leave it in seeMoreContainer), and don't add the second visibility test [12:55] Saviq, I need it for tests [12:55] Cimi, you don't need to test it, that's what I'm saying [12:55] Cimi, btw, anchors { left: parent.left; right: parent.right } usually read better [12:55] I need to test seeMore.more [12:55] and such [12:55] right [12:56] also, since we are here, I tested the container that goes away [12:56] adding one line of test doesn't hurt if it's basically easy and free to do [12:56] Cimi, + compare(seeMoreContainer.visible, true)? [12:56] mistake [12:56] is false [12:56] already corrected in second patch [12:57] bleh [12:57] not being to do padding in columns is bleh [12:59] Cimi, ok, I take that back [12:59] I pushed with the nested item [12:59] Cimi, if we have to do it like this, go back to anchors, it's awful with the hacked-in padding [12:59] Cimi, sorry [13:00] finally! :D [13:00] no worries [13:00] we discovered this ugly behaviour [13:01] /food === alan_g|lunch is now known as alan_g [13:16] when network says connected but it bloody doesn't work [13:16] any fix apart rebooting and praying? [13:37] Cimi, you can try rebooting without praying :) [13:44] mhr3, nope [13:44] doesn't like my prays [13:45] we need an online connectivity check [13:46] seeing the dash and previews and such full empty [13:46] blank pages [13:47] is weird [13:49] dednick, I have same issue with resolv.conf [13:51] and others [13:51] dammit [14:17] mhall119, it is possible to launch https://developer.ubuntu.com and looks broken (http is fine) [14:18] Saviq, which animation shall I use for the behaviour? [14:18] Cimi, which behaviour? [14:19] Saviq, height expansion [14:19] in textareass [14:19] Cimi, UbuntuAnimation with SnapDuration I'd say [14:20] Saviq, UbuntuAnimation { duration: UbuntuAnimation.SnapDuration } ? [14:20] Cimi: file a bug please, we need to fix the theme headers to use the same protocol as the original request (the css files are hard-coded to use http:// currently) [14:21] Cimi, yeah [14:21] Saviq, ok done [14:21] pushed [14:22] anyone remember how to disable automatic mtp mounting of devices? [14:22] greyback, setprop sys.usb.config=ptp,adb [14:22] greyback, setprop persist.sys.usb.config=ptp,adb [14:23] Saviq: nothing on the PC side, no? [14:23] greyback, nope [14:23] Saviq: ok, thanks [14:23] greyback, well, you could kill the mtp monitor [14:23] Cimi, yikes on http://bazaar.launchpad.net/~cimi/unity8/preview-text-summary-bottom-padding/revision/876, why didn't the previous approach work? [14:24] Saviq, previous approach wasn't counting on invisible textlabel... [14:24] titlelabel.. [14:24] unless a Label with text = "" is 0 px height [14:25] result seems similar, this looks just more correct [14:25] this whole thing makes my eyes water :P [14:25] Saviq, yeah, my code is like chopping fresh onions [14:26] let me see if height when text == "" is 0 [14:26] in this case we can remove that logic [14:26] Cimi, you can just do "height: visible ? implicitHeight : 0" [14:26] Cimi, that's actually what we did in CardHeader.qml [14:26] let me do the same for the seemore then [14:28] http://paste.ubuntu.com/7373290/ [14:28] Cimi: don't worry about that bug, a fix is in the pipeline already [14:28] mhall119, filed already [14:28] https://bugs.launchpad.net/ubuntudeveloperportal/+bug/1315016 [14:28] Ubuntu bug 1315016 in Ubuntu App Developer site "Theme is broken using https" [Undecided,New] === alan_g is now known as alan_g|tea === alan_g|tea is now known as alan_g [14:53] Saviq, how to restart shell from phone? restart unity8 as phablet? [14:53] Cimi, yes [14:53] it does not here [14:58] Cimi, how did you log in as phablet? [14:58] Saviq, sudo -u phablet -i [14:58] Cimi, and what does "restart unity8" say? [14:59] Saviq, hangs [14:59] Cimi, does not hang, but waits for unity8 to exit (it does take some time) [14:59] Cimi, you can also just go "pkill unity8" [14:59] Cimi, it will respawn [15:00] having a working wifi would dimplify my life here [15:00] on this ubuntu phone [15:14] I throw this phone [15:14] Saviq, bootstrap, no luck [15:14] I keep rebooting [15:14] Cimi, file a bug [15:15] feels like windows ME [15:15] ahah [15:15] * mhr3 wouldn't like to be close to Cimi now [15:15] mhr3, why not, I throw free Nexus 4s [15:15] Cimi, i'd be worried you throw it in my face :P [15:17] root@ubuntu-phablet:~# cat /etc/resolv.conf [15:17] # Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8) [15:17] # DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN [15:17] nameserver 127.0.1.1 [15:17] WHY [15:18] what's this [15:19] ah there;s dnsmasq === salem_ is now known as _salem === jhodapp is now known as jhodapp|lunch [15:34] * greyback akf for 45 mins [15:35] xnox, hey, can I bug you for some CMake pointers? [15:38] Saviq: go ahead! [15:39] xnox, so... I'm looking for a recommended way to override an executable (qmltestrunner) that we've been finding with find_program until now [15:39] xnox, problem with find_program is that once it finds something, it will keep it until the cache is removed [15:41] xnox, I started with add_executable(... IMPORTED ...), but that only really works for add_custom_command AFAICT, and we need to prefix with env anyway [15:42] Saviq: find it each time -> as in don't do find_program at all. [15:42] xnox, how can I find other than find_program? [15:42]