| kklimonda | yeah, too bad I get some other random issues with unity after restarting it... fun times | 00:00 |
|---|---|---|
| bschaefer | mhr3_, thanks you! Yes it does, but getting it to display the pretty pixels is step one :) | 00:00 |
| kklimonda | I guess unity is just telling me not to play games, and do work | 00:00 |
| kklimonda | (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:01 |
| bschaefer | kklimonda, :(, usually restarting the terminal fixes that | 00:02 |
| bschaefer | though i thought we fixed that issue... | 00:02 |
| kklimonda | has you fixed it in the last few days? | 00:02 |
| kklimonda | have* | 00:02 |
| bschaefer | kklimonda, nope that should have been like a year or so ago...so thats a different bug... | 00:03 |
| kklimonda | ah, I'll report it then when I get it to happen again | 00:03 |
| bschaefer | kklimonda, thank you! | 00:03 |
| bschaefer | kklimonda, also the fact that your settings don't change dynamically through ccsm is a bug (as it should be dynamic...) | 00:06 |
| kklimonda | bschaefer: I know, I'm really confused on why it's happening | 00:06 |
| bschaefer | yeah.. if you go to preferences and see what backend you're using? | 00:07 |
| bschaefer | as it should be set to gsettings | 00:07 |
| bschaefer | and the intergeneration box should be checked | 00:07 |
| bschaefer | (in ccsm) | 00:07 |
| kklimonda | where in ccsm? | 00:08 |
| bschaefer | kklimonda, bottom left | 00:08 |
| kklimonda | ah | 00:08 |
| bschaefer | above close, and advanced serach | 00:08 |
| bschaefer | search | 00:08 |
| bschaefer | yeah its in a strange spot | 00:08 |
| kklimonda | yeah, it's GSettings, and the integration box is checked | 00:08 |
| bschaefer | sadface | 00:08 |
| bschaefer | then im not sure what its not dynamic as i've never ran into that issue... | 00:09 |
| bschaefer | kklimonda, what version are you running of unity? | 00:09 |
| * bschaefer should have asked this first really... | 00:09 | |
| kklimonda | bschaefer: the one that comes with 14.04 | 00:10 |
| bschaefer | o so you've got the most update to date version hmm | 00:10 |
| bschaefer | (thats around for download) | 00:10 |
| kklimonda | 7.1.2+14.04.20131106.1-0ubuntu2, I just restarted it | 00:10 |
| bschaefer | thats the version i have installed...let me try running that version as im on trunk untiy | 00:11 |
| bschaefer | kklimonda, did you upgrade to 14.04 or clean install? | 00:12 |
| kklimonda | it's a clean install | 00:12 |
| bschaefer | as it still seems to work here hmm | 00:12 |
| bschaefer | kklimonda, ccsm is not a nice application at all | 00:14 |
| bschaefer | Trevinho, if you're still around, have you ever seen the problem where CCSM doesn't actually save the changes? | 00:14 |
| kklimonda | it isn't, but then i have used it a few times in the past, and never had this particular issue :) | 00:15 |
| bschaefer | yeah, its having problems saving the changes it seems... | 00:15 |
| bschaefer | kklimonda, could you try install 'dconf-editor' and see if it'll change it this way | 00:16 |
| bschaefer | once you've dconf-editor, press Ctrl+f (for find), then search for 'show-hud' | 00:18 |
| bschaefer | from there you should see the '<Alt>' key, if you just change it to '', it should stop the hud from opening right away | 00:18 |
| kklimonda | yeah, I just run apt-get dist-upgrade and unity exploded.. | 00:18 |
| bschaefer | :( | 00:18 |
| kklimonda | sigh, that will teach me ;) | 00:18 |
| kklimonda | can't load plugin 'ccp' | 00:19 |
| bschaefer | thats not a good error...umm, this is after a dist-upgrade and reboot? | 00:19 |
| bschaefer | compiz is having a problem starting (not unity this time!) | 00:19 |
| kklimonda | no, during dist-upgrade unity just segfaultet | 00:19 |
| kklimonda | segfaulted* | 00:20 |
| kklimonda | well, compiz | 00:20 |
| kklimonda | and then when I try to run in again, it hangs at this error | 00:20 |
| kklimonda | dconf-editor shows that the show-hud command is changed, and it works after I restart unity | 00:20 |
| bschaefer | but not right away? | 00:20 |
| bschaefer | ie. not dynamic. | 00:20 |
| kklimonda | yeah | 00:21 |
| bschaefer | that is very very strange... | 00:21 |
| kklimonda | I can make a clean user and see if it still happens | 00:21 |
| bschaefer | that isn't a bad idea | 00:21 |
| kklimonda | once I get my system into a usable state again :) | 00:21 |
| bschaefer | yeah :), umm getting back from that depends on the actual seg fault reason | 00:21 |
| bschaefer | but hopefully continuing with the upgrade | 00:22 |
| bschaefer | then a reboot...should fix it | 00:22 |
| kklimonda | ccp is not loading because the new libprotobuf changed ABI.. | 00:30 |
| kklimonda | (I had to downgrade some stuff) | 00:30 |
| kklimonda | it seems to be working under a new account, so I'll just assume something weird happened on my main | 00:34 |
| kklimonda | which is weird, this installation is like a week old.. | 00:34 |
| bschaefer | strange... | 00:37 |
| bschaefer | sounds like its looking in the wrong place... | 00:38 |
| bschaefer | well at lease now its working (though thats never fun to have to make a new account just because of that...) | 00:38 |
| bschaefer | kklimonda, if you get some time to file a bug against compiz? | 00:38 |
| kklimonda | 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 |
| kklimonda | bug 1273690 | 00:40 |
| ubot5 | bug 1273690 in unity (Ubuntu) "can't disable Alt+Button1 shortcut for moving windows around" [Undecided,New] https://launchpad.net/bugs/1273690 | 00:40 |
| bschaefer | kklimonda, sounds like it, it seems to be running of different files. ie, its reading one file and saving to a different file | 00:41 |
| bschaefer | then when you relog it looks at the saved one | 00:42 |
| kklimonda | mhm, sounds plausible | 00:43 |
| bschaefer | as all the settings are stored in a single compiled file hmm if you would be able to umm | 00:45 |
| bschaefer | kklimonda, get a strace off of the ccsm that fails to load dynamically | 00:45 |
| bschaefer | that way I can see if its reading/writing to the wrong files | 00:46 |
| bschaefer | kklimonda, or just upload that log to the bug | 00:46 |
| kklimonda | ok, I'll do it tomorrow when I get up :) | 00:46 |
| bschaefer | (it'll be a large file) | 00:46 |
| kklimonda | thanks for the help | 00:46 |
| kklimonda | yeah | 00:46 |
| bschaefer | kklimonda, thanks a lot! | 00:46 |
| bschaefer | and np! | 00:46 |
| Saviq | tsdgeos, 'fraid I overridden your review a bit on TextSummary | 08:23 |
| tsdgeos | Saviq: seen it, no worries, good stuff you found :) | 08:28 |
| Saviq | 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 |
| Saviq | tsdgeos, otherwise it'd grow and stay grown, even if the contents shrunk | 08:30 |
| tsdgeos | 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:30 |
| Saviq | tsdgeos, ok ;) | 08:31 |
| tsdgeos | actually no, needs a pot rebuild | 08:31 |
| Saviq | 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:32 |
| tsdgeos | no, never noticed it | 08:33 |
| tsdgeos | if we have a binding loop it may be that | 08:33 |
| tsdgeos | and maybe the way to break it is how you did | 08:33 |
| Saviq | tsdgeos, only I hate the solution ;) | 08:39 |
| Saviq | but well :| | 08:40 |
| Saviq | /biab | 08:40 |
| tsdgeos | Saviq: ok, pushed a branch of the image_preview thing that now is able to merge corrrectly | 08:51 |
| tsdgeos | mhr3_: Saviq: the actions definition is different in "preview definition example" and in "Preview JSONs" on the document | 09:05 |
| tsdgeos | which is right? the second one' | 09:05 |
| tsdgeos | ? | 09:05 |
| mhr3_ | tsdgeos, yea, second one | 09:16 |
| === mzanetti- is now known as mzanetti_ | ||
| mhr3_ | tsdgeos, i mean... both are correct :) | 09:17 |
| tsdgeos | mhr3_: ? | 09:20 |
| tsdgeos | ah you changed it | 09:21 |
| tsdgeos | come on! | 09:30 |
| tsdgeos | now other tests fail! | 09:30 |
| * tsdgeos jumps out of the window | 09:30 | |
| === lool- is now known as lool | ||
| mhr3_ | uh oh, developer down! | 09:38 |
| Cimi | Saviq, hey man | 09:44 |
| Cimi | Saviq, you asked me to change implicitHeight with height | 09:44 |
| Cimi | but it's implicitHeight also the minimum allowed height? | 09:44 |
| mhr3_ | sil2100, hey, can we land libunity without the train? | 09:46 |
| mhr3_ | sil2100, as in the old lib | 09:46 |
| tsdgeos | mhr3_: Saviq: where do i get the colour of the button for the action? | 09:54 |
| tsdgeos | i've seen some stuff as green, other as orange | 09:54 |
| mhr3_ | tsdgeos, design? | 09:54 |
| Cimi | wyluzuj poslady Saviq! | 09:54 |
| tsdgeos | mhr3_: design wants some green, some orange, so where does that come in the json? | 09:55 |
| Cimi | Saviq, (I don't know what it means, just told my gf you're picky with my reviews this morning :D) | 09:56 |
| mhr3_ | tsdgeos, i'm not convinced that should be configurable... way to make the previews eeek | 09:56 |
| mhr3_ | and inconsistent | 09:56 |
| tsdgeos | mhr3_: talk to design :D | 09:56 |
| mhr3_ | tsdgeos, pinged rosie | 09:59 |
| Mirv | 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 |
| ubot5 | bug 1276516 in Unity "unity trunk failing autopilot tests (02/2014)" [High,New] https://launchpad.net/bugs/1276516 | 10:42 |
| Mirv | bregma: to approach releasibility, we'd need some solution to those or how we'd validate the release via cu2d results. | 10:43 |
| Saviq | tsdgeos, the green part will be defined by the scope author | 10:44 |
| Saviq | tsdgeos, but don't worry about that yet | 10:44 |
| tsdgeos | Saviq: ? | 10:44 |
| tsdgeos | you mean a property | 10:44 |
| tsdgeos | ? | 10:44 |
| Saviq | Cimi, lol | 10:45 |
| Saviq | tsdgeos, yeah, but later | 10:45 |
| tsdgeos | ok | 10:45 |
| Saviq | tsdgeos, for now we're only going orange | 10:45 |
| tsdgeos | Saviq: there's also the width thing | 10:46 |
| tsdgeos | buttons are "smart" enough to give themselves a proper width | 10:46 |
| tsdgeos | but the OptionSelector doesn't seem to be able to do that | 10:46 |
| tsdgeos | so do we force say 30gu of width? | 10:46 |
| tsdgeos | or we have a property for that? | 10:46 |
| Saviq | 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:46 |
| tsdgeos | or try to fix optionselector to be smart? | 10:47 |
| Saviq | tsdgeos, optionselector? where do we use optionselector? you mean combo button? | 10:47 |
| tsdgeos | is there a combo button? | 10:47 |
| Saviq | tsdgeos, there isn't indeed | 10:47 |
| tsdgeos | well the docu says there is | 10:47 |
| Saviq | tsdgeos, there isn't in the SDK | 10:47 |
| Saviq | tsdgeos, hence we just need to go for multiple buttons now | 10:47 |
| tsdgeos | so i thought it meant option selector | 10:47 |
| Saviq | tsdgeos, nah, it's a different component that's "meant to be" in the SDK | 10:48 |
| tsdgeos | which confusingly we then have something called valueselector also | 10:48 |
| tsdgeos | Saviq: so just a row of buttons? or a column? | 10:48 |
| Saviq | tsdgeos, column, yeah | 10:48 |
| Saviq | tsdgeos, actually | 10:49 |
| Saviq | tsdgeos, take what's in Notification.qml | 10:49 |
| Saviq | tsdgeos, it's a poor man's combo button | 10:49 |
| tsdgeos | hmmm | 10:50 |
| tsdgeos | it's a column | 10:51 |
| tsdgeos | isn't it? | 10:51 |
| Saviq | tsdgeos, there's two buttons, and the left one opens the column | 10:51 |
| * tsdgeos reads the code more carefully | 10:53 | |
| Cimi | Saviq, length can be null? | 10:53 |
| Cimi | just wondering about the function | 10:54 |
| Cimi | also | 10:54 |
| Cimi | in case it's null | 10:54 |
| Saviq | Cimi, no, but foo["foo"] can be null | 10:54 |
| tsdgeos | Saviq: besides that, you saw that tests failed in mako now?¿ | 10:54 |
| Cimi | Saviq, modelData["length"] | 10:54 |
| Cimi | Saviq, I'm passing this to lengthToString(s) | 10:54 |
| Saviq | Cimi, let me see what I commented about | 10:54 |
| Saviq | Cimi, I commented about modelData["subtitle"] that it could be null | 10:55 |
| Cimi | Saviq, yes | 10:56 |
| Cimi | Saviq, but what happens if length is too? | 10:56 |
| Cimi | Saviq, it goes to that JS | 10:56 |
| Saviq | Cimi, ah well, yeah, you should return "" if it's not a number | 10:57 |
| Cimi | Saviq, I just have !== undefined | 10:57 |
| Saviq | Cimi, if (typeof s !== "number") | 10:57 |
| Cimi | cool | 10:57 |
| Cimi | thx | 10:57 |
| Cimi | I can drop undefined then | 10:57 |
| Saviq | tsdgeos, grr | 10:58 |
| tsdgeos | Saviq: i wonder how that happened, is because the tests were stopping before because the network ones failed? or? | 10:58 |
| Saviq | tsdgeos, no, a rewrite of the datetime indicator was released, probably changed the name or something, checking | 10:59 |
| Saviq | tsdgeos, or it's a valid failure and the indicator was broken | 10:59 |
| Saviq | Cimi, and well, me being picky, is that a new thing? | 11:04 |
| Saviq | Cimi, wanna know what "wyluzuj poślady" means? | 11:05 |
| Cimi | Saviq, that could be good :) | 11:05 |
| Saviq | Cimi, "loosen your cheeks", and it's not about your face ;) | 11:05 |
| Cimi | Saviq, she said at first that all polish people are picky :) | 11:05 |
| Cimi | Saviq, so what does it means? :) | 11:06 |
| Saviq | Cimi, look up ↑↑ | 11:07 |
| Cimi | Saviq, thought it was something related to smiling | 11:07 |
| Saviq | Cimi, in a sense... | 11:08 |
| Cimi | Saviq, she said it's something I can tell you in case we are friends | 11:09 |
| Cimi | Saviq, so I wrote you :) | 11:09 |
| Saviq | Cimi, yup | 11:10 |
| Saviq | Cimi, not offensive at all (or well, as usual - depends on how you say it) | 11:10 |
| Cimi | Saviq, I fixed your things and added test for NAN | 11:12 |
| tsdgeos | 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:12 |
| Cimi | guys, I'll be away for few mins, need to reconnect router with new cat 5e cables | 11:13 |
| Saviq | tsdgeos, we need to expand left button I believe | 11:17 |
| Saviq | tsdgeos, https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit | 11:17 |
| tsdgeos | why? | 11:17 |
| tsdgeos | Saviq: that is not "the left button" | 11:18 |
| tsdgeos | Saviq: each button is it's own widget | 11:18 |
| tsdgeos | if the expectation is that in that screenshot | 11:18 |
| tsdgeos | is that left button will do something like notification does | 11:18 |
| tsdgeos | we can't do it | 11:18 |
| tsdgeos | they don't know eachother | 11:18 |
| tsdgeos | or we need to rethink the actions thing | 11:19 |
| Saviq | tsdgeos, well, that's the question - don't they... | 11:19 |
| tsdgeos | Saviq: i don't see why, it'd just see left one expanding down | 11:19 |
| tsdgeos | and that's it | 11:19 |
| Saviq | 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 |
| Saviq | more buttons | 11:19 |
| Saviq | expanding from the elft | 11:19 |
| Saviq | left | 11:19 |
| tsdgeos | Saviq: i didn't understand at all what you meant | 11:20 |
| Saviq | tsdgeos, think of a "row of buttons" as one widget of type: "actions" | 11:21 |
| tsdgeos | which is not what how we were thinking of it, right? | 11:21 |
| Saviq | 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 |
| Saviq | tsdgeos, not before it was redesigned, no ;) | 11:21 |
| Saviq | tsdgeos, but I think it might've been bad thinking | 11:22 |
| Saviq | ETOOMUCHTHINKING | 11:22 |
| tsdgeos | but you don't want the left one to do expansion | 11:22 |
| tsdgeos | you want it to be a button and also expand | 11:22 |
| tsdgeos | otherwise how do you actually click "Add to queue" | 11:23 |
| tsdgeos | ? | 11:23 |
| Saviq | tsdgeos, oh yeah, well, that's the missing combo button | 11:23 |
| Saviq | tsdgeos, but we should be able to do something of the sort the Notification has | 11:23 |
| tsdgeos | well that is what i was saying, put two buttons together and let it be similar to a combobox | 11:24 |
| tsdgeos | i'll try to code it and we can talk over it later | 11:24 |
| tsdgeos | Saviq: what is actually interesting if you see https://docs.google.com/a/canonical.com/file/d/0B-a_7E3tDxOgUUY0MjFTQThBZFE/edit | 11:24 |
| tsdgeos | is why are the buttons one next to another | 11:25 |
| tsdgeos | and not on top of the other | 11:25 |
| tsdgeos | like all the other preview components | 11:25 |
| tsdgeos | who decides that? | 11:25 |
| Saviq | tsdgeos, that's the thing - I think that's a single widget | 11:25 |
| Saviq | +meant to be | 11:25 |
| Saviq | /should be | 11:25 |
| tsdgeos | agreed | 11:26 |
| Saviq | tsdgeos, so yeah, let's implement it with that in mind | 11:27 |
| tsdgeos | need a new json for it | 11:27 |
| Saviq | tsdgeos, no we don't | 11:27 |
| tsdgeos | ? | 11:27 |
| Saviq | tsdgeos, type: "actions" takes one or more actions | 11:27 |
| tsdgeos | how come | 11:27 |
| tsdgeos | yes | 11:27 |
| tsdgeos | and more actions | 11:27 |
| tsdgeos | is a combobox | 11:27 |
| Saviq | tsdgeos, well, that's the thing | 11:28 |
| Saviq | tsdgeos, not any moer | 11:28 |
| Saviq | moar | 11:28 |
| tsdgeos | but how do i put a button and a combo box in one actions | 11:28 |
| tsdgeos | with that json? | 11:28 |
| Saviq | tsdgeos, put more than 2 actions | 11:28 |
| tsdgeos | and i have 3 buttons | 11:28 |
| Saviq | tsdgeos, no | 11:28 |
| Saviq | tsdgeos, 2 buttons, left a combo | 11:28 |
| tsdgeos | doesn't make much sense | 11:28 |
| tsdgeos | what if i am in a tablet | 11:29 |
| Saviq | tsdgeos, you get two columnt | 11:29 |
| Saviq | columns | 11:29 |
| tsdgeos | why shouldn't i get to use all my width | 11:29 |
| Saviq | tsdgeos, the widget shouldn't change whether on tablet or phone at all | 11:29 |
| Saviq | tsdgeos, because that's wasting the width | 11:29 |
| tsdgeos | instead of a button and a combo? | 11:29 |
| Saviq | tsdgeos, that's output from Cape Town | 11:29 |
| tsdgeos | ok | 11:29 |
| Saviq | tsdgeos, no button to span the whole width of a phone | 11:29 |
| Saviq | tsdgeos, as that wastes space | 11:29 |
| tsdgeos | sure, i'm not saying that | 11:30 |
| tsdgeos | i'm saying 2 buttons (one a combo) when i could have 3 is a bit weird | 11:30 |
| tsdgeos | but ok | 11:30 |
| tsdgeos | if you want that | 11:30 |
| tsdgeos | i'll code that | 11:30 |
| Saviq | tsdgeos, if you want three, that's when you put two widgets - one with two actions, one with one action | 11:31 |
| tsdgeos | ok | 11:32 |
| mzanetti_ | man... I got a headache today | 11:32 |
| Saviq | 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:33 |
| Saviq | 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 |
| tsdgeos | insteresting | 11:35 |
| tsdgeos | 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 |
| Saviq | tsdgeos, yes | 11:36 |
| tsdgeos | ok | 11:36 |
| Saviq | tsdgeos, you get one button, two buttons, or a button and a combobox | 11:37 |
| Saviq | 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 |
| tsdgeos | Saviq: i think it limits what people can do | 11:41 |
| tsdgeos | but i'm not the designer so i'm ok | 11:41 |
| Saviq | tsdgeos, we're doing that all over the place don't we :) | 11:41 |
| cimi | tsdgeos, what you disagree? | 11:41 |
| cimi | 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 |
| cimi | tsdgeos, in some areas you have more experience than designers, so share your issue with me :) | 11:42 |
| tsdgeos | cimi: i'm not disagreeing with designers, i'm disagreeing with Saviq :D | 11:43 |
| cimi | tsdgeos, amazing! | 11:43 |
| cimi | lol | 11:43 |
| tsdgeos | i do think that in some case it may make sense to just have a combobox | 11:43 |
| tsdgeos | but since i am not designing the interface i don't know if that makes sense in the previews or not | 11:44 |
| tsdgeos | 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 |
| Saviq | tsdgeos, we could add a prop to actions | 11:44 |
| Saviq | tsdgeos, that would say "I want all to go into combo" | 11:44 |
| Saviq | tsdgeos, but yeah, we can add that later if needed | 11:45 |
| Saviq | tsdgeos, https://code.launchpad.net/~saviq/unity8/fix-indicator-tests/+merge/204889 | 11:47 |
| * tsdgeos runs the autopilot tests | 11:48 | |
| * tsdgeos dies of old age | 11:48 | |
| Saviq | tsdgeos, try to run them on emulator ;D | 11:49 |
| mzanetti_ | he'll be dead before the emulator has booted :P | 11:49 |
| tsdgeos | Saviq: you miss the guidelines thing | 11:50 |
| Saviq | tsdgeos, refresh | 11:50 |
| tsdgeos | ah, not anymore | 11:50 |
| Saviq | cimi, 56 if (s <= 0 || typeof(s) !== "number") return ""; | 11:54 |
| Saviq | cimi, I think those should be reversed | 11:54 |
| Saviq | cimi, we don't want to even look at whether s <= if it's NaN | 11:54 |
| Saviq | cimi, text: (modelData["subtitle"] && modelData["subtitle"] !== undefined) ? modelData["subtitle"] : "" | 11:55 |
| Saviq | cimi, text: modelData["subtitle"] || "" | 11:55 |
| Saviq | tsdgeos, just a small tweak to the docs: https://code.launchpad.net/~aacid/unity8/image_gallery_preview_widget/+merge/204712/comments/478927 | 12:01 |
| === MacSlow is now known as MacSlow|lunch | ||
| tsdgeos | Saviq: do you like the new styling i just pushed more? | 12:08 |
| Saviq | tsdgeos, +1 | 12:09 |
| Saviq | tsdgeos, I'll put * everywhere after the next release | 12:09 |
| tsdgeos | Saviq: so on the "combo - button" scenario, i'm still unsure of how we want the thing to work | 12:30 |
| tsdgeos | obviously pressing the right button is pressing it, that's easy | 12:30 |
| tsdgeos | but do we want the combo to have two sensitive areas, one for activate and one for "open"? | 12:30 |
| tsdgeos | and when the combo opens, do we want the first button to hide like in notificationss? | 12:30 |
| tsdgeos | 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 |
| tsdgeos | in the non "open" area to activate it | 12:31 |
| Saviq | tsdgeos, *we* do the simplest thing that makes sense | 12:33 |
| Saviq | tsdgeos, I think the combo button is desgined... somewhere... | 12:33 |
| Saviq | tsdgeos, Jouni should have them, he should be online in an hour or two (as should SDK folks) | 12:37 |
| tsdgeos | ok | 12:37 |
| tsdgeos | Saviq: no worky https://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5078/#showFailuresLink | 12:46 |
| tsdgeos | or maybe it did fix 1 | 12:47 |
| Saviq | tsdgeos, that's another issue... | 12:56 |
| Saviq | tsdgeos, they broke app launching in the latest image... | 12:56 |
| Saviq | tsdgeos, we'll have to wait for the next one | 12:56 |
| === alan_g is now known as alan_g|lunch | ||
| dandrader | is launchpad slow today or is it my connection? | 13:13 |
| dandrader | hmm, getting several launchpad "Oops!" pages now... | 13:14 |
| karni | o/ | 13:22 |
| === MacSlow|lunch is now known as MacSlow | ||
| Saviq | dandrader, oops from launchpad suggest LP issues, your connection shouldn't have any bearing, no? | 13:33 |
| Saviq | karni, hey | 13:33 |
| dandrader | Saviq, yes, I got those Oops after I notices the overall launchpad slowness | 13:33 |
| === mardy_ is now known as mardy | ||
| dandrader | noticed | 13:33 |
| karni | hi Saviq :) | 13:33 |
| Saviq | dandrader, so yeah, it seems pretty slow here | 13:34 |
| Saviq | mhr3_, so, scope::preview() only gives out a single PreviewModel now, is there a multi-column version anywhere yet? | 13:40 |
| === alan_g|lunch is now known as alan_g | ||
| mhr3_ | saviq, not finished yet | 13:47 |
| Saviq | aaagh stupid QtCreator... every single time I create a folder with the name of the file I want to create... | 13:51 |
| tsdgeos | Saviq: so arrpove yours that indeed seems to decrease the number of failing tests? | 13:56 |
| Saviq | tsdgeos, well, it fixes that one failure | 13:56 |
| Saviq | tsdgeos, so yeah | 13:56 |
| Saviq | tsdgeos, and we wait for improvement on the other side | 13:57 |
| mzanetti_ | Saviq: duude | 14:01 |
| Saviq | mzanetti_, what up? | 14:01 |
| mzanetti_ | I constantly get: mzanetti Nickname already in use, try a different one. | 14:01 |
| mzanetti_ | Saviq: do a whois mzanetti | 14:01 |
| Saviq | 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:02 | |
| Saviq | whois mzanetti | 14:03 |
| === dandrader is now known as dandrader|afk | ||
| mzanetti_ | well, with the / | 14:03 |
| mzanetti_ | :P | 14:03 |
| Saviq | yeah yeah :P | 14:03 |
| Saviq | need to log in to disconnect... | 14:03 |
| Saviq | mzanetti_, here | 14:04 |
| === mzanetti_ is now known as mzanetti | ||
| mzanetti | Saviq: cheers | 14:05 |
| mzanetti | still wondering how this is possible | 14:05 |
| mzanetti | but I never really got irc authentication anyways | 14:05 |
| Saviq | mzanetti, you have to explicitly ask NickServ to get your nick back | 14:09 |
| tsdgeos | ghost him! | 14:09 |
| tsdgeos | tedg: seen the comment i made in your anywhere MR? | 14:10 |
| tedg | tsdgeos, Yup, just did. | 14:10 |
| tsdgeos | ahhh | 14:10 |
| tsdgeos | tedg: that's going to conflict | 14:10 |
| tedg | K, I'll deal. | 14:11 |
| tsdgeos | with https://code.launchpad.net/~unity-team/unity8/unity8.previews_textSummary/+merge/204177 | 14:11 |
| tsdgeos | but that's not approved for the moment | 14:11 |
| tsdgeos | so do yours and see if we can sneak it in | 14:11 |
| tsdgeos | this delayed landing is a pain for merges | 14:12 |
| tsdgeos | because you have to think about billions of branches that may or may not conflict with yours | 14:12 |
| tedg | Hah, you're telling me! :-) | 14:12 |
| tedg | I'm starting to loose track of which branches I've told other people to run so they get the functionality they need. | 14:14 |
| === sdeziel is now known as sdeziel_ | ||
| Saviq | tsdgeos, http://pastebin.ubuntu.com/6879267/ ? | 14:17 |
| tsdgeos | that is what i have yes | 14:17 |
| tsdgeos | kind of | 14:18 |
| tsdgeos | your docu is better | 14:18 |
| === dandrader|afk is now known as dandrader | ||
| tedg | tsdgeos, So I ran it, then I deleted the changes that weren't related to my change. | 14:19 |
| tedg | tsdgeos, That should reduce the conflict | 14:19 |
| tsdgeos | maybe | 14:19 |
| tsdgeos | or not | 14:19 |
| tsdgeos | who knows :D | 14:19 |
| tedg | Well, yes, it should reduce the likelihood of conflict ;-) | 14:20 |
| karni | 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 |
| Saviq | karni, ok | 14:24 |
| Saviq | tsdgeos, PreviewActions.qml? | 14:27 |
| tsdgeos | ignore that | 14:27 |
| tsdgeos | Saviq: you're complaining on the "s" ? | 14:27 |
| Saviq | tsdgeos, lol | 14:27 |
| Saviq | tsdgeos, no, asking whether that's the name of the widget ;D | 14:27 |
| * Saviq should be less picky in reviews | 14:28 | |
| tsdgeos | i had PreviewActions | 14:28 |
| tsdgeos | moved to PreviewAction | 14:28 |
| tsdgeos | but i guess i have to move it to PreviewActions again | 14:28 |
| tsdgeos | given the new meaning we gave the json | 14:28 |
| Saviq | tsdgeos, yeah, plural :) | 14:28 |
| Saviq | tsdgeos, well, the json was always "actions" for one-or-more actions :) | 14:28 |
| tsdgeos | i know | 14:28 |
| tsdgeos | but didn't like it ^_^ | 14:29 |
| mhr3_ | tsdgeos, saviq, what's the conclusion on the colors? do we have new prop? | 14:29 |
| tsdgeos | what | 14:29 |
| Saviq | mhr3_, not yet | 14:29 |
| tsdgeos | mumble | 14:29 |
| tsdgeos | mumble: symbol lookup error: mumble: undefined symbol: _ZN6google8protobuf18GoogleOnceInitImplEPlPNS0_7ClosureE | 14:29 |
| tsdgeos | :S | 14:29 |
| Saviq | yay | 14:29 |
| Saviq | tsdgeos, yup, no mumble today | 14:29 |
| tsdgeos | i can trying using the android client on the Z10 | 14:30 |
| tsdgeos | let's see how it flies :D | 14:30 |
| Saviq | tsdgeos, I can't :P | 14:30 |
| Saviq | @unity: hangout today, mumble doesn't load with ↑↑ | 14:30 |
| Saviq | https://plus.google.com/hangouts/_/72cpiqua3mnkr7dti836vn0ui0 | 14:31 |
| dandrader | Saviq, will take me a while to jump on ghangout (have to turn on my laptop, etc) | 14:31 |
| Saviq | dandrader, sure | 14:31 |
| karni | SU notes: attending sprint sessions, letting Saviq kick ass and save the world before mwc, ha ;) reviewing code, etc | 14:31 |
| Saviq | karni, you'll have to put it in yourself ;P | 14:31 |
| tsdgeos | oh there's a new protobuf deb comin in dist-upgrade | 14:32 |
| tsdgeos | maybe fixes the thing | 14:32 |
| karni | Saviq: come again :)? | 14:32 |
| elopio | I'm trying to join too, but need to install the plugin. | 14:32 |
| sdeziel_ | Hi all | 14:32 |
| sdeziel_ | I'm running Trusty and installed some updates (one of them was lightdm) and rebooted. Since then, Unity doesn't autostart so I get a bare desktop. Any idea where to look for debugging? | 14:32 |
| Saviq | karni, in the doc I mean ;) | 14:32 |
| karni | Saviq: haha, I will | 14:32 |
| karni | guys, mind throwing the SU doc at me? | 14:33 |
| karni | found it | 14:34 |
| mzanetti | kgunn: -mumble +hangout | 14:36 |
| mzanetti | kgunn: https://plus.google.com/hangouts/_/72cpiqua3mnkr7dti836vn0ui0 | 14:36 |
| MacSlow | Saviq, hm... I'm not allowed to connect to that hangout... | 14:36 |
| Saviq | MacSlow, https://plus.google.com/hangouts/_/72cpiqua3mnkr7dti836vn0ui0?authuser=1 | 14:36 |
| Saviq | elopio, ↑↑ we're doing hangout today as mumble's dead | 14:38 |
| elopio | Saviq, I'm in, thanks. | 14:39 |
| Saviq | elopio, d'oh | 14:39 |
| elopio | mterry, ping. Do you have time now? | 14:45 |
| mterry | elopio, sure! Let me dig into how system-settings is called over dbus | 14:46 |
| kgunn | tsdgeos: so when would that qt release actually hit ? (...thinking of risk & mwc being right around the corner) | 14:48 |
| sdeziel_ | any idea on what's wrong with Unity not starting after login? | 14:48 |
| kgunn | altho...maybe not so big deal since Saviq says we're _virtually_ on that rel anyway | 14:48 |
| Saviq | kgunn, wait wait wait | 14:49 |
| Saviq | kgunn, we're not switching for mwc | 14:49 |
| Saviq | kgunn, we have 5.2 (and some of 5.2.1) in qt5-beta2 ppa | 14:49 |
| Saviq | kgunn, so there's little risk between that and full 5.2.1 | 14:49 |
| kgunn | ok, np.... | 14:50 |
| Saviq | kgunn, but we still have bugs to fix for it (although apparently not unity8 ones!) | 14:50 |
| kgunn | right, is that phonedations that makes the call for when to pull in a new Qt rel ? | 14:51 |
| Saviq | kgunn, QA rather | 14:51 |
| kgunn | mmm | 14:51 |
| Saviq | kgunn, basically when we get green on 5.2 and no bugs can be identified - that's when | 14:52 |
| kgunn | of course.... | 14:53 |
| kgunn | was wondering if there was even a "conformance test suite" in addition to that... | 14:53 |
| kgunn | guessing not | 14:53 |
| Saviq | karni, https://code.launchpad.net/~unity-team/unity8/new-scopes-title-alignment/+merge/204677/comments/479092 | 14:59 |
| Saviq | AAARGH | 15:03 |
| Saviq | mhr3_, action activation will happen on the preview object? | 15:16 |
| mhr3_ | yes | 15:17 |
| Saviq | mhr3_, any name for the method yet? | 15:17 |
| mhr3_ | saviq, didn't pick yet, feel free to | 15:18 |
| Saviq | mhr3_, we're using triggered(string, string, QVariantMap), how's that? | 15:18 |
| mhr3_ | fine with me | 15:19 |
| Saviq | mhr3_, I'm starting to feel like we need a "separator" widget... | 15:19 |
| Saviq | mhr3_, as I can't find a rule in the visuals for where to put it and where not | 15:20 |
| mhr3_ | :/ | 15:20 |
| mhr3_ | i can see scopes specifying margins in px soon | 15:21 |
| Saviq | mhr3_, let me see if katie has an idea | 15:21 |
| Saviq | <-- katie has quit (Ping timeout) FAIL | 15:21 |
| Saviq | ok actually found a rule... | 15:22 |
| Saviq | maybe | 15:22 |
| Saviq | mhr3_, btw, we discussed roleNames a while ago, and decided there's no point in keeping them in m_roles, as they will only ever get called once, so we're usually creating them in roleNames directly and letting it drop when going out of scope | 15:31 |
| mhr3_ | saviq, hm, not like it used up insane amounts of memory | 15:32 |
| Saviq | mhr3_, of course, just saying | 15:32 |
| mhr3_ | yea, will keep in mind | 15:32 |
| === alan_g is now known as alan_g|tea | ||
| mhr3_ | saviq | 15:39 |
| Cimi | Saviq, is this fine now? https://code.launchpad.net/~unity-team/unity8/unity8.previews_audioPlayer/+merge/203387 | 15:39 |
| mhr3_ | "card-size": "medium", // "small", "medium", "large", 12 ÷ 38 (in grid-units) when "category-layout": "vertical" | 15:39 |
| mhr3_ | should have been card-layout, right? | 15:40 |
| Saviq | mhr3_, of course | 15:40 |
| mhr3_ | i was confused if it wasn't supposed to be vertical-journal | 15:40 |
| mhr3_ | actually | 15:42 |
| mhr3_ | wasn't it supposed to be "category-layout": "vertical-journal"? | 15:42 |
| === alan_g|tea is now known as alan_g | ||
| mhr3_ | saviq, is there max for xollapsed-rows? | 15:48 |
| mhr3_ | collapsed* | 15:48 |
| Saviq | mhr3_, man, I should probably read what you write | 15:49 |
| mhr3_ | you should :P | 15:49 |
| Saviq | mhr3_, it should be journal | 15:49 |
| Saviq | mhr3_, in vertical size is s, m, l - only in journal you can say what height every row is | 15:49 |
| Saviq | mhr3_, spec defines collapsed-rows per category layout | 15:50 |
| mhr3_ | saviq, i wonder if we have a maximum value there | 15:51 |
| mhr3_ | 1024? :) | 15:51 |
| Saviq | mhr3_, no, like... 2 | 15:51 |
| mhr3_ | so, 1 or 2... lots of options | 15:51 |
| Saviq | mhr3_, indeed | 15:52 |
| mhr3_ | collapsed-rows Number of result rows displayed while the category is collapsed; possible values: any integer between 1 and 2 | 15:52 |
| mhr3_ | doesn't sound right :) | 15:52 |
| Saviq | lol | 15:52 |
| Saviq | mhr3_, it only affects grid and journal, too ;) | 15:52 |
| Saviq | mhr3_, sounds like we should make it bool | 15:52 |
| mhr3_ | saviq, i'm starting to think you should have been writing this ;) | 15:52 |
| Saviq | mhr3_, collapsed-2-rows: true ;) | 15:53 |
| mhr3_ | heh | 15:53 |
| tsdgeos | Saviq: can get lp:~aacid/unity8/action_preview_widget and run "make tryPreviewActions" | 15:59 |
| tsdgeos | and tell me what you think? | 15:59 |
| tsdgeos | I think it's not great, but unless I should be coding the combobox (which will take some time) I don't see anything better we can do | 15:59 |
| Saviq | tsdgeos, hmm yeah that looks fine, we'll only have two buttons at the most in our previews I think | 16:05 |
| tsdgeos | ook | 16:06 |
| Saviq | mhr3_, will I still have "id" and "type" in preview widget properties, or are you stripping them? | 16:20 |
| mhr3_ | saviq, they're separate roles, not in the big variantmap | 16:21 |
| Saviq | mhr3_, bleh :P | 16:21 |
| mhr3_ | you should be glad! | 16:25 |
| mhr3_ | it's guaranteed to always be there | 16:25 |
| mhr3_ | and always a string | 16:25 |
| Cimi | Saviq, ping? maybe I missed your answer | 16:40 |
| Cimi | Saviq, how is the preview widget now? | 16:40 |
| Saviq | Cimi, you didn't, just ACK'ed it | 16:41 |
| Cimi | Saviq, maybe tsdgeos could review it | 16:41 |
| Saviq | Cimi, he did already, didn't he | 16:42 |
| Saviq | Cimi, if you get him to do it - be my guest ;) | 16:43 |
| Cimi | Saviq, now again | 16:43 |
| tsdgeos | do what? | 16:47 |
| Saviq | tsdgeos, do a last review of audioPlayer (probably just the last commits would be enough) | 16:49 |
| Saviq | mhr3_, properties^Wattributes... widgetId, type... yay for consistency ;P | 16:50 |
| mhr3_ | consistency is relative :) | 16:51 |
| tsdgeos | Saviq: looks ok to me | 16:51 |
| === dandrader is now known as dandrader|lunch | ||
| pstolowski_ | sil2100, hey, any idea why https://code.launchpad.net/~marcustomlinson/unity-scopes-api/smartscopesproxy_fixes/+merge/204827 hasn't been merged yet? | 17:03 |
| tsdgeos | hmmm | 17:03 |
| tsdgeos | console.log(spy.signalArguments[0][0]); | 17:04 |
| tsdgeos | should give me the first paramter of the first signal that was spied, right? | 17:04 |
| greyback | Saviq: I'm still waiting for 2 branches of unity-mir to land. Did they fall off the CI train? | 17:23 |
| Saviq | greyback, http://i.dailymail.co.uk/i/pix/2009/05/25/article-1187119-051592F9000005DC-459_634x421.jpg | 17:24 |
| greyback | Saviq: in that case, can I squeeze this in: https://code.launchpad.net/~gerboland/unity-mir/fix-readme/+merge/204572 | 17:24 |
| Saviq | greyback, sure | 17:24 |
| greyback | Saviq: it needs a quick review plz | 17:24 |
| * greyback appalled Saviq reads the daily mail | 17:25 | |
| Saviq | greyback, you should stop supporting in-source builds | 17:25 |
| greyback | patches welcome :) | 17:26 |
| Saviq | greyback, why the -DCMAKE_INSTALL_PREFIX=/usr ? | 17:26 |
| greyback | Saviq: otherwise installs in /usr/local | 17:27 |
| Saviq | greyback, well, yeah, you shouldn't "make install" it anyway, should you.. | 17:27 |
| Saviq | greyback, but yeah, ok | 17:29 |
| greyback | Saviq: it's fast & dirty, just wanted to update the wrong info, tis all | 17:29 |
| Saviq | greyback, got it | 17:29 |
| Saviq | greyback, ACK | 17:30 |
| Saviq | mhall119, hey, so we've started generating docs from unity8 (doxygen style), think it'd make sense to publish them somewhere? | 17:31 |
| Saviq | mhr3_, hrmpf "Error: Unknown method return type: scopes_ng::PreviewModel*" | 18:02 |
| Saviq | but Q_DECLARE_METATYPE(scopes_ng::PreviewModel*)!! | 18:02 |
| Saviq | mzanetti, ↑ you solved that error for yourself recently with exactly that, right? | 18:02 |
| Saviq | ah hmm | 18:03 |
| mzanetti | not exactly this | 18:03 |
| mzanetti | it was a parameter, not the return type | 18:03 |
| mzanetti | but yeah. I guess its the same | 18:03 |
| karni | Saviq: I'll address the rest of the comments on card-overlay branch | 18:03 |
| Saviq | mzanetti, I think in this case it's because it's not wrapped in QVariant maybe | 18:04 |
| Saviq | karni, cool, I got previews *almost* working | 18:04 |
| karni | Saviq: fantastic :) | 18:04 |
| mzanetti | huh... no. its a pointer. don't wrap that in QVariants | 18:04 |
| === dandrader|lunch is now known as dandrader | ||
| Saviq | mzanetti, yeah, don't listen to me | 18:08 |
| Saviq | mzanetti, was thinking of different place in the code | 18:08 |
| mzanetti | :) no worries | 18:08 |
| Saviq | ah, it's not qmlRegistered | 18:08 |
| mhall119 | Saviq: why doxygen? | 18:10 |
| Saviq | mhall119, as opposed to? | 18:10 |
| mhall119 | QDoc | 18:11 |
| === alan_g is now known as alan_g|EOD | ||
| karni | Saviq: I think that unity8-card-overlay willl have to be against new-scopes. CardCarousel.qml is not present in unity8 (only contains DashCarousel, while it is CardCarousel.qml that should override whether overlay is shown on current item) | 18:17 |
| karni | whatcha think? | 18:17 |
| karni | s/against/rebased on new-scopes | 18:18 |
| Saviq | karni, https://code.launchpad.net/~unity-team/unity8/newscopes-card-overlay/+merge/204797 | 18:18 |
| karni | haha | 18:18 |
| * karni marks WIP | 18:18 | |
| karni | Saviq: shall I just continue work in that branch? | 18:18 |
| Saviq | karni, yup, is why I pushed it under unity-team | 18:21 |
| karni | thanks bug | 18:21 |
| karni | *bud :D | 18:22 |
| mhall119 | Saviq: I ask about qdoc because that's what all of the SDK uses already, and I have an importer for qdoc into the API website for publishing | 18:26 |
| mhall119 | for doxygen somebody would have to write an parser/importer to get it published to the API website | 18:26 |
| Saviq | mhall119, oh well, I thought doxygen was meant to be used... | 18:27 |
| Saviq | mhall119, fortunately they're compatible | 18:27 |
| mhall119 | in that case I can set you up on developer.ubuntu.com/api/unity/<version>/ for qdoc generated stuff | 18:28 |
| mhall119 | developer.u.c/api is just a reverse proxy too, so we can set up unity.u.c/api/ to point to it too | 18:28 |
| mhall119 | ...maybe | 18:29 |
| mhall119 | well, ok, on second thought that would take some work | 18:29 |
| mhall119 | mostly because of the website theme | 18:29 |
| Saviq | mhall119, if you can try and see how this will work out, that'd be nice | 18:31 |
| mhall119 | Saviq: worst case scenario, we deploy a separate instance of the API website just for Unity | 18:33 |
| mhall119 | it's all Juju charmed now, so that shouldn't be hard | 18:33 |
| mhall119 | (famous last words) | 18:33 |
| Saviq | :) | 18:34 |
| Saviq | mhr3_, http://paste.ubuntu.com/6880650/ | 18:34 |
| === boiko_ is now known as boiko | ||
| karni | Saviq: I assume the shader should fade in together with CardHeader (as if they were one thing) | 18:44 |
| karni | I'll do it for text first. Once that works, I'll include the shade effect. | 18:45 |
| Saviq | karni, yeah, fade in text, too | 18:49 |
| === jono is now known as Guest56083 | ||
| mzanetti | Saviq: another one of my cmake hacks :D https://code.launchpad.net/~mzanetti/unity8/make-single-testfunction/+merge/205035 | 19:11 |
| Saviq | mzanetti, huh, is $(FUNCTION) something cmake knows? | 19:12 |
| mzanetti | no | 19:12 |
| mzanetti | Saviq: well, sort of yes | 19:12 |
| mzanetti | its make | 19:12 |
| mzanetti | $(FOO) can be used to pass arguments to make | 19:12 |
| mzanetti | but not real arguments | 19:12 |
| mzanetti | only vial FOO=blabla | 19:12 |
| Saviq | mzanetti, will it work with ninja? | 19:13 |
| * mzanetti tries | 19:13 | |
| mzanetti | works with nmake according to stackoverflow | 19:13 |
| om26er | https://bugs.launchpad.net/unity8/+bug/1276747 | 19:14 |
| ubot5 | Launchpad bug 1276747 in autopilot (Ubuntu) "after starting a new app cannot get proxy object for unity8" [Undecided,New] | 19:14 |
| mhr3_ | saviq, eh, yea, sorry | 19:14 |
| Saviq | om26er, yeah, saw that - affected autopilot, too, we're not doing anything special around this | 19:14 |
| mhr3_ | saviq, will add to my previewstack branch | 19:14 |
| Saviq | mhr3_, nw, it excepts anyway | 19:14 |
| Saviq | ;) | 19:15 |
| mhr3_ | sucks :P | 19:15 |
| om26er | Saviq, thanks for that | 19:16 |
| mzanetti | Saviq: too bad... ninja doesn't like the $ | 19:17 |
| mzanetti | wants $$ | 19:17 |
| mzanetti | but then it breaks | 19:18 |
| elopio | mterry, https://code.launchpad.net/~elopio/unity8/url-dispatcher_test/+merge/205037 | 19:23 |
| Saviq | ok, finishing early today ;P | 19:33 |
| karni | Saviq: early you say :D 8:30 haha. If you happen to drop around, the isCurrentItem attached property doesn't seem to work for me :| http://paste.ubuntu.com/6881024/ | 19:46 |
| karni | btw your overlay is lovely | 19:47 |
| karni | font color: http://paste.ubuntu.com/6881030/ | 19:47 |
| Saviq | karni, "=== true" can probably be dropped | 20:04 |
| karni | ack | 20:04 |
| karni | just pushed the version that doesn't work with attached property isCurrentItem | 20:05 |
| karni | left a comment, don't know why it doesn't work. | 20:05 |
| Saviq | karni, http://developer.ubuntu.com/api/qml/sdk-1.0/Ubuntu.Components.Themes.PaletteValues/ use these for the default color | 20:05 |
| karni | if you put true/false, it toggles overlay on all items, so the attached property should work | 20:05 |
| karni | Saviq: I asked katie for those, she just said "white" in overlay xD thanks! | 20:05 |
| Saviq | karni, attached properties are only available on the direct delegate item | 20:06 |
| karni | the Loader in that case.. | 20:06 |
| Saviq | karni, yeah, for overlay white, but for the default - use one foregroundColor | 20:06 |
| * karni nods | 20:06 | |
| Saviq | karni, not Loader, Card? | 20:06 |
| Saviq | karni, ah I know what you mean | 20:07 |
| karni | that in that case, that'd be showHeader: isCurrentItem | 20:07 |
| karni | and that doesn't work either | 20:08 |
| karni | actually, it says ReferenceError: isCurrentItem is not defined | 20:08 |
| karni | we use Loader as the delegate in Carousel | 20:08 |
| karni | maybe that's the reason | 20:08 |
| Saviq | karni, no, you're right - ListView.isCurrentItem would only be available in the Loader | 20:08 |
| karni | parent.ListView.isCurrentItem works | 20:08 |
| karni | within Card { } in CardCarousel | 20:09 |
| karni | just found out by trial and error ;P | 20:09 |
| Saviq | karni, tricky ;) | 20:09 |
| karni | makes sense, as Loader would be parent of Card, but the Card becomes the delegate so.. not very intuitive | 20:09 |
| karni | hahahah, when you click - isCurrentItem changes. when you scroll the carousel, it doesn't :D | 20:10 |
| karni | challenge! | 20:10 |
| karni | let my try that on my own, tho! xD | 20:10 |
| karni | actually, got a session in 5 minutes, but I'll try to set current item from within Carousel component | 20:11 |
| Saviq | karni, showHeader: explicitlyScaled | 20:11 |
| karni | smart xD | 20:11 |
| Saviq | karni, that's what it's there for (but the name is tricky) Cimi, you and your prop names :P | 20:12 |
| karni | SLICK | 20:12 |
| karni | hahah | 20:12 |
| karni | yeah, naming is one of two worse problems in programming. the other is cache invalidation ;P | 20:12 |
| karni | *proper naming | 20:12 |
| Saviq | karni, to only use a single animation, you can go: opacity: header.opacity * 0.6 for the overlay | 20:18 |
| karni | yeah, I thought it'd be better to use one (or pull Shader under CardHeader with z: -1 ) | 20:19 |
| Saviq | karni, not sure what that has to do ;) | 20:23 |
| Saviq | karni, opacity is inherited, but z-order does not change parentship | 20:23 |
| karni | Saviq: I thought I could use one "Behavior" declaration, but I guess I can't (like, one animator, if that even matters) | 20:23 |
| karni | ok, now I get it heh, sorry :) | 20:24 |
| Saviq | karni, you could, though, set parent: of the header to the overlay, when in overlay mode | 20:24 |
| karni | :D | 20:24 |
| Saviq | karni, and sure, you could use a single Animation object | 20:24 |
| Saviq | karni, but that's effectively what opacity: header.opacity * 0.6 does | 20:24 |
| * karni nods, fixed | 20:25 | |
| Saviq | karni, with less code, 'cause you don't have to explicitly configure the Animation | 20:25 |
| karni | Correct, sir. Pushed | 20:25 |
| karni | Shall I leave it WIP to center align title within overlay (whish works in vertical mode, so Left aligned) ? | 20:26 |
| karni | but seriously, you should go enjoy your evening pal :) | 20:26 |
| Saviq | karni, not sure it should center in overlay | 20:26 |
| karni | Saviq: that's what's in design doc, katie added overlay section | 20:26 |
| Saviq | karni, I'll enjoy my week starting Saturday, when I'm off snowboarding in Austria, but first I need to earn it ;) | 20:27 |
| karni | :O wohooo \o/ | 20:27 |
| karni | Saviq: leaving for a week? you've certainly earned it | 20:27 |
| Saviq | karni, yeah, I just want to leave you guys in a state where you can mostly apply tweaks to the whole system, rather than have to implement huge portions of it | 20:28 |
| karni | :)) | 20:28 |
| karni | Appreciated | 20:29 |
| karni | Very much | 20:29 |
| kgunn | Saviq: booo...leaving | 20:29 |
| kgunn | enjoy | 20:29 |
| kgunn | hope there's snow | 20:30 |
| Saviq | karni, you should run stuff before pushing ;) | 20:30 |
| Saviq | opacity: opacity: header.opacity * 0.6 | 20:30 |
| karni | d'uh | 20:30 |
| karni | that's what happens when trying to work in middle of session, my bad :/ | 20:30 |
| karni | was sure I :s <enter>'ed ;P | 20:30 |
| Saviq | kgunn, hold your horses, I'm still here :D | 20:30 |
| karni | fixed, sorry about that :// | 20:31 |
| kgunn | Saviq: oh right...8 not 6 | 20:31 |
| Saviq | karni, re: margins, we should probably push them down into CardHeader, no need for them to be in two places | 20:34 |
| Saviq | karni, /me does | 20:34 |
| karni | :D was about to say "consider it done" | 20:34 |
| karni | ok. /me listens to the session | 20:34 |
| Saviq | karni, height: header.height is not good, we don't want to make overlays of different height in case there's only one line of title or missing subtitle | 20:41 |
| Saviq | karni, CardTool needs to give up headerHeight | 20:41 |
| karni | Saviq: ack | 20:43 |
| karni | I'll figure it out | 20:43 |
| karni | Saviq: I assume it'd be either 2 lines (for title) or 3 lines (for title+subtitle, if both are mapped) | 20:44 |
| karni | That sucks, cose we'll have 2 lines even if no title wraps to two lines | 20:45 |
| karni | :( | 20:45 |
| karni | but I guess that's better than have different height, yes | 20:46 |
| Saviq | karni, it sucks less than overlay going up and down | 20:46 |
| * karni agrees | 20:46 | |
| Saviq | karni, or well, maybe it depends | 20:46 |
| Saviq | karni, as in Carousel you only ever show a single overlay | 20:46 |
| karni | maybe I should ask katie? | 20:46 |
| karni | right | 20:46 |
| Saviq | karni, but in Grid you show them all | 20:46 |
| karni | good point | 20:46 |
| karni | actually, very good observation | 20:46 |
| Saviq | karni, so yeah, asking for design input would be good | 20:46 |
| karni | I'll do that | 20:46 |
| Saviq | karni, I moved most of your changes into unity8-card-overlay | 20:48 |
| Saviq | karni, and merged+pushed into newscopes-card-overlay | 20:49 |
| * karni looks | 20:50 | |
| karni | Looks great, Saviq, thank you | 20:51 |
| * Saviq really likes it how we managed to keep the diff between old and new scopes really small | 20:53 | |
| karni | Yea, you did it fantastic | 20:56 |
| === shiznix_ is now known as shiznix | ||
| bregma | tedg, I'm having trouble launching apps through upstart-app-launch, are there any cunning debug tricks? | 21:45 |
| tedg | bregma, Which branch are you using? | 21:46 |
| bregma | trunk ... in a Unity8 session on Mir on the desktop | 21:46 |
| tedg | bregma, K | 21:47 |
| tedg | bregma, So I'd set the glib debug flag | 21:47 |
| tedg | bregma, initctl set-env --global G_MESSSAGES_DEBUG=all | 21:47 |
| tedg | bregma, That'll put more interesting stuff in ~/.cache/upstart/application* | 21:47 |
| bregma | sounds like a good start | 21:48 |
| tedg | bregma, Wait, are you using trunk or main? | 21:48 |
| tedg | Like what's released in archive | 21:48 |
| bregma | whatever's in the release archive | 21:48 |
| tedg | K, that should be your next step then. | 21:48 |
| bregma | tedg, does upstart-app-launch/desktop-exec reset the environment or something? | 22:01 |
| tedg | bregma, It configures it, upstart creates a clean environment each time. | 22:02 |
| tedg | bregma, You can get it with initctl get-env | 22:03 |
| bregma | well, I'm setting the QT_QPA_PLATFORM explitcitly with initctl set-env (and env directives) but the app crashlog does not show anything but a few basic variables | 22:04 |
| tedg | set-env --global ? | 22:05 |
| tedg | What is generally best is to put that in an upstart job that is "start on starting dbus" so then it gets into the dbus env as well. | 22:05 |
| tedg | bregma, This is my ~/.config/upstart/glib-debug.conf as an example: http://paste.ubuntu.com/6881793/ | 22:06 |
| elopio | mterry: are you still here? | 22:51 |
| mterry | elopio, yup | 22:51 |
| elopio | I don't understand why url-dispatcher stops working when we do the initctl stop unity8 | 22:51 |
| mterry | elopio, I'm looking at things now too | 22:51 |
| elopio | is the unity process we launch on the tests missing something? | 22:51 |
| mterry | elopio, does autopilot know the right DBUS_SESSION_BUS_ADDRESS? | 22:51 |
| elopio | mterry: I bet not, but you said that comes from an environment variable, right? | 22:53 |
| mterry | elopio, you might be thinking of me saying about MIR_SOCKET | 22:53 |
| mterry | elopio, I'm talking about url-dispatcher being able to see dbus | 22:53 |
| mterry | elopio, I'm testing this idea now | 22:53 |
| mterry | elopio, no, that's fine | 22:54 |
| elopio | my previous tests restart unity8 and still can talk to the session dbus. But I barely know what I'm talking about :) | 22:55 |
| elopio | my guess is that initctl start unity8 does something we are not doing on the test's launch unity. | 22:56 |
| mterry | was just a though | 22:56 |
| mterry | hm | 22:56 |
| mterry | elopio, if I enter a bogus url or whatever to dispatcher, I get an error | 22:59 |
| mterry | I don't see that here | 22:59 |
| elopio | mterry: could the camera app be behind unity? | 23:01 |
| mterry | elopio, camera-app isn't running | 23:01 |
| elopio | mterry: it is here. | 23:02 |
| mterry | oh , let me try again | 23:02 |
| mterry | elopio, right you are! | 23:03 |
| mterry | How does AP launch unity8? I thought it was via upstart too | 23:04 |
| elopio | good :) | 23:05 |
| elopio | sadly, that leaves me as clueless as before, jaja. | 23:05 |
| elopio | mterry: it does initctl start unity8, but it also does a lot of other things I'm looking for the first time | 23:05 |
| elopio | os.unlink( | 23:05 |
| elopio | os.getenv('MIR_SOCKET', | 23:05 |
| elopio | os.path.join(os.getenv('XDG_RUNTIME_DIR', "/tmp"), | 23:05 |
| elopio | "mir_socket"))) | 23:05 |
| elopio | I don't know what's that about. | 23:05 |
| mterry | elopio, that's just cleaning up from previous runs | 23:14 |
| mterry | elopio, shouldn't be anything to worry about | 23:14 |
| kgunn | mterry: you got your wish...0.1.4 mir is landing now...its actually in devel-proposed image now | 23:19 |
| mterry | kgunn, sorta. I want 0.1.5 though! | 23:19 |
| mterry | elopio, odd results | 23:19 |
| kgunn | and it has the orientation argument in the configure_output | 23:19 |
| kgunn | mterry: it took so long.... | 23:19 |
| kgunn | i just remerged devel into the MP :) | 23:20 |
| mterry | kgunn, oh good! | 23:20 |
| kgunn | duflu set it up perfect so the so bumps were in devel...but the deb bump is only in trunk | 23:20 |
| kgunn | so i didn't have to wait | 23:20 |
| kgunn | enjoy! | 23:20 |
| mterry | kgunn, now we just have to get the new AP tests sorted. Neither one is working even against current image right now | 23:20 |
| kgunn | eegads | 23:20 |
| mterry | kgunn, tests are hard | 23:20 |
| kgunn | lol | 23:20 |
| kgunn | mterry: no kidding i keep telling people this | 23:21 |
| kgunn | theyu | 23:21 |
| kgunn | they're like hey...just get a guy to do a test for 2 weeks... | 23:21 |
| mterry | elopio, so when I manually run camera-app (sudo -u phablet -i camera-app) I get different results between the two runs (normal and AP) | 23:21 |
| kgunn | i'm like...uh, no...it takes maintenance | 23:21 |
| mterry | elopio, normal gives me "QUbuntu: Could not create application instance" | 23:21 |
| kgunn | ok, gonna drop for abit...later | 23:21 |
| mterry | elopio, AP gives me http://paste.ubuntu.com/6882127/ | 23:22 |
| mterry | kgunn, bye | 23:22 |
| mterry | elopio, why would we get those results? | 23:22 |
| mterry | elopio, neither brings up camera-app | 23:23 |
| elopio | mterry: I have no idea. | 23:24 |
| elopio | veebers should know more about this code, but I think new zeland is on holidays. | 23:24 |
| elopio | mterry: at what point of the autopilot test are you trying to launch the camera? | 23:26 |
| mterry | elopio, I'm doing my current testing in a separate adb terminal and running the camera-app manually either during AP test or during normal operation | 23:27 |
| mterry | elopio, not running it from the test, just during the test | 23:27 |
| elopio | right. It's weird, the set up of the unitytestcase must be playing tricks with the environment. | 23:28 |
| mterry | elopio, but it's weird that during AP test, it seems to get further | 23:29 |
| elopio | mterry: this is how we are starting unity on the tests: | 23:38 |
| elopio | /sbin/initctl start unity8 BINARY=/usr/bin/unity8 ARGS= XDG_DATA_DIRS=/usr/share/unity8/mocks/data:/usr/share/ubuntu-touch-surfaceflinger:/usr/local/share/:/usr/share/ LD_LIBRARY_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks/libusermetrics:/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks/LightDM/single QML2_IMPORT_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks QT_LOAD_TESTABILITY=1 | 23:38 |
| elopio | if I do that manually, and manually launch the camera, it doesn't appear in front. | 23:38 |
| mterry | ok, good | 23:38 |
| mterry | so we can start taking away things :) | 23:38 |
| mterry | oh we have quite a few mocks there | 23:39 |
| mterry | elopio, /usr/share/unity8/mocks/data in XDG_DATA_DIRS | 23:43 |
| mterry | hm | 23:44 |
| mterry | maybe something else too | 23:44 |
| mterry | elopio, that and dropping QML2_IMPORT_PATH=/usr/lib/arm-linux-gnueabihf/unity8/qml/mocks | 23:45 |
| elopio | mterry: yes! without those two, it works. | 23:46 |
| elopio | there is a fake application manager, maybe not updated to work with url dispatcher? | 23:49 |
| elopio | it seems to be faking too much. | 23:50 |
| mterry | elopio, why are we faking that much in AP mode? | 23:51 |
| * mterry looks at the delta for the mock and real thing | 23:51 | |
| elopio | mterry: I don't know. I'm a new comer. I added one fake, that was to avoid talking to the click servers | 23:52 |
| elopio | I suppose there are some good reasons for these others. | 23:52 |
| * elopio makes a mental note to always add a comment on the reason for using a fake. | 23:52 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
