[07:17] mzanetti, hey, it shouldn't be difficult to apply a different OOM score to the dash should it? see bug #1379296 [07:17] bug 1379296 in unity8 (Ubuntu) "unity8-dash should be excluded from app lifecycle management" [Critical,Triaged] https://launchpad.net/bugs/1379296 === dpm_ is now known as dpm [08:33] vesar: did JMulholland have a chance to try the bottom list thing? any input? [08:33] i think the wizard just crashed on me [08:34] when trying to select the language [08:36] _usr_sbin_unity-system-compositor.0.crash [08:42] MacSlow: do you know anything about a volume notification that appears one passing the first screen of the wizard? [08:42] feels pretty weird [08:43] Saviq: ack, can look into that today [08:43] mzanetti, too lat [08:43] e [08:44] mzanetti, https://code.launchpad.net/~unity-team/qtmir/dash-killed-less-likely/+merge/237915 [08:44] with a test and all! [08:44] * Saviq happy to know that he can still write some code (or at least copy-paste it :P) [08:45] hehe [08:45] congrats [08:48] tsdgeos, the welcome-wizard?! [08:48] MacSlow: yep [08:49] tsdgeos, did you press the any of the volume-keys? [08:49] MacSlow: i'm pretty sure i did not [08:49] MacSlow, tsdgeos, there's a bug [08:49] i've had that twice or three times already [08:49] Saviq: ah ok [08:49] ensureProcessABitMoreLikelyToBeKilledThanShellButNotMuch [08:49] tsdgeos, Saviq: oh... that's news to me [08:49] Saviq: bug # ? [08:49] bug #1379287 [08:49] bug 1379287 in indicator-sound (Ubuntu) "Blank "Volume" notification displayed at startup and when scrolling the indicator." [Undecided,Confirmed] https://launchpad.net/bugs/1379287 [08:50] update your sound indicator and it should be gone [08:50] Saviq, tsdgeos: ah... I'm pretty sure I know what's causing this [08:51] MacSlow, ted reverted the bubbles in i-sound trunk already, and has a new branch: https://code.launchpad.net/~ted/indicator-sound/synchronous-notification/+merge/237666 [08:51] Saviq, tsdgeos: my initial indicator-sound branch was reverted... yeah that [08:52] Saviq, tsdgeos: among the sync. notification/max-volume-warning work I'm also working on a new indicator-sound branch, which will avoid that from happening again [08:52] MacSlow, make sure you sync with ted's branch then [08:52] Saviq, yup [08:52] tsdgeos, you not working on the qmltest fail are you? [08:53] Saviq: the ShellWithPin::test_factoryReset ? no i'm testing touchOwnership for the last time [08:54] tsdgeos, kk, I'll look to fix the test then [08:58] tsdgeos, btw, I had a look at dash mem usage between pre and post more-in-mem... couldn't see a real difference... if anything it's using less memory now :P [08:58] :D [08:59] Saviq: ok, so i approved touchOwnership, do you want me to top approve or want someone to have another look? [08:59] tsdgeos, I think functional testing in silo will be enouh [08:59] enough [09:00] ok [09:00] there we go [09:21] mzanetti, I blame you https://code.launchpad.net/~saviq/unity8/fix-shellwithpin-test/+merge/237922 :P [09:21] Saviq: why would you do that? [09:22] mzanetti, blame you? because you reviewed mterry's branch which broke this [09:22] tablet-security [09:22] although somehow it passed in ci [09:26] Wellark, wasn't the "I see white before I see the SIM PIN dialog" supposed to be fixed already? === vrruiz_ is now known as rvr [09:35] Saviq: ask MacSlow [09:35] Saviq: my take on that is that let's leave it unfixed, as it will be a PITA [09:35] it's just a brief clitch [09:35] Wellark, well, *you* told me that it doesn't matter, because it will be fixed with your stuff ;P [09:36] Wellark, still, it's going to be the first thing you see on your phone when you turn it on... [09:36] Saviq: with my stuff we fixed the flickering while using the dialog [09:36] kk, we need to fix the flickering when opening, too [09:36] it used to flicker and sometimes break completely if you entered your pin wrong [09:36] Wellark, Saviq: such tiny glitches don't fit in the current notification-feature demand :) [09:36] yeah ok [09:36] MacSlow, not a feature ;P [09:36] Saviq: it's PITA to fix [09:37] and that too [09:37] MacSlow, can't we delay showing the notification until the model is loaded or something? [09:37] as the pin unlock is the only fullscreen snap decision [09:37] and yes, the clitch comes from the fact that the "fullscreen" property is not ready until the Loader finishes to load the pin-unlock snap decision [09:37] Saviq, I learned that one can't poke into models [09:38] Saviq, without really digging into it I can't tell if one could fix that or not [09:38] tsdgeos, I just saw him checking your implementation and expect him to get back to you with his comments soon. JMulholland^ [09:38] Saviq: there are just too many loaders involved [09:39] MacSlow, k, let's look into it once we got the more pressing things in [09:39] tsdgeos: We’ve taken a look, need to put together some feedback but impressive progress so far! [09:39] Saviq: +1 [09:39] Wellark, let's uninvolve them then ;P [09:39] Saviq, Wellark: yup [09:39] Saviq: well, why did you allow the unity8 tree to have Loaders everywhere? ;) [09:39] I hate the things [09:40] Wellark, because we like to be able to run apps [09:40] Saviq: just grep unity8 for "Loader" and see the results ;) [09:40] Saviq: I'm not saying they should be removed completely [09:40] Wellark, yes, again, because we like to be able to run apps [09:40] but I'm sensing a slight overuse ;) [09:41] anywayw [09:41] we are not even close to a point of starting to think such optimizations [09:41] so let's just have the loaders and acknowledge that we might have slight clitches [09:42] Saviq: btw, do we have any sane QML profiling tools already? [09:42] something that draws pretty pictures? [09:42] Wellark, you mean like QtCreator's profiler that's there from when we started? [09:43] Saviq: If I remember correctly the qtcreator profiler could not dig into the c++ side of the calls or anything? [09:43] Wellark, if you want C++ calls, there's any number of profiler tools out there [09:44] Saviq: both combined: the QML side and C++ side [09:45] Saviq: no dednick today? Do you know who I could assign bug #1369737 to? [09:45] not separately [09:45] Wellark, combined, no, not that I know of [09:45] bug 1369737 in unity8 (Ubuntu) "Send button does not enable until you press spacebar" [Critical,Confirmed] https://launchpad.net/bugs/1369737 [09:45] larsu, he's around, just went out to bring a friend to the emergency room.... [09:46] Saviq: oh, hope he's ok. I'll assign to him [09:46] larsu: are you close on the Final Solution of the toggle switches getting out of sync? [09:46] Saviq: thanks [09:46] Wellark: that's a question for dednick. We decided that he'll add a timeout on the switch and reset it if the service didn't change state [09:52] larsu, Wellark, there's a branch https://code.launchpad.net/~nick-dedekind/unity8/lp1336715.server-value-reassert/+merge/237822 [09:52] larsu: so this solution is not changing the semantics of the switches as seen from GAction API point of view? [09:53] Wellark: yes, the semantics are quite fine, the UI didn't react right [09:54] larsu: <3 === MacSlow is now known as MacSlow|hanout [10:16] Saviq: can you try to reproduce https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1251597 with http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7001/artifact/work/output/*zip*/output.zip ? [10:16] i can't [10:16] Ubuntu bug 1251597 in unity8 (Ubuntu) "Scope header sometimes appears in the middle of the scope" [High,Triaged] [10:17] tsdgeos, didn't try, but it makes sense, just link the bug'n'branch [10:17] ok [10:25] Cimi: https://code.launchpad.net/~unity-team/unity8/flickables-speed-workaround/+merge/234298 ? [10:25] larsu: https://code.launchpad.net/~larsu/unity8/stop-using-statusicon/+merge/234502 ? [10:26] tsdgeos, ouch [10:26] Saviq: i don't have a x-compile setup, but now that CI passes i guess we can just approve https://code.launchpad.net/~dobey/unity8/fix-cross/+merge/234818 ? [10:27] tsdgeos, well, not according to Colin... [10:27] tsdgeos, I'll have a chat with him it's good enough for now [10:27] ok [10:27] if [10:28] tsdgeos: should be fixed once https://code.launchpad.net/~larsu/ubuntu-ui-toolkit/icons/+merge/236497 lands [10:29] larsu: ok === MacSlow|hanout is now known as MacSlow [10:35] how do you handle showing times in the correct locale? messaging menu does it the wrong way according to bug #1372061 [10:35] bug 1372061 in Ubuntu Translations "SMS notification: time format not translatable" [Low,Triaged] https://launchpad.net/bugs/1372061 [10:35] it uses QDateTime::fromString() [10:35] the reporter suggest making the format string translatable, but that sounds wrong to me... [10:37] QLocale::toString() ? [10:41] larsu: QDateTime::toString ? [10:42] tsdgeos: sorry that's what I meant. [10:42] larsu: that one should work [10:43] tsdgeos: it doesn't respect locale according to the bug (which makes sense, we pass a fixed format into it) [10:43] larsu: well, don't :D [10:43] larsu: there's a version that does it right [10:44] tsdgeos: I think we did that because design... [10:44] larsu: if the available Qt::DateFormat don't suit you, then yes you'll have to make the string translatable [10:44] but someone from design should be told not to invent time formats [10:45] or at least if we're going to invent them [10:45] centralize them on the SDK [10:45] right, that makes a lot of sense [10:45] thanks for the info [10:47] oh look, I wonder who created a blueprint for that a year ago https://blueprints.launchpad.net/ubuntu-ui-toolkit/+spec/time-formatter [10:49] wohohoho [10:49] :D [10:58] Saviq: right, this is timeformatter.cpp in unity8 [10:58] Saviq: which already does the automatically-update-when-timezone-changes part [10:58] larsu, yeah, but shouldn't exist in unity8 at all [10:59] Saviq: unity8 is the only consumer of this right now. If we like the API and there's something else that wants to use it, let's move it to the toolkit [10:59] apparently we shouldn't yet, because it does the wrong thing with the format [10:59] ;) [11:00] larsu, sure, it does some wrong things, and some others - not at all [11:00] larsu, which is why it's not in the toolkit yet [11:05] Saviq: that's what I just said?! [11:05] larsu, possibly [11:06] do we have a color definition for the popver color in here? https://launchpadlibrarian.net/185198724/launcher_standard-menu.jpg [11:12] mzanetti, UbuntuColors.lightGrey I think? [11:12] mzanetti, actually probably not [11:12] mzanetti, since dash has f5f5f5 hardcoded [11:13] tsdgeos, fixed [11:14] Saviq: no f5f5f5 in UbuntuColors [11:14] lightGrey is 929292 === MacSlow is now known as MacSlow|lunch === dandrader_ is now known as dandrader [11:24] tsdgeos, do you know/recall what I have to pass to qmake in qtdeclarative-opensource-src-5.3.0 to have the tests built? [11:25] so it's developer-build on qtbase [11:25] not sure what in qtdeclarative [11:25] dandrader: you can also just go to the folder you want and run qmake [11:27] tsdgeos, I mean I did "apt-get souece qtdeclarative-opensource-src;mkdir qtdeclarative-build;cd qtdeclarative-build;qmake CONFIG+=debug ../qtdeclarative-opensource-src-5.3.0" [11:27] sure now go to tests and just run qmake there [11:27] or qmake CONFIG+=debug if you want [11:28] tsdgeos, no tests dir in there http://paste.ubuntu.com/8532319/ [11:28] dandrader: you're on the build dir [11:28] you need to run qmake over the source dir [11:28] or do [11:28] tsdgeos, I remember there was some magic variable I would pass to the qmake command to have the tests built as well [11:28] mkdir test_build [11:29] and run qmake there with ../omething/tests [11:29] dandrader: there probably is [11:29] tsdgeos, it took me quite a bit of research. but it was a while ago and I didn't write it down. so now I forgot it :/ [11:29] dandrader: anything against just running qmake? [11:30] Cimi: your merge is quite weird [11:30] Cimi: shows me changes in .po files [11:32] tsdgeos, I don't understand how the separate tests build dir would relate to the lib build dir [11:32] Holas [11:32] tsdgeos, but I will try it :) [11:34] tsdgeos, hmm, kinda works. but it only builds a very small percentage of the tests. it's missing the bulk of it, the so called "private tests" [11:35] tsdgeos, which are probably the tests that make use of the private apis [11:35] the search continues.... [11:36] dandrader: ah right, add private_tests to CONFIG and you should get them [11:41] tsdgeos, I redid... maybe was because was based on more things on memory [11:42] tsdgeos, nope :\ [11:42] Cimi: have you really merged unity8 and not some other branch? [11:42] tsdgeos, how I do now? I download the po for unity8 trunk and overwrite? [11:42] Cimi: just merge unity8 again :d [11:42] or that [11:43] tsdgeos, nothing to do :D [11:43] tsdgeos, I try to overwrite [11:44] err... why is there an activityIndactor above all tests now? [11:44] is that intentional? [11:47] yeah.... [11:47] makes them faster [11:47] .... [11:47] :D [11:47] mzanetti, it's to make waitForRendering(window) pass "immediately" [11:47] tsdgeos, is fine now! [11:48] mhm... [11:48] Cimi: good! [11:49] food === karni is now known as karni-afk === karni-afk is now known as karni === dandrader is now known as dandrader|afk === MacSlow|lunch is now known as MacSlow === dandrader|afk is now known as dandrader [13:15] * tsdgeos had an infection of tags [13:15] * tsdgeos nukes everything [13:15] careful in case you had been in contact with me branches [13:24] Has anyone else ever encountered an empty desktop with just a mouse cursor after an install of ubuntu 14.04? [13:30] Saviq: Cimi: so is https://code.launchpad.net/~unity-team/unity8/flickables-speed-workaround/+merge/237944 something we want for rtm or just unity9? [13:30] -1 [13:32] tsdgeos, rtm [13:32] Cimi: it has no critical bug linked [13:33] tsdgeos, well the toolkit has bug [13:33] well no bug at all [13:33] tsdgeos, not unity8... [13:33] tsdgeos, but it won't happen for rtm the toolkit [13:33] I highly doubt it... [13:36] Cimi: i'd still want a bug attached to it or something for everything we're going to land for rtm tbh [13:37] Zephyr1139: that often means that the unity plugin for compiz failed to start. You'd get more help from the folks in #ubuntu-desktop [13:37] tsdgeos, * Are there any related MPs required for this MP to build/function as expected? Please list. [13:37] * Did you perform an exploratory manual test run of your code change and any related functionality? [13:37] * Did you make sure that your branch does not contain spurious tags? [13:37] * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP? [13:37] * If you changed the UI, has there been a design review? [13:37] ops [13:37] https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1348557 [13:37] Ubuntu bug 1348557 in Ubuntu UI Toolkit "Make scrolling speed resolution independent" [High,Confirmed] [13:37] ahaha [13:39] tsdgeos, if is not rtm, I have to patch few of them... [13:39] Cimi: why added "import Ubuntu.Components 1.1" ? [13:39] tsdgeos, grid unit [13:40] Cimi: in qml/Dash/ScopesOverviewFavorites.qml ? [13:40] but you're right [13:40] why did it work? [13:41] tsdgeos, did I forget to add it here? [13:41] no no [13:41] i'm just asking why you added it [13:41] when it wasn't there before [13:41] but it's right that it's weird it did work before [13:42] tsdgeos, because it gets from the parent probably... [13:48] greyback, thanks. I'll ask there too. [13:55] Cimi: so your branch changes some of the defined values, i understand the benefit of having teh same everywhere, but those values maybe were there because they were better? what's your opinion [13:55] ? [13:55] tsdgeos, because who of us wrote then didn't know they could have done them resolution independent [13:56] them [13:56] me included (see carousel) [14:00] tsdgeos, linked to the bug report === mpt_ is now known as mpt [14:11] dednick: can you merge the expanded-panel-design branch? [14:12] tsdgeos: moved back to WIP. there are some problems with it still [14:13] dednick: ok [14:17] mterry, you know why I had to do more changes to fix the shell PIN test? [14:17] Saviq, I understand that the visuals were messed up, yeah. I'd like my tap() improvements to eventually make it in, but I'm fine with your branch instead. As long as the test gets fixed [14:18] mterry, already in my MP [14:18] mterry, it's not about the visuals even, it's the fact that it was unreliable still [14:18] Saviq, oh then I didn't understand that. I ran it a bunch and didn't see a failure [14:18] mterry, the popup got messed up, so the button inside it was messed up [14:18] mterry, yeah, well, it worked by chance :) [14:19] mterry, because the Popup tried to anchor.fill the root element [14:19] Saviq, sure, and it's a row so it can't [14:19] mterry, but the root element was a Row, that doesn't like anchoring horizontally [14:19] Saviq, but why was it clickable at all? [14:19] Saviq, I ran in xvfbtest mode, so I didn't see what it looked like [14:20] mterry, well, because it existed, and by chance had some size [14:20] mterry, in try, if you worked through the test, you could see stuff getting broken [14:20] mterry, and in test (no xvfb) the window just disappeared when Popup happened [14:23] jeez what happened with AP? :/ [14:24] don't act like you'd be surprised :P [14:24] ahah [14:32] @unity: I'm prepping a silo for unity8, qtmir, papi (?), let me know if there's anything that's not top-acked that should go in [14:32] Saviq: have my launcher-dconf branches? [14:33] Saviq, are you putting touchOwnership there? [14:33] dandrader, it's ACK'ed, so yeah [14:34] uuhh.. the kraken is about to be relased! [14:34] you never know, it's a huge one. might have scared you away :) [14:34] it should! [14:34] :) [14:34] :) [14:35] Saviq, release the Kraken! [14:35] ;) === karni is now known as karni-eod [14:39] lp:~mterry/unity8/greeter-profiles: 0.1.16 [14:40] Saviq, guh [14:40] Saviq, fixed [14:40] larsu, can you please run http://people.canonical.com/~msawicz/unity8/strip-u8-tags.py on your status-icon branch (and any local checkouts of unity8 you might have) [14:41] lp:~mterry/unity8/dbus-race-fix: 0.1.16 [14:41] and that's that [14:41] Saviq, did that tag sneak into trunk? I probably missed it on a merge from trunk [14:41] Saviq, dbus-race-fix tag deleted [14:42] Saviq: can I also just delete the local branches and check them back out? [14:42] mterry, yeah it did [14:42] larsu, sure you can :) [14:42] larsu, but the remote one you need to clear [14:42] Saviq: cool [14:43] Saviq: ah, right [14:43] Saviq, bzr has local hook support, right? Couldn't we whip up a hook to strip tags on push? [14:43] (for dev local machines) [14:43] mterry, it doesn't have a push hook [14:43] I realize having jenkins do it is harder [14:43] mterry, just a commit hook [14:43] Saviq, bummer [14:44] mterry, but yeah, we have a plan for a pre_push script or so [14:44] Saviq, even so... the latest version of strip tags is very fast. I probably wouldn't particular notice if it was in the commit hook [14:44] Saviq: pushing the branch after stripping the tags doesn't seem to work: No new revisions or tags to push. [14:45] larsu, you need to point the script at the remote branch [14:45] larsu, bzr doesn't keep history of tags [14:45] larsu, they're viral === dandrader is now known as dandrader|afk [14:45] which is why we're in this state in the first place :| [14:46] how do I do that? [14:46] larsu, on the command line [14:46] ah, the script accepts a branch param [14:46] larsu, yeah, a list [14:47] larsu, yeah, I know, it should complain if you don't, and it should have a --help, but you know, it was only supposed to live for a week :P [14:47] right [14:50] okay, done. [14:50] larsu, thanks [14:51] tsdgeos, we should bump qtmir in https://code.launchpad.net/~dandrader/qtmir/UbuntuKeyboardInfoQMLSingleton/+merge/236151 should we not? and depend on it in u8? [14:51] dandrader|afk, when you're back ↑ [14:54] Saviq: hmmm [14:55] probably === dandrader|afk is now known as dandrader [14:57] Saviq, ok, will do that now [14:59] @unity: here's my list, if something's missing please let me know http://pastebin.ubuntu.com/8533419/ [15:01] Saviq: the launcher-dconf one has related in the description... they are not *really* necessary, but would be nice to land together [15:02] Saviq, no greeter-profiles? [15:02] mterry, just discussing with ted [15:05] dandrader, sorry, touchOwnership will have to wait for another round, mterry's ↑ branch is ready to go with some indicator changes, and for yours we need to wait for Mir to unblock qtmir [15:05] Saviq bumped qtmir version as requested [15:05] Saviq :-( === dandrader is now known as dandrader|afk [15:17] dednick, https://code.launchpad.net/~nick-dedekind/unity8/lp1336715.server-value-reassert/+merge/237822/comments/583596 please [15:17] dandrader|afk, and this is fixed by your request, too https://code.launchpad.net/~saviq/unity8/fix-shellwithpin-test/+merge/237922 [15:17] Saviq: you mean in the proposal, or in the code? [15:17] dednick, code better [15:18] I knew it [15:20] dandrader|afk, ah you did abstain [15:20] as you were, then [15:22] Saviq: done. [15:23] dednick, thanks [15:23] although my spelling suck. one miunute [15:23] meh [15:23] dednick, it does ;) [15:24] dednick, and why 1500, that a magic value that Just Works™? and can't we get a "fuck off" signal from indicators instead and reset the value then? [15:25] Saviq: 1500=magic. and no, it would seem that we can't [15:25] since it's just dbus activations [15:26] :/ [15:28] larsu: is there a hardcoded timeout for unifying the action state change updates into a single signal? [15:29] larsu: re the state change problem we are seeing with activations. [15:30] larsu: as in -> state change true ->false -> true -> false -> true -> false" sends only one signal after x seconds? [15:53] hey tedg, can you please help us how to change role for audio? comment #4 https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1378920 [15:53] Ubuntu bug 1378920 in unity8 (Ubuntu) "Synchronous sound notifications pause video" [High,Triaged] [15:53] Cimi, I think rsalveti is the guy you want. [15:54] You tag the stream in GStreamer, but there may be a Qt API for it as well. [15:54] yeah, we can do that on Qt or QML [15:55] rsalveti, how? :) [15:55] rsalveti, so far, from what I can see in the code, we have just an Audio qml component [15:55] mzanetti: something broke in the launcher quicklist with utf chars [15:56] audioRole: MediaPlayer.alert [15:56] on the Audio component [15:56] we have that on system-settings [15:56] i get CA!mara [15:56] instead of Cámara [15:56] hmm [15:56] tsdgeos: and idea when it happened? [15:56] s/and/any/ [15:56] mzanetti: nope :/ [15:56] ok. will find out [15:56] thanks [15:57] tsdgeos, We made camera more exciting by adding ! [15:59] mpt, On your high volume mock up have a "headphones" label in the sound indicator, which we don't do today. [15:59] mpt, I mean the label over the slider. [16:00] mpt, I don't see that elsewhere in the spec though [16:01] tedg, https://wiki.ubuntu.com/Sound#notification [16:01] The visual design and wireframe+text aren’t in sync yet, sorry [16:02] mpt, Yes, I was meaning on the menu: https://wiki.ubuntu.com/Sound?action=AttachFile&do=get&target=sound-settings-volume-high.phone.png [16:02] mpt, Not the notification [16:02] oh, sorry [16:03] rsalveti, do we patch qt? [16:03] tedg, I’ll need to discuss that with Patti, but tbh that’s a bit down my priority list [16:04] mpt, Oh, wait, is that supposed to be a system settings screenshot? [16:04] rsalveti, I see this audioRole in system settings, but cannot find the property on qt doc [16:04] tedg, yes, System Settings [16:05] tedg, that’s why it continues onto “Phone calls:…” :-) [16:05] mpt, So then in the indicator we just need the caption, not the lable. [16:05] label [16:05] yep [16:05] mpt, Yeah, just noticed that. :-) [16:05] K, cool. === dandrader|afk is now known as dandrader === gatox is now known as gatox_lunch [17:07] larsu: hey. know much about g_dbus_action_group_get ? :) [17:20] Cimi: part of qt multimedia [17:21] Cimi: if you want to do in qt itself, you can check how that was done in telephony-service [17:21] yeah, the api is something we added, not upstream yet === gatox_lunch is now known as gatox === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader