[02:35] <ahoneybun> can I can take a screenshot in unity8?
[03:28] <duflu> ahoneybun: PrintScrn key I think
[03:28] <duflu> PrtScn / PrtSc key
[05:44] <ahoneybun> duflu: the issue is I have no way of getting that out though, I can't paste it in GIMP
[05:44] <duflu> ahoneybun: I think it's saved to your ~/Pictures or ~/Documents ?
[06:04] <ahoneybun> I never thought that it would just take it like that
[06:04] <ahoneybun> no UI when the key is pressed atm ?
[06:06] <ahoneybun> mm now I have to get a working File Manager
[06:11] <ahoneybun> mm nautilus looks fine in unity8
[06:14] <ahoneybun> ohhh
[06:14] <ahoneybun> fn key and prt sc
[06:14] <ahoneybun> thanks duflu
[06:14] <duflu> No problem
[06:15] <ahoneybun> silly me forgeting about function keys
[06:15] <ahoneybun> and now I have a reason to blog
[06:15] <ahoneybun> ...
[06:17] <duflu> Well, desktop keyboards probably won't need (or have) a Fn
[06:17] <duflu> Maybe laptops
[06:18] <ahoneybun> yea I;m on a HP Elitebook
[06:18] <ahoneybun> Intel only graphics
[06:20] <lpotter> the keyboard that came with my dell desktop machine has a Fn key
[06:21] <ahoneybun> mm no way to upload to Drive
[06:21] <ahoneybun> and since File Manager is broken I can't use flickr
[11:34] <dandrader> tsdgeos, got time for a code review? https://code.launchpad.net/~dandrader/unity8/cursorGridUnit/+merge/300130
[12:26] <tsdgeos> dandrader: sure
[12:26] <dandrader> tsdgeos, thanks
[12:27] <tsdgeos> dandrader: not test-able?
[12:27] <tsdgeos> auto-test-able?
[12:27] <dandrader> tsdgeos, don't know how or what to test there
[12:27] <tsdgeos> ok
[12:28] <dandrader> tsdgeos, there's "make tryCursor" at least....
[12:28] <dandrader> tsdgeos, see commit description with instructions on how to manually test it
[12:29] <dandrader> s/commit/MP
[13:15] <tsdgeos> dandrader|afk: did you think on passing the requestedCursor height by actually using the sourceSize property instead of using the url for serialize it?
[13:15] <tsdgeos> it wouldn't make much of a difference but seems "more proper"
[13:30] <dandrader> tsdgeos, that won't work with AnimatedSprite
[13:32] <tsdgeos> why?
[13:32] <dandrader> tsdgeos, it doesn't have this property
[13:32] <dandrader> tsdgeos, from reading the docs, it seems sourceSize mandates the size
[13:33] <dandrader> tsdgeos, but it should be only a preferred size, as the available cursor sizes might match the requested one
[13:34] <dandrader> tsdgeos, also, cursor image size is chosen solely by the preferred/requested height (no width involved as in a sourceSize case)
[13:35] <tsdgeos> dandrader: it doesn't really "mandate" as much as it's a strong suggestion
[13:35] <tsdgeos> you could use sourceSize: Qt.size(0, height)
[13:36] <tsdgeos> 0 is defined in the sourceSize property as "whatever goes well with the other parameter to maintain aspect ratio"
[13:36] <tsdgeos> but if you thought about it and decided against that's fine
[13:36] <tsdgeos> just wanted to know the reason
[13:36] <dandrader> tsdgeos, I suppose it will be sent as the last param here: requestImage(const QString &cursorThemeAndNameAndHeight, QSize *size, const QSize & /*requestedSize*/)
[13:37] <tsdgeos> yep
[13:37] <dandrader> tsdgeos, again, don't know how to use it on the AnimatedSprite case
[13:38] <tsdgeos> yep
[13:38] <tsdgeos> that's a good reason
[13:38] <tsdgeos> wonder why there's no sourceSize there
[14:05] <tsdgeos> dandrader: what do you think of moving the imageSource property up to the CursorImageInfo ?
[14:05] <tsdgeos> logically it seems to me as it belongs there
[14:05] <tsdgeos> it's the object that tells you how to "fill" the Image/AnimatedSprite given the mousePointer properties
[14:09] <tsdgeos> dandrader: also
[14:09] <tsdgeos> -        XcursorImage *xcursorImage = xcursorImages->images[0];
[14:09] <tsdgeos> + XcursorImage *xcursorImage = xcursorImages->images[i];
[14:09] <tsdgeos> is "just" an unrelated bugfix, right?
[14:09] <dandrader> tsdgeos, right
[14:10] <dandrader> tsdgeos, about the source location suggestion: yes, makes sense
[14:12] <tsdgeos> dandrader: the "height: units.gu(3)" in Shell.qml, should it be 4 (since afaik de wedault to 8px per gu)? or you've found 3 to be closer to what we want?
[14:12] <dandrader> tsdgeos, I chose 3 as it's the panel height
[14:13] <tsdgeos> ok
[14:13] <dandrader> tsdgeos, and empirically this seems to roughly match the pointer height in unity 7
[14:14] <dandrader> tsdgeos, are you able to try out a unity8-session on your laptop?
[14:15] <tsdgeos> dandrader: yes
[14:38] <dandrader> tsdgeos, pushed the imageSource location change. coundn't test as my test laptop refuses to show up lightdm right now. investigating...
[14:38] <tsdgeos> oka
[14:44] <mterry> tsdgeos, two of your qtmir branches seem to conflict in silo 77 -- system-includes and clazy-run.  I think system-includes conflicts with silo 76.  Not sure what clazy-run conflicts with
[14:44] <mterry> dandrader, removeDBusWindowStack conflicts in silo 77.  As does eglconvenience-retry for qtubuntu
[14:44] <mterry> oh whoops
[14:44] <tsdgeos> mterry: you mean they conflcit with eachother or with other stuff?
[14:44] <mterry> eglconvenience-retry is gerry's
[14:44] <mterry> tsdgeos, other stuff
[14:45] <mterry> tsdgeos, like you might want to rebase system-includes on mirSurfaceInputBounds?
[14:46] <mterry> mzanetti, onthefly-debugging creates a packaging build error -- I'm looking into it for ya
[14:46] <mterry> something about librarys and linking and something
[14:46] <mzanetti> meh
[14:47] <mzanetti> yeah, it requires libunity8-private in both, the main application and uqmlscene
[14:49] <tsdgeos> mterry: i guess i can try yes
[14:51] <tsdgeos> mterry: but honestly it's annoying to have to keep moving this branches up and up, my understanding from gerry is that we would land my silo first and then others would have to worry about merging with it
[14:51] <tsdgeos> but meh
[14:51] <mterry> tsdgeos, which silo?  Maybe I'm out of the loop
[14:51] <tsdgeos> no idea about which silo really :D
[14:52] <mterry> tsdgeos, hm, well system_includes isn't in any existing silos except for this one.  But the mirSurfaceInputBounds silo is ahead of us
[14:53] <tsdgeos> sure no worries, as said i'll merge it
[14:53] <mterry> I don't know what gerry promised  :)
[14:53] <tsdgeos> just need to test dandrader's cursor stuff first
[14:58] <tsdgeos> dandrader: ah, i actually just meant moving the qml property inside the object, not to C++ itself :D
[14:58] <dandrader> tsdgeos, too late :)
[14:59] <tsdgeos> dandrader: if you move it to C++ you need to emit imageSourceChanged from the other setters, no?
[14:59]  * dandrader checks
[14:59] <tsdgeos> from the ones that change m_themeName, m_cursorName, m_cursorHeight
[15:00] <dandrader> tsdgeos, it's all done centrally in CursorImageInfo::update()
[15:00] <tsdgeos> ah right
[15:00] <dandrader> tsdgeos, they all call it
[15:01] <tsdgeos> didn't see that
[15:12] <mterry> mzanetti, http://paste.ubuntu.com/19502610/ for your onthefly-debugging branch -- should fix debbuild
[15:12] <mterry> mzanetti, helps dpkg find libunity8-private
[15:13] <mzanetti> mterry, thanks a bunch :)
[15:13] <tsdgeos> mzanetti: merged mirSurfaceInputBounds to system_includes
[15:14] <mzanetti> mterry, pushed
[15:16] <mzanetti> tsdgeos, which branch?
[15:16] <mterry> mzanetti, I think he meant to ping me
[15:16] <tsdgeos> mzanetti: i merged https://code.launchpad.net/~dandrader/qtmir/mirSurfaceInputBounds into https://code.launchpad.net/~aacid/qtmir/system_includes/+merge/296598
[15:16] <tsdgeos> mzanetti: oh right sorry
[15:16] <mzanetti> ah ok
[15:16] <mterry> tsdgeos, there was a conflict, right?  (/me hopes I'm not crazy  :))
[15:17] <mzanetti> I only knew the mirSurfaceInputBounds (as we have that in unity too)
[15:17] <tsdgeos> mterry: yaeh some small in debian/rules and CMakeLists.txt
[15:17] <mzanetti> was seaching for that system_includes and thinking "Geez... did you really completely forget about this?"
[15:18] <mterry> :)
[15:18] <mterry> tsdgeos, so clazy_run also conflicted with something, but I didn't track down what
[15:19] <tsdgeos> mterry: well it for sure conclicts with the same branch
[15:19] <tsdgeos> since it's based on system_includes
[15:19] <tsdgeos> mterry: but i guess you can leave it out
[15:19] <tsdgeos> and i'll try to aim for the next silo :'(
[15:20] <tsdgeos> mterry: want me to change the prerequiste of https://code.launchpad.net/~aacid/qtmir/system_includes/+merge/296598  and resubmit? Someone will have to re-top-approve though
[15:21] <mterry> tsdgeos, or just rebase it on top of system_includes?
[15:21] <mterry> tsdgeos, sure I'll re-top  (or you can, hell)
[15:21] <mterry> just a rebase
[15:22] <tsdgeos> mterry: you can re-top approve plz https://code.launchpad.net/~aacid/qtmir/system_includes/+merge/300208
[16:26] <ahoneybun> https://www.flickr.com/photos/44748317@N08/28214927172/in/album-72157670987900966/
[16:26] <ahoneybun> :)
[16:45] <mterry> mzanetti, you need to also bzr add debian/shlibs.local
[16:45] <mzanetti> very good point
[16:45] <mterry> should have warned :)  I always forget that step myself
[16:46] <mzanetti> pushed. thanks
[16:49] <mterry> Saviq, so what was that about ubuntu-touch not installing Recommends?  Why do we do that?  General slimming or were we trying to avoid a specific set of packages from being pulled in automatically?
[16:50] <Saviq> mterry, more control over what's in the image, yeah
[16:50] <mterry> Saviq, it's not convergent!  We don't do that on the desktop images  :)
[17:28] <dandrader> mterry, that's about the standalone unity8-greeter, right? https://code.launchpad.net/~mterry/unity8/greeter-apps/+merge/299550
[17:30] <mterry> dandrader, yeah.  My focus with it is in preparation for using a split greeter on the phone in the encrypted-home case  (thus, worrying about the emergency dialer being able to launch, etc).  But should be featureful on desktop too (if you're running a unity8 session)
[17:30] <dandrader> mterry, so landing this by itself will be a NOOP on the phone?
[17:31] <mterry> dandrader, it should be yeah
[17:32] <mterry> Good prep work though.  Slowly getting a split greeter on phone in shape  :)
[17:33] <dandrader> looks like a good excuse to upgrade my test laptop do yakkety...
[17:34] <dandrader> mterry, , there's no stable-phone-overlay (or need for such) in yakkety I believe?
[17:37] <mterry> dandrader, xenial should be fine
[17:37] <mterry> dandrader, backported lightdm was for phone's benefit
[17:37] <mterry> dandrader, correct about yakkety and overlay though
[17:45] <dandrader> mterry, http://cdimage.ubuntu.com/releases/yakkety/alpha-1/source/ any idea why there are 4 different images (and the difference between them)?
[17:46] <mterry> dandrader, hah...  no?
[17:47] <mterry> dandrader, oh ...
[17:47] <mterry> dandrader, that's the source image?
[17:47] <mterry> dandrader, split over 4 dvds I'm guessing
[17:47] <dandrader> mterry, oh, I reached the wrong page then
[17:48] <mterry> dandrader, don't want to just upgrade in place?
[17:48] <dandrader> mterry, wanted to do a clean slate but I think I gonna do that
[17:49] <mterry> dandrader, I have a system that has been upgraded in place since precise  :-P
[18:08] <mterry> dandrader, you were asking about greeter-apps and lightdm.  You prompted me to finally create a testing silo for that MP.  Silo 55 has the goods (backported lightdm and the unity8 MP -- both building right now)
[18:09] <dandrader> mterry, ok