[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rdesktop-devel
Subject:    [rdesktop-devel] [ rdesktop-Patches-1569293 ] Release keyboard grab
From:       "SourceForge.net" <noreply () sourceforge ! net>
Date:       2008-09-21 18:58:32
Message-ID: E1KhU8e-0000hV-2x () d45xhf1 ! ch3 ! sourceforge ! com
[Download RAW message or body]

Patches item #1569293, was opened at 2006-10-02 15:34
Message generated for change (Comment added) made by wingel
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=381349&aid=1569293&group_id=24366

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Christer Weinigel (wingel)
Assigned to: Nobody/Anonymous (nobody)
Summary: Release keyboard grab with Ctrl-Alt.

Initial Comment:
I've been using VMware for a while and wanted to be
able to release the keyboard grab with Ctrl-Alt as
VMware allows me to do, so I hacked together this patch
to do so.  Besides allowing me to use Ctrl-Alt I also
added a lazy grabbing, so that rdesktop doesn't grab
the keyboard until I press a mouse button or the
keyboard.  

This patch has a few problems that have to be fixed if
you want to integrate this into rdesktop.

I haven't added any command line flags to
enable/disable ctrl-alt or the lazy grab.  I have added
variables in rdesktop.c that control these functions,
so this should be easy to do.

The combination ctrl-alt is hardcoded and the code that
checks for these keys is a bit ugly.  It'd be a lot
better if the key combination could be configured by
the user, for example, look at how VMware does this.

If I use full screen mode and press ctrl-alt to release
the focus and then switch to another application, the
rdesktop window still stays on top.  If I've read the
code correctly, I think this is because the window has
been created with the redirectoverrride flag.  For the
function to be useful with full screen, this has to be
fixed.


----------------------------------------------------------------------

>Comment By: Christer Weinigel (wingel)
Date: 2008-09-21 20:58

Message:
File Added: rdesktop-1.6.0-ctrlalt.patch

----------------------------------------------------------------------

Comment By: Christer Weinigel (wingel)
Date: 2007-04-09 00:07

Message:
Logged In: YES 
user_id=327600
Originator: YES

Sorry for the extra noise here.  I just decided to add a command line
option "-U" to enable ctrl-alt and to remove the separate "lazy" flag since
using ctrl-alt without lazy mode or vice versa was rather useless.  So,
here's yet another variant of the patch.

File Added: rdesktop-1.5.0-ctrlalt.patch

----------------------------------------------------------------------

Comment By: Christer Weinigel (wingel)
Date: 2007-04-08 23:40

Message:
Logged In: YES 
user_id=327600
Originator: YES

I have updated my ctrl-alt/lazy grabbing patch to work with
rdesktop-1.5.0.  

There is one small change in the logic of the code, instead of grabbing
the focus when any key except ctrl or alt is pressed it only grabs the
focus if a non-modifier key is pressed.  This allows me to press Ctrl-Alt
to ungrab the focus and then press Shift-Ctrl-Alt to move the rdesktop
window between different virtual desktops in metacity.

After playing around with the different command line parameters to
rdesktop, I figured out the magic incantation to get rdesktop to behave
just the way I want it to:

    rdesktop -D -g 1280x1024 host

If I do this, whenever rdesktop has focus I get a full screen window (on
my 1280x1024 screen) and everything looks just as if I was sitting on the
console of the Windows machine.  If the rdesktop window loses focus the
Linux window manager appears and I can switch to a different virtual
desktop without any problem.

File Added: rdesktop-1.5.0-ctrlalt.patch

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=381349&aid=1569293&group_id=24366

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
rdesktop-devel mailing list
rdesktop-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rdesktop-devel
[prev in list] [next in list] [prev in thread] [next in thread]