=== duflu_ is now known as duflu [03:00] Hey I need a hand with making script button with unity. I have made the button however I think my problem it requires root priv to exec [03:01] these are my scripts [03:01] http://pastebin.com/mhayTXeG [03:01] http://pastebin.com/LGGXbWGS [03:13] I have to go, but i anyone can help and sees this message could you pls email at mac.tzu@gmail.com === Wog is now known as Wug [09:24] dednick: did you finish running the tests for https://code.launchpad.net/~allanlesage/unity8/autopilot-indicator-page-title-matches-widget/+merge/196991 ? [09:25] tsdgeos: yeah, but i had some random failures in non-related tests, so I need to run again [09:25] dednick: seen the comment i made on the other indicator test? can you repro the issue? [09:25] tsdgeos: will take a look now [09:26] oki [09:30] tsdgeos: was that on device? [09:30] dednick: no [09:31] tsdgeos: and I'm presuming the thing you're talking about is the tab bar? [09:31] dednick: yes, the text is cut as you see on the screenshot [09:31] tsdgeos: what happens when you move? [09:32] tsdgeos: i can't repro [09:32] in other words [09:32] oh, it just happened. weird [09:32] well if i move it moves to a different indicator [09:32] and when i'm back it works [09:33] tsdgeos: hm. it only happened once... finding hard to repro [09:34] happens 90% of the cases here [09:34] the other times it travels from the right [09:34] which is also unxpected [09:35] tsdgeos: which qt ver you on? [09:35] 5.2.1? [09:35] no [09:35] 5.0.2 [09:35] as everyone is [09:35] hmm [09:36] tsdgeos: does it happen when you don't unlock? [09:37] yes [09:37] seems the unlock step is not needed [09:37] so litterally just "press hold on the messaging icon without move"? [09:37] yep [09:38] otoh it's an improvement [09:38] since if i doo that on the current master [09:38] i get the networking tex [09:38] t [09:39] ehhe [09:41] tsdgeos: and only first time you open indicators? [09:42] yes [09:49] tsdgeos: i think it may be a bug in the tabs [09:51] tsdgeos: i think because tabs are being inserted dynamically before the one which is currently selected, it's not adjusting. [09:52] tsdgeos: let me just confirm in a separate test [09:55] greyback: ricmm_ so we're gonna promote latest mir, looks like client abi broke...so just [09:55] need to invoke a rebuild for clients [09:56] is it only papi & unity-mir ? [09:56] or...does something else use client api? [09:57] kgunn: nothing else uses it [10:28] didrocks, hey, if you have a sec, can you tell me who to talk to about PPA issues? [10:29] Saviq: depends on the ppa, is it a general launchpad issue or something else ? [10:30] didrocks, rather general: https://launchpadlibrarian.net/162419907/buildlog_ubuntu-saucy-amd64.capnproto_0.4.0-0ppa~5~ubuntu13.10.3_UPLOADING.txt.gz [10:30] didrocks, it's running kernel 2.6 [10:30] didrocks, it's a virt ppa [10:30] didrocks, and tests fail there because of the old kernel [10:31] Saviq: ok, you need to ping the soyuz support channel then [10:50] tsdgeos: it's a ubuntu-ui-toolkit bug. I'll raise it with sdk [10:53] dednick: oki [10:54] dednick: can you link the bug in the review? [10:54] as a comment [10:54] tsdgeos: sure === _salem is now known as salem_ === anpok is now known as anpok|lunch [11:46] dednick: do you have a simple test you can attach to that bug? [11:46] tsdgeos: test for what? [11:46] dednick: for the misalignment [11:46] oh, you mean the sdk bug? [11:46] yep [11:47] tsdgeos: yeah [11:48] dednick: then you should probably add it to make it easier for the sdk guys to test it instead of them having to start unity8 [11:48] well - "simple" [11:48] simpler than compiling unity8 for someone that doesn't do it very often i hope :D [11:49] tsdgeos: I'm trying to add the test directly into the sdk branch, so they can just fix it from there... [11:49] ok [12:13] greyback: hello! === MacSlow is now known as MacSlow|lunch [12:13] sil2100: hey [12:13] greyback: just wanted to get a big upper confirmation - are all components for the dimention support for sidestage landed and ready for release? [12:14] greyback: since we saw the unity-mir merge landing already [12:15] sil2100: correct. unity-mir would need to released too, as without the experience is broken === alan_g is now known as alan_g|afk [12:16] greyback: ok, so the components to release are: platform-api, qtubuntu and unity-mir, right? [12:16] sil2100: correct === anpok|lunch is now known as anpok [13:11] tsdgeos, organicgrid conflicted === alan_g|afk is now known as alan_g === MacSlow|lunch is now known as MacSlow [13:56] booo [13:56] Saviq: tx [14:02] mhall119, good news, we found the issue with u8 deps on saucy [14:02] come on [14:02] 1: /home/tsdgeos_work/phablet/unity8/organicgrid/plugins/DashViews/organicgrid.cpp: multiple new lines at end of file [14:02] :D [14:02] tsdgeos, ;) [14:03] tsdgeos, should've `make testwhitespace` before ;) [14:03] or whatever [14:03] yeah i did before actually [14:03] the test still didn't get there [14:05] dandrader: organic grid is up at https://code.launchpad.net/~aacid/unity8/organicgrid/+merge/201586 you taking care of it? [14:06] yes, I can review it. === dandrader is now known as dandrader|lunch [14:17] Saviq: ah one thing i wanted to ask you, in the journals i print a warning when the delegate width/height do not match the width/height set to the journal [14:18] but in the organicgrid i think it's better not to do that, since you don't know if it'll end up in a big or a small delegate spot (well you know but it'd mean add an if in the qml side that i don't think gives us much) [14:18] Saviq: opinion? [14:18] dandrader|lunch: ↑↑ === dandrader|lunch is now known as dandrader [14:27] Saviq: \o/ [14:27] Saviq: so can I build it now? [14:28] mhall119, would be great if you tried [14:28] https://code.launchpad.net/~unity-team/+archive/ppa [14:29] Saviq: will do [14:29] Saviq, my main system is bricked atm and I can't connect to mumble atm as I'm missing the certificat password... [14:30] MacSlow: ?¿ [14:30] ouch [14:30] MacSlow: ok, fill in your stuff in the document :D [14:31] tsdgeos, I have kernel-panics all over the place after trying to update the UbuntuTouch emulator... fighting with this the whole day to solve... and on my laptop my mumble-setup only has the old certificate for mumble [14:31] You guys keeping some SU notes, right? I've been to a team that did Mumble standup, another one did IRC standup, current one does Hangout, and this one does Mumble again :) [14:38] karni: yes, we keep standup notes [14:38] good :) [14:42] mhall119, ah, and you need the sdk-team PPA, but that you probably have already === alan_g is now known as alan_g|tea [14:59] Saviq: the unity-team ppa isn't going to break my desktop unity is it? [15:03] mhall119, no === alan_g|tea is now known as alan_g [15:07] tsdgeos, can't run DashViews tests anymore [15:08] tsdgeos, $/builddir/make [try|test]Foo fails [15:08] (missing make targets) [15:09] ?¿ [15:09] really [15:09] ? [15:09] i did use that :D [15:10] tsdgeos: I noticed verticaljournal extends QQuickItem, thus does not have properties like: interactive, flicking, moving, pressDelay from Flickable (i.e. like GridView). Do we care? [15:10] dandrader: ping [15:10] dednick, pong [15:10] karni: yes, the vertical journal is not flickable, you have to put it inside a list to move it [15:10] tsdgeos: gotcha [15:10] karni: basically if you look at the dash, it's a list with things inside [15:11] the journal is one of those things [15:11] tsdgeos: ACK :) [15:11] dandrader: make testOrganicgrid just worked here [15:11] dandrader: doesn't work for you? [15:11] tsdgeos: Since I already have your attention - verticaljournal keeps track of it's implicit height, right? [15:11] karni: yes [15:11] so that I can tell whether it's clipped or not. great. [15:11] dandrader: howdy. just noticed something a bit odd with the DragHandles when using a hint. It looks like if you click for hint, then drag, the displacement is getting reset to 0. causes a bit of a flash as you drag. [15:12] ahhhhhhhh [15:12] dandrader: can see it either in indicators or tryDragHandle [15:12] its testOrganicgrid, not testOrganicGrid [15:12] dandrader: want me just to log a bug? [15:12] tsdgeos, can you fix that (and for the others as well)? [15:12] dandrader: actually i can't [15:12] since the name is autogenerated [15:12] * karni agrees testOrganicGrid would be more intuitive, that'd bite me too :) [15:13] oh :( [15:13] that's why you guys use the tab to autocomplete [15:13] hahah [15:13] make testO [15:13] tsdgeos, everywhere else is testFooBar, not testFoobar [15:13] dandrader: i know [15:13] tsdgeos, if there's a will, there's a way [15:13] dandrader: there is a way, i don't feel like doing it, moreover when you already approved the current naming [15:14] moving the goal posts is not fun [15:14] dednick, can you reproduce it with "make tryDragHandle"? [15:14] dandrader: ya [15:15] dandrader: i'll just create a bug and assign you. [15:15] bah [15:15] * tsdgeos shuts up and changes the capitalization [15:15] dednick, yeah, just saw the bug you described [15:16] with make tryDragHandle [15:16] that's definitely a regression [15:16] wasn't there before [15:16] dandrader: cool. i'll log it. [15:16] mhall119, there's a package missing in the ppa, just setting up the recipe [15:17] dandrader: pushed [15:17] tsdgeos, It slipped past me. If I had noticed it before I would not approve it :) [15:17] tsdgeos: verticaljournal has no currentIndex, like GridView in grid layout would, to track hilight/currently selected item. Do we care? :) [15:18] tsdgeos, the whole point of standardized make test/try targets is to make them predictable so that you don't have to look at the corresponding CMakelists.txt to find out how the heck to summon it [15:18] tsdgeos: Sorry for those questions, I'm just going through that code and saying out loud if I have any worries/doubts :) [15:18] karni: that's a good question, yes it has no index concept [15:18] you just have to known the name of the class and that's it [15:19] tsdgeos: Should I file a bug, so we don't forget? [15:19] tsdgeos: OR, it does not and will not have index concept? [15:19] Maybe that's what you meant. [15:19] karni: don't know, i guess we will need one to do keyboard navigation [15:19] other than keyboard navigation i don't see us needing one [15:20] maybe for previews too [15:20] since you can switch to next/previous there [15:20] tsdgeos: ok. I'll log a bug, just in case. Should I assign you, or leave it be for someone to pick it at some point? [15:21] tsdgeos, so... we don't need the workaround that you've merged yesterday any more... [15:21] tsdgeos, I'm tempted to uncommit it... [15:21] Saviq: the one for the python stuff? [15:21] tsdgeos, yes [15:21] tsdgeos, got fixed since yesterday... [15:22] karni: yeah file a bug, assign it to me, will think about it [15:22] okay [15:22] Saviq: is the change wrong? [15:22] tsdgeos, it's not, but unnecessary [15:22] tsdgeos, ok never mind, will put it in my "various fixes" branch [15:22] Saviq: well, reverting it is another unnecessary change then :D [15:23] but yeah, don't have an opinion [15:23] tsdgeos, that's why I'm saying uncommit ;) [15:23] ah [15:23] not revert [15:23] you mean uncommit and force push [15:23] since it's still the tip [15:23] tsdgeos, yes [15:23] brrrrrr [15:23] ok then no [15:24] tsdgeos, https://code.launchpad.net/~saviq/unity8/clean-deps/+merge/201608 though [15:25] * tsdgeos clicks [15:26] Saviq: don't need unity-mir at all? [15:26] ah [15:26] the libunity-mir-dev thing already wroks [15:27] tsdgeos, yup [15:27] Saviq: approved [15:28] tsdgeos, thanks, let's see what CI says [15:28] fail in otoo :D [15:28] otto [15:29] Saviq, the recent unity8 otto runs have unity8 crash files now. Unfortunately, I don't think they've been retraced so probably of no use [15:35] Saviq: just let me know when it's ready and I'll update/upgrade/rebuild [15:40] Saviq, wow, that last message I sent you was completely wrong. The unity8 otto tests are now passing. I was looking at a mako test and saw the crashdump. [15:43] fginther: are they? i saw lots of red [15:47] tsdgeos, I guess I may have jumped to soon, but the most recent run passed the otto test: http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2060/console [15:48] interesting [15:49] well maybe as it came it went away?¿ [15:49] tsdgeos, another recent run also had just one failure: http://s-jenkins.ubuntu-ci:8080/job/autopilot-testrunner-otto-trusty/2044/testReport/ [15:49] why was i away?¿ D: [15:49] tsdgeos, in make tryOrganicGrid, press "add" until you have 10 items (up to index 9) [15:50] do i make the window bigger? [15:50] tsdgeos, I wonder if the designer considered this situation [15:50] tsdgeos, looks bad if you ask me [15:50] this meaningless void on the right hand side [15:51] dandrader: the same whole that if you only have 4 items, no? [15:51] tsdgeos, yes [15:52] tsdgeos, but it just looks worse when you have that on the second component [15:52] dandrader: the "real" sizing of the delegates will probably make it a bit less bad [15:52] but yeah it's not ultra cool [15:53] i guess the designers idea is that it'll always there be "enough" stuff [15:53] tsdgeos, I don't know. it feels like its working against usability [15:53] but let me see what the gallery does [15:53] if i can start the phone :D [15:55] tsdgeos, maybe the order of items in a "module" should be different if there are less than 6 items === salem_ is now known as _salem [15:55] tsdgeos, because the layout when there are only 4 items in the module just doens't make sense [15:56] tsdgeos, and I'm starting to think designers out to agree on that point [15:56] s/out/ought [15:57] tsdgeos: thanks for looking into the currentIndex thing already. [15:57] dandrader: if you have the phone working (i need to reflash it) take 4 pictures with the camera and see what the gallery shows [15:59] tsdgeos, hmm, I also think I have to reflash mine (galaxy nexus). didn' t touch it since I got the nexus 10 on last sprint [15:59] dandrader: use the nexus10 then :D [15:59] should have the gallery app too [16:00] tsdgeos, I'm using it for the qml mir compositor work [16:00] ah [16:00] ok [16:00] well then let's wait 10 min until mine gets flashed [16:00] I'm flashing mine as well === _salem is now known as salem_ [16:01] tsdgeos, btw, the filling order works very well horizontally (if you make the tryOrganicGrid window very wide and start adding items to it) [16:02] the gallery-app is horizontal [16:02] maybe that's why it doesn't look bad there [16:03] tsdgeos: I wonder if it would be worth to expose columns (count) in vertical journal. It has the columnWidth property, but that means 'columns' has to be calculated in QML, even though you already do that in cpp. [16:04] karni: isn't it backwards? you know how much space you have and how much columns you need and then set the columnwidth? [16:04] well not really that [16:04] as far as i understand [16:04] you know the width [16:05] and you know your columns have to me between 10 and 20, so you calculate the proper sizes (10 and 20 are random numbers i just invented) [16:05] at least that's how i understand it [16:05] brb standup, sorry! [16:05] fginther: yeah the organicgrid otto tests passed :o [16:07] Saviq: do you have katie in there? [16:10] tsdgeos, I believe they've reached EOD there [16:10] but aren't they behind? [16:12] tsdgeos: d'oh, sure. you're right :) [16:12] dandrader: ok, i'll take a screenshot and send it via email to saviq, katie, you then, ok? [16:13] tsdgeos, ok, thanks! [16:14] tsdgeos: yep, we've hit eod [16:14] tsdgeos: Can I ask one question about your code? http://paste.ubuntu.com/6751191/ [16:17] tsdgeos: In my understanding, in RespGridView we subtract spacing/2 from width (say, left margin), and divide the rest over colWidth+colSpacing() (so, this excludes right margin, because that is included in the added columnSpacing already) [16:17] tsdgeos: If that doesn't make mush sense, or you don't feel like talking about pixel perfect shit, just be honest with me :D I'll leave hahah. [16:17] I just thought I'd bring up that small inconsistency I noticed. [16:18] karni: i think the difference is that responsivegridview takes into account the outer spacing while the journals do not [16:18] i.e. journal will put stuff at 0,0 [16:18] and responsivegridview probabbly not [16:19] the journal way is a bit more versatily since it allows you to have a different inner spacing and outer spacing [16:19] in case you want that [16:19] am i making sense? [16:20] oh, that makes sense. I must have not realized journal could put stuff at 0,0 instead spacing/2, 0 [16:20] tsdgeos: yes, makes sense. thanks [16:21] Wouldn't then [16:21] const int nColumns = qMax(1., floor((double)(width() + columnSpacing()) / (m_columnWidth + columnSpacing()))) [16:21] dandrader: mail sent, does it make sense to you? [16:21] be [16:21] const int nColumns = qMax(1., floor((double)(width()) / (m_columnWidth + columnSpacing()))) [16:22] nope [16:22] If you can put stuff at 0,0 (no left margin) [16:22] the last column doesn't get a columnspacing [16:22] oh [16:22] :D thanks tsdgeos [16:23] width = ncolumns * columnWidth + ncolumns * (columnSpacing - 1) [16:23] now do the math [16:23] * karni is really enjoying exploring this code base [16:23] and you end up with the other formula [16:23] correct === dandrader is now known as dandrader|afk [16:28] karni: cool, hope that it means the code is not so bad :D [16:29] tsdgeos: hahahahah, it's gooooooooood /me makes Bruce Almighty impression [16:29] :) [16:32] * tsdgeos pretends to know who is that Bruce something [16:32] tsdgeos: https://www.youtube.com/watch?v=nVCe0oWdnrE [16:34] ahh [16:34] :) [16:34] title translations always are creative here [16:34] that movie was called "Like God" [16:34] tsdgeos: I guess. That's why I always hunt for original titles :) [16:35] translation to Polish was, in this case, very adequate to the original. but yeah, I get your point, some are really terrible :) === dandrader|afk is now known as dandrader [16:59] dandrader: right the docu is missing [17:00] and the ascii art too [17:00] will do tomorrow [17:00] getting eod'ed here [17:00] ok === alan_g is now known as alan_g|EOD === salem_ is now known as _salem