thaolt / qscite

Automatically exported from code.google.com/p/qscite
GNU General Public License v2.0
0 stars 0 forks source link

Tab key press in QTerminal changes focus #29

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Open QSciTE
2. Activate the terminal
3. Press the tab key

Expected: the key press should be passed directly to the pseudo terminal

Actual: the key press seems to be intercepted by the GUI and causes the
focus to change

Original issue reported on code.google.com by jpsut...@gmail.com on 22 Aug 2008 at 5:23

GoogleCodeExporter commented 8 years ago
Ben, I've only tested this bug on Linux.  Could you verify it's a problem on OS 
X? 
I'll test it on Win32 later on.

Original comment by jpsut...@gmail.com on 22 Aug 2008 at 5:29

GoogleCodeExporter commented 8 years ago
Saw it on OS X a while ago; tried a few things that didn't work; for some reason
didn't file it at the time.

Original comment by bfrobin...@gmail.com on 22 Aug 2008 at 12:45

GoogleCodeExporter commented 8 years ago
I narrowed down the problem to SVN r147.  That is the first revision that the 
problem
showed up.

Original comment by jpsut...@gmail.com on 22 Aug 2008 at 1:27

GoogleCodeExporter commented 8 years ago
Most of the QTerminal-related changes in that revision look like my attempts to 
fix
this. The setTextInteractionFlags() in qterminal.cpp line 25 is probably the 
root cause.

Original comment by bfrobin...@gmail.com on 22 Aug 2008 at 2:07

GoogleCodeExporter commented 8 years ago
I commented that line out in my last commit.  It seems to have fixed it.  Why 
was
that line added originally?

Original comment by jpsut...@gmail.com on 22 Aug 2008 at 2:11

GoogleCodeExporter commented 8 years ago
I was hoping to disable drag-and-drop until I could make it behave correctly
(dragging selected text doesn't remove it, dropped text passed to the shell). It
didn't cross my mind that that was what had happened to Tab until you traced 
the bug
to that revision.

Original comment by bfrobin...@gmail.com on 22 Aug 2008 at 2:20

GoogleCodeExporter commented 8 years ago
Fix in r152 confirmed on Linux. Will test on OS X tonight.

Original comment by bfrobin...@gmail.com on 22 Aug 2008 at 2:55