Modified the UI to show controls in a single line

Change-Id: I6a08929ccbfea821f5b9a30097503c10d3bcf916

Author: praveen-skp

bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/WorkbenchMessages.java

@@ -745,7 +745,7 @@ public class WorkbenchMessages extends NLS {
 	// ==============================================================================
 	public static String PinEditorAction_toolTip;
 	public static String WorkbenchPreference_reuseEditors;
-	public static String WorkbenchPreference_reuseEditorsThreshold;
+	public static String WorkbenchPreference_reuseEditors_closing;
 	public static String WorkbenchPreference_reuseEditorsThresholdError;
 	public static String WorkbenchPreference_recentFiles;
 	public static String WorkbenchPreference_recentFilesError;

bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/dialogs/EditorsPreferencePage.java

@@ -18,20 +18,22 @@
 
 import static org.eclipse.swt.events.SelectionListener.widgetSelectedAdapter;
 
+import org.eclipse.jface.layout.GridDataFactory;
+import org.eclipse.jface.layout.GridLayoutFactory;
 import org.eclipse.jface.preference.FieldEditor;
 import org.eclipse.jface.preference.IPreferenceStore;
 import org.eclipse.jface.preference.IntegerFieldEditor;
 import org.eclipse.jface.preference.PreferencePage;
 import org.eclipse.jface.preference.StringFieldEditor;
 import org.eclipse.jface.util.IPropertyChangeListener;
+import org.eclipse.jface.widgets.LabelFactory;
 import org.eclipse.osgi.util.NLS;
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.Button;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Control;
-import org.eclipse.swt.widgets.Group;
 import org.eclipse.ui.IWorkbench;
 import org.eclipse.ui.IWorkbenchPreferenceConstants;
 import org.eclipse.ui.IWorkbenchPreferencePage;
@@ -47,9 +49,8 @@
  * The Editors preference page of the workbench.
  */
 public class EditorsPreferencePage extends PreferencePage implements IWorkbenchPreferencePage {
-	private static final int REUSE_INDENT = 20;
 
-	protected Group editorReuseGroup;
+	protected Composite editorReuseGroup;
 
 	private Button reuseEditors;
 
@@ -213,14 +214,9 @@ protected void updateValidState() {
 	 * preferences.
 	 */
 	protected void createEditorReuseGroup(Composite composite) {
-		editorReuseGroup = new Group(composite, SWT.LEFT);
-		GridLayout layout = new GridLayout();
-		// Line up with other entries in preference page
-		layout.marginWidth = 0;
-		layout.marginHeight = 0;
-		editorReuseGroup.setLayout(layout);
+		editorReuseGroup = new Composite(composite, SWT.LEFT);
+		editorReuseGroup.setLayout(GridLayoutFactory.fillDefaults().numColumns(3).spacing(0, 0).create());
 		editorReuseGroup.setLayoutData(new GridData(GridData.HORIZONTAL_ALIGN_FILL | GridData.GRAB_HORIZONTAL));
-		editorReuseGroup.setText(WorkbenchMessages.WorkbenchPreference_closeMode);
 
 		reuseEditors = new Button(editorReuseGroup, SWT.CHECK);
 		reuseEditors.setText(WorkbenchMessages.WorkbenchPreference_reuseEditors);
@@ -234,20 +230,20 @@ protected void createEditorReuseGroup(Composite composite) {
 		}));
 
 		editorReuseIndentGroup = new Composite(editorReuseGroup, SWT.LEFT);
-		GridLayout indentLayout = new GridLayout();
-		indentLayout.marginLeft = REUSE_INDENT;
-		indentLayout.marginWidth = 0;
-		editorReuseIndentGroup.setLayout(indentLayout);
-		editorReuseIndentGroup.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
+		editorReuseIndentGroup.setLayout(GridLayoutFactory.fillDefaults().create());
+		editorReuseIndentGroup
+				.setLayoutData(GridDataFactory.fillDefaults().grab(true, false).create());
 
 		editorReuseThresholdGroup = new Composite(editorReuseIndentGroup, SWT.LEFT);
-		layout = new GridLayout();
+		GridLayout layout = new GridLayout();
 		layout.marginWidth = 0;
 		editorReuseThresholdGroup.setLayout(layout);
-		editorReuseThresholdGroup.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
+		editorReuseThresholdGroup
+				.setLayoutData(GridDataFactory.fillDefaults().grab(true, false).create());
 
-		reuseEditorsThreshold = new IntegerFieldEditor(IPreferenceConstants.REUSE_EDITORS,
-				WorkbenchMessages.WorkbenchPreference_reuseEditorsThreshold, editorReuseThresholdGroup);
+		reuseEditorsThreshold = new IntegerFieldEditor(IPreferenceConstants.REUSE_EDITORS, "", //$NON-NLS-1$
+				editorReuseThresholdGroup);
+		reuseEditorsThreshold.getLabelControl(editorReuseThresholdGroup).dispose();
 
 		reuseEditorsThreshold.setPreferenceStore(WorkbenchPlugin.getDefault().getPreferenceStore());
 		reuseEditorsThreshold.setPage(this);
@@ -260,6 +256,8 @@ protected void createEditorReuseGroup(Composite composite) {
 		reuseEditorsThreshold.getTextControl(editorReuseThresholdGroup).setEnabled(reuseEditors.getSelection());
 		reuseEditorsThreshold.setPropertyChangeListener(validityChangeListener);
 
+		LabelFactory.newLabel(SWT.NONE).text(WorkbenchMessages.WorkbenchPreference_reuseEditors_closing)
+				.create(editorReuseGroup);
 	}
 
 	/**

bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/messages.properties

@@ -493,10 +493,10 @@ OpenPerspectiveDialogAction_tooltip=Open Perspective
 
 #---- General Preferences----
 PreferencePage_noDescription = (No description available)
-PreferencePageParameterValues_pageLabelSeparator = \ >\ 
+PreferencePageParameterValues_pageLabelSeparator = \ >\
 ThemingEnabled = E&nable theming
 ThemeChangeWarningText = Restart for the theme changes to take full effect
-ThemeChangeWarningTitle = Theme Changed 
+ThemeChangeWarningTitle = Theme Changed
 # --- Workbench -----
 WorkbenchPreference_openMode=Open mode
 WorkbenchPreference_closeMode=Close mode
@@ -730,8 +730,8 @@ PageLayout_missingRefPart=Referenced part does not exist yet: {0}.
 # Keys used in the reuse editor which is released as experimental.
 # ==============================================================================
 PinEditorAction_toolTip=Pin Editor
-WorkbenchPreference_reuseEditors=&Close editors automatically
-WorkbenchPreference_reuseEditorsThreshold=Number of opened editors before closi&ng:
+WorkbenchPreference_reuseEditors=&Close oldest if there are more than
+WorkbenchPreference_reuseEditors_closing=editors open
 WorkbenchPreference_reuseEditorsThresholdError=The number of opened editors should be more than 0.
 WorkbenchPreference_recentFiles=Size of &recently opened files list:
 WorkbenchPreference_recentFilesError=The size of the recently opened files list should be between 0 and {0}.