/srv/irclogs.ubuntu.com/2014/01/14/#ubuntu-unity.txt

=== duflu_ is now known as duflu
otakuHey 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 exec03:00
otakuthese are my scripts03:01
otakuhttp://pastebin.com/mhayTXeG03:01
otakuhttp://pastebin.com/LGGXbWGS03:01
otakuI have to go, but i anyone can help and sees this message could you pls email at mac.tzu@gmail.com03:13
=== Wog is now known as Wug
tsdgeosdednick: did you finish running the tests for https://code.launchpad.net/~allanlesage/unity8/autopilot-indicator-page-title-matches-widget/+merge/196991 ?09:24
dednicktsdgeos: yeah, but i had some random failures in non-related tests, so I need to run again09:25
tsdgeosdednick: seen the comment i made on the other indicator test? can you repro the issue?09:25
dednicktsdgeos: will take a look now09:25
tsdgeosoki09:26
dednicktsdgeos: was that on device?09:30
tsdgeosdednick: no09:30
dednicktsdgeos: and I'm presuming the thing you're talking about is the tab bar?09:31
tsdgeosdednick: yes, the text is cut as you see on the screenshot09:31
dednicktsdgeos: what happens when you move?09:31
dednicktsdgeos: i can't repro09:32
dednickin other words09:32
dednickoh, it just happened. weird09:32
tsdgeoswell if i move it moves to a different indicator09:32
tsdgeosand when i'm back it works09:32
dednicktsdgeos: hm. it only happened once... finding hard to repro09:33
tsdgeoshappens 90% of the cases here09:34
tsdgeosthe other times it travels from the right09:34
tsdgeoswhich is also unxpected09:34
dednicktsdgeos: which qt ver you on?09:35
dednick5.2.1?09:35
tsdgeosno09:35
tsdgeos5.0.209:35
tsdgeosas everyone is09:35
dednickhmm09:35
dednicktsdgeos: does it happen when you don't unlock?09:36
tsdgeosyes09:37
tsdgeosseems the unlock step is not needed09:37
dednickso litterally just "press hold on the messaging icon without move"?09:37
tsdgeosyep09:37
tsdgeosotoh it's an improvement09:38
tsdgeossince if i doo that on the current master09:38
tsdgeosi get the networking tex09:38
tsdgeost09:38
dednickehhe09:39
dednicktsdgeos: and only first time you open indicators?09:41
tsdgeosyes09:42
dednicktsdgeos: i think it may be a bug in the tabs09:49
dednicktsdgeos: i think because tabs are being inserted dynamically before the one which is currently selected, it's not adjusting.09:51
dednicktsdgeos: let me just confirm in a separate test09:52
kgunngreyback: ricmm_ so we're gonna promote latest mir, looks like client abi broke...so just09:55
kgunnneed to invoke a rebuild for clients09:55
kgunnis it only papi & unity-mir ?09:56
kgunnor...does something else use client api?09:56
greybackkgunn: nothing else uses it09:57
Saviqdidrocks, hey, if you have a sec, can you tell me who to talk to about PPA issues?10:28
didrocksSaviq: depends on the ppa, is it a general launchpad issue or something else ?10:29
Saviqdidrocks, rather general: https://launchpadlibrarian.net/162419907/buildlog_ubuntu-saucy-amd64.capnproto_0.4.0-0ppa~5~ubuntu13.10.3_UPLOADING.txt.gz10:30
Saviqdidrocks, it's running kernel 2.610:30
Saviqdidrocks, it's a virt ppa10:30
Saviqdidrocks, and tests fail there because of the old kernel10:30
didrocksSaviq: ok, you need to ping the soyuz support channel then10:31
dednicktsdgeos: it's a ubuntu-ui-toolkit bug. I'll raise it with sdk10:50
tsdgeosdednick: oki10:53
tsdgeosdednick: can you link the bug in the review?10:54
tsdgeosas a comment10:54
dednicktsdgeos: sure10:54
=== _salem is now known as salem_
=== anpok is now known as anpok|lunch
tsdgeosdednick: do you have a simple test you can attach to that bug?11:46
dednicktsdgeos: test for what?11:46
tsdgeosdednick: for the misalignment11:46
dednickoh, you mean the sdk bug?11:46
tsdgeosyep11:46
dednicktsdgeos: yeah11:47
tsdgeosdednick: then you should probably add it to make it easier for the sdk guys to test it instead of them having to start unity811:48
dednickwell - "simple"11:48
tsdgeossimpler than compiling unity8 for someone that doesn't do it very often i hope :D11:48
dednicktsdgeos: I'm trying to add the test directly into the sdk branch, so they can just fix it from there...11:49
tsdgeosok11:49
sil2100greyback: hello!12:13
=== MacSlow is now known as MacSlow|lunch
greybacksil2100: hey12:13
sil2100greyback: just wanted to get a big upper confirmation - are all components for the dimention support for sidestage landed and ready for release?12:13
sil2100greyback: since we saw the unity-mir merge landing already12:14
greybacksil2100: correct. unity-mir would need to released too, as without the experience is broken12:15
=== alan_g is now known as alan_g|afk
sil2100greyback: ok, so the components to release are: platform-api, qtubuntu and unity-mir, right?12:16
greybacksil2100: correct12:16
=== anpok|lunch is now known as anpok
Saviqtsdgeos, organicgrid conflicted13:11
=== alan_g|afk is now known as alan_g
=== MacSlow|lunch is now known as MacSlow
tsdgeosbooo13:56
tsdgeosSaviq: tx13:56
Saviqmhall119, good news, we found the issue with u8 deps on saucy14:02
tsdgeoscome on14:02
tsdgeos1: /home/tsdgeos_work/phablet/unity8/organicgrid/plugins/DashViews/organicgrid.cpp: multiple new lines at end of file14:02
tsdgeos:D14:02
Saviqtsdgeos, ;)14:02
Saviqtsdgeos, should've `make testwhitespace` before ;)14:03
Saviqor whatever14:03
tsdgeosyeah i did before actually14:03
tsdgeosthe test still didn't get there14:03
tsdgeosdandrader: organic grid is up at https://code.launchpad.net/~aacid/unity8/organicgrid/+merge/201586 you taking care of it?14:05
dandraderyes, I can review it.14:06
=== dandrader is now known as dandrader|lunch
tsdgeosSaviq: 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 journal14:17
tsdgeosbut 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
tsdgeosSaviq: opinion?14:18
tsdgeosdandrader|lunch: ↑↑14:18
=== dandrader|lunch is now known as dandrader
mhall119Saviq: \o/14:27
mhall119Saviq: so can I build it now?14:27
Saviqmhall119, would be great if you tried14:28
Saviqhttps://code.launchpad.net/~unity-team/+archive/ppa14:28
mhall119Saviq: will do14:29
MacSlowSaviq, my main system is bricked atm and I can't connect to mumble atm as I'm missing the certificat password...14:29
tsdgeosMacSlow: ?¿14:30
tsdgeosouch14:30
tsdgeosMacSlow: ok, fill in your stuff in the document :D14:30
MacSlowtsdgeos, 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 mumble14:31
karniYou 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:31
tsdgeoskarni: yes, we keep standup notes14:38
karnigood :)14:38
Saviqmhall119, ah, and you need the sdk-team PPA, but that you probably have already14:42
=== alan_g is now known as alan_g|tea
mhall119Saviq: the unity-team ppa isn't going to break my desktop unity is it?14:59
Saviqmhall119, no15:03
=== alan_g|tea is now known as alan_g
dandradertsdgeos, can't run DashViews tests anymore15:07
dandradertsdgeos, $/builddir/make [try|test]Foo fails15:08
dandrader(missing make targets)15:08
tsdgeos?¿15:09
tsdgeosreally15:09
tsdgeos?15:09
tsdgeosi did use that :D15:09
karnitsdgeos: 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
dednickdandrader: ping15:10
dandraderdednick, pong15:10
tsdgeoskarni: yes, the vertical journal is not flickable, you have to put it inside a list to move it15:10
karnitsdgeos: gotcha15:10
tsdgeoskarni: basically if you look at the dash, it's a list with things inside15:10
tsdgeosthe journal is one of those things15:11
karnitsdgeos: ACK :)15:11
tsdgeosdandrader: make testOrganicgrid just worked here15:11
tsdgeosdandrader: doesn't work for you?15:11
karnitsdgeos: Since I already have your attention - verticaljournal keeps track of it's implicit height, right?15:11
tsdgeoskarni: yes15:11
karniso that I can tell whether it's clipped or not. great.15:11
dednickdandrader: 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:11
dandraderahhhhhhhh15:12
dednickdandrader: can see it either in indicators or tryDragHandle15:12
dandraderits testOrganicgrid, not testOrganicGrid15:12
dednickdandrader: want me just to log a bug?15:12
dandradertsdgeos, can you fix that (and for the others as well)?15:12
tsdgeosdandrader: actually i can't15:12
tsdgeossince the name is autogenerated15:12
* karni agrees testOrganicGrid would be more intuitive, that'd bite me too :)15:12
karnioh :(15:13
tsdgeosthat's why you guys use the tab to autocomplete15:13
karnihahah15:13
tsdgeosmake testO<TAB>15:13
dandradertsdgeos, everywhere else is testFooBar, not testFoobar15:13
tsdgeosdandrader: i know15:13
dandradertsdgeos, if there's a will, there's a way15:13
tsdgeosdandrader: there is a way, i don't feel like doing it, moreover when you already approved the current naming15:13
tsdgeosmoving the goal posts is not fun15:14
dandraderdednick, can you reproduce it with "make tryDragHandle"?15:14
dednickdandrader: ya15:14
dednickdandrader: i'll just create a bug and assign you.15:15
tsdgeosbah15:15
* tsdgeos shuts up and changes the capitalization15:15
dandraderdednick, yeah, just saw the bug you described15:15
dandraderwith make tryDragHandle15:16
dandraderthat's definitely a regression15:16
dandraderwasn't there before15:16
dednickdandrader: cool. i'll log it.15:16
Saviqmhall119, there's a package missing in the ppa, just setting up the recipe15:16
tsdgeosdandrader: pushed15:17
dandradertsdgeos, It slipped past me. If I had noticed it before I would not approve it :)15:17
karnitsdgeos: verticaljournal has no currentIndex, like GridView in grid layout would, to track hilight/currently selected item. Do we care? :)15:17
dandradertsdgeos, 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 it15:18
karnitsdgeos: Sorry for those questions, I'm just going through that code and saying out loud if I have any worries/doubts :)15:18
tsdgeoskarni: that's a good question, yes it has no index concept15:18
dandraderyou just have to known the name of the class and that's it15:18
karnitsdgeos: Should I file a bug, so we don't forget?15:19
karnitsdgeos: OR, it does not and will not have index concept?15:19
karniMaybe that's what you meant.15:19
tsdgeoskarni: don't know, i guess we will need one to do keyboard navigation15:19
tsdgeosother than keyboard navigation i don't see us needing one15:19
tsdgeosmaybe for previews too15:20
tsdgeossince you can switch to next/previous there15:20
karnitsdgeos: 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:20
Saviqtsdgeos, so... we don't need the workaround that you've merged yesterday any more...15:21
Saviqtsdgeos, I'm tempted to uncommit it...15:21
tsdgeosSaviq: the one for the python stuff?15:21
Saviqtsdgeos, yes15:21
Saviqtsdgeos, got fixed since yesterday...15:21
tsdgeoskarni: yeah file a bug, assign it to me, will think about it15:22
karniokay15:22
tsdgeosSaviq: is the change wrong?15:22
Saviqtsdgeos, it's not, but unnecessary15:22
Saviqtsdgeos, ok never mind, will put it in my "various fixes" branch15:22
tsdgeosSaviq: well, reverting it is another unnecessary change then :D15:22
tsdgeosbut yeah, don't have an opinion15:23
Saviqtsdgeos, that's why I'm saying uncommit ;)15:23
tsdgeosah15:23
Saviqnot revert15:23
tsdgeosyou mean uncommit and force push15:23
Saviqsince it's still the tip15:23
Saviqtsdgeos, yes15:23
tsdgeosbrrrrrr15:23
Saviqok then no15:23
Saviqtsdgeos, https://code.launchpad.net/~saviq/unity8/clean-deps/+merge/201608 though15:24
* tsdgeos clicks15:25
tsdgeosSaviq: don't need unity-mir at all?15:26
tsdgeosah15:26
tsdgeosthe libunity-mir-dev thing already wroks15:26
Saviqtsdgeos, yup15:27
tsdgeosSaviq: approved15:27
Saviqtsdgeos, thanks, let's see what CI says15:28
tsdgeosfail in otoo :D15:28
tsdgeosotto15:28
fgintherSaviq, the recent unity8 otto runs have unity8 crash files now. Unfortunately, I don't think they've been retraced so probably of no use15:29
mhall119Saviq: just let me know when it's ready and I'll update/upgrade/rebuild15:35
fgintherSaviq, 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:40
tsdgeosfginther: are they? i saw lots of red15:43
fginthertsdgeos, 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/console15:47
tsdgeosinteresting15:48
tsdgeoswell maybe as it came it went away?¿15:49
fginthertsdgeos, another recent run also had just one failure: http://s-jenkins.ubuntu-ci:8080/job/autopilot-testrunner-otto-trusty/2044/testReport/15:49
tsdgeoswhy was i away?¿ D:15:49
dandradertsdgeos, in make tryOrganicGrid, press "add" until you have 10 items (up to index 9)15:49
tsdgeosdo i make the window bigger?15:50
dandradertsdgeos, I wonder if the designer considered this situation15:50
dandradertsdgeos, looks bad if you ask me15:50
dandraderthis meaningless void on the right hand side15:50
tsdgeosdandrader: the same whole that if you only have 4 items, no?15:51
dandradertsdgeos, yes15:51
dandradertsdgeos,  but it just looks worse when you have that on the second component15:52
tsdgeosdandrader: the "real" sizing of the delegates will probably make it a bit less bad15:52
tsdgeosbut yeah it's not ultra cool15:52
tsdgeosi guess the designers idea is that it'll always there be "enough" stuff15:53
dandradertsdgeos, I don't know. it feels like its working against usability15:53
tsdgeosbut let me see what the gallery does15:53
tsdgeosif i can start the phone :D15:53
dandradertsdgeos, maybe the order of items in a "module" should be different if there are less than 6 items15:55
=== salem_ is now known as _salem
dandradertsdgeos, because the layout when there are only 4 items in the module just doens't make sense15:55
dandradertsdgeos, and I'm starting to think designers out to agree on that point15:56
dandraders/out/ought15:56
karnitsdgeos: thanks for looking into the currentIndex thing already.15:57
tsdgeosdandrader: if you have the phone working (i need to reflash it) take 4 pictures with the camera and see what the gallery shows15:57
dandradertsdgeos, hmm, I also think I have to reflash mine (galaxy nexus). didn' t touch it since I got the nexus 10 on last sprint15:59
tsdgeosdandrader: use the nexus10 then :D15:59
tsdgeosshould have the gallery app too15:59
dandradertsdgeos, I'm using it for the qml mir compositor work16:00
tsdgeosah16:00
tsdgeosok16:00
tsdgeoswell then let's wait 10 min until mine gets flashed16:00
dandraderI'm flashing mine as well16:00
=== _salem is now known as salem_
dandradertsdgeos, btw, the filling order works very well horizontally (if you make the tryOrganicGrid window very wide and start adding items to it)16:01
tsdgeosthe gallery-app is horizontal16:02
tsdgeosmaybe that's why it doesn't look bad there16:02
karnitsdgeos: 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:03
tsdgeoskarni: isn't it backwards? you know how much space you have and how much columns you need and then set the columnwidth?16:04
tsdgeoswell not really that16:04
tsdgeosas far as i understand16:04
tsdgeosyou know the width16:04
tsdgeosand 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
tsdgeosat least that's how i understand it16:05
karnibrb standup, sorry!16:05
tsdgeosfginther: yeah the organicgrid otto tests passed :o16:05
tsdgeosSaviq: do you have katie in there?16:07
dandradertsdgeos, I believe they've reached EOD there16:10
tsdgeosbut aren't they behind?16:10
karnitsdgeos: d'oh, sure. you're right :)16:12
tsdgeosdandrader: ok, i'll take a screenshot and send it via email to saviq, katie, you then, ok?16:12
dandradertsdgeos, ok, thanks!16:13
greybacktsdgeos: yep, we've hit eod16:14
karnitsdgeos: Can I ask one question about your code? http://paste.ubuntu.com/6751191/16:14
karnitsdgeos: 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
karnitsdgeos: 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
karniI just thought I'd bring up that small inconsistency I noticed.16:17
tsdgeoskarni: i think the difference is that responsivegridview takes into account the outer spacing while the journals do not16:18
tsdgeosi.e. journal will put stuff at 0,016:18
tsdgeosand responsivegridview probabbly not16:18
tsdgeosthe journal way is a bit more versatily since it allows you to have a different inner spacing and outer spacing16:19
tsdgeosin case you want that16:19
tsdgeosam i making sense?16:19
karnioh, that makes sense. I must have not realized journal could put stuff at 0,0 instead spacing/2, 016:20
karnitsdgeos: yes, makes sense. thanks16:20
karniWouldn't then16:21
karniconst int nColumns = qMax(1., floor((double)(width() + columnSpacing()) / (m_columnWidth + columnSpacing())))16:21
tsdgeosdandrader: mail sent, does it make sense to you?16:21
karnibe16:21
karniconst int nColumns = qMax(1., floor((double)(width()) / (m_columnWidth + columnSpacing())))16:21
tsdgeosnope16:22
karniIf you can put stuff at 0,0 (no left margin)16:22
tsdgeosthe last column doesn't get a columnspacing16:22
karnioh16:22
karni:D thanks tsdgeos16:22
tsdgeoswidth = ncolumns * columnWidth + ncolumns * (columnSpacing -  1)16:23
tsdgeosnow do the math16:23
* karni is really enjoying exploring this code base16:23
tsdgeosand you end up with the other formula16:23
karnicorrect16:23
=== dandrader is now known as dandrader|afk
tsdgeoskarni: cool, hope that it means the code is not so bad :D16:28
karnitsdgeos: hahahahah, it's gooooooooood /me makes Bruce Almighty impression16:29
karni:)16:29
* tsdgeos pretends to know who is that Bruce something16:32
karnitsdgeos: https://www.youtube.com/watch?v=nVCe0oWdnrE16:32
tsdgeosahh16:34
karni:)16:34
tsdgeostitle translations always are creative here16:34
tsdgeosthat movie was called "Like God"16:34
karnitsdgeos: I guess. That's why I always hunt for original titles :)16:34
karnitranslation to Polish was, in this case, very adequate to the original. but yeah, I get your point, some are really terrible :)16:35
=== dandrader|afk is now known as dandrader
tsdgeosdandrader: right the docu is missing16:59
tsdgeosand the ascii art too17:00
tsdgeoswill do tomorrow17:00
tsdgeosgetting eod'ed here17:00
dandraderok17:00
=== alan_g is now known as alan_g|EOD
=== salem_ is now known as _salem

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!