This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
[ 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] Proposed patch
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 ;) Regards
Created attachment 84460 [details] An better patch
IMHO this second patch is much better. Please review it when possible. Regards
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 [1] of bugs that were patched by community. I think that's OK, isn't it? [1] http://wiki.netbeans.org/NetFIXIssues
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
core-main/rev/45bf306ec44f
Btw. the same problem I found while editing a form file (in the edit mode). After cancel the fix the focus returns to [Edit][Design] buttons.
Integrated into 'main-golden', will be available in build *200907100200* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-golden/rev/45bf306ec44f User: Jiri Rechtacek <jrechtacek@netbeans.org> 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. ;) Regards
v.
The fix has been ported into the release67_fixes repository. http://hg.netbeans.org/release67_fixes/rev/3988cd215d53
As Standa points out here: http://netbeans.org/bugzilla/show_bug.cgi?id=168397#c14 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.