[ BUILD # : 200906241340 ]
[ JDK VERSION : 1.6.* ]
1. Open any SQL editor
2. Ctrl + F to open Find feature
3. Press ESC to cancel the find
The focus goes to Connection combo box instead editor.
Created attachment 84408 [details]
I attached an proposal patch. I am not sure if it is the better way to solve it, but define combo as not focusable, as
did with buttons, is not a choice since it broke the shortcut. Ideas are welcome.
Maybe we can track it by NetFIX if you agree ;)
Created attachment 84460 [details]
An better patch
IMHO this second patch is much better. Please review it when possible.
Thanks for the patch. I'm going to review it and let you know asap. Thanks
If you don't mind, I wish to mark this as tracked by NetFIX program.
Thanks in advance.
Jirko, Michel only wants to add this bug to the pool  of bugs that were patched by community. I think that's OK,
Michel, your patch looks good and works for me. I'm integrating into codeline. I also add NETFIX into issue's keywords.
Thanks for the fix
Btw. the same problem I found while editing a form file (in the edit mode). After cancel the fix the focus returns to
Integrated into 'main-golden', will be available in build *200907100200* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
User: Jiri Rechtacek <email@example.com>
Log: #168108: SQL editor should focus at editor after close Find feature instead Connection combo box; (applied hmichel's patch)
@jrechtacek I filed issue #168397 and I will take a look at it too and thanks for quick review and integration.
BTW I will mark this as 67patch-candidate, probably it could be transplanted to 6.7 code base for patch 2 if we have one
since it probably is late for patch 1. I will verify it as soon I finish the download. ;)
The fix has been ported into the release67_fixes repository.
As Standa points out here:
The patch applied in this issue is also suffering from the flaw that I have explained when rollbacking the fix in 168397. Namely when instantiating toolbar presenter for this action the AWT waits for the editor pane to be fully initialised. This is not very good from the performance point of view. Please consider rollbacking this change as well.
I am just reopening the issue - please have a look and potentially revert ... Thanks.
Tried to reproduce with 7.4 and this bug does not appear anymore.
Closing as WORKSFORME.