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.

Bug 16932 - in Add Shortcut - keyboard focus should be set to text field
Summary: in Add Shortcut - keyboard focus should be set to text field
Status: CLOSED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Window System (show other bugs)
Version: 3.x
Hardware: PC Linux
: P4 blocker (vote)
Assignee: Peter Zavadsky
URL:
Keywords: A11Y
Depends on:
Blocks:
 
Reported: 2001-10-25 05:32 UTC by _ jrichard
Modified: 2008-12-22 20:51 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
patch to set initial focus to the text field for the add shortcut dialog (1.50 KB, patch)
2001-10-27 15:38 UTC, _ jrichard
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description _ jrichard 2001-10-25 05:32:17 UTC
When adding a new shortcut, the Add Shortcut dialog comes up
with no keyboard focus.  It is much more convienent if the 
Add Shortcut dialog came up with focus already set to 
the text field.
Comment 1 _ jrichard 2001-10-27 15:38:42 UTC
Created attachment 3173 [details]
patch to set initial focus to the text field for the add shortcut dialog
Comment 2 Jiri Mzourek 2001-10-30 14:34:56 UTC
Component changed.
Comment 3 Jiri Mzourek 2001-10-30 15:56:07 UTC
reassigned
Comment 4 _ jrichard 2001-10-31 03:25:31 UTC
If this is patched, it would be worth adding a note that
this might be fixed in JDK 1.4 beta 3+.  I tried beta 2, but
it still requires the patch for the focus to work.
Comment 5 Peter Zavadsky 2001-10-31 13:29:25 UTC
Fixed in [main-trunk].

Fix:
core/../core/NbPresenter.java [1.49].

Note: The solution is general for all dialogs created via
DialogDescriptor (NotifyDescriptor). When added message component is
of JComponent type first default request focus is called (so for our
case directly to the text field is set), just in case it fails is
requested focus for message component directly.
Comment 6 _ jrichard 2001-11-17 18:07:41 UTC
Using the 16-Nov build, I still don't see this for the Keyboard
Shortcuts dialog.  Is the fix merged?  

Also, see 17794, the "New" wizard has the same problem.
Comment 7 Peter Zavadsky 2001-11-19 13:00:49 UTC
I have last build and it works. Closing as workforme.

For sure if we are talking about the same thing;

1) Go to Tools | Keybard Shortcuts... and invoke it.
2) Select some actions , button Add... enables.
3) Press the button  and Add Shortcut dialog appears.
4) The textfield has the focus, caret is there and if you
type something you set the sortcut.

That's as it shuould be.

Please reopen if you have something else in mind.

Comment 8 _ jrichard 2001-11-20 03:01:53 UTC
OK, I do see this fix on Solaris (I'm primarily testing on Solaris 2.6)
but only with JDK 1.3.1.  With JDK 1.4 beta 3, I see quite
unusual focus handling on both Solaris and Linux.


Do you think this could be a bug in JDK 1.4 beta 3?  The
focus handing should be backwardly compatable, right?  If
so, I don't suppose it is easy to point the swing team to
this code in netbeans?

Feel free to close this if it looks like a JDK bug...
Comment 9 Peter Zavadsky 2001-11-20 08:25:35 UTC
You are right John, it occures now on jdk1.4. But I should first look
in if the problem causes our code. Added keyword jdk_1.4.
Comment 10 Peter Zavadsky 2001-11-20 10:35:10 UTC
Closing as later.
Note: This issue will be possible to fix when we'll came to jdk1.4
purelly, which is not now.
Now we can't take the advantage of the new 1.4 code, dealing with
focus policy. There are incompatibilties between jdk1.3 and 1.4 focus
dealing and one of it is related to this issue. i.e. default focus
traversability.

See:
http://java.sun.com/j2se/1.4/docs/api/java/awt/doc-files/FocusSpec.html#Incompatibilities
Comment 11 Jiri Mzourek 2001-11-20 12:32:52 UTC
added A11Y keyword
Comment 12 Marian Mirilovic 2002-01-11 17:07:05 UTC
reopen
Comment 13 Peter Zavadsky 2002-01-17 14:08:12 UTC
It seems fix for #17794 solves also this issue.
Comment 14 Marian Mirilovic 2002-05-30 14:21:19 UTC
verified in [nb_dev](20020530)