[07:42] Mirv: are at we at the state of caring about new patches that fix the "desktop" use case for the dbus patchset? [07:42] in case we are, there's updated https://codereview.qt-project.org/#/c/110325/ and https://codereview.qt-project.org/#/c/110643 that is new and should be applied before [07:44] tsdgeos: well sure, since it'd be a bit unfortunate to land something to rtm that can't be landed to w, so if we have something that would be acceptable to both that'd be great. [07:44] Mirv: so try applying those too please :) [07:44] trying [07:46] tsdgeos: 110325 has a new title but it's the one that replaces the older WIP-Deliver-events-in-threads-regardless-of-QCoreApp.patch, right? [07:47] Mirv: i think so yes, that's probably why the new one is there, i think it was kind of spawned into two [07:51] tsdgeos: 110325 now depends on adding something to qevdevtouchhandler_p.h that doesn't exist on 5.4 [07:52] Mirv: it's fine, don't add it then [07:52] that particular patch to the file i mean [07:53] it's not a dependency, it's "fixing" the file, since we don't have it, nothing to fix [07:53] ok [07:59] tsdgeos: hmm. notifyInternal2 function in https://codereview.qt-project.org/#/c/110325/7/src/corelib/kernel/qcoreapplication.cpp,unified also doesn't exist in 5.4 (hunks 3 and 4). also regarding notify_helper it's quite different in 5.4 http://pastebin.ubuntu.com/10837225/ so I'm not sure if it needs changing (hunk 6) [08:00] Mirv: that's why i said that https://codereview.qt-project.org/#/c/110643/1//ALL goes before [08:01] tsdgeos: oh, right, I did that, then I had a previous error and tried switching it back. thanks :) [08:01] ok, seems it's rebased again [08:04] cleaned 018 from the old builds and pushing a new one [08:18] :) [08:24] Saviq: do you think we could do "a small landing" with what we have at the moment? to get 5.5 build out there and those small things? [08:24] tsdgeos, we could, usre [08:24] sure, even [08:25] I *think* [08:25] :D [08:25] we're in Final Freeze since yesterday [08:25] or well there's this release thing [08:25] right [08:25] i wonder when w is going to open [08:25] we need a name! [08:26] Mirv, what's the archive state atm, can we land anything? and will vivid accept anything before the release next week? [08:26] tsdgeos: as I understand it there will be like a standby ppa till w is open and the initial imports done [08:27] but I think that is just for phone [08:38] dednick: are you adding the deletes to https://code.launchpad.net/~nick-dedekind/unity8/1436982.message-freeze/+merge/255702 ? [08:38] tsdgeos: i am. but i'm adding unit tests for the async calls. [08:38] dednick: cool [08:45] Saviq: archive is closed regarding us, we'll start landings to https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay today [08:47] devel-proposed will be built using that PPA too [08:48] could someone help a scope developer? -> http://askubuntu.com/questions/610196/scope-developing-how-to-change-scope-result-inside-perform-action [08:49] Mirv, okies [08:49] pstolowski, ↑ askubuntu for you guys [09:15] tsdgeos: https://launchpadlibrarian.net/203564985/buildlog_ubuntu-vivid-amd64.qtbase-opensource-src_5.4.1%2Bdfsg-2ubuntu6~rtm~test1_BUILDING.txt.gz error: type 'QThread' is not a direct base of 'QEvdevTabletHandlerThread' [09:16] hmmm [09:16] let me check [09:18] dpm, Saviq, replied. btw that reminded me of https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1335761 cause he is likely to hit it; do you recall if we concluded anything about that bug? [09:18] Ubuntu bug 1335761 in unity8 (Ubuntu RTM) "[Dash] Activating a preview with a scope URI for the current scope doesn't send you back to the results view" [Medium,Confirmed] [09:18] also tsdgeos ^ [09:19] Mirv: can you replace the QThread(parent) in there by QDaemonThread(parent) ? [09:19] src/platformsupport/input/evdevtablet/qevdevtablet.cpp line 295 [09:19] and i'll comment in the MR [09:20] thanks pstolowski! [09:20] tsdgeos: ok [09:23] adding to the patch [09:24] pstolowski: have you tried this lately? maybe it was fixed when we made temp scopes open temp scopes? [09:24] or has nothing to do? [09:28] tsdgeos, maybe, i'm not sure. no idea what what;s the best scope right now to exhibit this problem, the steps for youtube from the bug report don't seem to be applicable any,ore [09:29] pete-woods, hey, has 'Search' button been removed from youtube previews for channels? this is re https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1335761 ? [09:29] Ubuntu bug 1335761 in unity8 (Ubuntu RTM) "[Dash] Activating a preview with a scope URI for the current scope doesn't send you back to the results view" [Medium,Confirmed] [09:31] pstolowski: hmm. I haven't changed youtube in quite some time [09:33] pete-woods, i cannot follow the steps from this report, because there is no search button in previews.. [09:34] pstolowski: I think you had to preview a channel, or something like that [09:34] but I ended up taking all that out [09:34] obviously my bug description has now become junk [09:37] pstolowski: actually, it looks like it works now! [09:37] pete-woods, also some time ago we change the semantics of handling of scope:// uris in the shell; the are not preview-able, they just start a new search [09:37] yes [09:37] this is what I wanted! [09:37] so you can click through playlists [09:37] and it shows the contents [09:37] pete-woods, i suppose before you had an intermediate step of displaying a preview of a channel, now you don't have that [09:37] pstolowski: yeah, but you could't expand the playlist [09:38] as the search button URI didn't work [09:38] oh right [09:38] pstolowski: the navigation is a bit broken now, thouygh [09:38] when you hit back from inside in item in a playlist, you get dumped all the way out [09:38] but I guess that's the same in a lot of places in the dash [09:38] you get out of the youtube scope? [09:39] pstolowski: yeah [09:39] or if it's favourited, you get taken back to home [09:39] yeah i see it [09:39] pstolowski: I think we would really benefit from a history stack in the dash [09:40] yes [09:41] ok, to sum up, i'm not sure a Search button from the preview would work. what many scopes including youtube use is canned queries in search results, which always worked fine [10:10] tsdgeos, unfortunately changes to temp scopes didn't fix the issue; i've just commented on the bug report [10:10] k [10:13] pstolowski: what happens in that case signal wise? [10:13] are we just getting a "search changed"? [10:14] tsdgeos, i havent' debugged it this time, but i think my comment #4 is still valid [10:15] ok [10:15] well i guess until noone really wants to do this [10:15] it's no high prio :D [10:18] mzanetti: https://code.launchpad.net/~mzanetti/unity8/fix-dnd-cancelling/+merge/256462/comments/639298 [10:18] tsdgeos, uh oh. thanks a lot for catching this [10:19] mzanetti: honestly i think we should block clicks so you can't open an app while you're dragging an icon [10:19] tsdgeos, well, people do this, as the above ubuntuask question & comments to this bug show ;) [10:19] pstolowski: then tell the powers that be to make it higher prio D: [10:20] mzanetti: but still the cancellation should work fine in case it happens [10:20] tsdgeos, i think it's pretty useful for scopes to display a preview button that re-does the search with speicifc params [10:20] pstolowski: agreed [10:21] tsdgeos, yeah, definitely... the canellation might happen for different reasons. also tapping outside with another finger to close the launcher [10:21] I guess notifications, exiting apps and whatnot can cause it too [10:22] tsdgeos, I can't launch another app while draggin one [10:23] mzanetti: be in the app cscope [10:23] mzanetti: you can click icons in the background [10:23] it's kind of confusing [10:23] uh oh [10:23] * mzanetti tries [10:24] indeed... that should not work [10:24] otoh... the fact that we both notice it now for the first time :D [10:25] yaeh [10:25] well it's not too often that you use two fingers [10:26] mzanetti: i mean that's actually the other way i have to cancel the drag, hwo do you do it otherwise? [10:26] tsdgeos, when on the greeter, swipe the launcher away to the left [10:27] ah it's on the greeter [10:27] well... doesn't matter... that's one way I found to repro a cancel [10:27] right [10:27] the other way i found was opnenig an app :D [10:27] right :) [10:55] tsdgeos, fixed [10:55] k [11:15] mzanetti: Did you do any launcher-parity work already in a branch I should be picking up? [11:15] MacSlow, nope, don't have a branch. what would you start with? [11:16] mzanetti: I think the app-launching animation [11:16] ack [11:17] mzanetti: trello-card is also filled with the most current info from Design now [11:18] mzanetti: still need to pull all animation-spec info from Design [11:19] MacSlow, looking at that design picture I see conflicting specs with what they requested for the phone [11:19] MacSlow, for instance the quicklist was explicitly requested to not be next to the panel, but above/below the pressed icon [11:19] now in this one it's the old style again [11:20] mzanetti, I've seen the "above" version too [11:20] mzanetti: I'm expecting to do more details-info hunting in the coming days [11:21] MacSlow, so for things that are just not there yet, I guess whatever design comes up with is fine. but before changing existing things, please talk to me first [11:21] mzanetti: I want to keep asking question on a case-by-case/feature-by-feature basis [11:21] mzanetti: ack === alan_g is now known as alan_g|lunch [12:04] mzanetti, do you know what "lockout capability" means wrt to launcher-feature-parity? Lockout as in "log out of session"?! === MacSlow is now known as MacSlow|lunch [12:32] MacSlow|lunch, no, always-show === alan_g|lunch is now known as alan_g [12:47] mzanetti: that's the other bug we found https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1445472 [12:47] Ubuntu bug 1445472 in unity8 (Ubuntu) "Can click on the Dash while dragging a Launcher icon" [Undecided,New] [12:51] mzanetti: i'm on bug finding mode today https://code.launchpad.net/~mzanetti/unity8/fix-dnd-cancelling/+merge/256462/comments/639341 [12:54] Saviq, will change to Super_L only after the change has landed. Otherwise people won't be able to test it: https://code.launchpad.net/~dandrader/unity8/homeKey/+merge/256658 [12:56] MacSlow|lunch, lockout: make the launcher stay on screen [12:56] instead of auto-hiding [12:58] dandrader, sure [12:59] tsdgeos, can't repro [12:59] it is a bit tricky [12:59] i agree [12:59] ah, got it... but not sure what I did [12:59] mzanetti: let me repeat what needs to be done [13:00] basically you need to be moving the drag finger and the "cancel" finger at the same time [13:00] and get lucky the drag finger would put it in the "next spot" i guess [13:00] yeah, that's how I understand the comment [13:00] right. the "lucky" part is a bit odd :D [13:01] :D [13:02] tsdgeos, I think I found a way.... seems to happen if you get the cancel during a progressive autoscroll === MacSlow|lunch is now known as MacSlow [13:05] Saviq, mzanetti: hm... ok, odd feature-name then :) [13:15] tsdgeos, can you give it another try? [13:16] * mzanetti goes friday labbing [13:16] mzanetti: sure, just after flashing stable finishes [13:16] tsdgeos, thanks [13:30] uh oh [13:30] my BQ is in an infinite reboot loop [13:30] after flashing stable [13:37] and disconnecting the usb cable fixed it [13:37] :S [13:38] :) [13:50] Saviq, design is ok if we just use art header horizontal for the configuration cards [14:04] mzanetti, we do already have badge-counters in the unity8-lauchner as far as I can tell. Only top-right or bottom-right corner is unclear. Right?! [14:05] MacSlow, I hear bottom-right for the first time right now [14:06] MacSlow, where does it say bottom-right? [14:07] mzanetti, we do have badge-counters for all I know... at least I see them in the settings-icon when updates arrive on the device... in mockups I've seen contradicting top-right and bottom-right variations... email to design regarding this (among other things) is on its way to them . [14:07] MacSlow, pretty sure the bottom-right is a mistake [14:07] MacSlow, on the bottom there is a progress bar [14:07] we already have that, just not used anywhere yet [14:08] mzanetti, hm... the progress-bar is meant to e i nthe middle o fthe icon... [14:08] mzanetti, the "available updates" counter on the settings-icon... is that not a badge-counter?! [14:08] MacSlow, do a "make tryShell" [14:09] there you'll see all the things [14:09] mzanetti, ok [14:10] MacSlow, yes, the settings thing is the badge-counter [14:13] mzanetti, general question... the spreadsheet "N/A" means "none available" or "not applicable" [14:13] yes [14:14] mzanetti, that was a this-or-that question... not yes-or-no :) [14:15] dunno... means it's not there [14:15] :D [14:15] mzanetti, ok :) [14:15] mzanetti, "blocked on design" this means to me then [14:15] MacSlow, if you refer to the ones in the "Visual Design" column, feel free to question that if you think there is a visual design needed [14:16] MacSlow, for example in row 17 I would disagree that a visual design is not applicable. becuase I already had one when I implemented that [14:17] anhow, as it's already implemented we probably don't *really* need it any more now [14:17] although it should be archived in that design website... [14:18] mzanetti, I'll focus first on really non-implemented pieces first, before I start polishing things... since I know that's far more design-feedback-intense [14:18] Cimi, so no work for us yet, then? [14:18] Saviq, I would say so [14:18] Cimi, cool, DONE [14:18] :D [14:19] Cimi, ok, so to fix the visual effect, jump on the shape card then [14:19] Cimi, or let's size it after standup === dandrader is now known as dandrader|lunch === dandrader|lunch is now known as dandrader === alan_g is now known as alan_g|EOW === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader [19:30] is it just me or does it really take forever to download an image to flash [19:31] ? === dandrader is now known as dandrader|afk === dandrader|afk is now known as dandrader