Code review comment for lp:~andreas-pokorny/mir/fix-1580774
Revision history for this message
| Andreas Pokorny (andreas-pokorny) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
> I still think power_mode needs to be ignored. 'used' is the only field that
> matters. If you're moving the mouse then your used displays should be already
> on or turning on as a result of the mouse movement.
>
> A monitor that is 'used' but also 'off' (which probably never exists anyway)
> should not in theory disallow the cursor moving to that monitor. In fact, you
> need the cursor to be able to reach it to ensure it turns on.
Hmm I fully agree. But I am unsure if that works out - especially since unity8 and usc may have a different sense of cursor placement. Unity8 only extracts the relative motion, and may have a different resulting location.. Yes this is wrong we should have a better way to express that. So I think we should only do that change on lp:mir and let 0.22 and 0.23 use the logic as is.
* Needs discussion * Who would turn on the screen in that scenario?