| cwayne | greyback, ping! | 00:27 |
|---|---|---|
| greyback | cwayne: pong | 00:27 |
| cwayne | greyback, hey, did you want more testing on those unity-mir fixes, or are they pushed? | 00:28 |
| greyback | cwayne: since I didn't get a mail from you, I pushed the code and have kicked off a build | 00:28 |
| greyback | should land in about 20-25 mins | 00:30 |
| cwayne | greyback, awesome, thanks! | 00:31 |
| greyback | cwayne: again, thanks for making sure it was ok | 00:31 |
| cwayne | greyback, np, so this is in the ppa right? not in the archive that is | 00:34 |
| greyback | cwayne: yeah the demo-stuff ppa. | 00:35 |
| cwayne | perfect | 00:36 |
| Longfellow | is there an easy walk thru on how to create a shortcut on unity for a program compiled from source? | 05:06 |
| === jono is now known as Guest99797 | ||
| Longfellow | anyone in here? | 05:10 |
| RAOF | Lots of people. | 05:16 |
| RAOF | Longfellow: You're after a .desktop file; install alacarte and it will help you create one. | 05:17 |
| RAOF | Longfellow: Does your desktop file point to the right thing? Also, this is probably more an #ubuntu question (as it's support) | 05:20 |
| Longfellow | yeah i was in #ubuntu and no one responded to my question and saw this channel and thought i'd give it a shot | 05:23 |
| === Wig is now known as Wug | ||
| karni | o/ | 07:28 |
| karni | Saviq_: hrm. I see the changes showed up when I dist-upgraded the image, but the branch is not merged. Have you worked on it elsewhere? https://code.launchpad.net/~unity-team/unity8/themeing-font-and-mascot/+merge/207282 | 07:50 |
| Saviq | karni, nope, it's not merged anywhere?? | 08:11 |
| karni | Saviq: hrm. I guess someone set the card colors in the Apps scope, that could have confused me | 08:12 |
| Saviq | karni, probably yeah :) | 08:12 |
| karni | Saviq: no stress! :) | 08:12 |
| karni | Saviq: ok, I'll work on it then | 08:12 |
| Saviq | karni, ok thanks, I started, but it quickly became a little more than I wanted to bite yesterday | 08:14 |
| karni | Saviq: any diff you want to send over, or you just started? | 08:14 |
| Saviq | karni, nah, not really | 08:15 |
| karni | Saviq: yeah, took me a while yesterday (while doing other stuff in the mean time). I thought it'd be a smaller task. | 08:15 |
| karni | okay, I got it then. | 08:15 |
| tsdgeos | Saviq: which bug/feature/thing should i target next? | 08:31 |
| Saviq | tsdgeos, have a look at https://bugs.launchpad.net/unity8/+bug/1127665 | 08:32 |
| ubot5 | Launchpad bug 1127665 in unity8 (Ubuntu) "[SIDESTAGE] Dash isn't visible when only sidestage app running " [Medium,Triaged] | 08:32 |
| Saviq | tsdgeos, which https://bugs.launchpad.net/unity8/+bug/1213034 is related to, too | 08:32 |
| ubot5 | Launchpad bug 1213034 in Unity 8 "Can't dismiss keyboard by tapping outside of search entry" [High,Opinion] | 08:33 |
| tsdgeos | Saviq: i don't see how those two are related | 08:34 |
| Saviq | tsdgeos, well, if there's no dash behind sidestage-only app, dismissing it would be weird | 08:34 |
| Saviq | tsdgeos, if it's there, dismissing it just means you go back to the dash | 08:34 |
| tsdgeos | Saviq: i'm still sleepy, but i don't get it :D | 08:38 |
| tsdgeos | one is "dash should be besible when there's not main stage app" | 08:38 |
| Saviq | tsdgeos, sure, they don't really require one another | 08:38 |
| tsdgeos | other is "i want do dismiss dash search by clicking somewhere out of the dash search" | 08:38 |
| Saviq | tsdgeos, ooopf | 08:39 |
| Saviq | tsdgeos, wrong bug? | 08:39 |
| tsdgeos | maybe? | 08:39 |
| Saviq | tsdgeos, https://bugs.launchpad.net/unity8/+bug/1129290 | 08:39 |
| ubot5 | Launchpad bug 1129290 in Unity 8 "[SIDESTAGE] can't dismiss the sidestage through grab when only sidestage app is running " [Medium,Triaged] | 08:39 |
| Saviq | that's better ;D | 08:39 |
| Saviq | tsdgeos, sorry ;) | 08:39 |
| tsdgeos | ok | 08:39 |
| tsdgeos | those are more related | 08:39 |
| tsdgeos | i'm a bit scared that this is a bit harder than the usual bug and will at least need fixes in some non-unity8 layer | 08:40 |
| tsdgeos | but having a look | 08:40 |
| Saviq | tsdgeos, yeah, if you see it too complex, let me know | 08:41 |
| Saviq | mzanetti, btw, https://bugs.launchpad.net/ubuntu/+source/ca-certificates/+bug/1258286 | 08:51 |
| ubot5 | Launchpad bug 1258286 in nss (Ubuntu Trusty) "CAcert should not be trusted by default" [Undecided,New] | 08:51 |
| tsdgeos | Saviq: i don't think we can do the first one without ¿heavy? architectural changes. The Shell needs to be on top so it can catch left edge, right edge, but then if it is on top *and* we show the Dash, the Dash will be on top of the sidestage app, no? | 08:51 |
| tsdgeos | Saviq: which means, we need to make the Dash an app :D | 08:51 |
| Saviq | tsdgeos, well, or clip it | 08:51 |
| Saviq | tsdgeos, that'd be the only "quick" way I can think of | 08:52 |
| tsdgeos | hmmmm, ok | 08:52 |
| tsdgeos | can try to see if that works | 08:52 |
| mzanetti | Saviq: intersting... a bit sad tho too.. | 08:55 |
| Saviq | mzanetti, yeah, seems they tried to bite too big a cake | 08:55 |
| Saviq | mzanetti, or maybe that model is just incompatible with a trusted CA | 08:56 |
| Saviq | mhr3, hey, on https://bugs.launchpad.net/ubuntu/+source/unity-scopes-shell/+bug/1282249 | 08:56 |
| ubot5 | Launchpad bug 1282249 in Unity 8 "After the first preview, actions gets unresponsive" [Critical,Triaged] | 08:56 |
| Saviq | mhr3, what should I react to when you're replacing the preview? modelReset? | 08:56 |
| mzanetti | Saviq: ah, btw... I played around with the MWC image for a while yesterday: Previews have a bunch of issues. Are they known of should I report them? | 08:57 |
| Saviq | mzanetti, depends on the issue ;) | 08:57 |
| mzanetti | dark overlay at bottom missing | 08:57 |
| mzanetti | app preview's screenshots not scrollable horzontally | 08:58 |
| Saviq | https://bugs.launchpad.net/unity8/+bug/1281995 | 08:58 |
| ubot5 | Launchpad bug 1281995 in Unity 8 "Previews should have headers and "back" buttons at the top, and no "see through" at the bottom" [Medium,Triaged] | 08:58 |
| mzanetti | app preview's author name missing in header | 08:58 |
| Saviq | https://bugs.launchpad.net/unity8/+bug/1281709 | 08:58 |
| ubot5 | Launchpad bug 1281709 in Unity 8 "Preview with gallery widget and multiple images: image is cut off" [Undecided,Fix committed] | 08:58 |
| mhr3 | Saviq, no, i do remove all + insert | 08:58 |
| Saviq | mhr3, so what do I react to? | 08:58 |
| Saviq | mzanetti, that I don't know about, please file against unity-scope-click | 08:59 |
| mzanetti | ack | 08:59 |
| mhr3 | Saviq, i don't see why it wouldn't work | 08:59 |
| mzanetti | but for the rest it looks freakin awesome. Good job on the new scopes everyone! | 08:59 |
| Saviq | mhr3, it's not that | 09:00 |
| Saviq | mhr3, we're triggering a "please wait" layer | 09:00 |
| Saviq | mhr3, on action activation | 09:00 |
| mzanetti | (which btw confuses the shit out of mzanetti) | 09:00 |
| mzanetti | :D | 09:00 |
| Saviq | mhr3, and that needs to go away when the preview gets replaced | 09:01 |
| Saviq | mhr3, this was a way to avoid people tapping on the same action repeatedly | 09:03 |
| mzanetti | Saviq: here you go: https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/1282460 | 09:05 |
| ubot5 | Launchpad bug 1282460 in unity-scope-click (Ubuntu) "Wrong header in app previews" [Undecided,New] | 09:05 |
| Saviq | mzanetti, thanks, tweaked the title and description slightly | 09:08 |
| mzanetti | Saviq: doesn't this stuff use the Preview code I wrote any more? | 09:08 |
| mzanetti | Saviq: there should be an AppHeader.qml ready to be used | 09:08 |
| Saviq | mzanetti, nope, it's completely redone | 09:09 |
| mzanetti | ok | 09:09 |
| Cimi | Saviq, ciao | 09:10 |
| Saviq | Cimi, o/ | 09:10 |
| mhr3 | Saviq, ah, i see | 09:12 |
| Cimi | Saviq, remember the paste of yesterday? | 09:12 |
| Saviq | Cimi, my log does | 09:12 |
| mhr3 | Saviq, what if i gave you a new prop which would be "actionBeginProcessed" | 09:12 |
| mhr3 | Being* | 09:12 |
| Saviq | mhr3, sure, that would work | 09:13 |
| Cimi | Saviq, when you have time to give me feedback | 09:15 |
| Cimi | :) | 09:15 |
| Saviq | Cimi, btw, when you start with "something like that is ugly, right?" don't expect me to contradict you much ;) | 09:16 |
| Cimi | Saviq, I'm not expecting :) | 09:17 |
| Cimi | Saviq, if I were sure I wouldn't ask | 09:17 |
| Cimi | I don't know how to do it smartly, that's all | 09:17 |
| Saviq | Cimi, just combine CardTool and CarouselTool, no need for two separate components | 09:17 |
| Saviq | Cimi, otherwise looks fine | 09:18 |
| === shiznix_ is now known as shiznix | ||
| Cimi | Saviq, you told me you didn't want to have cardtool to switch category | 09:18 |
| Saviq | Cimi, better than having CardTool and CarouselTool separate | 09:19 |
| Saviq | Cimi, and I slept on it ;) | 09:19 |
| Cimi | hah | 09:20 |
| Saviq | mzanetti, did you get anywhere with the crash investigation? | 09:23 |
| mzanetti | Saviq: not really far... stack traces indicate its something in mir | 09:28 |
| Saviq | mzanetti, don't hesitate to include them guys | 09:28 |
| mzanetti | Saviq: what I noticed too: when unlocking the screen, it hangs for about 2 secs | 09:29 |
| mzanetti | which makes swiping away the greeter quite nasty | 09:29 |
| mzanetti | anyone already on that? | 09:29 |
| Saviq | mzanetti, no, didn't see that before | 09:30 |
| mzanetti | Saviq: I only see that with the complete mwc image (including sample data etc). didn't notice that before when I manually installed the ppas and branches | 09:31 |
| Saviq | mzanetti, might be that something gets refreshed | 09:31 |
| karni | Saviq: I have a question. How do you work on stuff that you want to test with new-scopes, but are intended to be merged to trunk? do you force new scopes and just work on the trunk branch? or do some rebasing-shenanigans? | 09:31 |
| mzanetti | Saviq: actually its worse: drag the greeter half away and release it so that it snaps back. on the last pixel it hangs already, printing this to the logs: http://paste.ubuntu.com/6964506 | 09:36 |
| mzanetti | so yeah. it seems to recreate all the scopes stuff when the greeter is locked. and because we only lock it after waking up (instead of before going to sleep) we notice the hanging when turning it on | 09:37 |
| Saviq | mzanetti, that would mean that if you wait for a few seconds after powering up, it'd be fine again, which it isn't | 09:44 |
| Saviq | mzanetti, I think the scopes refresh their data when unlocking | 09:44 |
| Saviq | mhr3, comment ↑? | 09:44 |
| Saviq | mzanetti, or actually yeah, both happens, really | 09:45 |
| mzanetti | Saviq: same if you have one app open and drag it a bit from left to right and back | 09:45 |
| mzanetti | it stutters, printing the same scope messages | 09:45 |
| mhr3 | Saviq, no, they don't refresh | 09:46 |
| mhr3 | Saviq, i see that i'm setting loaded to false as soon as the preview realizes that it will be replaced | 09:49 |
| mhr3 | should i be doing that? | 09:49 |
| Saviq | mhr3, think it's fine | 09:49 |
| mhr3 | Saviq, there will be a delay between that and it removing and adding new widgets though | 09:50 |
| Saviq | mhr3, that's fine, we can put the activity indicator on then | 09:50 |
| mhr3 | comes back to the activity indicator | 09:50 |
| mhr3 | Saviq, your tiny patch was broken btw | 09:51 |
| mhr3 | the one that added it back | 09:51 |
| Saviq | mhr3, it was incomplete, yeah | 09:51 |
| mhr3 | it was never displayed when the preview was loading | 09:51 |
| mhr3 | because of the fadeIn thing | 09:51 |
| Saviq | mhr3, does click scope report loaded=true yet? | 09:52 |
| === dandrader is now known as dandrader|afk | ||
| mhr3 | Saviq, no.. c++ lambda madness in there | 09:53 |
| mhr3 | tvoss will check :) | 09:53 |
| Saviq | mhr3, so no activity indicator yet anyway | 09:53 |
| mhr3 | right | 09:53 |
| mhr3 | Saviq, columns? :) | 09:54 |
| Saviq | mhr3, yeah, tweaking one other thing first | 09:54 |
| Cimi | Saviq, maybe we can rename cardtool into something else | 09:54 |
| Saviq | Cimi, no, leave it | 09:54 |
| Saviq | Cimi, it's all about cards | 09:54 |
| Saviq | karni, I usually merge new-scopes, then shelve the new changes, uncommit, unshelve, commit | 09:57 |
| * karni parses | 09:58 | |
| karni | gotcha | 09:58 |
| mhr3 | Saviq, https://code.launchpad.net/~mhr3/unity-scopes-shell/processing-actions/+merge/207384 | 10:00 |
| Saviq | mhr3, see, newlines in init list good! | 10:01 |
| mhr3 | Saviq, btw did rosie ping you yesterday? she said that pretty much everything card-related is using too large font | 10:01 |
| Saviq | mhr3, yeah, but implementing the whole ruleset is not trivial, still hope to get it in, though | 10:02 |
| mhr3 | ok | 10:02 |
| Saviq | mhr3, top-ack, or do we want to check that it works first? ;) | 10:04 |
| mhr3 | Saviq, tests say it does :P | 10:05 |
| Saviq | mhr3, I like it how we got away from libunity(-core) and things can actually happen now ;D | 10:09 |
| mhr3 | all hail my awesome unit tests! :) | 10:10 |
| mhr3 | not even scopes-api has so good end-to-end testing | 10:10 |
| mhr3 | which is sad really :P | 10:10 |
| Saviq | ;) | 10:12 |
| tsdgeos | Saviq: the clipping seems to work | 10:16 |
| tsdgeos | i only need to make sure i don't break everything | 10:16 |
| Saviq | tsdgeos, yup, awesome | 10:16 |
| tsdgeos | Saviq: i'm taking we still want disabled (and a bit opaque) when the sidesstage is shown? | 10:17 |
| tsdgeos | s/opaque/transparent | 10:17 |
| Saviq | tsdgeos, yeah, darkened and disabled, tapping on it should bring the dash in I think | 10:17 |
| Saviq | tsdgeos, as should dragging the sidestage away | 10:18 |
| tsdgeos | so tapping on dash "minimizes" the sidestage? | 10:18 |
| tsdgeos | ok, can try | 10:18 |
| Saviq | tsdgeos, FYI small addition bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/660 | 10:18 |
| tsdgeos | ah | 10:19 |
| tsdgeos | there's one thing i wanted to do yesterday | 10:19 |
| Saviq | mhr3, so where do I set the number of columns on? | 10:19 |
| tsdgeos | and i forgot | 10:19 |
| tsdgeos | the pageheader mousearea should be an abstractbutton | 10:19 |
| tsdgeos | so we get the haptic ZZZZZZ when clicking on it | 10:19 |
| mhr3 | Saviq, the preview stack, has widgetColumnCount prop | 10:19 |
| Saviq | tsdgeos, ah so we only get it on AbstractButton do we?! | 10:20 |
| mhr3 | or the preview model itself, since you don't care about the stack these days yet | 10:20 |
| tsdgeos | Saviq: think so yes | 10:20 |
| Saviq | mzanetti, ^^ there, reason for using AB instead of MouseArea ;P | 10:20 |
| Saviq | mhr3, I wonder if it should be higher, though... | 10:20 |
| tsdgeos | or we need to add the HapticsEffect | 10:20 |
| tsdgeos | ourselves | 10:20 |
| === dandrader|afk is now known as dandrader | ||
| Saviq | mhr3, like it's fairly static, and it'd be better if the previewModel knew it already before it's created? | 10:21 |
| tsdgeos | will try to get that done in a sec (AB thing) | 10:21 |
| mhr3 | Saviq, the model is created empty, so doesn't matter much | 10:21 |
| mzanetti | Saviq: heh | 10:21 |
| mzanetti | speaking of the haptics effect. Am I the only one that thinks it is waaaay to strong? | 10:22 |
| Saviq | it is ;) | 10:22 |
| mhr3 | Saviq, also, shouldn't orientation affect it? doesn't seem that static to me | 10:22 |
| Saviq | mhr3, I mean it spans all preview stacks/models | 10:22 |
| mhr3 | right | 10:23 |
| Saviq | mhr3, so either I store it somewhere and bind to all the preview stacks/models, or you do | 10:23 |
| mhr3 | then again previewstack is owned by qml and is therefore detached from everything else | 10:23 |
| Saviq | mhr3, mhm | 10:23 |
| Saviq | mhr3, ok | 10:23 |
| Cimi | do you know a search string that returns me 4-5 elements? | 10:24 |
| Cimi | always getting 8 :) | 10:24 |
| tsdgeos | Saviq: https://code.launchpad.net/~aacid/unity8/new-scopes-haptic-back/+merge/207390 | 10:27 |
| Saviq | tsdgeos, just push to new-scopes | 10:27 |
| Saviq | (next time) | 10:27 |
| tsdgeos | ok | 10:27 |
| karni | Saviq: how can I force new-scopes when I run from unity8 trunk? UNITY_FORCE_NEW_SCOPES=1 ./builddir/tools/unity-scope-tool doesn't work. | 10:28 |
| Cimi | Saviq, http://paste.ubuntu.com/6964688/ | 10:28 |
| Saviq | karni, you can't "really" force new scopes (i.e. you can't have Cards) | 10:29 |
| Saviq | karni, so only try* in trunk makes sense | 10:29 |
| Saviq | test*/try* | 10:29 |
| karni | oh ok.. | 10:29 |
| karni | thank you | 10:29 |
| Saviq | Cimi, MP please | 10:29 |
| karni | Cimi: in CardCarousel.qml, you remove CardTool { id: cardTool ..}, and in line 25,26 reference cardTool.* | 10:30 |
| mhr3 | hmm, should remove the FORCE flag now | 10:31 |
| Cimi | karni, the item cardTool is passed to DashRenderers | 10:31 |
| Cimi | karni, line 208 of that diff | 10:32 |
| karni | Cimi: I don't know the detail, but I guess what I said is not a problem then. | 10:32 |
| Cimi | karni, it is instantiated in genericscopeview | 10:32 |
| Cimi | karni, and sent in a different way | 10:32 |
| Cimi | works | 10:32 |
| Cimi | thanks though | 10:32 |
| karni | I see | 10:32 |
| mhr3 | Saviq, quickie https://code.launchpad.net/~mhr3/unity-scopes-shell/no-force-new-scopes/+merge/207398 | 10:40 |
| mhr3 | karni, fyi ^ | 10:41 |
| * karni looks | 10:41 | |
| karni | mhr3: ack | 10:41 |
| Saviq | mhr3, wanna check it out http://paste.ubuntu.com/6964770/ ? | 10:45 |
| mhr3 | Saviq, should work on desktop, right? | 10:47 |
| Saviq | mhr3, yup | 10:47 |
| mhr3 | if i stretch the window | 10:47 |
| Saviq | mhr3, works with scopes here | 10:47 |
| mhr3 | Saviq, in that case +1 :) | 10:47 |
| mhr3 | Saviq, i'm seeing odd crashes with scope-tool, not sure if it's just me | 10:50 |
| mhr3 | Saviq, maybe cause i don't have demo ppa's uitk? | 10:50 |
| mhr3 | Saviq, http://paste.ubuntu.com/6964780/ shortly after startup and things showing up | 10:50 |
| Saviq | mhr3, or thumbnailer | 10:51 |
| Saviq | mhr3, actually album art fetcher | 10:51 |
| Saviq | mhr3, the threaded thing didn't get merged yet | 10:51 |
| mhr3 | Saviq, no, shouldn't be, i didn't enable those scopes | 10:51 |
| mhr3 | got just scopes in my envvar | 10:51 |
| Saviq | mhr3, I only saw that trace with the music scope before | 10:52 |
| mhr3 | but it doesn't have the actual albumart code | 10:52 |
| mhr3 | those traces did | 10:52 |
| mhr3 | Saviq, i've actually seen it yesterday when playing with the activity indicator in previews | 10:53 |
| mhr3 | and your code is doing something similar | 10:53 |
| mhr3 | cause the previewModel can be null | 10:53 |
| karni | Saviq: I have to admit it's a bit of pain to work on trunk for Card background features. there's only tryCardTool in trunk, and all tryCard*stuff is in new-scopes. | 10:54 |
| karni | wait.. I did merge, I should rebuild | 10:55 |
| Saviq | karni, huh? | 10:55 |
| Saviq | karni, tryCard is there in trunk | 10:55 |
| Saviq | karni, I tested your changes on trunk yesterday | 10:55 |
| karni | oh, it is | 10:55 |
| karni | Saviq: that's all I needed ;) thanks.. | 10:55 |
| mhr3 | Saviq, eeeh, nevermind, it was the albumart thing, my env wasn't what i expected :/ | 10:56 |
| Saviq | mhr3, oof | 10:56 |
| Saviq | mhr3, columns pushed | 10:58 |
| mhr3 | yey! | 10:58 |
| mhr3 | Saviq, so, can we merge the albumart fix? | 10:58 |
| Saviq | tsdgeos, ↑ | 10:59 |
| Saviq | weren't you reviewing that? | 10:59 |
| tsdgeos | don't remember :D | 10:59 |
| tsdgeos | url? | 10:59 |
| Saviq | tsdgeos, https://code.launchpad.net/~jpakkane/unity8/threadfix/+merge/206948 | 10:59 |
| tsdgeos | ah | 11:00 |
| tsdgeos | i only complained about it | 11:00 |
| tsdgeos | that probably counts as reviweing | 11:00 |
| mhr3 | heh | 11:00 |
| mhr3 | it does :) | 11:00 |
| Saviq | tsdgeos, indeed! ;D | 11:00 |
| Saviq | mhr3, so, activity.visible: !loaded && !processingAction ? | 11:00 |
| Saviq | or no | 11:00 |
| Saviq | well, yes | 11:01 |
| Saviq | !(loaded || processingAction) | 11:01 |
| Saviq | no! | 11:01 |
| Saviq | loaded && !processingAction | 11:01 |
| tsdgeos | Saviq: ok, i'll review | 11:01 |
| mhr3 | Saviq, well... you can't interact with the preview when processingAction == true, and you want to know that it's being reloaded, so i'd just base it off loaded | 11:02 |
| Saviq | mhr3, "can't interact" - that's the thing | 11:02 |
| Saviq | mhr3, we need to prevent the interaction | 11:02 |
| Saviq | mhr3, and ideally show the user that it's busy | 11:02 |
| mhr3 | performing the action should be fast enough, getting the new preview is what will take longer | 11:03 |
| mhr3 | Saviq, so i'd say don't overcomplicate it | 11:03 |
| tsdgeos | there's something that's breaking the logging | 11:04 |
| Saviq | mhr3, "should be" | 11:04 |
| tsdgeos | and i get all the stuff in a single line :-S | 11:04 |
| Saviq | tsdgeos, scopes ↑↑ | 11:04 |
| Saviq | mhr3, scopes log errors with no \n | 11:04 |
| Saviq | and/or warnings | 11:04 |
| tsdgeos | Saviq: but why would that mess with console.log ? | 11:04 |
| tsdgeos | i.e. i'm getting | 11:04 |
| Saviq | tsdgeos, because console.log prints at the end of a line with no \n | 11:04 |
| tsdgeos | stagesOuterContainer.x 2129.573974609375stages.x 915.4135131835938stagesOuterContainer.x 2129.141357421875 | 11:04 |
| Saviq | tsdgeos, ah | 11:05 |
| Saviq | tsdgeos, not sure then | 11:05 |
| mhr3 | qWarning needs \n? | 11:05 |
| Saviq | tsdgeos, maybe your terminal got confused | 11:05 |
| Saviq | mhr3, no | 11:05 |
| mhr3 | if so then whoops :) | 11:05 |
| tsdgeos | might be | 11:05 |
| mhr3 | when no whoops | 11:05 |
| mhr3 | s/when/then/ | 11:05 |
| Saviq | mhr3, so, action means a road trip to the scope | 11:05 |
| Saviq | mhr3, don't you think that might take long enough that the activity indicator should kick in? | 11:06 |
| Saviq | mhr3, maybe after 50ms or so? | 11:06 |
| mhr3 | Saviq, i'd maybe start desaturing the buttons, wouldn't show the indicator | 11:06 |
| tsdgeos | Saviq: that change we're doing in the art thing | 11:06 |
| Saviq | mhr3, and something we can do *now*? ;P | 11:07 |
| tsdgeos | we're blocking the thread | 11:07 |
| mhr3 | Saviq, heh :) | 11:07 |
| mhr3 | Saviq, wasn't qml making that thing easy? :P | 11:07 |
| tsdgeos | but that's ok | 11:07 |
| Saviq | mhr3, weren't we supposed to work off of designs? :P | 11:07 |
| tsdgeos | since we're doing QQmlImageProviderBase::ForceAsynchronousImageLoading so it should be a different thread than the main one | 11:07 |
| mhr3 | Saviq, no dash designers, we do the design now ;) | 11:08 |
| Saviq | tsdgeos, yeah | 11:08 |
| tsdgeos | i honestly don't see why this would change anything | 11:08 |
| tsdgeos | since we change from blocking in non main thread | 11:08 |
| tsdgeos | to blocking in a thread that is not the not the main thread D: | 11:08 |
| Saviq | mhr3, ↑ explain | 11:08 |
| tsdgeos | but if you guys says it fixes it | 11:08 |
| Saviq | tsdgeos, it's not about blocking | 11:08 |
| Saviq | tsdgeos, it's about crashing | 11:08 |
| mhr3 | tsdgeos, it was spinning the loop of that thread | 11:08 |
| tsdgeos | Saviq: sure i know | 11:09 |
| * Saviq shuts up | 11:09 | |
| mhr3 | tsdgeos, which meant is started processing other events in that thread etc, and that could obviously horribly break something | 11:09 |
| tsdgeos | mhr3: i don't see why, since that thread is ad-hoc for image loading, i don't see why it would go wrong | 11:09 |
| tsdgeos | but if it fixes it, it fixxes it | 11:10 |
| tsdgeos | Saviq: are we still eventually going to rip that up of unity8 or? | 11:10 |
| Saviq | tsdgeos, yes, definitely | 11:10 |
| mhr3 | tsdgeos, well, you know it's just a bad idea to keep respinning the mainloop | 11:10 |
| Saviq | greyback, hey, alive yet? | 11:10 |
| tsdgeos | mhr3: but that is *not* the mainloop in there | 11:10 |
| tsdgeos | it's a separate thread for image loading | 11:10 |
| tsdgeos | so you're not respinning the mainloop | 11:10 |
| mhr3 | tsdgeos, but there is *a* mainloop processing events for that thread | 11:11 |
| mhr3 | which might be requests for another image | 11:11 |
| mhr3 | maybe it could break some implicit ordering of things, and qt didn't handle that | 11:11 |
| greyback | Saviq: yes yes | 11:11 |
| tsdgeos | mhr3: ok | 11:12 |
| Saviq | greyback, could you mark the bugs as Fix Committed for those that we have fixed for mwc? like https://bugs.launchpad.net/unity8/+bug/1281199 I believe? | 11:12 |
| ubot5 | Launchpad bug 1281199 in unity8 (Ubuntu) "Can only pull launcher out in landscape from top left on flo mwc image" [Critical,In progress] | 11:12 |
| greyback | Saviq: yep, just am pulling updates to confirm | 11:13 |
| Saviq | greyback, ok great | 11:13 |
| Saviq | thanks | 11:13 |
| karni | Saviq: I suppose that's ready for new-scopes, still needs tests for trunk https://code.launchpad.net/~unity-team/unity8/themeing-font-and-mascot/+merge/207282 | 11:13 |
| Saviq | karni, awesome, thaks | 11:14 |
| karni | Addressed your review comments | 11:14 |
| karni | :) | 11:14 |
| Saviq | karni, don't put internal properties on root objects in a component | 11:14 |
| karni | hrm | 11:15 |
| Saviq | karni, remember that root component's props are basically its API | 11:15 |
| karni | right.. | 11:15 |
| Saviq | karni, we usually put those somewhere lower down the hierarchy | 11:16 |
| Saviq | karni, or sometimes in a separate QtObject { id: priv } | 11:16 |
| karni | Saviq: you mean these should have been aliases? not sure how can I bind from Card without putting properties at CardHeader root | 11:16 |
| Saviq | karni, CardHeader they need to be there | 11:16 |
| Saviq | karni, but not in Card.qml | 11:16 |
| karni | oh | 11:16 |
| karni | now I understand. | 11:17 |
| Saviq | karni, one more thing: no mascot shape when summary and art is mapped, as we're putting those on a default, white background | 11:18 |
| karni | Saviq: for instance, inOverlay - making readonly is good, or should I hide it anyway? | 11:18 |
| Saviq | karni, if no user of Card needs access to it, hide it | 11:18 |
| karni | ack | 11:18 |
| Saviq | mzanetti, awesome, can you include Mir folks in the crash investigation? | 11:20 |
| Saviq | karni, so basically property bool useMascotShape: background.visible | 11:20 |
| mzanetti | Saviq: I'm just merging the right-edge unity-mir branch with trunk, seeing that ricmm fixed something in there related to crashing apps. will make sure it still happens with that, and then forward things to the Mir people | 11:20 |
| karni | Saviq: ack | 11:21 |
| Saviq | mzanetti, ok awesome, | 11:21 |
| Saviq | karni, so you probably don't even need the useMascotShape prop anywhere in card.qml, just bind CardHeader.useMascotShape directly to background.visible | 11:21 |
| karni | Saviq: just did that :) | 11:22 |
| Saviq | karni, and maybe s/useMascotShape/shapeMascot/ | 11:22 |
| karni | Saviq: fontcolor, I think I need. for the summary. make it readonly would suffice? | 11:22 |
| karni | or hide in QtObject ? | 11:22 |
| Saviq | karni, just put it on the Label | 11:23 |
| Saviq | karni, same with the functions | 11:23 |
| karni | ok :D | 11:23 |
| Saviq | karni, you put them on the root object, which means they're accessible to Card users, which they shouldn't be | 11:23 |
| Saviq | karni, just hide them in Label { id: summary } | 11:23 |
| karni | yep. thanks Saviq | 11:23 |
| Saviq | karni, and then bind for CardHeader | 11:23 |
| Saviq | karni, we're not breaking overlay are we:? | 11:24 |
| karni | yes, gotcha | 11:24 |
| Saviq | karni, we don't want summary to be white if overlay is enabled | 11:24 |
| karni | good catch :| | 11:24 |
| karni | I'll make sure that works. | 11:25 |
| mzanetti | Saviq: seems its fixed by now | 11:31 |
| mzanetti | it behaves somewhat odd still, but doesn't crash any more | 11:31 |
| * mzanetti pushes the merged branch to the mwc image | 11:32 | |
| karni | Saviq: I'm confident I'm not breaking overlay (summary is now grey even when overlay enabled), but I'm having trouble testing card-background: "color:///#112233" -- has JSON spec changed? | 11:38 |
| karni | do I need to make it card-background: { "type": "color", sth: "#aarrggbb" }? | 11:39 |
| karni | mhr3: ↑ | 11:40 |
| mhr3 | karni, do you have unity-plugin-scopes from yesterdya? | 11:40 |
| karni | I am up to date. I'm working on trunk, using tryCard | 11:41 |
| mhr3 | oh.. trycard.. that isn't going through the plugin | 11:41 |
| karni | mhr3: I assume it's in the archive | 11:41 |
| mhr3 | so saviq will know | 11:41 |
| === MacSlow is now known as MacSlow|lunch | ||
| karni | oh | 11:41 |
| karni | qml/Dash/Card.qml:48:41: QML QQuickImage: Protocol "color" is unknown | 12:01 |
| karni | maybe it's because this doesn't go through the plugin | 12:01 |
| mhr3 | karni, probably, use the object instead | 12:04 |
| mhr3 | {"type": "color", "elements": ["#fooo"]} | 12:04 |
| karni | mhr3: JSON spec doesn't say how to define color using an object. oh.. "elements", very intuitive ;D | 12:04 |
| karni | mhr3: thanks!! | 12:04 |
| karni | Not complaining, but not working well either. Can't see the background change. | 12:06 |
| tsdgeos | greyback: there is a bug in sidestage in which if i start an app and hide it before it is really shown, then the side stage autoshows, but is black, you aware of that? | 12:06 |
| karni | not complaining - in the logs, I mean | 12:06 |
| greyback | tsdgeos: you start MS or SS app? | 12:07 |
| tsdgeos | greyback: SS | 12:07 |
| tsdgeos | greyback: be on dash, start notes, swipe sidestage away when it appears before notes actually renders anything, SS autoreapers when notes is ready to draw, but all i have is a black rectangle | 12:07 |
| karni | ignore what I just said. that was horizontal card. | 12:08 |
| greyback | tsdgeos: logging a bug about it wouldn't hurt. TBH if you swipe away an app before it's fully shown, everything gets confused | 12:08 |
| tsdgeos | greyback: hmmm, ok, is there a way i can know that the app isn't really there yet? because i'm adding a "press outside SS minimizes SS if there's no MS", and maybe i can disable it if the SS app is not really there yet to minimize this effect | 12:09 |
| mhr3 | saviq, click has a branch that fixes the preview loading | 12:10 |
| greyback | tsdgeos: you could listen to ApplicationManager.onFocusedApplicationIdChanged (or maybe ApplicationManager.onFocusedRequested) - a focus change by Mir is only way you can tell if app has actually appeared | 12:12 |
| tsdgeos | :/ | 12:13 |
| karni | Saviq: FYI still no tests, but addressed all comments. https://code.launchpad.net/~unity-team/unity8/themeing-font-and-mascot/+merge/207282 | 12:21 |
| karni | Saviq: do we hope to decrease font size in Cards? I had to take back that 0.5 instead of 1 gu spacing, because the mascot didn't align left with the summary properly and, while the text did fit the Cards better, mascot and summary where not aligned. so, either we decrease Card font sizes (so that stuff looks better), or decrease the spacing/margins in Card.qml | 12:23 |
| karni | s/hope/plan :P | 12:23 |
| Saviq | karni, I was actually wanting to ask you, if you still had time | 12:32 |
| karni | whitespace (Failed) - could tell me where it fails :P | 12:32 |
| karni | Saviq: hit me | 12:33 |
| Saviq | karni, to try and get as much of https://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit?pli=1#heading=h.5uwk9y3j0hso in as possible | 12:33 |
| Saviq | karni, one thing I'm not sure of is whether this is meant to be static in a category or not | 12:33 |
| Saviq | i.e. based on mapping or content | 12:34 |
| * karni nods | 12:34 | |
| karni | Saviq: I'll try to ask katie | 12:34 |
| Saviq | karni, katie's not with us as of today :/ | 12:34 |
| Saviq | karni, try with rosie | 12:34 |
| karni | as of today? | 12:34 |
| karni | still on IRC lol | 12:34 |
| karni | too bad.. | 12:34 |
| karni | ok | 12:34 |
| karni | If only rosie was on IRC. I think it's not her "thang" | 12:35 |
| Saviq | karni, no, she's "rosie" :) | 12:36 |
| Saviq | karni, but I think we can assume it's meant to be per-category, not per-card | 12:36 |
| karni | Saviq: Should I defer the theming branch tests for later and focus on font sizes now? | 12:36 |
| Saviq | karni, yeah | 12:36 |
| karni | yes, I'd say the same | 12:36 |
| Saviq | karni, so into CardTool, then | 12:36 |
| karni | Saviq: ok. going for font work, and static per category | 12:36 |
| * karni nod | 12:37 | |
| karni | I know she's rosie, she's just on on IRC. | 12:37 |
| karni | or not in #canonical at least | 12:37 |
| Saviq | karni, ok, I misunderstood "thang" ;) | 12:38 |
| karni | I meant, not her thing ^^ | 12:38 |
| karni | oh hahahah | 12:38 |
| karni | yeah, I just read what I wrote. you could have misunderstood that easily :) | 12:38 |
| Saviq | mhr3, k | 12:41 |
| karni | pstolowski: so, we got response from victor. Want me to reply? basically, we want small cards and question is - if we can get the grooveshark in music scope in the image today | 12:44 |
| pstolowski | karni, ok, I'll MP my branch. I think it should be reviewed quickly, but whether it lands on the image today is not up to me | 12:46 |
| karni | ack | 12:46 |
| Saviq | karni, hmm, one last thing: I don't think we should have mascot shaped in overlay either | 12:49 |
| Saviq | karni, as that results in shape-in-shape again | 12:49 |
| mhr3 | karni, pstolowski, should be fine to land today | 12:49 |
| karni | mhr3: nice | 12:50 |
| karni | Saviq: good pointe about shape in shape | 12:50 |
| karni | will fix | 12:50 |
| Saviq | karni, thanks | 12:50 |
| Saviq | karni, other than that - looks great! | 12:50 |
| karni | \o/ | 12:50 |
| * Saviq looks at code now ;D | 12:50 | |
| karni | HAHAHAH | 12:51 |
| Saviq | karni, 8+ readonly property bool inOverlay: template["overlay"] === true is still on root Card object - move down please :) | 12:51 |
| Saviq | karni, it's only used in CardHeader by now, so again - no need for that prop outside of it? | 12:52 |
| karni | yes, no need. I guess I cleaned up stuff on the way, should have moved it. | 12:52 |
| karni | moving now | 12:52 |
| Saviq | karni, i.e. http://paste.ubuntu.com/6965233/ | 12:53 |
| Saviq | karni, that, and no shape in overlay, and I'm merging into new-scopes | 12:53 |
| karni | Saviq: ack | 12:54 |
| tsdgeos | Saviq: do you have time to play a bit with lp:~aacid/unity8/new-scopes-show-dash-app-running ? I think it works ok-ish (i.e. not worse than what we had that seems a bit broken imho) but a second set of eyes never hurts | 12:55 |
| Saviq | tsdgeos, will do | 12:56 |
| tsdgeos | ok, lunch! | 12:56 |
| karni | Saviq: pushed single commit to that MP | 12:56 |
| karni | wait.. did I remove that line or not xD | 12:57 |
| karni | le'sigh. fixed ;) | 12:57 |
| kgunn | mzanetti: Saviq read scrollback...do the mir guys need to hop on something ? | 12:57 |
| mzanetti | kgunn: yeah, still not completely fixed. | 12:58 |
| Saviq | kgunn, mzanetti, crash not triggered any more, though, right? | 12:58 |
| Saviq | so lower prio | 12:58 |
| mzanetti | Saviq: still happens :/ | 12:58 |
| Saviq | mzanetti, ah | 12:58 |
| mzanetti | Saviq: it recovers if you don't touch the device for 2 secs after an app crashes | 12:59 |
| mzanetti | Saviq: but if you do something that requires updating the screenshot in those 2 secs it'll still crash | 12:59 |
| mzanetti | so its a bit better than before, but still happening | 12:59 |
| mzanetti | kgunn: ^ | 12:59 |
| kgunn | is this phone or screen shotting in general ? | 12:59 |
| mzanetti | kgunn: phone | 12:59 |
| kgunn | mzanetti: there a bug ? may as well get someone looking | 13:00 |
| dandrader | greyback, tada, a simple fix for that side stage bug that doesn't touch anything else: https://code.launchpad.net/~dandrader/unity8/sideStage_lp1281088_take2/+merge/207430 | 13:00 |
| mzanetti | kgunn: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1281728 | 13:01 |
| ubot5 | Launchpad bug 1281728 in unity8 (Ubuntu) "Unity8 random freeze on demo image" [Critical,In progress] | 13:01 |
| greyback | dandrader: will look now. What else you at? Polishing animations? | 13:02 |
| Saviq | karni, no need to call getFontColor() twice, just bind summary.color? | 13:02 |
| dandrader | greyback, polishing those stage animaitons causes a mild maddening, hence the simple fix. :) but yeah, I could tackle them now | 13:02 |
| Saviq | summary.font.color I mean | 13:03 |
| greyback | dandrader: well I just don't want to clash with your work. I can focus on other things and leave the polish to you | 13:03 |
| karni | Saviq: pushed. with summary.color, though. | 13:03 |
| === alan_g is now known as alan_g|lunch | ||
| karni | I beleive it's the same thing? | 13:03 |
| dandrader | greyback, what are you working on | 13:03 |
| dandrader | ? | 13:03 |
| Saviq | karni, yeah, summary.color is correct | 13:03 |
| mzanetti | kgunn: who would be right Mir guy to help me out here? | 13:03 |
| karni | Saviq: phew. done then. | 13:03 |
| kgunn | mzanetti: wanna join #ubuntu-mir | 13:04 |
| greyback | dandrader: atm I am testing the side stage and getting familiar with its code again. But I can leave it to you if you want, there's other bits to do | 13:05 |
| Saviq | karni, CardHeader.inOverlay should be false by default, no? | 13:05 |
| greyback | dandrader: one tip I'll give you though: I suspect takes Mir >1 frame to react to an order from unity8. E.g. if unity8 asks mir to hide a surface, it may not happen in that frame, so you need to factor that in | 13:06 |
| karni | Saviq: it's always set from Card.qml, but should you consider that better default - just pushed it. | 13:06 |
| Saviq | karni, it's used outside of Card, too | 13:06 |
| karni | Saviq: you're right. in case it wasn't set. | 13:06 |
| Saviq | karni, we're using it in previews, too | 13:06 |
| karni | ah, ack | 13:06 |
| karni | so yeah, fixed. thanks for your comments :) | 13:07 |
| Saviq | let's hope that's all ;D | 13:07 |
| karni | haha | 13:07 |
| dandrader | greyback, yeah, I was working on waiting a bit (on the screenshot <-> live surface swap) until taking further actions, such as starting a drag, because of that. | 13:08 |
| dandrader | greyback, solves issues but the complexity grows, so it was tricky to refactor into something manageable | 13:08 |
| greyback | dandrader: I know | 13:09 |
| === MacSlow|lunch is now known as MacSlow | ||
| dandrader | greyback, well, I spent the whole week fiddling with the side stage code, so I'm pretty familiar with all its glory by now. | 13:12 |
| === gatox is now known as gatox_away | ||
| greyback | dandrader: keep at it then. I'll do other bits | 13:12 |
| mhr3 | saviq, how about doing this now? the limitation no longer applies http://paste.ubuntu.com/6965323/ | 13:12 |
| Saviq | mhr3, we still need something to do opacity=0 I believe | 13:13 |
| dandrader | greyback, ok | 13:13 |
| Saviq | mhr3, will add that when I do the activity indicator / processingAction soon | 13:13 |
| mhr3 | saviq, maybe, it looks ok without it thought | 13:13 |
| mhr3 | saviq, and it's pretty clever it pre-fetches the next preview | 13:14 |
| mhr3 | i was like whaaaat :) | 13:14 |
| Saviq | mhr3, ;D | 13:14 |
| greyback | dandrader: ok, confirm that fixes bug 1281088 <- the transition between the 2 apps isn't smooth | 13:14 |
| ubot5 | bug 1281088 in Unity 8 "From Gallery, open Camera. A blank Side-stage appears on top" [Critical,In progress] https://launchpad.net/bugs/1281088 | 13:14 |
| Saviq | mhr3, it shouldn't, actually ;) | 13:14 |
| mhr3 | lol | 13:14 |
| Saviq | mhr3, unless you move it | 13:14 |
| Saviq | mhr3, it has cacheBuffer: 0, so delegates outside of view should not be loaded | 13:15 |
| dandrader | greyback, didn't get it. you expected that fix to also make the transition between apps smooth? | 13:15 |
| mhr3 | saviq, i had a console.log there and it was fetching two when opening preview | 13:15 |
| mhr3 | anyway, /me food | 13:16 |
| greyback | dandrader: so in gallery, I tap camera. Gallery slides to the right, white rectangle scales to fill screen (camera placeholder). But then white rectangle removed, revealing gallery app. Few frames later, camera app appears. | 13:16 |
| dandrader | greyback, right | 13:17 |
| greyback | dandrader: so what white rectangle needs to stay on screen longer | 13:17 |
| dandrader | greyback, sure, but that's a whole different issue | 13:17 |
| greyback | dandrader: ok, just pointing it out :) | 13:17 |
| greyback | dandrader: I'm gonna merge that into new-scopes now | 13:18 |
| dandrader | greyback, ok. be back in a bit | 13:18 |
| === dandrader is now known as dandrader|afk | ||
| karni | Saviq: you mind if I surface component["foo"]'s as top level properties in Card? I just think that working with 'price' and 'title' is so much more convenient than writing compoent['price/title/etc'] each time | 13:21 |
| karni | that, or QtObject { id: priv } :) | 13:21 |
| karni | Saviq: I meant in CardTool | 13:21 |
| Saviq | karni, not top-level | 13:22 |
| Saviq | karni, somewhere down - fine | 13:22 |
| karni | ok | 13:22 |
| Saviq | karni, don't bother with prices yet | 13:22 |
| Saviq | karni, we need to redo it anyway | 13:22 |
| karni | plus, template, not component | 13:22 |
| karni | ok | 13:22 |
| Saviq | karni, just take title, subtitle, summary into account | 13:22 |
| * karni nod | 13:22 | |
| MacSlow | There's a build-dependency glitch with unity8 on jenkins. My builds fail because pbuilder-satisfydepends cannot resolve some libunity-mir packages... http://pastebin.ubuntu.com/6965379 Has anybody else seen this yet? | 13:27 |
| Saviq | MacSlow, huh, libmirserver12 is soo old | 13:30 |
| Saviq | MacSlow, which job? | 13:30 |
| MacSlow | Saviq, http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2356/ | 13:31 |
| Saviq | MacSlow, triggered without the local repo http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2361/console | 13:31 |
| Saviq | MacSlow, let's see how that goes | 13:31 |
| MacSlow | Saviq, ok... fingers crossed | 13:32 |
| Saviq | karni, fwiw qml/Dash/Card.qml: bad whitespace in line 143 | 13:33 |
| Saviq | karni, it does say which | 13:33 |
| karni | Saviq: make test tells you which line? | 13:35 |
| Saviq | karni, yes | 13:35 |
| Saviq | karni, it should | 13:36 |
| karni | That line looks legit to me, Saviq o_O | 13:36 |
| Saviq | karni, leading space/tabs? | 13:36 |
| === _salem is now known as salem_ | ||
| karni | maybe a tab crept in, and not visible | 13:36 |
| karni | I suck with vi :P | 13:36 |
| karni | Saviq: replaced whitespace with spaces, bzr diff tells me nothing changed | 13:37 |
| Saviq | karni, leave it ;) | 13:37 |
| karni | ack | 13:37 |
| Saviq | greyback, are you fixing bugs or only finding new ones? ;D | 13:37 |
| karni | haha | 13:38 |
| karni | hi cwayne | 13:38 |
| greyback | Saviq: both! | 13:38 |
| Saviq | cwayne, o/ | 13:38 |
| * Saviq is always scared when people wave straight when I join the channel :D | 13:38 | |
| karni | cwayne: btw - the work that I said I was involved in didn't even land. so, stuff will look better. I saw icons already there, so they only need to get card background color right ;) | 13:38 |
| karni | Saviq: hahah | 13:38 |
| greyback | dandrader|afk: can you give this a quick look https://code.launchpad.net/~gerboland/unity8/sidestage-fix-sidestage-popping-out | 13:39 |
| Saviq | karni, cwayne, they need assets... we just removed the shape-in-shape when on background, just to see that they have square icons there ;D | 13:39 |
| karni | Saviq: ideally, I guess, these icons should have transparent background. but as long as it matches card background, we'll be fine ;D | 13:40 |
| Saviq | karni, it won't ;) | 13:40 |
| karni | :O | 13:40 |
| karni | karni says not good x) | 13:41 |
| Saviq | karni, espeically when there will be gradients ;) | 13:41 |
| Saviq | karni, but there should be assets for all of that | 13:41 |
| Saviq | mhr3, did you guys ask for assets (icons) for scopes scope? | 13:41 |
| karni | we needz transparent backgrounds | 13:42 |
| mhr3 | saviq, yea, it's being fixed as we speak | 13:42 |
| karni | yeah.. gradien't won't work well | 13:42 |
| karni | I mean, the square icon on a gradient :P | 13:42 |
| mhr3 | s/fixed/collected/ | 13:42 |
| Saviq | mhr3, awesome | 13:42 |
| karni | mhr3: sweet | 13:42 |
| karni | Saviq: XS is small, XXS is x-small ? | 13:43 |
| Saviq | mhr3, we just merged no-shape-when-background and light-font on dark background | 13:43 |
| mhr3 | yey! | 13:43 |
| karni | \o/ | 13:43 |
| Saviq | karni, yes, go one level up with everything, we seem to have failed to agree on those names... | 13:43 |
| Saviq | which is kind of weird since we got them from design folk in the first place ;P | 13:44 |
| karni | ack. yes, that's what I thought from the beginnig. these fint sizees in the doc didn't make much sense to me. | 13:44 |
| Saviq | xnox, hey, is it ubuntu-settings-components missing something in debian/control or unity8 in its depends http://bazaar.launchpad.net/~ubuntu-settings-components-team/ubuntu-settings-components/trunk/view/head:/debian/control ? | 13:45 |
| Saviq | xnox, I'm getting qtdeclarative5-ubuntu-settings-components:armhf : Depends: qtdeclarative5-ubuntu-settings-components-assets:armhf (= 0.1+14.04.20131217-0ubuntu1) but it is not installable | 13:45 |
| cwayne | Saviq, karni cool,t hanks! | 13:45 |
| xnox | seb128: See Saviq ^ | 13:45 |
| seb128 | xnox, why me? | 13:45 |
| xnox | seb128: i don't deal with ubuntu-settings-components =) aren't you? | 13:46 |
| seb128 | xnox, not really, but I can have a look I guess | 13:46 |
| seb128 | xnox, I deal with ubuntu-system-settings | 13:46 |
| Saviq | xnox, seb128, FIGHT! | 13:46 |
| seb128 | lol | 13:46 |
| Saviq | xnox, seb128, I can do it, just need to know what ;D | 13:46 |
| seb128 | Saviq, when do you get that error? | 13:46 |
| * xnox sees no mud or oil pits around. | 13:46 | |
| karni | Saviq: hrm. titleLabel is already "small", subtitleLabel is already "x-small". only weights need adjusting? | 13:47 |
| Saviq | karni, potentially, yes | 13:47 |
| karni | ack | 13:47 |
| Saviq | seb128, trying to x-build unity8 | 13:47 |
| seb128 | Saviq, where? | 13:47 |
| xnox | Saviq: are you installing from proposed, and armhf build didn't finish/publish yet? (cause i386 builds arch:all and would finish earlier typically) | 13:47 |
| Saviq | seb128, in sbuild | 13:47 |
| xnox | Saviq: well pastebin the full log.... | 13:48 |
| seb128 | Saviq, what xnox said, feels like archs mismatch, i386 done but armhf not yet | 13:48 |
| xnox | Saviq: the most cryptic message is usually not the cause ;-) | 13:48 |
| Saviq | xnox, seb128, http://paste.ubuntu.com/6965470/ | 13:49 |
| Saviq | xnox, seb128, no, -assets is Arch: all I think | 13:49 |
| Saviq | and it's trying to get :armhf | 13:49 |
| Saviq | yeah | 13:50 |
| seb128 | Saviq, right, but it tries to get the exact same version | 13:50 |
| seb128 | Saviq, if i386 built first you get an -asset newer than your armhf binary | 13:50 |
| === dandrader|afk is now known as dandrader | ||
| Saviq | seb128, they're both from December... | 13:50 |
| seb128 | Saviq, qtdeclarative5-ubuntu-settings-components:armhf : Depends: qtdeclarative5-ubuntu-settings-components-assets:armhf (= 0.1+14.04.20131217-0ubuntu1) | 13:50 |
| xnox | seb128: he is cross-compiling without -proposed, thus there shouldn't be any arch-scew. | 13:51 |
| seb128 | right | 13:51 |
| Saviq | seb128, yeah, :armhf | 13:51 |
| Saviq | seb128, there is no -assets:armhf | 13:51 |
| seb128 | cross compilation issue | 13:51 |
| seb128 | xnox, it's for you :p | 13:51 |
| xnox | Saviq: seb128: it could be that -assets don't declare Multi-arch:foreign and thus it treis to install assets:armhf instead of assets:all | 13:51 |
| karni | Saviq: Regular - DemiBold, Medium - Normal, Light - Light | 13:51 |
| Saviq | I DID PING HIM FIRST DIDN'T I!?!?!? ;D | 13:51 |
| Saviq | karni, no | 13:51 |
| karni | Saviq: does that make sense? mapping to.. | 13:51 |
| Saviq | Regular == Normal, Medium == DemiBold, Light == Light | 13:51 |
| seb128 | Saviq, yeah, he tried to sneak out! | 13:51 |
| dandrader | greyback, looking at it now -> https://code.launchpad.net/~gerboland/unity8/sidestage-fix-sidestage-popping-out | 13:51 |
| xnox | seb128: yeah sorry about hat. | 13:51 |
| karni | Saviq: ack | 13:51 |
| greyback | dandrader: ta | 13:52 |
| seb128 | Saviq, but what he said, is the asset using "Multi-arch:foreign" | 13:52 |
| * seb128 checks | 13:52 | |
| Saviq | no it's not | 13:52 |
| xnox | Saviq: let me rebuild components-setting-assets locally here and check if that helps. | 13:52 |
| seb128 | that's the issue | 13:52 |
| karni | Saviq: god.. title weight depends on whether there's overflow to second line. that sounds like per-card behavior, but cards looking differently in same category.. that'd be weird. would it not? | 13:55 |
| Saviq | karni, it would | 13:55 |
| Saviq | karni, disregard that, assume always wrap | 13:55 |
| Saviq | karni, and comment on spec | 13:55 |
| karni | ack | 13:55 |
| Saviq | karni, we might need a "how many title lines are available" | 13:55 |
| * karni nod | 13:56 | |
| Saviq | karni, as a configuration option | 13:56 |
| Saviq | fginther`, hey, think we should drop the local repo from -ci jobs yet? | 13:58 |
| Saviq | xnox, seb128, pfft! https://code.launchpad.net/~xnox/ubuntu-settings-components/xcompile/+merge/207442 | 14:00 |
| Saviq | ah, 5 minutes ago ;p; | 14:00 |
| dandrader | greyback, looks good. works fine | 14:01 |
| greyback | dandrader: cool, please land? | 14:01 |
| dandrader | greyback, ? | 14:02 |
| greyback | dandrader: could you please push it to new-scopes? | 14:02 |
| dandrader | greyback, sure | 14:02 |
| greyback | thanks | 14:02 |
| cwayne | oooh stuffs getting pushed what is it | 14:03 |
| karni | Saviq: well.. that was less then expected. I went through font sizes and weights, and that's basically it heh: http://paste.ubuntu.com/6965516/ | 14:03 |
| Saviq | karni, lol really :D | 14:03 |
| karni | Saviq: unless you want me to look into "Padding between text baselines", though I initially got no idea how to approach that | 14:03 |
| karni | Saviq: yes xD | 14:03 |
| Saviq | karni, good, merging | 14:04 |
| karni | Saviq: ok, leaving it with you then. | 14:04 |
| Saviq | karni, no, I don't even understand the deal with baselines, so ;) | 14:04 |
| Saviq | karni, well, there's anchors.baseline | 14:04 |
| Saviq | karni, and baselineOffset | 14:04 |
| Saviq | karni, so it's relatively easy to do | 14:04 |
| karni | hrm | 14:04 |
| karni | I can try | 14:05 |
| xnox | Saviq: well i'm hoping landing robots will build that for me on armhf such that i can test it =) | 14:05 |
| Saviq | xnox, it works | 14:06 |
| Saviq | xnox, xbuilt here | 14:06 |
| Saviq | seb128, where test plan for system settings? | 14:07 |
| === gatox_away is now known as gatox | ||
| seb128 | Saviq, https://wiki.ubuntu.com/Process/Merges/TestPlan/ubuntu-system-settings | 14:08 |
| Saviq | seb128, care to add in https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0Au6idq7TkpUUdC05a2ZQSmgwU2NFYnJQOE9qMDRYa3c&usp=drive_web#gid=1 ? | 14:08 |
| seb128 | Saviq, the fix from xnox? | 14:09 |
| seb128 | oh, that table | 14:09 |
| seb128 | Saviq, sure | 14:09 |
| Saviq | seb128, no, links to your test plans / checklists | 14:09 |
| seb128 | Saviq, done | 14:10 |
| Saviq | seb128, thanks | 14:10 |
| seb128 | yw | 14:10 |
| Saviq | seb128, https://wiki.ubuntu.com/Process/Merges/Checklists/Ubuntu-Settings-Components https://wiki.ubuntu.com/Process/Merges/TestPlans/Ubuntu-Settings-Components see if that suits you please | 14:12 |
| seb128 | Saviq, +1 | 14:15 |
| Saviq | seb128, thanks | 14:15 |
| Saviq | seb128, as core dev, care to +1 https://code.launchpad.net/~xnox/ubuntu-settings-components/xcompile/+merge/207442 please? | 14:17 |
| xnox | Saviq: i am coredev.... why would that need any acking? | 14:17 |
| seb128 | Saviq, done | 14:17 |
| seb128 | xnox, peer review | 14:17 |
| Saviq | xnox, process | 14:17 |
| xnox | Saviq: i can just dput into the archive =) | 14:17 |
| Saviq | xnox, sure you can ;) | 14:17 |
| xnox | seb128: "peer review" -> "seb says i trust any xnox change re-crosscompile metadata" ?! =)))) | 14:18 |
| seb128 | lol | 14:18 |
| === alan_g|lunch is now known as alan_g | ||
| karni | Saviq: regarding font *sizes* - didn't you mention to mhr3 today everything was too large regarding font sizes in Cards? I mean.. Call subtitles in Scopes scope look like "Canonical...", literally all, which looks a bit meh. | 14:18 |
| seb128 | Saviq, I +1 it but I can't change the status | 14:18 |
| Saviq | seb128, nw, will just land it | 14:19 |
| seb128 | k | 14:19 |
| mhr3 | karni, yes, rosie yesterday said that everything is too big | 14:20 |
| mhr3 | karni, in cards, in previews... everywhere | 14:20 |
| karni | is xx-small defined? | 14:20 |
| karni | the constant, I mean. | 14:20 |
| karni | Saviq: re what mhr3 just said - can I be of help? the fact that souce matches design doc doesn't mean it looks right :| | 14:21 |
| karni | *source :D | 14:21 |
| Saviq | karni, mhr3, they couldn't agree on font sizes, so I'm sorry, but no, we won't be defining a new font size before it shows up in the SDK | 14:22 |
| karni | ack | 14:22 |
| Saviq | there is no xx-small font in the sdk, so we can't use it | 14:22 |
| karni | Saviq: you want me to talk to sdk team? | 14:22 |
| Saviq | karni, we did already, there was a font thread | 14:23 |
| Saviq | of which nothing came out | 14:23 |
| karni | do they even have an IRC channel ;D | 14:23 |
| Saviq | so no, not this week | 14:23 |
| karni | Saviq: I see :/ | 14:23 |
| karni | ok | 14:24 |
| Saviq | karni, #sdk on canonical | 14:24 |
| Cimi | Saviq, I think there's a bug in cardtool tests | 14:24 |
| Saviq | tsdgeos_, works fine, two things: still can't dismiss by grabbing, and dash scales down, which I don't think should happen | 14:24 |
| Cimi | Saviq, function updateAreas unset layoutArea.text | 14:24 |
| tsdgeos_ | Saviq: yeah i know about the grabbing, and i did like more the scale down than without | 14:24 |
| tsdgeos_ | helps understanding it's not going to work if you click on the button | 14:24 |
| tsdgeos_ | together with the opacity | 14:25 |
| Cimi | Saviq, which is needed by layoutSelector onSelectedIndexChanged | 14:25 |
| Saviq | tsdgeos_, I think it should be darkened, not transparented, btw | 14:25 |
| Saviq | tsdgeos_, that should be enough to suggest it's unavailable | 14:25 |
| Saviq | tsdgeos_, obviously the flickering of app screenshots doesn't help, but :/ | 14:26 |
| tsdgeos_ | yeah | 14:26 |
| tsdgeos_ | Saviq: ok, i'll try to do that | 14:26 |
| tsdgeos_ | and work on the grabbing | 14:26 |
| tsdgeos_ | which honestly not sure makes sense since you can just click | 14:26 |
| tsdgeos_ | but oh well | 14:26 |
| Saviq | Cimi, it wouldn't work at all if that was the case | 14:26 |
| Saviq | tsdgeos_, sure, but the handle is there, so... | 14:26 |
| Saviq | Cimi, onSelectedIndexChanged replaces layoutArea.text | 14:27 |
| Cimi | Saviq, but uses it | 14:27 |
| Saviq | sil2100, icanhassilo for row 50 please? | 14:27 |
| Cimi | Saviq, var current = JSON.parse(layoutArea.text); | 14:27 |
| Cimi | so JSON will parse empty string and complain, failing my test | 14:27 |
| fginther` | Saviq, ack, I'll work on that today for the unity8 stack | 14:28 |
| Saviq | fginther`, thanks | 14:28 |
| sil2100 | Saviq: sure ;) | 14:28 |
| Saviq | Cimi, as long as element is selected, it won't reset to empty | 14:28 |
| Saviq | Cimi, so if you need a new item in the layout selector, put it there | 14:29 |
| === larsu_ is now known as larsu | ||
| sil2100 | Saviq: assigned! | 14:31 |
| Saviq | sil2100, thanks! | 14:32 |
| === ken_ is now known as kenvandine | ||
| Saviq | karni, there is no such thing as template["subtitle"] | 14:35 |
| Saviq | karni, components["subtitle"] you wanted | 14:35 |
| karni | OUCH :/ | 14:36 |
| karni | Saviq: correct. also, I just learned on #sdk there's xx-small, trying it now | 14:36 |
| mhr3 | saviq, karni, can't we at least lower the sizes of things that can be lowered? | 14:37 |
| Saviq | karni, is there? | 14:37 |
| Saviq | karni, ah maybe it was XXL before | 14:37 |
| karni | yes. and stuff looks ridiculously small. | 14:37 |
| karni | let me show you | 14:37 |
| mhr3 | the running apps font size was using correct size, so clearly there must be some support | 14:38 |
| karni | Saviq: mhr3: http://ubuntuone.com/3z1RKtWhK9blHpfrxeqdqw | 14:40 |
| karni | is this better? | 14:40 |
| karni | note that Card title is smaller - so it's also smaller on Apps screen (app titles) | 14:40 |
| Saviq | sil2100, hum, it says silo ready landing-016, but the sheet for that is empty? | 14:41 |
| Saviq | karni, yikes ;? | 14:41 |
| mhr3 | karni, how about screenshot from the phone? | 14:41 |
| karni | been a while I tried ./run-on-device, will try | 14:42 |
| mhr3 | karni, just change the qml in /usr/share/unity8 :) | 14:42 |
| karni | or that | 14:42 |
| karni | heh ;] sure | 14:42 |
| Saviq | karni, won't work with mwc image, not enough space | 14:42 |
| karni | ack | 14:43 |
| * Saviq really needs to employ click chroots to cross-build and push to device... | 14:43 | |
| karni | np, will push the file manually | 14:43 |
| sil2100 | Saviq: let me see | 14:43 |
| Cimi | Saviq, https://code.launchpad.net/~unity-team/unity8/new-scopes.carousel-dinamic-fallback/+merge/207451 | 14:43 |
| sil2100 | Saviq: hmmmm | 14:45 |
| === MacSlow_ is now known as MacSlow | ||
| sil2100 | Saviq: bug bug, let me see why it can't do the assignment | 14:45 |
| sil2100 | Saviq: hah, found the bug | 14:45 |
| tsdgeos_ | dandrader: i'm working on https://code.launchpad.net/~aacid/unity8/new-scopes-show-dash-app-running/+merge/207446 (you said you wanted to sync with me, right?) | 14:47 |
| tsdgeos_ | Saviq: yeah darkening works nice too | 14:47 |
| dandrader | tsdgeos_, yeah, just skimmed through that branch | 14:47 |
| dandrader | tsdgeos_, shoudn't clash with my stuff | 14:48 |
| tsdgeos_ | nice | 14:48 |
| Saviq | tsdgeos_, good | 14:48 |
| sil2100 | Saviq: fixed ;) | 14:48 |
| Saviq | sil2100, thanks | 14:48 |
| sil2100 | Saviq: thanks for noticing! | 14:48 |
| Saviq | sil2100, difficult to not notice :D | 14:49 |
| Saviq | Cimi, can you look at why greeter selects phone wallpaper on tablet? | 14:49 |
| MacSlow | Saviq, without the local archive everything in the build failed... http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2361/console | 14:50 |
| Saviq | MacSlow, ugh | 14:51 |
| Cimi | Saviq, damn I don't have nexus 10 here | 14:51 |
| Saviq | MacSlow, let me fix | 14:51 |
| Cimi | Saviq, I'll have a look though in the code | 14:51 |
| MacSlow | Saviq, the build before... with the archive worked fine... except for the libunity-mir dependency not being met. | 14:52 |
| Saviq | Cimi, you should be able to find out on desktop, too (IIRC use a wrong value for your own wallpaper) | 14:52 |
| Saviq | MacSlow, yeah, my fault, should've removed the hook | 14:52 |
| sil2100 | ;) | 14:52 |
| Saviq | MacSlow, kicked again | 14:52 |
| MacSlow | Saviq, all hooks or just the "mbs_archive" one? | 14:53 |
| Saviq | MacSlow, mbs | 14:53 |
| dandrader | mzanetti, are those the unity8 crashes you're experiencing http://paste.ubuntu.com/6965783/ | 14:55 |
| dandrader | ? | 14:55 |
| Saviq | dandrader, no | 14:55 |
| mzanetti | dandrader: no. But I've seen this too. I think rebooting helps in this case | 14:55 |
| Saviq | dandrader, that's different, hwc gets confused, I think a restart should help (or a reboot) | 14:55 |
| dandrader | hmm, ok | 14:55 |
| Saviq | tsdgeos_, pretty! | 14:56 |
| === jhodapp|afk is now known as jhodapp | ||
| karni | mhr3: had to mwc-reflash my device, stuff broke. | 15:02 |
| karni | will try in a sec | 15:02 |
| mhr3 | karni, once you have it, just show it to rosie | 15:02 |
| mhr3 | she'll know if it's ok | 15:02 |
| * karni nod | 15:03 | |
| greyback | dandrader: https://code.launchpad.net/~gerboland/unity8/launcher-greeter-reenable <- mind a quick look, it's a one liner | 15:03 |
| mzanetti | greyback: this breaks authentication | 15:04 |
| mzanetti | greyback: if the greeter is password-protected, you can bypass it with the launcher this way | 15:04 |
| dandrader | greyback, one lines but pretty non-obvious | 15:05 |
| dandrader | liner* | 15:05 |
| mzanetti | greyback: from the top of my head, without trying it, I'd say "greeter.shown" needs to be replaced with "greeter.locked" | 15:06 |
| dandrader | greyback, so you found yourself a reviewer! :) /me moves along | 15:06 |
| greyback | mzanetti: good point, hadn't thought of that | 15:06 |
| Cimi | Saviq, how do I test it? | 15:08 |
| Cimi | Saviq, on the desktop | 15:08 |
| Saviq | Cimi, IIRC change your wallpaper to a broken one in dconf-editor | 15:08 |
| Saviq | Cimi, or hack the code so it doesn't read from gsettings | 15:08 |
| Saviq | or from accountservice, for that matter | 15:08 |
| Saviq | Cimi, GreeterContent.qml:43 | 15:09 |
| Saviq | Cimi, probably easiest | 15:09 |
| Cimi | Saviq, changes for me °_° | 15:15 |
| === daek_ is now known as Daekdroom | ||
| Saviq | Cimi, hmm :/ | 15:15 |
| karni | what now.. phablet-screenshot doesn't work? | 15:16 |
| Cimi | Saviq, I put greetercontent bg to default bg | 15:16 |
| Cimi | Saviq, and shell.qml I fixed backgroundGsettings | 15:16 |
| Cimi | Saviq, when I change dimensions, the bg changes for me | 15:16 |
| Saviq | cwayne, are you storing background for the guest user in accountservice by any chance? | 15:17 |
| Cimi | Saviq, I believe then that on the tablet it might be the *gsettings* bg stored that is broken | 15:17 |
| cwayne | Saviq, we're doing a dbus call to set it there, ya | 15:17 |
| Saviq | cwayne, for guest user, too? | 15:17 |
| cwayne | for the greeter you mean? | 15:17 |
| cwayne | well, we're leaving the guest user as default | 15:18 |
| Saviq | cwayne, right | 15:18 |
| Saviq | cwayne, somehow we're getting the wrong background for the guest user | 15:18 |
| tsdgeos_ | Saviq: so added the sidestage handle thing, it was pretty easy actually, so if you can give a final test, i'll commit to new-scopes (and i'll need a new bug to squash) | 15:18 |
| Saviq | cwayne, it's using the phone one instead of the tablet one | 15:18 |
| Saviq | tsdgeos_, will do | 15:18 |
| Saviq | tsdgeos_, next one for you would be https://bugs.launchpad.net/unity8/+bug/1281995 | 15:19 |
| ubot5 | Launchpad bug 1281995 in Unity 8 "Previews should have headers and "back" buttons at the top, and no "see through" at the bottom" [Medium,Triaged] | 15:19 |
| Saviq | tsdgeos_, for now go for "$scopename preview" in the header | 15:19 |
| tsdgeos_ | ok | 15:19 |
| Saviq | tsdgeos_, you might want to strip trailing "s" from $scopename | 15:19 |
| tsdgeos_ | brrrr | 15:20 |
| tsdgeos_ | ok will see | 15:20 |
| Saviq | sil2100, can you reconfigure silo 16 please | 15:21 |
| Saviq | mzanetti, could you https://code.launchpad.net/~saviq/unity8/sidestage-threshold/+merge/207260 | 15:21 |
| Saviq | cwayne, I *think* you're setting the guest background, too, which you should just not set at all | 15:24 |
| Saviq | cwayne, can you give me the dbus command for setting it? | 15:24 |
| karni | Saviq: mhr3: You've got mail. As far as I'm concerned, we have lots of space, and we'd have friggin tiny fonts, hardly readable. | 15:25 |
| karni | But hey, I'm not a designer ;D | 15:25 |
| cwayne | Saviq, i'm not setting the guest one | 15:25 |
| Saviq | grr | 15:25 |
| Saviq | cwayne, can you get me the dbus command? I want to see if it ends up there somehow | 15:25 |
| cwayne | dbus-send --print-reply --system --dest=org.freedesktop.Accounts /org/freedesktop/Accounts/User${uid_string} org.freedesktop.Accounts.User.SetBackgroundFile string:/custom/usr/share/backgrounds/$user.png | 15:26 |
| Saviq | cwayne, mterry, just noticed, btw, that on creation the correct background isn't loaded (the default is used) | 15:26 |
| Saviq | s/creation/startup/ | 15:26 |
| greyback | mzanetti: slightly embarrassing question: how do I get up a password on the greeter? | 15:27 |
| mterry | Saviq, curious. You mean you boot up the device and see the wrong background? | 15:27 |
| mzanetti | greyback: I think launching it with -p or -k should do the trick | 15:27 |
| mzanetti | Saviq: ack | 15:27 |
| Saviq | mterry, yeah, the default is shown instead of the "George Constanza" one | 15:27 |
| Saviq | mterry, (on the MWC image, that is) | 15:27 |
| greyback | mzanetti: launching unity8 with those switches? | 15:27 |
| mzanetti | greyback: yep | 15:28 |
| greyback | mzanetti: ok thanks | 15:28 |
| mzanetti | greyback: well, no | 15:28 |
| mzanetti | greyback: ./run has those | 15:28 |
| Saviq | mterry, what's more, if you "log in" with the guest user and lock again, you're back to the greeter for the first user, but with default background | 15:28 |
| cwayne | greyback, hey, is there supposed to be an update in demo-stuff for the blank sidestage thing? i'm not seeing any updates still | 15:28 |
| mterry | Saviq, does it ever correct itself? You say "on startup" like other times it does work | 15:28 |
| mzanetti | greyback: what it does is to LD_LIBRARY_PATH stuff | 15:28 |
| greyback | mzanetti: understood | 15:28 |
| mzanetti | greyback: I think run_on_device knows that too | 15:28 |
| mterry | Saviq, this is tablet? I'm seeing correct behavior on phone | 15:28 |
| greyback | cwayne: haven't kicked off a rebuild yet, but will do so now | 15:28 |
| Saviq | mterry, yes, tablet (you get multiple users on phone?) | 15:29 |
| mterry | Saviq, no, was just confirming that the background worked on boot | 15:29 |
| mterry | (on phone) | 15:29 |
| mterry | Saviq, I see. So on startup, wrong background, but if you switch users, it starts working | 15:29 |
| Saviq | mterry, yes | 15:29 |
| mterry | Saviq, I can look at that today | 15:31 |
| Cimi | Saviq, here | 15:34 |
| Saviq | Cimi, yeah | 15:35 |
| Cimi | tsdgeos_, so we have the previewrating widget, for input | 15:35 |
| Cimi | tsdgeos_, how shall I name the rating/review for display? | 15:35 |
| Cimi | tsdgeos_, I'm wondering if I named the rating widget incorrectly | 15:36 |
| Cimi | tsdgeos_, and should have been PreviewRatingReviewInput | 15:36 |
| Cimi | against PrevireRatingReviewDisplay | 15:36 |
| Saviq | mterry, while you're at it, see why/whether http://bazaar.launchpad.net/~unity-team/unity8/new-scopes/revision/671 was needed/correct | 15:36 |
| Cimi | or we have PreviewReviews etc | 15:36 |
| * Saviq is confused with all the bgs | 15:36 | |
| Cimi | feedback is appreciated from json authors, or from Saviq | 15:36 |
| Saviq | Cimi, it's only a name ;) | 15:36 |
| Saviq | Cimi, deal with it | 15:37 |
| Saviq | Cimi, FWIW, I don't like PRRI | 15:37 |
| Cimi | Saviq, I'll call PreviewCimi then, it's only as name | 15:37 |
| mterry | Saviq, on first blush, I don't like that change | 15:37 |
| Cimi | when you give a 5 star review the widget is called PreviewCimi, for 1-star reviews PreviewTsdgeos | 15:38 |
| mhr3 | karni, in apps it looks smaller than the design | 15:38 |
| Saviq | mterry, it helps ;) | 15:38 |
| mhr3 | karni, do we have something in between the current and your screenshots? | 15:38 |
| Saviq | mterry, I think what happens is that background settles on phone before the view is resized | 15:38 |
| * Cimi daily rain affects mental sanity | 15:38 | |
| mhr3 | karni, well.. camera shots :) | 15:38 |
| karni | mhr3: no, mine was 1 step lower | 15:39 |
| karni | :S | 15:39 |
| tsdgeos_ | Cimi: the UX document says PreviewRatingReviewInput and PrevireRatingReviewDisplay, i'd go with that unless the json is wildly different | 15:39 |
| Saviq | tsdgeos_, Cimi, let's not do "RatingReview", Rating or Review is enough | 15:39 |
| mterry | Saviq, I'm sure it does, hopefully it wasn't made willy nilly ;) But ideally shell.background should be the definitive background always | 15:39 |
| karni | mhr3: I agree Apps scope has larger titles in desicns | 15:39 |
| === alan_g is now known as alan_g|tea | ||
| karni | design visual | 15:39 |
| mhr3 | sigh | 15:40 |
| Saviq | mterry, well, it does settle on phone (I _think_ the "check if image from gsettings is correct" thing gets confused and overwrites the defaultBackground binding) | 15:40 |
| karni | We don't, but I think we *should* have something in between those sizes | 15:40 |
| tsdgeos_ | greyback: so with my new "show dash when only SS is shown", it happens that the running apps icon doesn't get updated correctly, any idea how i can fix that? | 15:40 |
| mzanetti | Saviq: re https://code.launchpad.net/~saviq/unity8/sidestage-threshold/+merge/207260 | 15:40 |
| mzanetti | I thought there should be a sidestage on N7? | 15:41 |
| Cimi | tsdgeos_, Review or Rating | 15:41 |
| Saviq | mzanetti, in landscape | 15:41 |
| Cimi | tsdgeos_, I'd say rating | 15:41 |
| tsdgeos_ | Cimi: Rating | 15:41 |
| Saviq | mzanetti, 1920/18 | 15:41 |
| Cimi | tsdgeos_, I'll rename current to RatingInput | 15:41 |
| Saviq | mzanetti, we just changed GU on N7 | 15:41 |
| Cimi | tsdgeos_, and add Display | 15:41 |
| mzanetti | Saviq: does it autorotate? | 15:41 |
| tsdgeos_ | Cimi: ok | 15:41 |
| Saviq | mzanetti, no | 15:41 |
| mzanetti | phew :D | 15:41 |
| Saviq | mzanetti, it's locked on landscape | 15:42 |
| Saviq | mzanetti, yeah ;) | 15:42 |
| karni | mhr3: lol, what a dirty, dirty thought. we have fontScale to scale fonts down in Carousel. we could scale these up in grid x))) | 15:42 |
| karni | like, 1.4 heheheh | 15:42 |
| Saviq | mzanetti, it'll be a ride with shell rotation and sidestage going on/off/on/off/on/off | 15:42 |
| mhr3 | karni, just don't tell Saviq ;) | 15:42 |
| Saviq | karni, DON'T | 15:42 |
| karni | buahahahahah | 15:42 |
| karni | Saviq: I wouldn't do that ;) | 15:42 |
| greyback | tsdgeos_: see Dash/Apps/RunningApplicationGrid - there's the updateScreenshots signal there, if you can fire that the icons will grab a screenshot | 15:42 |
| tsdgeos_ | greyback: ok, tx | 15:43 |
| karni | in any case. small is to big, x-small is to small. | 15:43 |
| mhr3 | karni, try (small + x-small) / 2 :) | 15:45 |
| Saviq | karni, top hint: phablet-screenshot foo.png | 15:45 |
| karni | hahahah you know that's not how it works ;D | 15:45 |
| karni | Saviq: what's what I did. black screenshots, bro :/ | 15:46 |
| Saviq | karni, N4? | 15:46 |
| karni | yes | 15:46 |
| karni | this used to work before | 15:46 |
| Saviq | karni, also, you forgot to take off the foil of your screen ;) | 15:46 |
| Saviq | karni, worked for me yesterday, not sure what's what, then :/ | 15:46 |
| karni | Saviq: lol. it's solely dev device, sits on my desk. so, yeah xD | 15:46 |
| Cimi | greyback, I need to set up that famous tomboy note | 15:46 |
| Cimi | (hah) | 15:46 |
| greyback | Cimi: :D | 15:47 |
| Cimi | tsdgeos_, free karma https://code.launchpad.net/~cimi/unity8/previews.PreviewRatingInput_rename/+merge/207476 | 15:48 |
| Saviq | didrocks, can you reconfigure silo 16 please (row 50) | 15:49 |
| Saviq | mzanetti, doubts about the threshold still? | 15:50 |
| mzanetti | Saviq: no daubts, but the checklist requires me to run it on the N10. still trying to turn it on (battery drained) | 15:50 |
| Saviq | mzanetti, pfft | 15:51 |
| mzanetti | :D | 15:51 |
| sil2100 | Saviq: reconfig still needed? | 15:52 |
| Saviq | sil2100, yes please | 15:52 |
| Saviq | pstolowski, what's the deal with bug #1281602 ? | 15:52 |
| ubot5 | bug 1281602 in unity-scope-mediascanner (Ubuntu) "When searching videoaggregator scope, local videos are shown at the bottom" [Undecided,In progress] https://launchpad.net/bugs/1281602 | 15:52 |
| karni | Saviq: I think the background luminosity 0.7 treshold for font color switch is not ideal, I may tweak that tomorrow. | 15:52 |
| Saviq | pstolowski, not unity8 but unity-scopes-shell, if anything | 15:52 |
| Saviq | karni, sure, let's see | 15:52 |
| === alan_g|tea is now known as alan_g | ||
| sil2100 | Saviq: doing | 15:53 |
| Saviq | sil2100, thank you | 15:53 |
| Saviq | sil2100, btw, ogra dput'ed -session into the silo, will the whole silo get copied over on merge? | 15:54 |
| Saviq | sil2100, or should it be added to "additional source packages to land"? | 15:54 |
| sil2100 | Into that silo? | 15:54 |
| sil2100 | Saviq: you need to mention it in the additional source packages to land | 15:55 |
| sil2100 | And I need to reconfigure it then ;) | 15:55 |
| sil2100 | Saviq: tell me once you do that and I'll reconfigure again | 15:55 |
| Saviq | sil2100, sorry, never done that before :) | 15:55 |
| Saviq | sil2100, done | 15:55 |
| Saviq | karni, you didn't have the auto-switch on your phone | 15:56 |
| Saviq | karni, it's white for weather channel | 15:56 |
| sil2100 | Saviq: no problem, reconfiguring :) | 15:56 |
| Saviq | karni, it looks fine | 15:56 |
| karni | Saviq: saw my last screenshot in the thread? grey font on orange groveshark is.. so-so. | 15:57 |
| sil2100 | uuuuh, bug! | 15:57 |
| karni | oh right | 15:57 |
| karni | d'oh xD thanks Saviq ! | 15:57 |
| Saviq | karni, it's white on grooveshark, too :) | 15:57 |
| karni | perfect | 15:58 |
| karni | Saviq: yeah, I didn't have the switch on my phone, my bad | 15:58 |
| mhr3 | karni, i was just checking the L, but for the twc blue i'm getting L = 0.274 | 15:59 |
| mhr3 | karni, so why is it grey? | 15:59 |
| karni | mhr3: you looking at the pics I sent? | 16:00 |
| mhr3 | karni, yea | 16:00 |
| karni | mhr3: I didn't have the switch on my phone, the code wasn't there | 16:00 |
| karni | Saviq just said it's white on TWC blue :) | 16:00 |
| mhr3 | oh... you should have said that | 16:00 |
| Saviq | mhr3, it's fine, twc and grooveshark is white | 16:00 |
| Saviq | mhr3, he did | 16:00 |
| karni | mhr3: I just posted another e-mail to that thread ;) | 16:01 |
| Saviq | mhr3, refresh your inbox ;) | 16:01 |
| mhr3 | ok, so just me being blind | 16:01 |
| Saviq | tsdgeos_, hmm I can't pull the sidestage in after dismissing it | 16:01 |
| karni | no, my bad, mhr3 :) I sent it later. | 16:01 |
| mhr3 | ok | 16:01 |
| tsdgeos_ | Saviq: oh, do you have to be able of doing that? | 16:01 |
| Saviq | tsdgeos_, yeah, and I actually am, if I dismiss with tap - only not when dismissed with handle | 16:02 |
| tsdgeos_ | i see | 16:02 |
| tsdgeos_ | ok, i'll check that | 16:02 |
| Saviq | tsdgeos_, I think it's only missing the clip, 'cause dash goes dark | 16:02 |
| Saviq | tsdgeos_, but app doesn't come in | 16:02 |
| tsdgeos_ | may be | 16:03 |
| mhr3 | karni, what size are the category headers using? | 16:03 |
| karni | mhr3: honestly, dunno. just joined my teams SU. | 16:03 |
| * mhr3 digs | 16:04 | |
| Saviq | MacSlow, AAARRGHHH, the hooks are not propagated to downstream jobs, you'll have to wait for fginther` to fix the jobs... | 16:04 |
| MacSlow | Saviq, yeah... just saw it too... | 16:04 |
| Saviq | MacSlow, your qmltests segfaulted, though... | 16:05 |
| MacSlow | Saviq, odd... it did not two builds ago today | 16:06 |
| Saviq | MacSlow, maybe a one-time issue | 16:06 |
| MacSlow | Saviq, I can't reproduce these failures locally | 16:06 |
| Saviq | MacSlow, the usual | 16:06 |
| Saviq | sil2100, uh oh, reconfigure failed? | 16:07 |
| sil2100 | Saviq: yes... looking into that ;/ | 16:07 |
| MacSlow | Saviq, I'm out of ideas what else I could to avoid this. | 16:07 |
| Saviq | MacSlow, leave it for now | 16:08 |
| Saviq | MacSlow, let's wait for the jobs to be reconfigured and for MWC to be past us | 16:08 |
| sil2100 | Saviq: phew, works now | 16:09 |
| Saviq | sil2100, oh good | 16:09 |
| fginther` | Saviq, is this beyond the change to remove the local archive? | 16:09 |
| Saviq | fginther`, not sure what you mean? we want to drop the local archive, since nothing interesting is getting built there... | 16:10 |
| fginther` | Saviq, right, I just want to make sure the isn't another change needed | 16:10 |
| Saviq | fginther`, not that I know of, no | 16:10 |
| Saviq | fginther`, we can drop notify-osd from the packages | 16:11 |
| fginther` | Saviq, ack, I'm working on that now | 16:11 |
| fginther` | ah | 16:11 |
| fginther` | right | 16:11 |
| Saviq | fginther`, so dropping local repo and the hook for it, and the test_packages notify-osd | 16:11 |
| karni | hehe. < jounih> karni: yeah after applying your files, hold on, we're doing some testing. We may need to increase the actual font sizes in the SDK | 16:11 |
| cwayne | wooo! | 16:16 |
| cwayne | no more blank sidestage | 16:16 |
| Cimi | tsdgeos_, for rating display | 16:19 |
| Cimi | tsdgeos_, listview or repeater? | 16:19 |
| Cimi | and/or flickable | 16:19 |
| tsdgeos_ | Cimi: depends, is the rating model finite or technically infinite? | 16:20 |
| Cimi | who knoes | 16:21 |
| Cimi | *knows | 16:21 |
| Cimi | mhr3, Saviq ^ | 16:21 |
| mzanetti | Saviq: hmm... your branch failed in jenkins with this: Use of uninitialized value $json in concatenation (.) or string at debian/scopes-recommends-generator line 17. | 16:21 |
| Saviq | Cimi, technically infinite | 16:21 |
| tsdgeos_ | Cimi: let's go with a list then | 16:22 |
| Saviq | Cimi, in theory you'll be able to press "see more" and load more reviews from the server | 16:22 |
| Cimi | Saviq, but how shall we pick up the reviews to display? | 16:22 |
| Cimi | ok | 16:22 |
| Saviq | Cimi, we won't | 16:22 |
| Saviq | Cimi, the remote side will | 16:23 |
| Cimi | we hope users won't review our apps :D | 16:23 |
| Saviq | Cimi, no, we'll only get the first few | 16:23 |
| Saviq | Cimi, and have a see more button that will load a few more at a time | 16:23 |
| Cimi | Saviq, so repeater is fine then? | 16:23 |
| Saviq | Cimi, I'd say yes | 16:23 |
| Saviq | tsdgeos_, ↑ | 16:23 |
| tsdgeos_ | ok | 16:23 |
| Saviq | Cimi, if the user loads 500 reviews (manually clicking 100 times on the button or so) | 16:24 |
| Saviq | Cimi, I don't think we should optimize for that use case ;) | 16:24 |
| mhr3 | saviq, karni, there's something weird, the category headers use fontSize: medium, and that is smaller than when using small in cards | 16:25 |
| karni | huh | 16:26 |
| mhr3 | or well.. similar at least | 16:26 |
| Saviq | mhr3, the differences are quite small there | 16:27 |
| karni | I need to run now :/ mhr3, feel free to leave msgs on IRC, I'll get back to you guys tom. morning, k? | 16:27 |
| karni | Saviq: Can we get back to font size fixes tom. morning? I need to run now :( | 16:27 |
| mzanetti | Saviq: hmm... your branch failed in jenkins with this: Use of uninitialized value $json in concatenation (.) or string at debian/scopes-recommends-generator line 17. | 16:28 |
| Saviq | karni, sure | 16:28 |
| mzanetti | do you know what that is? | 16:28 |
| Saviq | mzanetti, I know what that is, not why it happened | 16:28 |
| karni | Saviq: good stuff today :) have a good evening! o/ | 16:29 |
| Saviq | sil2100, did you manage to reconfigure our silo? | 16:29 |
| * mzanetti triggers a rebuild... all the other passed | 16:29 | |
| Saviq | mzanetti, you mean the threshold one? | 16:29 |
| mzanetti | Saviq: yep | 16:29 |
| Saviq | mzanetti, where https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1221/console ? | 16:30 |
| Saviq | mzanetti, I just see x failing here | 16:30 |
| greyback | mzanetti: https://code.launchpad.net/~gerboland/unity8/launcher-greeter-reenable makes more sense to you now? I tested it with locked greeter and it prevents launcher appearing, as it should | 16:31 |
| mzanetti | greyback: haven't tested it, but are you sure it still shows up on the phone now even if there is password protection? | 16:32 |
| mzanetti | greyback: I believe greeter should never be locked in that case, but better testing it | 16:33 |
| greyback | mzanetti: this is for tablet anyway | 16:33 |
| mzanetti | so this is not merged to trunk? | 16:33 |
| Saviq | kgunn, mzanetti, is anyone looking into the crash on mir side? | 16:33 |
| mzanetti | Saviq: alf | 16:33 |
| Saviq | mzanetti, ok great, thanks | 16:33 |
| greyback | mzanetti: nope, since your code will eventually replace the trunk, it's just enough to fix the bug for mwc IMO | 16:34 |
| mzanetti | greyback: well, I would merge trunk before proposing my branch to merge. so it would end up there too. | 16:34 |
| mzanetti | greyback: in any case. it looks good to me from what I know. haven't tested it | 16:34 |
| greyback | mzanetti: ok | 16:35 |
| mzanetti | Saviq: approved | 16:40 |
| Saviq | mzanetti, thanks | 16:40 |
| mzanetti | Saviq: JFI: uninstalling apport works around the crash | 16:40 |
| Saviq | mzanetti, HUH | 16:40 |
| Saviq | mzanetti, ah got it | 16:40 |
| mzanetti | seems apport tracing the crashing app delays some threads in mir so that the shell doesn't get the "appRemoved()" in time | 16:40 |
| Saviq | mzanetti, yeah, understood | 16:40 |
| mzanetti | and might request a screenshot for it | 16:40 |
| Saviq | mzanetti, we should be looking on SIGSEGV for it | 16:40 |
| tsdgeos_ | anyoen | 16:40 |
| Saviq | ME ME ME | 16:41 |
| tsdgeos_ | has an idea of why the tablet is shutting me down? | 16:41 |
| tsdgeos_ | Connection to 127.0.0.1 closed by remote host. | 16:41 |
| tsdgeos_ | Connection to 127.0.0.1 closed. | 16:41 |
| tsdgeos_ | makes debugging kind of hard | 16:41 |
| Saviq | tsdgeos_, "shuttting you down"? | 16:41 |
| tsdgeos_ | kicks me out of ssh | 16:41 |
| Saviq | tsdgeos_, mtp? | 16:41 |
| Saviq | tsdgeos_, should only happen once you connect | 16:41 |
| tsdgeos_ | happens randomly | 16:41 |
| tsdgeos_ | makes my life miserable | 16:41 |
| Saviq | tsdgeos_, mtp kicks in and restarts adb | 16:41 |
| Saviq | tsdgeos_, :/ | 16:42 |
| Saviq | mhr3, wanna check that out http://paste.ubuntu.com/6966317/ for preview activity indicator? | 16:43 |
| Saviq | mhr3, apps still stuck at !loaded for me | 16:43 |
| mhr3 | saviq, not landed yet | 16:43 |
| tsdgeos_ | and now mir being stupid again | 16:43 |
| tsdgeos_ | terminate called after throwing an instance of 'boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<boost::system::system_error> >' | 16:44 |
| tsdgeos_ | i don't care mate | 16:44 |
| Saviq | mhr3, kind of tricky to verify the multiple-preview issue | 16:44 |
| mhr3 | saviq, what do you mean by multiple preview issue? | 16:47 |
| Saviq | mhr3, https://bugs.launchpad.net/ubuntu/+source/unity-scopes-shell/+bug/1282249 | 16:48 |
| ubot5 | Launchpad bug 1282249 in Unity 8 "After the first preview, actions gets unresponsive" [Critical,Triaged] | 16:48 |
| mhr3 | ah | 16:48 |
| mhr3 | saviq, btw | pastebin -f diff ;) | 16:49 |
| mhr3 | *pastebinit | 16:49 |
| Cimi | Saviq, mhr3 there's something wrong | 16:51 |
| Saviq | mhr3, never worked for me | 16:51 |
| Cimi | Saviq, mhr3 rating display accepts half star rating | 16:51 |
| Cimi | but when you do a review, you can not set half star | 16:51 |
| cwayne | greyback, trying out the sidestage fix, working out well so far! | 16:51 |
| Saviq | Cimi, mhr3, right, we don't do average ratings this way, do we? | 16:52 |
| Cimi | half star rating in the json makes sense only if it's the average of all views | 16:52 |
| Saviq | Cimi, sounds like it can be dropped | 16:52 |
| Cimi | Saviq, I think we're showing all reviews here | 16:52 |
| Cimi | like, one by one | 16:52 |
| Saviq | Cimi, yeah | 16:52 |
| mhr3 | i'm ok to drop | 16:52 |
| mzanetti | tsdgeos_: fwiw, when we had the adb disconnect issue on the N10 in the beginning, sshing in through WiFi worked around that quite well | 16:53 |
| greyback | cwayne: let us know if other things break it, I find it unlikely everything is perfect | 16:53 |
| Cimi | Saviq, mhr3 I already have code for this, so I can leave it | 16:53 |
| cwayne | greyback, nothing is perfect :) | 16:53 |
| Cimi | Saviq, the rating branch supported half icons | 16:53 |
| cwayne | greyback, for example, launching a sidestage app still launches the mainstage app as well | 16:54 |
| greyback | cwayne: how'd you do that? | 16:54 |
| tsdgeos_ | mzanetti: ok, tx | 16:55 |
| tsdgeos_ | Saviq: i'm having quite pproblems on getting the thing back dragged in, what you are dragging is not the sidestage handle but something else (twoo many things in this right edge) | 16:55 |
| Saviq | cwayne, not launches | 16:55 |
| Saviq | cwayne, focuses | 16:55 |
| Saviq | cwayne, that's per design | 16:55 |
| cwayne | greyback, agh, trying to reproduce, and i cant get the launcher to come up | 16:55 |
| Saviq | cwayne, if you have *any* main stage app | 16:56 |
| Saviq | cwayne, it will be brought back whenever you focus any side stage one | 16:56 |
| cwayne | oh | 16:56 |
| cwayne | ok then | 16:56 |
| Saviq | cwayne, which might change with the whole side stage/right edge rework | 16:58 |
| Saviq | cwayne, and especially with tsdgeos_'s branch, which actually shows the dash when you only have a side stage app | 16:59 |
| cwayne | ooooh | 16:59 |
| * Saviq finds it difficult to type "unit"... y types itself at the end | 16:59 | |
| cwayne | is that gonna be in for mwc? | 16:59 |
| cwayne | Saviq, i do the same exact thing, lol | 16:59 |
| Saviq | cwayne, should land today even, maybe | 16:59 |
| Saviq | cwayne, was one last issue to fix | 17:00 |
| mhr3 | saviq, hmm general qml question - you're setting the Previews.Preview instance id to "preview", but that thing is inside a delagate, so there will be multiple of those, what is that id really pointing to? | 17:00 |
| Saviq | mhr3, in the scope of the current delegate it's pointing to that delegate | 17:00 |
| Saviq | mhr3, outside of it should be undefined | 17:00 |
| mhr3 | aaah, right all of it is inside the delegate | 17:01 |
| mhr3 | nevermind then | 17:01 |
| Saviq | mhr3, or maybe it might point at the delegate component, but I wouldn't bet on it ;) | 17:01 |
| Saviq | as pointing at a component... STOOPID | 17:01 |
| mhr3 | saviq, anyway, i'm not sure about waiting for loaded to be true before showing the preview, that completely defeats the purpose of earlypreviews (which we don't have yet :P) | 17:03 |
| Saviq | mhr3, ah, you didn't say that | 17:03 |
| Saviq | mhr3, so you think we should just show it empty, with the activity indicator | 17:04 |
| Saviq | mhr3, and fade in the individual widgets? | 17:04 |
| Saviq | until loaded, at which point we hide the indicator and disable the mouse area | 17:04 |
| mhr3 | i'm not sure we need the fade in tbh | 17:05 |
| Saviq | mhr3, we do | 17:05 |
| mhr3 | the images are fading in anyway | 17:05 |
| Saviq | mhr3, but other widgets aren't | 17:05 |
| mhr3 | and text doesn't really have to fade in | 17:05 |
| Saviq | mhr3, we can see | 17:05 |
| Saviq | mhr3, so no fade in for now | 17:05 |
| Saviq | mhr3, http://paste.ubuntu.com/6966424/ | 17:06 |
| mhr3 | see ^^, nice :) | 17:06 |
| * Saviq might have been using -f patch before... | 17:06 | |
| Saviq | or sthg | 17:06 |
| mhr3 | saviq, - fadeIn.start() | 17:07 |
| mhr3 | saviq, and also, mousearea.enabled: !previewmodel.processingAction | 17:07 |
| Saviq | mhr3, http://paste.ubuntu.com/6966430/ | 17:08 |
| mhr3 | eh, !! | 17:08 |
| Saviq | mhr3, ready was doing it before | 17:08 |
| Saviq | mhr3, dropped ready now | 17:08 |
| Saviq | ++preview. dammit | 17:08 |
| Saviq | http://paste.ubuntu.com/6966434/ | 17:08 |
| mhr3 | saviq, that was my point, don't mix loaded + processing | 17:08 |
| mhr3 | use only processing | 17:08 |
| Saviq | mhr3, so no activity indicator when loading? | 17:09 |
| mhr3 | saviq, no, no disabled buttons when not processing action | 17:09 |
| Saviq | mhr3, activity indicator === disabled buttons | 17:09 |
| Saviq | mhr3, unless we drop the activity indicator, buttons need to be disabled | 17:10 |
| mhr3 | hm, what if preview is slow to provide some last bit of data yet you already have button which you want to click? | 17:10 |
| Saviq | mhr3, then we need to move the indicator up to the header or something | 17:10 |
| Saviq | mhr3, so it's not IN YOUR FACE | 17:10 |
| mhr3 | meh, in your face is fine :) | 17:11 |
| Saviq | no it's not | 17:11 |
| mhr3 | but ok, whatev | 17:11 |
| Saviq | mhr3, if we move it up, suggesting there's more happening, but you'll still be able to interact | 17:11 |
| Saviq | mhr3, I'm fine | 17:12 |
| Saviq | mhr3, but with it in the center it says "you can't touch this" | 17:12 |
| mhr3 | i'm not sure where up you'd put it? | 17:12 |
| Saviq | mhr3, in the header (when we have it) | 17:12 |
| mhr3 | would look too empty if there's nothing in the preview yet | 17:12 |
| Saviq | mhr3, so maybe we need the ready: count > 0, too? | 17:12 |
| mhr3 | maybe that would be best | 17:13 |
| Saviq | mhr3, so while there's nothing in the preview, the indicator is in the middle | 17:13 |
| Saviq | mhr3, but while it's loading more, it will move to the header? | 17:13 |
| mhr3 | wfm | 17:13 |
| mhr3 | not sure design would be happy :) | 17:13 |
| Saviq | mhr3, we're design now, remember? | 17:13 |
| mhr3 | ah right :) | 17:13 |
| Saviq | mhr3, can we leave it as is for now | 17:14 |
| mhr3 | sure | 17:14 |
| Saviq | mhr3, since we don't have the header anyway | 17:14 |
| mhr3 | indeed | 17:14 |
| mhr3 | saviq, feel free to push | 17:14 |
| mhr3 | ralsina_, we need tvoss' preview-reply branch to land | 17:15 |
| mhr3 | ralsina_, otherwise click previews will show just an indicator | 17:15 |
| Saviq | indeed | 17:15 |
| ralsina_ | alecu: I am in a call, can you check what mhr3 said? | 17:18 |
| ralsina_ | mhr3: I don't know about that preview-reply branch... | 17:19 |
| mhr3 | it's approved and ready to land | 17:19 |
| mhr3 | needs to land | 17:19 |
| ralsina_ | mhr3: have a link? | 17:22 |
| mhr3 | ralsina_, https://code.launchpad.net/~thomas-voss/unity-scope-click/fix-preview-proxy-leak/+merge/207436 | 17:22 |
| ralsina_ | ok, did not recognize it as "preview-reply" :-) | 17:22 |
| ralsina_ | mhr3: ok, I'll land that silo | 17:23 |
| mhr3 | thx | 17:23 |
| pstolowski | Saviq, right, #1281602 is unity-scopes-shell | 17:23 |
| ralsina_ | mhr3: in about 15' sorry, on a big call | 17:23 |
| fginther` | Saviq, the job update to remove the local archive is complete | 17:30 |
| Saviq | fginther`, awesome, thanks | 17:30 |
| === robru_ is now known as robru | ||
| cwayne | Saviq, you need help testing something on flo? | 18:00 |
| Saviq | cwayne, non-mwc, though | 18:00 |
| Saviq | cwayne, we changed grid to 18px on flo | 18:00 |
| Saviq | cwayne, which resulted in multi-user greeter on portrait flo | 18:01 |
| Saviq | cwayne, so I'm fixing that | 18:01 |
| Saviq | cwayne, but in general it might be useful to see what's what | 18:01 |
| === alan_g is now known as alan_g|EOD | ||
| Saviq | cwayne, you could check with 18 (DPR 2.25) in /etc/ubuntu-touch-session.d/flo.conf | 18:02 |
| mterry | Saviq, so I have a branch for the background stuff | 18:52 |
| mterry | Saviq, regarding your commit changing shell.background to shell.defaultBackground. You did that because all the fake user's default backgrounds were being set to the real user's shell background? | 18:53 |
| Saviq | mterry, guest's were left alone, cwayne said | 19:20 |
| Saviq | mterry, and that's where the problem was | 19:20 |
| mterry | Saviq, guest's was left alone? I don't parse | 19:22 |
| mterry | Saviq, anyway, it's a harmless to good change. My branch will include it | 19:22 |
| Saviq | mterry, I meant that the user's whose background was wrong, it was not set in accountsservice | 19:39 |
| Saviq | mterry, so the default background was incorrect | 19:40 |
| mterry | Saviq, right. defaultBackground = shell.background means "take the default greeter background as the user's gsettings background", so all the tablet user's defaults would be the user's set shell background | 19:40 |
| Saviq | mterry, sure, but there isn't one set in gsettings | 19:41 |
| Saviq | mterry, or well | 19:41 |
| mterry | Saviq, then it should have fallen back to default... | 19:41 |
| Saviq | mterry, probably the default in gsettings is _phone | 19:42 |
| Saviq | mterry, so that's what was used | 19:43 |
| mterry | Saviq, well the change is harmless/fine. Using gsettings was just assumed to be closer to what the user wanted than the default in case we couldn't load normal user background. But default is fine too | 19:46 |
| * mterry is just working on test stuff for branch, then will propose | 19:46 | |
| Saviq | mterry, yeah, well, gsettings *should* indeed be used as default, but we need two keys - portrait and landscape | 19:48 |
| mterry | Interesting point. That is a future convergence problem | 19:49 |
| Saviq | mterry, indeed | 19:53 |
| Saviq | mterry, so yeah, don't include that change in your branch against trunk | 19:53 |
| Saviq | mterry, as it's incorrect in the long run | 19:53 |
| mterry | Saviq, if you like. It's not like it's super bad to have a slightly less optimal fallback wallpaper | 19:56 |
| Saviq | mterry, it is for customization | 19:56 |
| Saviq | mterry, when people override the default wallpaper in an .override | 19:56 |
| Saviq | mterry, and it's being ignored due to aspect ratio | 19:56 |
| mterry | Saviq, oh that's right, they have problems overriding accountsservice | 19:56 |
| mterry | They can do that via overrides too though... ah well | 19:57 |
| mterry | oh but not for background | 19:57 |
| mterry | stupid accountsservice | 19:57 |
| cwayne | we do a dbus call in an upstart script | 19:57 |
| mterry | Saviq, cwayne: https://code.launchpad.net/~mterry/unity8/ensure-selected-greeter/+merge/207548 | 20:18 |
| cwayne | oh, neat | 20:19 |
| cwayne | i was just gonna stage my way around that, but this works :) | 20:19 |
| Saviq | mterry, cheers | 20:20 |
| Saviq | cwayne, I'll test quickly and merge into new-scopes | 20:21 |
| cwayne | Saviq, neato, thanks | 20:21 |
| Saviq | mterry, works! | 20:36 |
| mterry | Saviq, awesome | 20:37 |
| mterry | kgunn, did I end up missing the merge window on alpha-greeter after all? Ah well | 20:52 |
| kgunn | mterry: sorry ? | 20:52 |
| kgunn | just need some context... | 20:53 |
| mterry | kgunn, I see the alpha-greeter unity-mir branch isn't merged. Last night you mentioned it needed a quick trunk-merge to make it into the landing silo | 20:53 |
| mterry | Just curious if it made it | 20:53 |
| kgunn | mterry: yeah...took in 3 other branches...1 of which broke unit test (1 of 2 :) | 20:54 |
| kgunn | so we're trying to chase a fix | 20:54 |
| kgunn | of course it only fails on the builders | 20:54 |
| kgunn | typical | 20:54 |
| kgunn | so its still in the silo | 20:55 |
| mterry | Fair enough | 20:55 |
| === redj_ is now known as redj | ||
| === fginther` is now known as fginther | ||
| === Debolaz is now known as Guest79143 | ||
| === cyphase_ is now known as cyphase | ||
| === salem_` is now known as _salem | ||
| ESphynx | Hey guys, is there a bug right now on Trusty about the Alt key/Release? It's seriously annoying that the 'Please Type your command' menu keeps showing up when I press alt to access my app's menus | 22:23 |
| ESphynx | it seems to behaves worst than in Saucy... | 22:24 |
| ESphynx | (I really don't get why one would want Alt to do this at all, can't you just type in the dash that pops up when you press the Start key?) | 22:24 |
| ESphynx | and by worst I mean it pops up even when I do 'Alt-F' | 22:25 |
| greyback | Saviq: I'm hoping you're not still around | 22:47 |
| cwayne | Saviq, i'm hoping you are -- we seem to have broken the apps scope | 22:47 |
| Saviq | greyback, cwayne, I'm lingering | 22:56 |
| Saviq | cwayne, what about the apps scope? | 22:57 |
| Saviq | greyback, where does unity-mir decide on surface width (main vs. side stage)? | 22:57 |
| cwayne | Saviq, its empty | 22:58 |
| greyback | Saviq: it doesn't decide surface width, the app itself in qtubuntu does. | 22:58 |
| cwayne | https://bugs.launchpad.net/ubuntu/+source/unity-scope-click/+bug/1282803 | 22:58 |
| ubot5 | Launchpad bug 1282803 in unity-scope-click (Ubuntu) "Apps scope empty" [Undecided,New] | 22:58 |
| greyback | Saviq: unity-mir only positions | 22:58 |
| Saviq | greyback, ohkay, so that's where we need to fix 60GU → 100GU for sidestage vs. no sidestage | 22:58 |
| greyback | Saviq: we're making sidestage 100GU in width? | 22:58 |
| Saviq | greyback, no, the threshold for it | 22:59 |
| Saviq | https://code.launchpad.net/~saviq/unity8/sidestage-threshold/+merge/207260 | 22:59 |
| Saviq | ricmm, we're talking here | 22:59 |
| Saviq | ricmm, but it's just placement, apparently qtubuntu decides on the size (why oh why...) | 23:00 |
| ricmm | Saviq: qtubuntu decides on the initial size as reported to the client | 23:01 |
| ricmm | but unity-mir can always resize | 23:01 |
| ricmm | sadly, theres no way to signal this resizing to the application (yet) | 23:01 |
| greyback | but it doesn't | 23:01 |
| ricmm | in a good world, the surface creation should signal back to the client process | 23:02 |
| ricmm | with the geometry it was created in | 23:02 |
| ricmm | by the displayt server / ap mnager | 23:02 |
| greyback | https://bugs.launchpad.net/bugs/1277228 meant I could not have unity-mir do it | 23:03 |
| ubot5 | Launchpad bug 1277228 in Mir "Shell overriding surface creation parameters width/height causes device reset" [High,New] | 23:03 |
| ricmm | wow, deviec reset | 23:03 |
| ricmm | whiny system | 23:03 |
| greyback | Saviq: src/platforms/ubuntu/ubuntucommon/screen.cc - check out the constants near the top | 23:04 |
| Saviq | greyback, already done | 23:04 |
| Saviq | ricmm, https://code.launchpad.net/~saviq/qtubuntu/sidestage-threshold/+merge/207569 | 23:04 |
| ricmm | why 100 ? | 23:05 |
| ricmm | to account for n7 depth | 23:05 |
| ricmm | ? | 23:05 |
| Saviq | ricmm, yes | 23:05 |
| Saviq | ricmm, n7 portrait is 106 wide at 1080 | 23:06 |
| Saviq | at 21 | 23:06 |
| Saviq | but anyway, | 23:06 |
| Saviq | side stage only makes sense when main stage is bigger than side stage | 23:06 |
| Saviq | and side stage is 40 | 23:06 |
| Saviq | so decided to go for 100 | 23:06 |
| Saviq | ricmm, does qtubuntu have citrain checklist? | 23:07 |
| ricmm | no | 23:11 |
| Saviq | ricmm, how do we land qtubuntu these days then? | 23:12 |
| ricmm | Saviq: what happens with N10 ? | 23:12 |
| ricmm | at 100 | 23:12 |
| Saviq | ricmm, nothing changes | 23:12 |
| ricmm | Saviq: I approve it and we give it to sergio, afaik | 23:12 |
| Saviq | ricmm, N10 is 128GU side | 23:12 |
| Saviq | wide | 23:12 |
| ricmm | ok | 23:12 |
| Saviq | 2560@20pxpgu | 23:12 |
| Saviq | cwayne, any idea what happened with the click scope? | 23:21 |
| cwayne | Saviq, apparently it's an ABI mismatch with u-d-m | 23:22 |
| Saviq | cwayne, indeed there's a newer version in distro | 23:23 |
| cwayne | mmcc is working on it | 23:24 |
| Saviq | cwayne, so we need a no-change rebuild of u-s-c | 23:24 |
| Saviq | cwayne, ok cool beanz | 23:24 |
| Saviq | nice timing, too! | 23:25 |
| cwayne | hm? | 23:26 |
| mhr3 | Saviq, go to bed! | 23:28 |
| Saviq | cwayne, on the last day ;) | 23:28 |
| cwayne | Saviq, heh, always :) | 23:28 |
| Saviq | mhr3, who says I'm not in one!? | 23:29 |
| mhr3 | Saviq, oh, then close the laptop! | 23:29 |
| cwayne | mhr3, he's got ya there | 23:29 |
| Saviq | mhr3, but I won't see! | 23:30 |
| mhr3 | Saviq, that's the point | 23:30 |
| Saviq | mhr3, ¡pfft! | 23:31 |
| mhr3 | Saviq, what's up with rightedge? | 23:31 |
| mhr3 | seems a bit old :) | 23:32 |
| Saviq | mhr3, seems the recipe doesn't get kicked | 23:33 |
| mhr3 | Saviq, failed 8hours ago | 23:34 |
| Saviq | ah maybe because it failed again | 23:34 |
| Saviq | conflicts yay | 23:34 |
| * Saviq fixes | 23:34 | |
| Saviq | robru, could you please reconfigure silo 16 (row 50) | 23:34 |
| Saviq | ? | 23:34 |
| robru | Saviq, on it | 23:36 |
| Saviq | robru, thanks! | 23:36 |
| Saviq | mhr3, uh oh, conflict bad | 23:36 |
| robru | Saviq, ok, it's reconned, please build | 23:37 |
| mhr3 | :/ | 23:37 |
| Saviq | robru, thanks | 23:37 |
| cwayne | Saviq, so are we setting the GU on flo to 18px? | 23:47 |
| Saviq | cwayne, yes | 23:47 |
| Saviq | cwayne, got a fix in the pipe | 23:48 |
| cwayne | Saviq, neato gang | 23:48 |
| Saviq | cwayne, yeah, it's much better like that indeed :) | 23:48 |
| cwayne | oh yeah, its like night and day | 23:48 |
| cwayne | it really is amazing how far the flo image has come in the last week | 23:48 |
| * greyback ears tingled, that almost sounded like a compliment | 23:50 | |
| cwayne | :D | 23:50 |
| cwayne | greyback, sidestage is miles ahead of what it was 2 weeks ago, good job :) | 23:50 |
| greyback | it has a bit more to go | 23:50 |
| greyback | and tbh, I think I've broken manta left edge a bit | 23:51 |
| Saviq | if only all that wasn't going away next week.. | 23:54 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!