[00:00] yeah, too bad I get some other random issues with unity after restarting it... fun times [00:00] mhr3_, thanks you! Yes it does, but getting it to display the pretty pixels is step one :) [00:00] I guess unity is just telling me not to play games, and do work [00:01] (when I restart unity, from time to time, gnome-terminal gets into a weird state where I can't alt-tab to it - i can still click on it to get focus though) [00:02] kklimonda, :(, usually restarting the terminal fixes that [00:02] though i thought we fixed that issue... [00:02] has you fixed it in the last few days? [00:02] have* [00:03] kklimonda, nope that should have been like a year or so ago...so thats a different bug... [00:03] ah, I'll report it then when I get it to happen again [00:03] kklimonda, thank you! [00:06] kklimonda, also the fact that your settings don't change dynamically through ccsm is a bug (as it should be dynamic...) [00:06] bschaefer: I know, I'm really confused on why it's happening [00:07] yeah.. if you go to preferences and see what backend you're using? [00:07] as it should be set to gsettings [00:07] and the intergeneration box should be checked [00:07] (in ccsm) [00:08] where in ccsm? [00:08] kklimonda, bottom left [00:08] ah [00:08] above close, and advanced serach [00:08] search [00:08] yeah its in a strange spot [00:08] yeah, it's GSettings, and the integration box is checked [00:08] sadface [00:09] then im not sure what its not dynamic as i've never ran into that issue... [00:09] kklimonda, what version are you running of unity? [00:09] * bschaefer should have asked this first really... [00:10] bschaefer: the one that comes with 14.04 [00:10] o so you've got the most update to date version hmm [00:10] (thats around for download) [00:10] 7.1.2+14.04.20131106.1-0ubuntu2, I just restarted it [00:11] thats the version i have installed...let me try running that version as im on trunk untiy [00:12] kklimonda, did you upgrade to 14.04 or clean install? [00:12] it's a clean install [00:12] as it still seems to work here hmm [00:14] kklimonda, ccsm is not a nice application at all [00:14] Trevinho, if you're still around, have you ever seen the problem where CCSM doesn't actually save the changes? [00:15] it isn't, but then i have used it a few times in the past, and never had this particular issue :) [00:15] yeah, its having problems saving the changes it seems... [00:16] kklimonda, could you try install 'dconf-editor' and see if it'll change it this way [00:18] once you've dconf-editor, press Ctrl+f (for find), then search for 'show-hud' [00:18] from there you should see the '' key, if you just change it to '', it should stop the hud from opening right away [00:18] yeah, I just run apt-get dist-upgrade and unity exploded.. [00:18] :( [00:18] sigh, that will teach me ;) [00:19] can't load plugin 'ccp' [00:19] thats not a good error...umm, this is after a dist-upgrade and reboot? [00:19] compiz is having a problem starting (not unity this time!) [00:19] no, during dist-upgrade unity just segfaultet [00:20] segfaulted* [00:20] well, compiz [00:20] and then when I try to run in again, it hangs at this error [00:20] dconf-editor shows that the show-hud command is changed, and it works after I restart unity [00:20] but not right away? [00:20] ie. not dynamic. [00:21] yeah [00:21] that is very very strange... [00:21] I can make a clean user and see if it still happens [00:21] that isn't a bad idea [00:21] once I get my system into a usable state again :) [00:21] yeah :), umm getting back from that depends on the actual seg fault reason [00:22] but hopefully continuing with the upgrade [00:22] then a reboot...should fix it [00:30] ccp is not loading because the new libprotobuf changed ABI.. [00:30] (I had to downgrade some stuff) [00:34] it seems to be working under a new account, so I'll just assume something weird happened on my main [00:34] which is weird, this installation is like a week old.. [00:37] strange... [00:38] sounds like its looking in the wrong place... [00:38] well at lease now its working (though thats never fun to have to make a new account just because of that...) [00:38] kklimonda, if you get some time to file a bug against compiz? [00:40] yeah, I wonder if it's related to that other bug I reported, where ccsm is not saving changes I make (I tried to disable "Initiate Window Move" but it's always being reverted to the default value if I disable it, and when I set a different combination it's just ignored) [00:40] bug 1273690 [00:40] bug 1273690 in unity (Ubuntu) "can't disable Alt+Button1 shortcut for moving windows around" [Undecided,New] https://launchpad.net/bugs/1273690 [00:41] kklimonda, sounds like it, it seems to be running of different files. ie, its reading one file and saving to a different file [00:42] then when you relog it looks at the saved one [00:43] mhm, sounds plausible [00:45] as all the settings are stored in a single compiled file hmm if you would be able to umm [00:45] kklimonda, get a strace off of the ccsm that fails to load dynamically [00:46] that way I can see if its reading/writing to the wrong files [00:46] kklimonda, or just upload that log to the bug [00:46] ok, I'll do it tomorrow when I get up :) [00:46] (it'll be a large file) [00:46] thanks for the help [00:46] yeah [00:46] kklimonda, thanks a lot! [00:46] and np! [08:23] tsdgeos, 'fraid I overridden your review a bit on TextSummary [08:28] Saviq: seen it, no worries, good stuff you found :) [08:30] tsdgeos, did you ever notice weirdness with childrenRect when it's supposed to "shrink"? I had to do lines 11-12 of https://code.launchpad.net/~unity-team/unity8/unity8-card-overlay/+merge/204790 in Card.qml for it to behave properly [08:30] tsdgeos, otherwise it'd grow and stay grown, even if the contents shrunk [08:30] Saviq: i'm going to approve ted's the table is anywhere MR https://code.launchpad.net/~ted/unity8/on-the-screen/+merge/203851 [08:31] tsdgeos, ok ;) [08:31] actually no, needs a pot rebuild [08:32] tsdgeos, the other possibility is there's a binding loop and it just doesn't propagate up (which, btw, Qt complains about), but I found no way to break that loop [08:33] no, never noticed it [08:33] if we have a binding loop it may be that [08:33] and maybe the way to break it is how you did [08:39] tsdgeos, only I hate the solution ;) [08:40] but well :| [08:40] /biab [08:51] Saviq: ok, pushed a branch of the image_preview thing that now is able to merge corrrectly [09:05] mhr3_: Saviq: the actions definition is different in "preview definition example" and in "Preview JSONs" on the document [09:05] which is right? the second one' [09:05] ? [09:16] tsdgeos, yea, second one === mzanetti- is now known as mzanetti_ [09:17] tsdgeos, i mean... both are correct :) [09:20] mhr3_: ? [09:21] ah you changed it [09:30] come on! [09:30] now other tests fail! [09:30] * tsdgeos jumps out of the window === lool- is now known as lool [09:38] uh oh, developer down! [09:44] Saviq, hey man [09:44] Saviq, you asked me to change implicitHeight with height [09:44] but it's implicitHeight also the minimum allowed height? [09:46] sil2100, hey, can we land libunity without the train? [09:46] sil2100, as in the old lib [09:54] mhr3_: Saviq: where do i get the colour of the button for the action? [09:54] i've seen some stuff as green, other as orange [09:54] tsdgeos, design? [09:54] wyluzuj poslady Saviq! [09:55] mhr3_: design wants some green, some orange, so where does that come in the json? [09:56] Saviq, (I don't know what it means, just told my gf you're picky with my reviews this morning :D) [09:56] tsdgeos, i'm not convinced that should be configurable... way to make the previews eeek [09:56] and inconsistent [09:56] mhr3_: talk to design :D [09:59] tsdgeos, pinged rosie [10:42] bregma: heya. I started looking at unity stack a bit yesterday. compiz needed some changelog syncs. it's working nicely (I updated nux + compiz + unity) on my machine, but I filed a bug #1276516 about the autopilot situation on cu2d. [10:42] bug 1276516 in Unity "unity trunk failing autopilot tests (02/2014)" [High,New] https://launchpad.net/bugs/1276516 [10:43] bregma: to approach releasibility, we'd need some solution to those or how we'd validate the release via cu2d results. [10:44] tsdgeos, the green part will be defined by the scope author [10:44] tsdgeos, but don't worry about that yet [10:44] Saviq: ? [10:44] you mean a property [10:44] ? [10:45] Cimi, lol [10:45] tsdgeos, yeah, but later [10:45] ok [10:45] tsdgeos, for now we're only going orange [10:46] Saviq: there's also the width thing [10:46] buttons are "smart" enough to give themselves a proper width [10:46] but the OptionSelector doesn't seem to be able to do that [10:46] so do we force say 30gu of width? [10:46] or we have a property for that? [10:46] Cimi, re: implicitHeight vs. height, it's not "minimal", it's only "default", but in this case the widget won't adapt if you change its height, so it's not really default, it's the height it's meant to be [10:47] or try to fix optionselector to be smart? [10:47] tsdgeos, optionselector? where do we use optionselector? you mean combo button? [10:47] is there a combo button? [10:47] tsdgeos, there isn't indeed [10:47] well the docu says there is [10:47] tsdgeos, there isn't in the SDK [10:47] tsdgeos, hence we just need to go for multiple buttons now [10:47] so i thought it meant option selector [10:48] tsdgeos, nah, it's a different component that's "meant to be" in the SDK [10:48] which confusingly we then have something called valueselector also [10:48] Saviq: so just a row of buttons? or a column? [10:48] tsdgeos, column, yeah [10:49] tsdgeos, actually [10:49] tsdgeos, take what's in Notification.qml [10:49] tsdgeos, it's a poor man's combo button [10:50] hmmm [10:51] it's a column [10:51] isn't it? [10:51] tsdgeos, there's two buttons, and the left one opens the column [10:53] * tsdgeos reads the code more carefully [10:53] Saviq, length can be null? [10:54] just wondering about the function [10:54] also [10:54] in case it's null [10:54] Cimi, no, but foo["foo"] can be null [10:54] Saviq: besides that, you saw that tests failed in mako now?¿ [10:54] Saviq, modelData["length"] [10:54] Saviq, I'm passing this to lengthToString(s) [10:54] Cimi, let me see what I commented about [10:55] Cimi, I commented about modelData["subtitle"] that it could be null [10:56] Saviq, yes [10:56] Saviq, but what happens if length is too? [10:56] Saviq, it goes to that JS [10:57] Cimi, ah well, yeah, you should return "" if it's not a number [10:57] Saviq, I just have !== undefined [10:57] Cimi, if (typeof s !== "number") [10:57] cool [10:57] thx [10:57] I can drop undefined then [10:58] tsdgeos, grr [10:58] Saviq: i wonder how that happened, is because the tests were stopping before because the network ones failed? or? [10:59] tsdgeos, no, a rewrite of the datetime indicator was released, probably changed the name or something, checking [10:59] tsdgeos, or it's a valid failure and the indicator was broken [11:04] Cimi, and well, me being picky, is that a new thing? [11:05] Cimi, wanna know what "wyluzuj poślady" means? [11:05] Saviq, that could be good :) [11:05] Cimi, "loosen your cheeks", and it's not about your face ;) [11:05] Saviq, she said at first that all polish people are picky :) [11:06] Saviq, so what does it means? :) [11:07] Cimi, look up ↑↑ [11:07] Saviq, thought it was something related to smiling [11:08] Cimi, in a sense... [11:09] Saviq, she said it's something I can tell you in case we are friends [11:09] Saviq, so I wrote you :) [11:10] Cimi, yup [11:10] Cimi, not offensive at all (or well, as usual - depends on how you say it) [11:12] Saviq, I fixed your things and added test for NAN [11:12] Saviq: can't copy directly Notifications since it expands with left button instead of right, so you meant adapt right? I was thinking to do a base class that both Notifications.qml and this would use, but it'd be a work since behaviour is not exactly the same, so you think it's worth or just copy/adapt the code and wait for the combobox to land in SDK? [11:13] guys, I'll be away for few mins, need to reconnect router with new cat 5e cables [11:17] tsdgeos, we need to expand left button I believe [11:17] tsdgeos, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit [11:17] why? [11:18] Saviq: that is not "the left button" [11:18] Saviq: each button is it's own widget [11:18] if the expectation is that in that screenshot [11:18] is that left button will do something like notification does [11:18] we can't do it [11:18] they don't know eachother [11:19] or we need to rethink the actions thing [11:19] tsdgeos, well, that's the question - don't they... [11:19] Saviq: i don't see why, it'd just see left one expanding down [11:19] and that's it [11:19] tsdgeos, I think we can make a "smart combo button" i.e. one button - on the right, two buttons - next to each other [11:19] more buttons [11:19] expanding from the elft [11:19] left [11:20] Saviq: i didn't understand at all what you meant [11:21] tsdgeos, think of a "row of buttons" as one widget of type: "actions" [11:21] which is not what how we were thinking of it, right? [11:21] tsdgeos, first one of those actions would be the button on the right, second - the button on the left, any more - expanded from the left one [11:21] tsdgeos, not before it was redesigned, no ;) [11:22] tsdgeos, but I think it might've been bad thinking [11:22] ETOOMUCHTHINKING [11:22] but you don't want the left one to do expansion [11:22] you want it to be a button and also expand [11:23] otherwise how do you actually click "Add to queue" [11:23] ? [11:23] tsdgeos, oh yeah, well, that's the missing combo button [11:23] tsdgeos, but we should be able to do something of the sort the Notification has [11:24] well that is what i was saying, put two buttons together and let it be similar to a combobox [11:24] i'll try to code it and we can talk over it later [11:24] Saviq: what is actually interesting if you see https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit [11:25] is why are the buttons one next to another [11:25] and not on top of the other [11:25] like all the other preview components [11:25] who decides that? [11:25] tsdgeos, that's the thing - I think that's a single widget [11:25] +meant to be [11:25] /should be [11:26] agreed [11:27] tsdgeos, so yeah, let's implement it with that in mind [11:27] need a new json for it [11:27] tsdgeos, no we don't [11:27] ? [11:27] tsdgeos, type: "actions" takes one or more actions [11:27] how come [11:27] yes [11:27] and more actions [11:27] is a combobox [11:28] tsdgeos, well, that's the thing [11:28] tsdgeos, not any moer [11:28] moar [11:28] but how do i put a button and a combo box in one actions [11:28] with that json? [11:28] tsdgeos, put more than 2 actions [11:28] and i have 3 buttons [11:28] tsdgeos, no [11:28] tsdgeos, 2 buttons, left a combo [11:28] doesn't make much sense [11:29] what if i am in a tablet [11:29] tsdgeos, you get two columnt [11:29] columns [11:29] why shouldn't i get to use all my width [11:29] tsdgeos, the widget shouldn't change whether on tablet or phone at all [11:29] tsdgeos, because that's wasting the width [11:29] instead of a button and a combo? [11:29] tsdgeos, that's output from Cape Town [11:29] ok [11:29] tsdgeos, no button to span the whole width of a phone [11:29] tsdgeos, as that wastes space [11:30] sure, i'm not saying that [11:30] i'm saying 2 buttons (one a combo) when i could have 3 is a bit weird [11:30] but ok [11:30] if you want that [11:30] i'll code that [11:31] tsdgeos, if you want three, that's when you put two widgets - one with two actions, one with one action [11:32] ok [11:32] man... I got a headache today [11:33] tsdgeos, I'll add a comment to the dash spec, 'cause it doesn't deal with that, but I think it's the best route [11:35] tsdgeos, found the issue with the tests - datetime takes longer to load now it seems, and the test didn't wait for it [11:35] insteresting [11:36] Saviq: so just to confirm, with this new definition on how the json works, you can never have just a combobox, if you want a combobox, you'll always hace a button on it's right too, correct? [11:36] tsdgeos, yes [11:36] ok [11:37] tsdgeos, you get one button, two buttons, or a button and a combobox [11:41] tsdgeos, if you disagree with that, comment please on https://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1&disco=AAAAAH2p9p4 [11:41] Saviq: i think it limits what people can do [11:41] but i'm not the designer so i'm ok [11:41] tsdgeos, we're doing that all over the place don't we :) [11:41] tsdgeos, what you disagree? [11:42] tsdgeos, "I'm not a designer so I'm ok" is not the answer I want to hear from anyone working on my team :) [11:42] tsdgeos, in some areas you have more experience than designers, so share your issue with me :) [11:43] cimi: i'm not disagreeing with designers, i'm disagreeing with Saviq :D [11:43] tsdgeos, amazing! [11:43] lol [11:43] i do think that in some case it may make sense to just have a combobox [11:44] but since i am not designing the interface i don't know if that makes sense in the previews or not [11:44] and if it does, we can add it later, is not that as if the code i'm writing now won't even change [11:44] tsdgeos, we could add a prop to actions [11:44] tsdgeos, that would say "I want all to go into combo" [11:45] tsdgeos, but yeah, we can add that later if needed [11:47] tsdgeos, https://code.launchpad.net/~saviq/unity8/fix-indicator-tests/+merge/204889 [11:48] * tsdgeos runs the autopilot tests [11:48] * tsdgeos dies of old age [11:49] tsdgeos, try to run them on emulator ;D [11:49] he'll be dead before the emulator has booted :P [11:50] Saviq: you miss the guidelines thing [11:50] tsdgeos, refresh [11:50] ah, not anymore [11:54] cimi, 56 if (s <= 0 || typeof(s) !== "number") return ""; [11:54] cimi, I think those should be reversed [11:54] cimi, we don't want to even look at whether s <= if it's NaN [11:55] cimi, text: (modelData["subtitle"] && modelData["subtitle"] !== undefined) ? modelData["subtitle"] : "" [11:55] cimi, text: modelData["subtitle"] || "" [12:01] tsdgeos, just a small tweak to the docs: https://code.launchpad.net/~aacid/unity8/image_gallery_preview_widget/+merge/204712/comments/478927 === MacSlow is now known as MacSlow|lunch [12:08] Saviq: do you like the new styling i just pushed more? [12:09] tsdgeos, +1 [12:09] tsdgeos, I'll put * everywhere after the next release [12:30] Saviq: so on the "combo - button" scenario, i'm still unsure of how we want the thing to work [12:30] obviously pressing the right button is pressing it, that's easy [12:30] but do we want the combo to have two sensitive areas, one for activate and one for "open"? [12:30] and when the combo opens, do we want the first button to hide like in notificationss? [12:31] and when the combo is opened, does clicking on one of the opened options activate it? or just select it and close the combo and then you have to click on the combo? [12:31] in the non "open" area to activate it [12:33] tsdgeos, *we* do the simplest thing that makes sense [12:33] tsdgeos, I think the combo button is desgined... somewhere... [12:37] tsdgeos, Jouni should have them, he should be online in an hour or two (as should SDK folks) [12:37] ok [12:46] Saviq: no worky https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5078/#showFailuresLink [12:47] or maybe it did fix 1 [12:56] tsdgeos, that's another issue... [12:56] tsdgeos, they broke app launching in the latest image... [12:56] tsdgeos, we'll have to wait for the next one === alan_g is now known as alan_g|lunch [13:13] is launchpad slow today or is it my connection? [13:14] hmm, getting several launchpad "Oops!" pages now... [13:22] o/ === MacSlow|lunch is now known as MacSlow [13:33] dandrader, oops from launchpad suggest LP issues, your connection shouldn't have any bearing, no? [13:33] karni, hey [13:33] Saviq, yes, I got those Oops after I notices the overall launchpad slowness === mardy_ is now known as mardy [13:33] noticed [13:33] hi Saviq :) [13:34] dandrader, so yeah, it seems pretty slow here [13:40] mhr3_, so, scope::preview() only gives out a single PreviewModel now, is there a multi-column version anywhere yet? === alan_g|lunch is now known as alan_g [13:47] saviq, not finished yet [13:51] aaagh stupid QtCreator... every single time I create a folder with the name of the file I want to create... [13:56] Saviq: so arrpove yours that indeed seems to decrease the number of failing tests? [13:56] tsdgeos, well, it fixes that one failure [13:56] tsdgeos, so yeah [13:57] tsdgeos, and we wait for improvement on the other side [14:01] Saviq: duude [14:01] mzanetti_, what up? [14:01] I constantly get: mzanetti Nickname already in use, try a different one. [14:01] Saviq: do a whois mzanetti [14:02] mzanetti_, I am! [14:02] * mzanetti_ wonders why you can hold that one even though I am identified @ NickServ [14:02] * tedg wonders if IRC explodes if you try to whois yourself [14:03] whois mzanetti === dandrader is now known as dandrader|afk [14:03] well, with the / [14:03] :P [14:03] yeah yeah :P [14:03] need to log in to disconnect... [14:04] mzanetti_, here === mzanetti_ is now known as mzanetti [14:05] Saviq: cheers [14:05] still wondering how this is possible [14:05] but I never really got irc authentication anyways [14:09] mzanetti, you have to explicitly ask NickServ to get your nick back [14:09] ghost him! [14:10] tedg: seen the comment i made in your anywhere MR? [14:10] tsdgeos, Yup, just did. [14:10] ahhh [14:10] tedg: that's going to conflict [14:11] K, I'll deal. [14:11] with https://code.launchpad.net/~unity-team/unity8/unity8.previews_textSummary/+merge/204177 [14:11] but that's not approved for the moment [14:11] so do yours and see if we can sneak it in [14:12] this delayed landing is a pain for merges [14:12] because you have to think about billions of branches that may or may not conflict with yours [14:12] Hah, you're telling me! :-) [14:14] I'm starting to loose track of which branches I've told other people to run so they get the functionality they need. === sdeziel is now known as sdeziel_ [14:17] tsdgeos, http://pastebin.ubuntu.com/6879267/ ? [14:17] that is what i have yes [14:18] kind of [14:18] your docu is better === dandrader|afk is now known as dandrader [14:19] tsdgeos, So I ran it, then I deleted the changes that weren't related to my change. [14:19] tsdgeos, That should reduce the conflict [14:19] maybe [14:19] or not [14:19] who knows :D [14:20] Well, yes, it should reduce the likelihood of conflict ;-) [14:24] Saviq: re: your comment, I did that for title only because that was critical fix for us. I'd be happy to continue work in anothe branch, if that's okay with you. https://code.launchpad.net/~unity-team/unity8/new-scopes-title-alignment/+merge/204677 [14:24] karni, ok [14:27] tsdgeos, PreviewActions.qml? [14:27] ignore that [14:27] Saviq: you're complaining on the "s" ? [14:27] tsdgeos, lol [14:27] tsdgeos, no, asking whether that's the name of the widget ;D [14:28] * Saviq should be less picky in reviews [14:28] i had PreviewActions [14:28] moved to PreviewAction [14:28] but i guess i have to move it to PreviewActions again [14:28]