Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 314497

Summary: Changing value of runtime or configuration combo box (using the keyboard) in new EJB project wizard changes focus to the project name field
Product: [WebTools] WTP EJB Tools Reporter: Hari Shankar <hshanka>
Component: jst.ejbAssignee: Hari Shankar <hshanka>
Status: RESOLVED FIXED QA Contact: Kaloyan Raev <kaloyan>
Severity: normal    
Priority: P3 CC: ccc, jsholl, neil.hauge
Version: 3.2Flags: jsholl: pmc_approved? (david_williams)
jsholl: pmc_approved? (raghunathan.srinivasan)
jsholl: pmc_approved? (naci.dai)
deboer: pmc_approved+
neil.hauge: pmc_approved+
jsholl: pmc_approved? (kaloyan)
cbridgha: review+
Target Milestone: 3.2 RC3   
Hardware: PC   
OS: Windows XP   
Whiteboard: PMC_approved
Attachments:
Description Flags
patch none

Description Hari Shankar CLA 2010-05-26 12:11:16 EDT
Build Identifier: 32

Open the new EJB project wizard, and using the keyboard, move the cursor to the runtime or the configuration combo, and using the keyboard again, change the value of the combo (use up arrow or down arrow keys). The project name control gets the focus

Reproducible: Always
Comment 1 Hari Shankar CLA 2010-05-26 12:13:27 EDT
Created attachment 170035 [details]
patch

The reason the focus was shifting was because an explicit setFocus is called in the createProjectNameGroup. This method gets called whenever the facet combo selection is changed.
Comment 2 Chuck Bridgham CLA 2010-05-26 13:54:01 EDT
approve
Comment 3 Jason Sholl CLA 2010-05-26 17:57:16 EDT
    * Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 

This is impacting accessibility.

    * Is there a work-around? If so, why do you believe the work-around is insufficient? 

Yes, extra key strokes

    * How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? 

This is UI only.

    * Give a brief technical overview. Who has reviewed this fix? 

Chuck reviewed this

    * What is the risk associated with this fix? 

None
Comment 4 Carl Anderson CLA 2010-05-27 13:32:27 EDT
Committed to HEAD for WTP 3.2 RC3