Bug 41926 - Ctrl+V getting the wrong focus
Summary: Ctrl+V getting the wrong focus
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: 3.0 M3   Edit
Assignee: Douglas Pollock CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 41944 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-08-25 11:47 EDT by Tod Creasey CLA
Modified: 2003-08-26 12:26 EDT (History)
2 users (show)

See Also:

Attachments
Workbench Patch (910 bytes, patch)
2003-08-26 11:30 EDT, Douglas Pollock CLA 		no flags Details | Diff
AbstractTextEditor & TextOperationAction Patch (2.85 KB, patch)
2003-08-26 11:32 EDT, Douglas Pollock CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tod Creasey CLA 2003-08-25 11:47:13 EDT 
20030825

When I try and do a paste into a find dialog Ctrl+V always pastes into the 
editor behind it and not into the dialog.

STEPS
1) Open an editor
2) Copy something
3) Hit Ctrl+F to open the Find Dialog
4) Highlight on of the fields
5) Hit Ctrl+V - it will paste into the dialog behind
Comment 1 Tod Creasey CLA 2003-08-26 08:47:57 EDT 
*** Bug 41944 has been marked as a duplicate of this bug. ***
Comment 2 Tod Creasey CLA 2003-08-26 09:13:37 EDT 
Marking as M3
Comment 3 Douglas Pollock CLA 2003-08-26 11:30:49 EDT 
Created attachment 5857 [details]
Workbench Patch

Workbench shouldn't consume key events to disabled actions.
Comment 4 Douglas Pollock CLA 2003-08-26 11:32:33 EDT 
Created attachment 5858 [details]
AbstractTextEditor & TextOperationAction Patch

TextOperationAction checks to see if its associated Control is in focus before
declaring itself enabled.  AbstractTextEditor provides a public method for
checking a Control against its associated Control.
Comment 5 Douglas Pollock CLA 2003-08-26 11:38:05 EDT 
The last patch is a bit of a hack.  Some discussion needs to be made about the
way in which dialogs will be handled (in general) by the new command
architecture.  The current slate of actions do not support awareness of control
focus.
Comment 6 Tod Creasey CLA 2003-08-26 12:06:18 EDT 
We have updated the test for a dialog by checking the parent rather than the 
modality. This is still just a temporary workaround and needs to be addressed 
by the context work.
Comment 7 Douglas Pollock CLA 2003-08-26 12:26:07 EDT 
This fix turns out to be the plan for M3, with no hacks.  Future releases will
allow commands in dialogs, but that is an enhancement (there is no current
regression in functionality).