[Konsole-devel] Review Request 114242: konsole (kde4) - place new window under cursor possition

[Kurt Hindenburg]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114242/#review45302
-----------------------------------------------------------

Ship it!


Thanks - I don't have a dual monitor setup ATM - it works fine here w/ 1 monitor - session management still works - I'll commit it.

- Kurt Hindenburg


On Dec. 2, 2013, 6:14 p.m., David Heidelberger wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114242/
> -----------------------------------------------------------
> 
> (Updated Dec. 2, 2013, 6:14 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> Use cases
> 1) When you drag a tab from Konsole open on screen and drag tab into second screen. Actual state: it always open on primary screen, ignoring cursor position.
> 2) When you open konsole window, it should be opened near your cursor
> 
> This is something really simplify life of users with two and more monitors.
> 
> 
> Diffs
> -----
> 
>   src/MainWindow.cpp 964c277 
> 
> Diff: http://git.reviewboard.kde.org/r/114242/diff/
> 
> 
> Testing
> -------
> 
> http://ixit.cz/konsole-multi-screen-one-line-fix/ (including video, how naturally it works)
> 
> Draging tabs, creating windows.
> 
> Dual monitor setup (different resolutions tested).
> 
> This change respect fullscreen settings (so, when is position set to fullscreen, it'll remember). Also placing respect screen boundaries, so window is always rendered into visible area on actual screen (so nothing between two screens or in not visible area, if isn't resized manually)
> 
> 
> Thanks,
> 
> David Heidelberger
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20131207/d27f94e9/attachment.html>