Bug 302133 - Strange tab sequence in Breakpoint properties UI
diff --git a/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/debug/ui/breakpoints/JavaBreakpointConditionEditor.java b/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/debug/ui/breakpoints/JavaBreakpointConditionEditor.java
index 9fac782..629cefa 100644
--- a/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/debug/ui/breakpoints/JavaBreakpointConditionEditor.java
+++ b/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/debug/ui/breakpoints/JavaBreakpointConditionEditor.java
@@ -236,7 +236,7 @@
* @return top level control
*/
public Control createControl(Composite parent) {
- Composite controls = SWTFactory.createComposite(parent, parent.getFont(), 3, 1, GridData.FILL_HORIZONTAL, 0, 0);
+ Composite controls = SWTFactory.createComposite(parent, parent.getFont(), 2, 1, GridData.FILL_HORIZONTAL, 0, 0);
fConditional = new Button(controls, SWT.CHECK);
fConditional.setText(PropertyPageMessages.JavaBreakpointConditionEditor_0);
fConditional.setLayoutData(new GridData(SWT.BEGINNING, SWT.CENTER, false, false));
@@ -247,9 +247,10 @@
setDirty(PROP_CONDITION_ENABLED);
}
});
- fWhenTrue = SWTFactory.createRadioButton(controls, PropertyPageMessages.JavaBreakpointConditionEditor_1);
+ Composite radios = SWTFactory.createComposite(controls, controls.getFont(), 2, 1, GridData.FILL_HORIZONTAL, 0, 0);
+ fWhenTrue = SWTFactory.createRadioButton(radios, PropertyPageMessages.JavaBreakpointConditionEditor_1);
fWhenTrue.setLayoutData(new GridData());
- fWhenChange = SWTFactory.createRadioButton(controls, PropertyPageMessages.JavaBreakpointConditionEditor_2);
+ fWhenChange = SWTFactory.createRadioButton(radios, PropertyPageMessages.JavaBreakpointConditionEditor_2);
fWhenChange.setLayoutData(new GridData());
fWhenTrue.addSelectionListener(new SelectionAdapter() {
public void widgetSelected(SelectionEvent e) {
diff --git a/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/breakpoints/StandardJavaBreakpointEditor.java b/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/breakpoints/StandardJavaBreakpointEditor.java
index 416d7af..5efd0d7 100644
--- a/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/breakpoints/StandardJavaBreakpointEditor.java
+++ b/org.eclipse.jdt.debug.ui/ui/org/eclipse/jdt/internal/debug/ui/breakpoints/StandardJavaBreakpointEditor.java
@@ -64,7 +64,7 @@
}
protected Control createStandardControls(Composite parent) {
- Composite composite = SWTFactory.createComposite(parent, parent.getFont(), 5, 1, 0, 0, 0);
+ Composite composite = SWTFactory.createComposite(parent, parent.getFont(), 4, 1, 0, 0, 0);
fHitCountButton = SWTFactory.createCheckButton(composite, PropertyPageMessages.JavaBreakpointPage_4, null, false, 1);
fHitCountButton.setLayoutData(new GridData());
fHitCountButton.addSelectionListener(new SelectionAdapter() {
@@ -82,9 +82,10 @@
}
});
SWTFactory.createLabel(composite, "", 1); // spacer //$NON-NLS-1$
- fSuspendThread = SWTFactory.createRadioButton(composite, PropertyPageMessages.JavaBreakpointPage_7, 1);
+ Composite radios = SWTFactory.createComposite(composite, composite.getFont(), 2, 1, GridData.FILL_HORIZONTAL, 0, 0);
+ fSuspendThread = SWTFactory.createRadioButton(radios, PropertyPageMessages.JavaBreakpointPage_7, 1);
fSuspendThread.setLayoutData(new GridData());
- fSuspendVM = SWTFactory.createRadioButton(composite, PropertyPageMessages.JavaBreakpointPage_8, 1);
+ fSuspendVM = SWTFactory.createRadioButton(radios, PropertyPageMessages.JavaBreakpointPage_8, 1);
fSuspendVM.setLayoutData(new GridData());
fSuspendThread.addSelectionListener(new SelectionAdapter() {
public void widgetSelected(SelectionEvent e) {