=== _salem is now known as salem_ === salem_ is now known as _salem === jasoncwarner_ is now known as jasoncwarner === iahmad is now known as iahmad|afk [08:16] pstolowski: ping [08:16] tsdgeos, pong [08:16] see the first change at https://code.launchpad.net/~stolowski/unity8/category-reordering/+merge/189263 [08:16] you're adding an include that's already there :D [08:17] like, two lines above ;-) [08:17] tsdgeos, indeed :) thanks [08:20] tsdgeos, pushed [08:24] pstolowski: const & the vector of onCategoryOrderChanged to avoid copying the uints all the time? [08:26] tsdgeos, agree, I'm not sure why didn't I do it :/ [08:29] pstolowski: on the qWarning("beginMoveRows failed"); thing, maybe add old_pos and target_pos? i don't think this will ever fail by reading the code of beginMoveRows, but if it fails, we'll want to know what the indexes were [08:33] pstolowski: also i don't like this change [08:33] http://paste.ubuntu.com/6212868/ [08:34] it's just making things slower [08:34] since basically contains has to look for the thing in the set [08:34] which is what << does already [08:34] so if it's not there [08:34] you end up doing "two searches" [08:37] otoh contains does not detach and << always detaches even if not necessary [08:39] so not sure anymore :D [08:39] feel free to ignore [08:40] * tsdgeos feels like he's talking to himself [08:40] hiya [08:41] how much work would it be to create a desktop session for unity8? or what kind of work would be involved? [08:49] dholbach: we don't do much of the typical "desktop management" on the desktop, so not sure it'd be much useful [08:50] Cimi, ping [08:50] tsdgeos, I just thought that it'd help with testing, trying it out, it might be good for app devs to test their click apps there too and it will be required on the path to convergence too - so I was just wondering what would need to be done to offer a session [08:55] dholbach: not sure to be honest, but i don't see how it'd help to test click apps if it doesn't even do proper alt+tab, etc, anyway Saviq is your man a proper answer for that [08:56] dholbach, we've not even *tried* to run proper unity8 on the desktop under Mir [08:56] dholbach, which is a requirement for being able to launch any apps there [08:58] Saviq, ok - shall I ask again after saucy release? :-) [08:58] Saviq, those autopilot issues with the sdk branch have been fixed. I'm going to get that reviewed today. [08:58] finally. [08:59] So the filters can be looked at again. [09:03] nic-doffay, cool [09:10] tsdgeos, what did you mean with your last comment about saving on lookup? [09:11] pstolowski: we don't need the variable at all, so remove the indexOf call? [09:12] tsdgeos, ah, ok, indeed [09:15] nic-doffay, pong [09:16] tsdgeos, can I throw https://code.launchpad.net/~unity-team/unity8/albumart-provider/+merge/189923 on you afterwards? [09:16] Saviq: sure [09:16] * tsdgeos throws it to himself [09:18] mzanetti: : I'm getting an error running qmltests on jenkins for ubuntu-settings-components "qmltestrunner: could not find a Qt installation of ''" [09:18] mzanetti: could be missing something from deps? [09:19] dednick: guess mzanetti stil at devdays, not sure how online he'll be [09:19] tsdgeos: ah. ok [09:19] dednick: sounds as if qt5-default was not installed maybe? [09:19] tsdgeos, just experienced this when playing with category reordering branch - http://ubuntuone.com/5CJqELstdZQw6dRR44acoy [09:20] Saviq: ^ ? [09:20] tsdgeos, (it's my branch with all trunk changes in) [09:20] pstolowski: wops [09:20] pstolowski: not good :_/ [09:20] dednick, that would be caused by missing qt5-default [09:20] pstolowski: i guess not reproduceable? [09:20] dednick, where? [09:21] Saviq: http://10.97.0.26:8080/job/ubuntu-settings-components-qmltests-saucy/4/console [09:21] tsdgeos, it still works and stays like that, till I drag all the contents up/down, then it's back to normal [09:21] pstolowski: yeah the LVWPH "fixes" itself in some cases [09:21] in others just crashes :D [09:21] Saviq: ok, it is missing from packaging [09:21] dednick, it shouldn't be there, arguably... [09:22] Saviq: can you see if you notice anything else? http://bazaar.launchpad.net/~ubuntu-settings-components-team/ubuntu-settings-components/trunk/view/head:/debian/control [09:22] dednick, we don't have it, do we [09:22] pstolowski: but of course it should never happen, just that not sure how to end up where you ended up to fix it [09:22] "we" == unity8 [09:22] Saviq: it does [09:22] dednick, ah, build deps, yeah [09:22] Saviq: qtdeclarative5-test-plugin ? [09:22] tsdgeos, well, I saw it before a couple of times, but not easily reproducible [09:23] dednick, yeah, and tools [09:24] Cimi, can you pull on that branch and see if the tests pass? === iahmad|afk is now known as iahmad [09:24] Saviq: qtbase5-dev-tools or qtdeclarative5-dev-tools? [09:24] Cimi, they pass on jenkins. [09:25] Which means it's some sort of local issue. [09:25] dednick, declarative [09:25] nic-doffay, the test is wrong [09:26] dednick, you'll need qtquick too, for testing [09:26] nic-doffay, you're clicking on the inverse mouse area, directly [09:26] pstolowski: so what do you do when you reproduce it? just search random words or is there a pattern? [09:26] nic-doffay, of course it caches it [09:26] dednick, 'cause qmluitests only install build deps [09:26] nic-doffay, you should instead click on coordinates relatives to where you want it to have the mouse area [09:27] and verify it really s [09:27] nic-doffay, on my test and nick [09:27] nic-doffay, touches are not detected a little bit down the search box till half the screen [09:28] Cimi, I disagree. [09:29] Saviq: thanks [09:29] Cimi, what do you mean a little bit down the search box? [09:30] nic-doffay, 20% y of the screen [09:30] I'm trying to ascertain why this works for me and not for you. [09:30] nic-doffay, for example, when ou have an opened app [09:30] nic-doffay, and you're on the app lens [09:30] nic-doffay, right to the miniature of the opened app [09:31] nic-doffay, nothing happens there [09:31] tsdgeos, didn't noticed a pattern yet; just search a couple of times [09:31] ok [09:31] Cimi, but the rest now works for you? [09:31] I'm still not following what you mean. [09:32] nic-doffay, it was worning [09:32] tsdgeos, I've updated category-reordering branch [09:32] nic-doffay, i didn't test today [09:32] Cimi, but it wasn't working yesterday? [09:32] dednick, can you explain nic-doffay what's wrong? my english might suck [09:32] pstolowski: tx [09:34] pstolowski: any reason RoleRendererHint: uses index for the query and not realIndex? or just a merge mistake? [09:34] Cimi: yeah [09:35] tsdgeos, ah, merge mistake, it was added to trunk after my changes, thanks for spotting [09:35] tsdgeos, fixing [09:37] 1) have home lens open. [09:37] 2) tap search panel, search bar opens at top. [09:37] 3) type "plop", no results [09:37] 4) half way between keyboad and search bar, tap on screan. [09:37] 5) keyboard does not close. [09:37] 6) tap "just" above the keyboard, keyboard closes. [09:37] nic-doffay: ^ [09:38] dednick, cool going to give it a go. [09:38] and screan obviously = screen :) [09:38] dednick, other conditions work properly though? [09:39] nic-doffay: it's very hit and miss. it seems if you tap anywhere where there could be a dash category row (even if there are no results), it doesnt close. [09:41] tsdgeos, pushed [09:41] nic-doffay: sometimes closes randomly if i tap the search bar again. sometimes not, sometimes tap on panel closes... [09:42] dednick, hmm that most likely is an inverse mouse area issue imo [09:42] The second one you mentioned. [09:42] Saviq, what are your thoughts on all of this? ^ [09:43] nic-doffay, none atm, sorry [09:45] nic-doffay: i have no idea how the InverseMouseArea works in the case of z-order press events. I would have thought tapping ANYWHERE that is not the searchbar would close it, but it seems that everything else that accepts press events and is above in z-order is getting them. [09:46] dednick, I'm going to try change the z order to see if it makes any difference. [09:47] nic-doffay: i think zorder is only for children on same object. [09:47] dednick, well that approach worked for the filters. [09:47] Everything below the filter selectors drop down cannot be clicked. [09:47] So I'm hoping it will help. [09:48] nic-doffay: we have filters? [09:48] need a friggin cancel search button as well... [09:48] dednick, yeah we talked yesterday with mhr3 [09:49] dednick, that there's a hidden feature in unity7 - you can still tap on the activity indicator to cancel ;) [09:49] dednick, there's a branch waiting to land. [09:49] Saviq: heh. does that dismiss the search bar though? [09:50] I can work on this bug if you want [09:50] Saviq, do you have another bug for nic-doffay in case? [09:50] Cimi, this might not be anything to do with the Unity8 code. [09:52] Cimi, he's got some https://launchpad.net/~nicolas-doffay/+assignedbugs ;) [09:53] nic-doffay, move to another one :) [09:53] nic-doffay, I'll have a look in three hours [09:53] Cimi, what's the point of passing it over? [09:53] nic-doffay, so keep going [09:54] nic-doffay, start drawing rectangles filling the inverse mouse areas [09:54] nic-doffay, with different colors [09:54] nic-doffay, so you see where they are [09:54] and stuff like this.. [09:54] nic-doffay, another point is that I had a chat with oren and he explained me how it should behave [09:55] Cimi, how does it differ to the bug report that he filed? [09:56] Cimi, I don't think the issue is as simple as you think either. [09:56] nic-doffay, ok so keep debugging and find out why it's not working [09:56] nic-doffay, we cannot accept half fixes [09:56] or well, we shouldn't [09:57] Cimi, who said we were going to? [09:57] You're making an awful lot of assumptions here. === alan_g is now known as alan_g|tea [09:57] nic-doffay, hey sorry for upsetting you [09:57] pstolowski: i think your beginMoveRows is wrong [09:58] Cimi, I'm still confused, you told me yesterday it didn't work at all. [09:58] pstolowski: beginMoveRows is braindead and takes into account the thing itself [09:58] so you need something like [09:58] But today it apparently has other issues according to dednick [09:58] int newModelIndex = newIndex > oldIndex ? newIndex+1 : newIndex; [09:59] like we have in the launcherlistmodel [09:59] launchermodel sorry [09:59] nic-doffay, I might have explained badly [09:59] nic-doffay, yesterday I had same issues of dednick [09:59] Cimi, ok right got it. [10:01] MacSlow, I read your review and you said this works for you. Have you run into any of dednick 's issues? (If you've been following this at all0 [10:02] MacSlow: 1) have home lens open. [10:02] 2) tap search panel, search bar opens at top. [10:02] 3) type "plop", no results [10:02] 4) half way between keyboad and search bar, tap on screan. [10:02] 5) keyboard does not close. [10:02] 6) tap "just" above the keyboard, keyboard closes. [10:04] dednick, btw does it dismiss the search entry after those steps? [10:04] tsdgeos, yeah, I saw it in the qt docs, fixing [10:04] Even if the keyboard doesn't close properly? === alan_g|tea is now known as alan_g [10:13] tsdgeos, pushed [10:13] cool [10:13] nic-doffay: yeah. if you close the popup first. [10:17] nic-doffay: not really sure why you have 2 inverse areas though. wont the other one always catch everything not in itself and close? [10:18] pstolowski: one last question [10:19] pstolowski: the "// populate category order vector with 0..n" part in ::data [10:19] dednick, nah [10:19] the one is for the top bit [10:19] which is a different layer [10:19] pstolowski: does it make more sense somewhere else? [10:19] Sorry, let me try explain in more detail. [10:19] The one for the pageHeader, the other is for the rest of the area below the page header. [10:20] So the first is to catch the inverse of the textEntry in the pageHeader [10:20] the other is for everything below. [10:20] dednick, it's a small area in the case of the first inverse area, but I figured it should be added anyway. [10:20] nic-doffay: but the sensitive area is the root item. "Everything not in this area" includes the other InverseMouseArea region [10:21] dednick, I don't think so. I recall doing a print to test if the presses were caught, they were not [10:21] but let me check again... [10:22] dednick, confirmed. [10:25] nic-doffay: where you testing on device or on desktop? it works for me on desktop [10:26] dednick, I've had issues on the device before which I mentioned to Saviq but the desktop works 100% for me. [10:26] Which is what leads me to believe it's an issue with the InverseMouseArea SDK component. [10:27] On desktop, everything is caught, and the focus is disabled. [10:27] nic-doffay, I'll try to replicate that... [10:27] MacSlow, ta [10:28] MacSlow, try on the device though, the desktop works for me every time... [10:28] Unless you were already? O_o [10:29] nic-doffay, I always try it on the device... unless it's just some API-change with tests [10:30] MacSlow, ergh ok. [10:30] I hope something does break, otherwise this is even more confusing. [10:32] nic-doffay, not that it should matter... I'm testing it on a GalaxyNexus... what device did dednick try it on? [10:33] MacSlow: same [10:33] it's obviously only an issue on the device. [10:33] It probably has something to do with the keyboard [10:34] dednick, nic-doffay: I can confirm this odd behaviour here too [10:34] tsdgeos, this was the best place I could think of.. the problem is it has to happen when category model is fully established. and we can't do this in onCategoryOrderChanges because we may never get this signal (or get it after call to data()) [10:34] MacSlow, mind just adding that as a review comment. Thanks dednick thanks for the feedback too. [10:34] nic-doffay, just doing that [10:34] MacSlow, does it work fine on desktop out of interest? [10:35] (sorry for being a pain) [10:35] I just want to make 100% sure this behaviour is the same for multiple people. [10:35] pstolowski: hooking up to onRowCountChanged/modelreset/layoutchanged? [10:35] nic-doffay, well there's no osk when I run it on the desktop [10:36] pstolowski: though otoh in ::data is just easier [10:36] indeed [10:36] pstolowski: but means we destroy the ordering when the rowCount changes, is that what is expected? [10:36] nic-doffay, but as soon as you type the tap outside doesn't close the searchbar anymore [10:38] MacSlow, true. [10:38] MacSlow, but it should disable the search bar focus still. [10:39] MacSlow, so you won't be able to type any more... [10:39] MacSlow, the behaviour apparently differs with the component on desktop and mobile. [10:40] nic-doffay, the keyboard-focus is gone (for the desktop-case) [10:40] tsdgeos, we don't support changes to category model after the scope is created, so this shouldn't be an issue [10:41] pstolowski: so basically the row count never changes after it has been set up? [10:41] nic-doffay, updated the MR with all the new info [10:41] tsdgeos, actually let me double check that [10:42] MacSlow, thanks it's helpful. I'm going to give Tomas a shout to see if he can shed more light on this. [10:43] mhr3, category model will never get updates after scope is set up, correct? [10:43] pstolowski, it's better if you don't assume that [10:44] pstolowski: mhr3: the question is not that if it does not change, the question is, if row count changes, is it ok to "reset" the ordering? [10:44] sure [10:44] mhr3, tsdgeos : I don't see any handling of that in categories.cpp (for the main category model) [10:44] mhr3, tsdgeos : only the change of the entire model is handled [10:45] pstolowski, well model reset is row count change, no? [10:45] well.. can be [10:46] mhr3, adding rows to existing model won't trigger model.changed in unitycore, will it? [10:47] mhr3, anyway, we don't support that in libunity afair [10:47] pstolowski, but there are ways it can happen, restarting scopes etc [10:53] pstolowski: mhr3: so how do we deal with that? [10:54] tsdgeos, if the cat model changes, forget about the overridden order and wait for new signal that will override it [10:54] tsdgeos, is that what you're asking^? [10:58] tsdgeos, I'l add m_categoryOrder.clear() for the case when categories model changes [10:59] tsdgeos, just to start off with a default order [11:01] ok === MacSlow is now known as MacSlow|lunch [11:51] mhr3: how did you reproduce bug #1236249 ? The bluetooth indicator and the setting in system-settings vanishes when I turn off bluetooth [11:51] bug 1236249 in unity8 (Ubuntu) "unity8 crashed with SIGSEGV in UnityMenuModelPrivate::itemState()" [High,Confirmed] https://launchpad.net/bugs/1236249 [11:54] larsu, there's a mir bug describing the exact steps, but if you use sf instead of mir you get this bt [11:54] mhr3: ah thanks, I also just managed to get the crash when activating the action on the bus manually [11:55] larsu, mhr3: to reproduce I think you should use the bluetooth panel in system settings [11:55] larsu, do you need link to that mir bug? [11:55] mhr3: only if you don't need to search for it ;) (like I said, I can reproduce already) [11:56] larsu, mhr3: it's only the switch in the battery which has the vanishing issue [11:56] larsu, cool... i would have to :) [11:56] seb128: ah of course! Sorry about that. (I spent too much time in the battery panel today) [11:56] mhr3: ya, don't worry about it then [11:57] larsu, lol, I know what you mean, I had the same issue a couple of time, looking at the wrong panel [11:58] seb128: tbh, I think hiding the option and the indicator is wrong... [11:59] larsu, what option/indicator? [11:59] bluetooth [11:59] oh, bluetooth [11:59] the option in settings is a bug [11:59] due to the indicator object being hidding [11:59] charles is going to fix [11:59] ah cool [11:59] the indicator is a mpt's design decision to spare panel space on the small screen === alan_g is now known as alan_g|lunch === MacSlow|lunch is now known as MacSlow [12:16] Saviq, a little explanation [12:16] Saviq, why online category in Videos dash are processed by GenericScopeView? [12:16] and not DashVideos.qml [12:17] Cimi, DashVideos is deprecated [12:17] Cimi, if it's still there at all - should go [12:17] Saviq, ok [12:17] Cimi, same for DashMusic, for that matter [12:18] Saviq, ok gotcha === _salem is now known as salem_ [12:36] Saviq, I thought UbuntuShape was able to take ShaderEffectSource [12:36] Saviq, how do I fill the video renderer with black otherwise? [12:37] Cimi, make the whole UShape black [12:37] Cimi, and overlay a cropped image [12:37] Saviq, ah yeah [12:37] Cimi, no need to shape the image [12:37] Saviq, so why we said the ubuntushape didn't support that? [12:38] Cimi, because it should do it internally [12:38] Cimi, right, so for now you can put the "film" image in the UbuntuShape [12:38] Cimi, and overlay a cropped image [12:38] Saviq, what should do internally? [12:39] Cimi, UShape should take both the background image [12:39] Cimi, and the overlay image [12:39] Cimi, or well... maybe not [12:39] Cimi, so, it's probably even easier not to use UShape at all [12:39] Cimi, but have an UShaped image already [12:40] Cimi, and just overlay a cropped image on top [12:40] Cimi, I think the "mounting" is a left over from back when it was supposed to be slightly more complicated - or maybe we simply didn't take the needed step-back [12:41] to see how to do it the simplest [12:41] Saviq, shaderEffectSource support :) [12:41] Cimi, no UShape at all [12:42] -Cimi, Image { source: "background.png"; width: foo; height: bar; Image { source: the_source; anchors { fill: parent; topMargin: units.gu(5); bottomMargin: units.gu(5) }; fillMode: Image.PreserveAspectCrop; sourceSize.height: height } [12:42] } [12:43] Cimi, where background.png is simply the mount background, already "ubuntu-shaped" [12:43] Cimi, get what I mean? [12:43] Saviq, sure [12:43] Saviq, but we need videos that have aspect ratio not squared [12:44] Cimi, not sure what you mean there? [12:44] dednick: I've half-fixed bug #1236249 - can you please have a look at the other half? (see last comment) [12:44] bug 1236249 in unity8 (Ubuntu) "unity8 crashed with SIGSEGV in UnityMenuModelPrivate::itemState()" [High,Confirmed] https://launchpad.net/bugs/1236249 [12:44] Cimi, right now everything in Videos will be square, only movie posters are meant to be portrait - but we don't have those [12:44] Saviq, ok [12:44] Cimi, and posters don't need to be mounted [12:45] Cimi, so all in all - you can implement the end solution already [12:45] Saviq, I need that mounted shape asses [12:45] Cimi, yes [12:46] Cimi, jounih will get it for you [12:50] ci borked again? [12:52] pstolowski: so that should cover all regarding ordering and rowcount changes, right? [12:58] dednick, lost time again :/ [13:05] mhr3: can you check if indicator-datetime is running? === vila is now known as vila-laaaate-lun [13:05] dednick, it's not [13:05] mhr3: hm. musta crashed [13:06] larsu: i'll take a look [13:06] any log to check? [13:06] dednick: thanks [13:08] Saviq, music renderer is instead a square with more text? [13:09] Cimi, left-aligned [13:09] Cimi, so it's pretty close to what we have already [13:09] ok [13:09] Cimi, can you start with carousel delegate though? [13:09] Cimi, we need to put the black overlay with song name on there [13:10] Cimi, like we had for people lens [13:10] Saviq, ah ok [13:10] Saviq, for a reason I missed this [13:10] Saviq, I thought the carousel just needed to be loaded with count > 5 [13:10] Cimi, I think the reason is because it's missing from the visual design spec... [13:10] Cimi, that, too, yes [13:10] Cimi, oh well, it's not there in the visual design [13:11] tsdgeos, right === vila-laaaate-lun is now known as vila-late-lunch [13:15] mhr3: dont know. /var/crash ? [13:15] dednick, if there was something indicator-related you'd have it already ;) [13:15] Cimi, for the overlay in carousel - there's no design I'm afraid [13:16] ok [13:16] Cimi, so start with the grid indeed [13:16] Cimi, and I'll try and get you a design in the mean time === alan_g|lunch is now known as alan_g [13:18] Saviq, songs names in carousel? the visual spec doesn't have that, something new? [13:18] mhr3, it's described in the UX spec [13:19] mhr3, "The hero item in a carousel (the biggest thumbnail) has an overlay that will show the song name so songs will be differentiated through the title." [13:20] yey for even more inconsistencies in the specs [13:21] Cimi, let's go for left-aligned two lines with right-eliding to start with - you'll chose the right font size [13:21] pstolowski: ok, approving then, the problems you're having with LVWPH are not your fault, and if this makes them easier to repro that may even be a good thing. Agreed? [13:21] hey...where is my datetime indicator [13:21] ! [13:21] dednick, desktop? [13:22] Saviq: phone [13:22] and process is running. grr [13:22] dednick, oh, is consistently there for me... [13:22] Saviq: mine is normally. i just restarted my phone and poof [13:24] Saviq, dimensions? [13:24] hm. actually, process was on desktop. no ssh on phone :( [13:24] Saviq, should be as big as now or smaller? [13:25] Cimi, check out the https://docs.google.com/a/canonical.com/presentation/d/1X3_G6reCpcFJxzxn1Xtv9wK5n3-wOW-6z2l0qSZ8-sg/present#slide=id.gefe99b49_00 [13:25] kgunn: video standup today? [13:26] Cimi, https://lh6.googleusercontent.com/DiFH5zJxUPhXES9dICX4ydlmQzpkKFMNeJlq3NsPhPIUAzbAH8XjX4YlZ0eii1w5csbAX2ZqnW9l6bjLSDzzd8Y1oCJf2qKzJYZ49lDeg5dPDbCiCEI-3eyi8Fs says 18GU [13:27] Saviq, but this is carousel [13:27] tsdgeos: yeah...we should [13:28] Cimi, ah grid is there, too - 11GU [13:28] yeah [13:28] Saviq, total 16 with text [13:28] Cimi, yup === vila-late-lunch is now known as vila [13:34] tsdgeos, yup, thanks! [13:35] Saviq, can you give me the link of the image of video with grid? [13:35] I right click and cannot see it [13:35] might use inspector in case.. [13:36] Cimi, https://lh5.googleusercontent.com/YJsbwT4BhY1HuNX9vaLwtOruPFxSRVWs8zIcVoqHUTLFdi4ETM3SUDrJKb0sHn3AOoh7TFnGSos7MidzXW5fbgKLwcfYQRDjOXvUU3R8be6s-z4dYfLLSUM1Nts ? [13:36] Saviq, thanks! [13:40] mhr3: hm. not sure about that datetime problem. looks like it crashes, but would expect to come back up. [13:43] dednick, yea, it's back now [13:43] but it took a while [13:44] what triggers dbus service to restart process? [13:44] mhr3: ^ ? [13:44] dednick, a call to that service [13:44] any call to that service [13:45] dednick: do you have any idea why the volume slider in the sound menu doesn't work when I add a parameter to the volume action? [13:45] dednick: it shouldn't have any influence... [13:48] larsu: er, you match the type in the menu item? [13:49] nic-doffay, you joining hangout? [13:52] MacSlow, https://code.launchpad.net/~mterry/unity8/hide-greeter-on-app-launch/+merge/189948 [13:52] MacSlow: curious, what's the worst scenario in terms of rendering performance ? [13:53] * mterry hugs MacSlow [13:53] dednick: yeah, nothing but the parameter type changed [13:53] mterry, tkx [13:54] dednick: I've got to run now unfortunately. If you want to have a look, the branch is at https://code.launchpad.net/~larsu/indicator-sound/volume-action-activation [13:54] dednick: it is supposed to fix bug #1236292 [13:54] bug 1236292 in Unity 8 "Using volume buttons on phone triggers spurious popup" [High,Triaged] https://launchpad.net/bugs/1236292 [13:54]