[Gtkradiant] [Bug 183] New: some toggles have to be fired twice

[gtkradiant@zerowing.idsoftware.com]

http://zerowing.idsoftware.com/bugzilla/show_bug.cgi?id=183

           Summary: some toggles have to be fired twice
           Product: GtkRadiant
           Version: 1.1-TA.1-nightly
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: editor
        AssignedTo: ttimo@idsoftware.com
        ReportedBy: ttimo@idsoftware.com


Reported with patch by Gef:

The attached patch fixes the problem I was having with toggles needing to
be triggered twice to toggle them back on.
eg: 
Patch's are drawn
-> ctrl-p 
Patch's are not drawn
-> ctrl-p
Patch's are not drawn
-> ctrl-p 
Patch's are drawn


The patch just comments out the if(!keydown) section that is
intended to handle keyup events. From my understanding of the event handling
for these particular events is that the value of keydown is passed to the 
function
that handles the events and is handled there anyway.
The consequence of the code block that is that every second time a toggle key 
was
pressed, keydown would == false, and would skip the normal handling of the 
keypress.


>From what I can tell, commenting out this section of code has no affect on the 
functioning
of the movement keys which the code block is intended to handle.
Obviously since I'm the only person who was having this problem the patch would
need to be verified for ppl who were not having this prob. I have no idea why
I'm the only person who was having this prob either, I thought it may be the 
gtk/glib 
version 1.2.10 that i'm using. *shrug*


Gef :]

--------------
I tried and could not reproduce on my dev box. I am going to rebuilt Gtk 1.2.10 
on my test machine and see if I have the same problem. I can't reproduce the 
issue on win32 either.



------- You are receiving this mail because: -------
Whoops!  I have no idea!