From 4a3f51c8e9e3bdb39e17c53183fdf146d935eaa1 Mon Sep 17 00:00:00 2001
From: Evgeny Fradkin <Eugene.Fradkin@gmail.com>
Date: Sat, 3 Mar 2018 00:07:16 +0300
Subject: [PATCH] #1599 Mock Data wizard settings page is a bit redesigned

---
 .../mockdata/MockDataWizardPageSettings.java  | 22 ++++++++++++++-----
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/plugins/org.jkiss.dbeaver.ext.mockdata/src/org/jkiss/dbeaver/ext/mockdata/MockDataWizardPageSettings.java b/plugins/org.jkiss.dbeaver.ext.mockdata/src/org/jkiss/dbeaver/ext/mockdata/MockDataWizardPageSettings.java
index 071669d981..fa7a625772 100644
--- a/plugins/org.jkiss.dbeaver.ext.mockdata/src/org/jkiss/dbeaver/ext/mockdata/MockDataWizardPageSettings.java
+++ b/plugins/org.jkiss.dbeaver.ext.mockdata/src/org/jkiss/dbeaver/ext/mockdata/MockDataWizardPageSettings.java
@@ -91,13 +91,19 @@ public class MockDataWizardPageSettings extends ActiveWizardPage<MockDataExecute
                 }
             };
 
-            Group settingsGroup = UIUtils.createControlGroup(
-                    composite, MockDataMessages.tools_mockdata_wizard_page_settings_group_settings, 4, GridData.FILL_HORIZONTAL, 0);
-
-            this.entityNameText = UIUtils.createLabelText(settingsGroup, "Entity", "", SWT.BORDER | SWT.READ_ONLY);
+            Composite entityPlaceholder = UIUtils.createPlaceholder(composite, 2);
+            this.entityNameText = UIUtils.createLabelText(entityPlaceholder, "Entity", "", SWT.NONE | SWT.READ_ONLY);
             GridData gd = new GridData(GridData.HORIZONTAL_ALIGN_BEGINNING);
             gd.widthHint = 230;
-            this.entityNameText.setLayoutData(gd);
+            gd.verticalIndent = 5;
+            gd.horizontalIndent = 9;
+            entityPlaceholder.setLayoutData(gd);
+
+            Group settingsGroup = UIUtils.createControlGroup(
+                    composite, MockDataMessages.tools_mockdata_wizard_page_settings_group_settings, 4, GridData.FILL_HORIZONTAL, 0);
+            gd = new GridData(GridData.FILL_HORIZONTAL);
+            gd.verticalIndent = 5;
+            settingsGroup.setLayoutData(gd);
 
             this.removeOldDataCheck = UIUtils.createCheckbox(
                     settingsGroup,
@@ -121,11 +127,14 @@ public class MockDataWizardPageSettings extends ActiveWizardPage<MockDataExecute
 
         {
             Group generatorsGroup = UIUtils.createControlGroup(composite, "Generators", 5, GridData.FILL_BOTH, 0);
+            GridData gd = new GridData(GridData.FILL_BOTH);
+            gd.verticalIndent = 5;
+            generatorsGroup.setLayoutData(gd);
 
             columnsTableViewer = new TableViewer(generatorsGroup, SWT.BORDER | SWT.SINGLE | SWT.FULL_SELECTION);
             final Table table = columnsTableViewer.getTable();
 
-            GridData gd = new GridData(GridData.FILL_VERTICAL);
+            gd = new GridData(GridData.FILL_VERTICAL);
             gd.widthHint = 230;
             table.setLayoutData(gd);
             table.setHeaderVisible(true);
@@ -398,6 +407,7 @@ public class MockDataWizardPageSettings extends ActiveWizardPage<MockDataExecute
             }
 
             entityNameText.setText(DBUtils.getObjectFullName(mockDataSettings.getEntity(), DBPEvaluationContext.DML));
+            propsEditor.getControl().setFocus();
 
             // select the attributes table item
             final Table table = columnsTableViewer.getTable();
-- 
GitLab