Bug 480806

Summary:	Remove unnecessary setFocus() calls that tried to avoid overriding the default button
Product:	[Eclipse Project] JDT	Reporter:	Markus Keller <markus.kell.r>
Component:	UI	Assignee:	Markus Keller <markus.kell.r>
Status:	NEW ---	QA Contact:
Severity:	enhancement
Priority:	P3	CC:	markus.kell.r, noopur_gupta
Version:	4.6
Target Milestone:	---
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/59048
Whiteboard:
Bug Depends on:	471329, 479727
Bug Blocks:

Description Markus Keller

2015-10-27 14:55:24 EDT

Follow-up to bug 471329, bug 479727.

Remove unnecessary setFocus() calls that tried to avoid overriding the default button. Also check MessageDialog and subclasses.

Test on all platforms:
- Mac: first control/button should get focus by default
- Win/GTK: first control should get focus unless it's a push button. In that case, the default button has to get the default focus.

Comment 1 Eclipse Genie

2015-10-27 14:59:44 EDT

New Gerrit change created: https://git.eclipse.org/r/59048

Comment 2 Markus Keller

2015-10-27 15:04:36 EDT

(In reply to Eclipse Genie from comment #1)
> New Gerrit change created: https://git.eclipse.org/r/59048

This is just a first cut that fixes the problems mentioned in bug 471329. Need to check all setFocus() calls.