/srv/irclogs.ubuntu.com/2015/03/06/#ubuntu-unity.txt

=== mzanetti_ is now known as mzanetti
dandradergreyback, mir currently doesn't provide the display's physical size (although it has API for that). Do you mind if I made qtmir fallback to querying ro.sf.lcd_density android property (in case the mir property is empty, as it is now)?11:02
greybackdandrader: why do we need it?11:03
dandradergreyback, I would like EdgeDragArea's properties to be based on physical sizes instead of grid units11:04
greybackI'd rather Mir fixes the API. We shouldn't have to work around Mir faults11:04
dandradergreyback, sure, they will fix it. but might take some time. I don't want to be blocked by that11:04
greybackbut for middle-ground, make it a giant REMOVEME11:04
Saviqdandrader, /me not sure we want to make edge areas oblivious of GRID_UNIT_PX11:18
dandraderSaviq, why? a user's finger lives in the physical world11:19
Saviqas the idea behind switching e.g. a big phone from 50 to 40 GU wide is to make touch targets bigger, too11:19
Saviqand how is the edge touch target different... especially if these days we have them smart about non-edge-swipes11:19
Saviqdandrader, yeah, but the user's finger might just be bigger/older/less dexterous11:20
SaviqI do think we might actually need a per-device factor, to potentially cater for worse touch hw11:21
dandraderSaviq, but does that make the finger grid unit based?11:21
Saviqdandrader, it does make it the user's decision on what size the grid is11:21
dandraderSaviq, making it based on physical units doesn't mean it's worse for bigger/older/less dexterous fingers. I don't see the relationship11:22
Saviqdandrader, you sure?11:23
Saviqdandrader, don't get me wrong, I don't have data on this or anything11:23
Saviqdandrader, I'm just raising a concern11:24
Saviqdandrader, already on arale it's tricky to drag from the edge because they're "sharp"11:24
Saviqso if we make the edge area smaller because it's more dense, it might get even trickier11:25
dandraderSaviq, physical units means mm, not pixels!11:25
Saviqdandrader, I know, but we're not doing anything in the phone in mm11:26
Saviqdandrader, that's why we have the grid, some people like their mm bigger than others ;)11:26
Saviqthat's our way of dealing with dpi difference11:26
dandraderSaviq, a gesture is not a graphical UI item, it's an action that happens in the physical world. you edge-swipe the same way regardless of the screen dpi or how big the ui elements are11:27
Saviqdandrader, well that's the thing I'm not so sure of, but ok :)11:28
dandraderSaviq, edge-swipe properies have to be made so that false-negatives don't happen anyway, as they're very frustrating11:28
dandraderSaviq, do you want to make grid units also scale timers? like make animations slower? :-)11:29
dandraderSaviq, as the big old finger is also slower11:29
Saviqdandrader, that's reaching, isn't it ;)11:29
Saviqdandrader, I'm just saying that if you increase touch targets (i.e. item sizes) everywhere in the UI by switching to the "zoomed" mode or whatever we call it11:30
Saviqthat might be an indication that we should make the edge swipes somewhat more forgiving, too11:30
Saviqdunno, not really sure it's us who should have the discussion anyway (without some user testing data)11:31
=== dandrader is now known as dandrader|afk
=== MacSlow is now known as MacSlow|lunch
=== dandrader|afk is now known as dandrader
=== marcusto_ is now known as marcustomlinson
=== MacSlow|lunch is now known as MacSlow
dandraderSaviq, how's landing-006 looking so far?14:04
Saviqdandrader, just flashed again after the tweaks from Gerry and Robert14:05
Saviqelopio, hey, on cleaning up py2, you might also have a look at libautopilot-qt, it pulls qt4 onto the phone whenever I install unity8-autopilot... and IIUC there's a dep loop between libautopilot-qt and autopilot-qt{4,5}14:21
elopioSaviq: can you please report a bug? I will tell jfunk to put it close to the top, as this unneeded deps makes running tests in read-only harder than it should be.14:23
Saviqelopio, sure, doing14:23
Saviqelopio, hmm actually it seems like it might be our issue after all, we should depend autopilot-qt5 instead14:25
Saviqelopio, could you include that change in your branch?14:26
elopioSaviq: instead of libautopilot-qt?14:29
Saviqelopio, yeah, libautopilot-qt says "This is a compatibility package depending on both the Qt4 and Qt5 drivers."14:30
Saviqand then autopilot-qt5 says "Replaces: libautopilot-qt (<< 1.4+14.10.20140724.1-0ubuntu2)"14:30
elopioSaviq: ok, but autopilot-qt5 doesn't install qttestability-autopilot14:31
Saviqelopio, then you might need to add that (even if it's weird that it doesn't...)14:31
elopioSaviq: yes. I don't fully understand this, I'd prefer to do it in a different branch.14:32
Saviqah right, qttestability-autopilot needs either of autopilot-qt{4,5}14:32
elopioI'll give it a try.14:32
Saviqelopio, it's fine, there's no way to determine which of autopilot-qt{4,5} it needs14:32
Saviqelopio, so you need to add both autopilot-qt5 and qttestability-autopilot14:33
elopioSaviq: yes, but I'll make a new branch, if you don't mind.14:33
Saviqelopio, your call14:33
Saviqelopio, I can take care of that then14:34
Saviqelopio, thought it'd make sense in the same branch, but if you think not, I'll do14:34
elopioSaviq: don't worry, I'm have the changes and I'll push.14:34
Saviqk14:34
Saviqwe'll know soon enough14:34
elopioSaviq: the toolkit depends on libautopilot-qt, so we'll need to change it there too to be useful. I'll file bugs about that.14:39
Saviqelopio, sure, same as with py2 ;)14:39
Saviqelopio, http://paste.ubuntu.com/10550731/14:40
elopiooh shit :)14:40
Saviqso yeah, I think we didn't get the memo to transition from libautopilot-qt14:40
elopiowe can't make autopilot-qt5 depend on qttestability-autopilot, because there would be a loop in there.14:45
elopiomaybe a libautopilot-qt5 makes sense.14:45
elopioI'll talk to veebers on monday. He should know.14:45
dandradermzanetti, are you still reviewing the shellRotation branch?15:39
mzanettidandrader, not right now... but given we can't go into vivid with that it's not totally high priority atm is it?15:40
mzanettidandrader, I'll continue next week for sure. but feel like having to fix that darn dpr thing15:40
dandradermzanetti, we can't land right now becuse it needs the autopilot test paulliu15:42
mzanettidandrader, ack. so yes, I'll continue reviewing, but it's not on the top of my priority list atm15:43
dandradermzanetti, as for landing in vivid, as far as I understood with would just merge it to trunk and create a separate branch from which we would cherry pick changes for vivid15:43
dandraderSaviq, right? ^15:43
mzanettiyeah... that sounds about right15:43
dandradermzanetti, sure15:43
Saviqdandrader, we'd like to do that, yes, but we don't have buy-in from foundations yet15:44
dandraderSaviq, or we could do the other way round: have a branch like lp:unity8/devel and cherry pick non-feature stuff onto lp:unity8, as mir guys were doing in the past15:45
mzanettilet's call it staging15:45
mzanetti:D15:45
Saviqdandrader, long story short, we don't want that, not without automation running on those15:46
Saviqdandrader, we have a long-standing request in the CI team backlog for that, actually15:48
Saviqdandrader, but ultimately, without an archive to put the built packages into, we're doomed, 'cause as soon as you change something in unity-api, for example, you can't build against vivid15:49
Saviqso we need a target for the releases, which we'd like to be a PPA on top of vivid15:49
=== jhodapp_ is now known as jhodapp
=== dandrader is now known as dandrader|lunch
=== jfunk is now known as jfunk-lunch
=== alan_g is now known as alan_g|EOW
=== dandrader|lunch is now known as dandrader
=== jfunk-lu_ is now known as jfunk

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!