[09:28] <mzanetti> greyback: good morning. I addressed all your comments in the qtmir merge.
[09:28] <greyback> mzanetti: cool thanks
[09:29] <greyback> mzanetti: ah sorry I didn't realize there was a unity8 branch to go with it
[09:30] <mzanetti> greyback: no worries
[09:30] <mzanetti> greyback: if you want to test, it should be enough to copy Shell.qml from that unity8 branch to the device
[09:30] <mzanetti> no need to build a package
[09:30] <greyback> mzanetti: yep
[09:37] <mzanetti> huh... why does "df -h" not show my partitions any more?
[09:38] <mzanetti> only virtual file systems in there
[11:10] <facundobatista> Hola
[11:12] <mzanetti> Saviq: hey, want to have a look at those so we can get it approved before you're away: https://code.launchpad.net/~unity-team/unity8/two_see_more_bugfix/+merge/234340/comments/572776
[11:12] <mzanetti> its really minor things
[12:44] <greyback> mzanetti: lifecycle fixes approved
[13:07] <MacSlow> Saviq, can you have a quick look at https://code.launchpad.net/~macslow/unity-notifications/synchronous-notification/+merge/229060
[13:08] <MacSlow> Saviq, I'm confused about that "automatic" merge that just happened
[13:09] <Saviq> MacSlow, you pushed to trunk
[13:09] <Saviq> MacSlow, https://code.launchpad.net/~unity-api-team/unity-notifications/trunk
[13:09] <MacSlow> How?
[13:09] <MacSlow> Crap!
[13:11] <Saviq> MacSlow, will you resolve or shall I?
[13:11] <MacSlow> just doing that
[13:29] <MacSlow> Saviq, hm... kind of stuck with fixing the accidental merge...
[13:30] <Saviq> MacSlow, can I overwrite trunk?
[13:30] <Saviq> MacSlow, ugh, you overwrote?
[13:31] <Saviq> MacSlow, https://code.launchpad.net/~ps-jenkins/unity-notifications/ubuntu-utopic-proposed should be the good thing
[13:31] <MacSlow> Saviq, correct
[13:32] <Saviq> MacSlow, so yeah, take that branch and overwrite trunk
[13:32] <Saviq> MacSlow, and fix your bash history and push locations ;)
[13:33] <MacSlow> Saviq, yeah... hell knows how that happened... I always explicitly do "bzr push lp:~macslow/bla..."
[13:40] <MacSlow> Saviq, hm... fixed... but lp:unity-notifications on launchpad did not yet update
[13:43] <Saviq> MacSlow, indeed, it will catch up
[13:45] <MacSlow> phew... lp also updated now
[13:48] <mzanetti> greyback: thanks!
[14:10] <greyback> Saviq: we need an opinion:        void fn(int var) { Q_UNUSED(var); }            OR           void fn(int) {}
[14:12] <dandrader> Saviq, is that part of our word-of-mouth coding guidelines/ http://paste.ubuntu.com/8327396/
[14:12] <dandrader> ?
[14:13] <Saviq> dandrader, greyback, Q_UNUSED generally leads to Q_UNUSED being left around when it is, in fact, used
[14:14] <Saviq> I don't have a strong opinion but was leaning towards /**/
[14:14] <dandrader> Saviq, you mean later someone actually starts using a parameter but forgets to remove the corresponding Q_UNUSED() line?
[14:14] <Saviq> dandrader, yes
[14:15] <dandrader> Saviq, greyback, hmm, makes sense
[14:20] <mterry> mzanetti, do you know how to anchor an element of the shell to the OSK?  Normally you can ask for that behavior if you have a MainView, but we don't
[14:20] <Saviq> dandrader, greyback, I'm not even sure we need the /* */, after all you're only an F4 away from the header
[14:21] <mzanetti> mterry: Qt.inputMethod.rect or something like this
[14:21] <Saviq> mzanetti, not in shell
[14:21] <mzanetti> yeah, works in apps too
[14:21] <Saviq> unless it's back with qtmir?
[14:21] <mzanetti> oh... in the shell
[14:21] <Saviq> mzanetti, Qt.inputMethod didn't work before qtmir, does it work again now?
[14:21] <dandrader> Saviq, "F4", are you talking Qt-Creator?
[14:21] <mzanetti> Saviq: mterry: there is an InputMethod item in Shell.qml
[14:21] <mzanetti> you can probably anchor to that
[14:21] <Saviq> dandrader, yes, but $your_ide's_shortcut works, too ;)
[14:22]  * dandrader is old school and doesn't use such niceties
[14:22] <mterry> mzanetti, interesting, will try thanks!
[14:22] <dandrader> mostly because vim doesn't do it reliably :D
[14:23] <mterry> Saviq, aren't you supposed to be on holiday?
[14:23] <Saviq> mterry, starting tomorrow
[14:23] <mterry> Saviq, hrm...  OK then, carry on  ;)
[14:32] <mzanetti> Saviq: https://code.launchpad.net/~unity-team/unity8/two_see_more_bugfix/+merge/234340/comments/572865
[14:32] <Saviq> mzanetti, yeah, saw that, will have a look
[14:33] <mzanetti> Saviq: ok... I see (re sync MP)
[14:33] <mzanetti> Saviq: is silo 11 about to land?
[14:34] <Saviq> mzanetti, you tell me ;)
[14:34] <mzanetti> ah ok
[14:34] <Saviq> mzanetti, but no
[14:34] <mzanetti> I thought you'll land that still today
[14:34] <Saviq> mzanetti, there's a bunch of unapproved branches
[14:34] <mzanetti> ack
[14:48] <dednick> mzanetti, satorisany idea how to catch individual role changes for model data in qml?
[14:48] <dednick> Saviq: ^
[14:48] <dednick> * any idea
[14:48] <mzanetti> dednick: ?
[14:49] <mzanetti> emit dataChanged(index, index, QVector<int> roles)
[14:49] <dednick> mzanetti: doesnt seem to work with Connections { target: modelData; onXXXChanged: {} }
[14:49] <mzanetti> ah
[14:49] <mzanetti> no. that won't do
[14:49] <mzanetti> hmm
[14:49] <mzanetti> dednick: you can do this:
[14:49] <dednick> or onModelDataChanged
[14:49] <mzanetti> property var foobar: model.barbaz
[14:49] <mzanetti> onFooBarChanged
[14:49] <dednick> mzanetti: yeah. though so
[14:49] <mzanetti> not totally elegant though
[14:49] <dednick> mzanetti: ok, thanks
[14:53] <Saviq> MacSlow, https://code.launchpad.net/~unity-team/unity8/notifications-audio-no-play-empty/+merge/234486
[14:54] <MacSlow> Saviq, on it...
[14:56] <Saviq> MacSlow, no worries, not pressing
[14:56] <Saviq> @unity ↑ important to note; bool("") [14:56] <Saviq> I'd say a QTBUG really
[14:57] <kgunn> wtf...
[14:57] <mzanetti> I think QUrl("") actually only sets the path
[14:58] <mzanetti> so there might be a valid authority or protocol string somewhere
[15:01] <Saviq> dandrader|afk, mterry, paulliu http://pastebin.ubuntu.com/8327743/
[15:01] <Saviq> mzanetti, think it defaults to file:// ?
[15:01] <MacSlow> Saviq, it is a bit odd
[15:01] <MacSlow> Saviq, I mean the need for this kind of fix
[15:02] <mterry> Saviq, sigh... fixed
[15:02] <mzanetti> Saviq: likely, but I didn't verify
[15:02] <Saviq> MacSlow, well, that's a different thing, our media components didn't deal well with playing an empty string (to be fixed)
[15:02] <Saviq> MacSlow, but still we shouldn't even call play() on it
[15:02] <mzanetti> Saviq: actually file:/// to be precise :D
[15:03] <Saviq> mzanetti, you're kidding?
[15:03] <mzanetti> ah no... "" might be 2 / only
[15:03] <Saviq> mzanetti, that would be your slash
[15:03] <Saviq> mzanetti, meaning your root directory
[15:03] <mzanetti> :D
[15:04] <MacSlow> Saviq, btw... approved
[15:04] <Saviq> MacSlow, t
[15:04] <Saviq> x
[15:04] <MacSlow> Saviq, jenkis will take a while I guess
[15:04] <Saviq> MacSlow, yeah
[15:05] <MacSlow> Saviq, but since your branch is so fresh it should cleanly merge
[15:05] <Saviq> MacSlow, yeah well, let's see if something in silo 11 conflicts
[15:11] <greyback> dandrader|afk: FYI https://code.launchpad.net/~dandrader/unity-api/splash/+merge/233288 fails on CI
[15:15] <mzanetti> Saviq: hey, do we actually have the gesture cancellation in place yet?
[15:16] <mzanetti> we don't, right?
[15:16] <paulliu> Saviq: ok. removed tags. Thanks.
[15:16] <mzanetti> Saviq: IMO bugs like these are are misleading because they aren't so much of an issue any more when that is fixed: https://bugs.launchpad.net/ubuntu-ux/+bug/1368702
[15:17] <greyback> mzanetti: we don't have gesture cancellation
[15:20] <Saviq> mzanetti, no
[15:21] <Saviq> mzanetti, we don't have it I mean, and correct, that bug would be less important if we had it
[15:21] <mzanetti> Saviq: https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1368702/comments/1
[15:22] <Saviq> dandrader, do you have the new python version of the strip tags script?
[15:22] <Saviq> dandrader, 'cause you seem to have the most of the two bad tags that are still travelling around
[15:22] <dandrader> Saviq, I'lll download it again
[15:23] <Saviq> dandrader, http://people.canonical.com/~msawicz/unity8/strip-u8-tags.py
[15:23] <Saviq> dandrader, and remember you need to strip both local and remote, as tag deletion doesn't travel with commits
[15:24] <Saviq> josharenson, please strip tags on lp:~josharenson/unity8/fix_panel_minute_count and any local checkouts you might have
[15:24] <Saviq> josharenson, http://people.canonical.com/~msawicz/unity8/strip-u8-tags.py
[15:24] <josharenson> saviq, ack...
[15:24] <josharenson> has that script been updated recently?
[15:25] <dandrader> Saviq, could you name a branch that has bad tags?
[15:25] <dandrader> Saviq, I'm running it and it's not printing anything
[15:26] <dandrader> Saviq, ah, finally got some tags in lp:~dandrader/unity8/real_header_splashscreen
[15:26] <dandrader> s/got/found
[15:29] <dandrader> Saviq, ahhhh! strip-u8-tags.py  doesn't run on the current dir by default. you have to specify "strip-u8-tags.py  ."
[15:29] <Saviq> dandrader, http://pastebin.ubuntu.com/8327743/
[15:29] <dandrader> Saviq, now it's explained why I wans't finding anything locally
[15:29] <Saviq> dandrader, oops, sorry, should probably make it work like that :)
[15:31] <dandrader> Saviq, or print out an error like "please specify a directory"
[15:31] <Saviq> dandrader, yeah, you know... that script was only supposed to live for a week or so :P
[15:31] <Saviq> then it got rewritten in py, and was supposed to only live another week...
[15:32] <dandrader> Saviq, all my branches should be clean now
[15:33] <Saviq> dandrader, thanks
[15:33] <josharenson> greyback, ready when you are
[15:34] <greyback> josharenson: https://plus.google.com/hangouts/_/canonical.com/gerry
[15:34] <Saviq> josharenson, no, as long as you had the .py version
[15:40] <dandrader> greyback, oh, right. should be fixed now (let's see what jenkins says next)
[15:49] <mterry> greyback, how do I find out where the OSK is on screen (from the shell code) -- we used to have OSKController, now we have InputMethod, but it doesn't seem to give info on where on the screen is actually being drawn
[15:59] <greyback> dandrader: thanks
[16:00] <greyback> mterry: hmm. OSK is actually a maximized surface like any other (non-fullscreen) app surface, with lots of transparency. Shell doesn't know from that how big the OSK is
[16:01] <Saviq> greyback, that's why we have the keyboard communicator socket?
[16:01] <greyback> mterry: if InputMethod doesn't tell you, I'm not sure there's any other place to get that info right now
[16:01] <Saviq> or... had...
[16:01] <greyback> Saviq: yep
[16:01] <mterry> greyback, I looked into how anchorToKeyboard works and it's doing "Qt.inputMethod.visible ? Qt.inputMethod.keyboardRectangle.height : 0"
[16:01] <Saviq> greyback, do we have access to ↑ now?
[16:01] <mterry> greyback, does that seem likely to work?
[16:01] <mterry> Didn't in my quick test, but I'm probably doing something wrong, since that's what the SDK uses
[16:02] <greyback> mterry: yeah that works for apps. I don't see any reason why it would not work for unity8 too, there may be somethinkg missing in the QPA for it
[16:02] <Saviq> greyback, it didn't work before qtmir because shell was never focused
[16:02] <greyback> mterry: if it's what the SDK uses, I'm sure we can make it work for shell
[16:02] <Saviq> and it *did* work before that because Qt didn't care
[16:02] <Saviq> and then it started caring
[16:03] <greyback> Saviq: now the unity8 "Window" should be focused, or at least we can persuade Qt that it is
[16:03] <mterry> greyback, so you're saying there may be qtmir work for this?
[16:03] <Saviq> greyback, yeah, that'd be grand
[16:03] <greyback> mterry: a little yeah
[16:04] <mterry> greyback, OK.  I think I'll go with a workaround for now, then file a qtmir bug about it for proper solution
[16:05] <greyback> mterry: thanks
[19:32] <mterry> greyback, btw that Qt.inputMethod.keyboardRectangle.height trick did work after all -- forgot to get back to you on that
[19:32] <greyback> mterry: it works? Oh great
[19:32]  * greyback draws satisfying line through task in his TODO list
[20:22] <dandrader> greyback, what would be the best way to refer to an image in /usr/lib/$ARCH/qt5/qml/Ubuntu/Components/Themes/Ambiance/artwork/
[20:23] <dandrader> greyback, ^^^ that's about your comment https://code.launchpad.net/~dandrader/unity8/real_header_splashscreen/+merge/233286/comments/571898
[20:24] <dandrader> from unity8
[20:37] <greyback> dandrader: nothing easy springs to mind. But it's late here, so I'm not thinking hard right now
[20:38] <dandrader> greyback, I think the only way would be to have a wrapper component there in Ubuntu/Components/Themes/Ambiance so unity8 could import it
[20:38] <dandrader> like just an Image with a preset source
[20:39] <dandrader> greyback, so easier to copy over like Florian did
[20:39] <greyback> dandrader: could this be a component in the SDK, instead of copying all this stuff over? A kinda private component
[20:41] <dandrader> greyback, are you talking about PageHeadStyle and MainViewStyle?
[20:41] <greyback> yep
[20:41] <dandrader> greyback, they are :D
[20:41] <dandrader> greyback, but for the splash screen we needed modified versions of them
[20:41] <dandrader> greyback, so they were copied over and modified
[20:42] <dandrader> greyback, I believe mainly so that it's not tied to the theme of the qml scene but to a hardcoded one
[20:42] <dandrader> greyback, florian left some todo's there about how it should be ideally in the future
[20:43] <dandrader> greyback, but the way the sdk is implemented at the moment does not support such flexibility
[20:43] <dandrader> greyback, thus the copy-over-and-modify approach
[20:44] <greyback> dandrader: is a pity. I just forsee design changes in SDK happening that break this trick
[20:44] <dandrader> greyback, error prone as it will have to be updated every time their sdk counterparts change, but it-just-works(Tm)
[20:44] <dandrader> greyback, ditto
[20:54] <greyback> dandrader: I'm not going to block this. I just dislike copying the images, so would like to explore all options before just copying