Redesign dialogs

JetBrains · Mar 14, 2013 · a6d5ad4 · a6d5ad4
1 parent d519538
commit a6d5ad4
Show file tree

Hide file tree

Showing 7 changed files with 127 additions and 74 deletions.
diff --git a/idea/src/org/jetbrains/jet/plugin/framework/JSLibraryDescription.java b/idea/src/org/jetbrains/jet/plugin/framework/JSLibraryDescription.java
@@ -28,18 +28,14 @@
 import com.intellij.openapi.vfs.VirtualFile;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
-import org.jetbrains.jet.plugin.JetPluginUtil;
 import org.jetbrains.jet.plugin.framework.ui.CreateJavaScriptLibraryDialog;
 import org.jetbrains.jet.plugin.framework.ui.FileUIUtils;
 import org.jetbrains.jet.utils.KotlinPaths;
 import org.jetbrains.jet.utils.PathUtil;
 
 import javax.swing.*;
 import java.io.File;
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
 
 public class JSLibraryDescription extends CustomLibraryDescription {
     public static final LibraryKind KOTLIN_JAVASCRIPT_KIND = LibraryKind.create("kotlin-js-stdlib");
@@ -69,15 +65,24 @@ public NewLibraryConfiguration createNewLibrary(@NotNull JComponent parentCompon
                 return null;
             }
 
-            String copyIntoPath = dialog.getCopyIntoPath();
-            if (copyIntoPath != null) {
-                List<File> copyFiles = new ArrayList<File>();
+            Map<File, String> copyToPaths = new HashMap<File, String>();
+            if (dialog.isCopyLibraryFiles()) {
+                String copyIntoPath = dialog.getCopyLibraryIntoPath();
+                assert copyIntoPath != null;
 
-                if (dialog.isCopyLibraryFiles()) copyFiles.add(libraryFile);
-                if (dialog.isCopyECMA3()) copyFiles.add(paths.getJsLibJsPath());
+                copyToPaths.put(libraryFile, copyIntoPath);
+            }
+
+            if (dialog.isCopyJS()) {
+                String copyIntoPath = dialog.getCopyJsIntoPath();
+                assert copyIntoPath != null;
+
+                copyToPaths.put(paths.getJsLibJsPath(), copyIntoPath);
+            }
 
+            if (!copyToPaths.isEmpty()) {
                 Map<File,File> copiedFiles =
-                        FileUIUtils.copyWithOverwriteDialog(parentComponent, JAVA_SCRIPT_LIBRARY_CREATION, copyIntoPath, copyFiles);
+                        FileUIUtils.copyWithOverwriteDialog(parentComponent, JAVA_SCRIPT_LIBRARY_CREATION, copyToPaths);
                 if (copiedFiles == null) {
                     return null;
                 }

diff --git a/idea/src/org/jetbrains/jet/plugin/framework/ui/CopyIntoPanel.java b/idea/src/org/jetbrains/jet/plugin/framework/ui/CopyIntoPanel.java
@@ -19,7 +19,6 @@
 import com.intellij.openapi.fileChooser.FileChooserDescriptorFactory;
 import com.intellij.openapi.project.Project;
 import com.intellij.openapi.ui.TextFieldWithBrowseButton;
-import com.intellij.openapi.vfs.VirtualFile;
 import com.intellij.ui.DocumentAdapter;
 import com.intellij.util.EventDispatcher;
 import org.jetbrains.annotations.NotNull;
@@ -38,7 +37,11 @@ public class CopyIntoPanel {
 
     private boolean hasErrorsState;
 
-    public CopyIntoPanel(@Nullable Project project, @NotNull VirtualFile contextDirectory) {
+    public CopyIntoPanel(@Nullable Project project, @NotNull String defaultPath) {
+        this(project, defaultPath, null);
+    }
+
+    public CopyIntoPanel(@Nullable Project project, @NotNull String defaultPath, @Nullable String labelText) {
         copyIntoField.addBrowseFolderListener(
                 "Copy Into...", "Choose folder where files will be copied", project,
                 FileChooserDescriptorFactory.createSingleFolderDescriptor());
@@ -49,8 +52,20 @@ protected void textChanged(final DocumentEvent e) {
             }
         });
 
+        if (labelText != null) {
+            String text = labelText.replace("&", "");
+            int mnemonicIndex = labelText.indexOf("&");
+            char mnemonicChar = mnemonicIndex != -1 && (mnemonicIndex + 1) < labelText.length() ? labelText.charAt(mnemonicIndex + 1) : 0;
+
+            copyIntoLabel.setText(text);
+            copyIntoLabel.setDisplayedMnemonic(mnemonicChar);
+            copyIntoLabel.setDisplayedMnemonicIndex(mnemonicIndex);
+        }
+
         copyIntoLabel.setLabelFor(copyIntoField.getTextField());
-        copyIntoField.getTextField().setText(FileUIUtils.getDefaultLibraryFolder(project, contextDirectory));
+        copyIntoField.getTextField().setText(defaultPath);
+
+        copyIntoField.getTextField().setColumns(40);
 
         updateComponents();
     }

diff --git a/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaLibraryDialog.form b/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaLibraryDialog.form
@@ -14,7 +14,7 @@
         </constraints>
       </vspacer>
       <grid id="46726" layout-manager="GridLayoutManager" row-count="2" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
-        <margin top="15" left="0" bottom="0" right="0"/>
+        <margin top="10" left="0" bottom="0" right="0"/>
         <constraints>
           <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
         </constraints>
@@ -26,7 +26,8 @@
               <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
-              <text value="Make local copy of library (could be stored in VCS) "/>
+              <selected value="true"/>
+              <text value="&amp;Make local copy of library (could be stored in VCS) "/>
             </properties>
           </component>
           <grid id="e47d8" binding="copyIntoPanelPlace" layout-manager="BorderLayout" hgap="0" vgap="0">

diff --git a/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaLibraryDialog.java b/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaLibraryDialog.java
@@ -45,7 +45,7 @@ public CreateJavaLibraryDialog(@Nullable Project project, @NotNull String title,
         ChooseCompilerSourcePanel compilerSourcePanel = new ChooseCompilerSourcePanel();
         compilerSourcePanelPlace.add(compilerSourcePanel.getContentPane(), BorderLayout.CENTER, 0);
 
-        copyIntoPanel = new CopyIntoPanel(project, contextDirectory);
+        copyIntoPanel = new CopyIntoPanel(project, FileUIUtils.createRelativePath(project, contextDirectory, "lib"));
         copyIntoPanel.addValidityListener(new ValidityListener() {
             @Override
             public void validityChanged(boolean isValid) {

diff --git a/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaScriptLibraryDialog.form b/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaScriptLibraryDialog.form
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <form xmlns="http://www.intellij.com/uidesigner/form/" version="1" bind-to-class="org.jetbrains.jet.plugin.framework.ui.CreateJavaScriptLibraryDialog">
-  <grid id="cbd77" binding="contentPane" layout-manager="GridLayoutManager" row-count="4" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+  <grid id="cbd77" binding="contentPane" layout-manager="GridLayoutManager" row-count="3" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
     <margin top="0" left="0" bottom="0" right="0"/>
     <constraints>
       <xy x="48" y="54" width="436" height="297"/>
@@ -10,13 +10,13 @@
     <children>
       <vspacer id="bb37c">
         <constraints>
-          <grid row="3" column="0" row-span="1" col-span="1" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
+          <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="6" hsize-policy="1" anchor="0" fill="2" indent="0" use-parent-layout="false"/>
         </constraints>
       </vspacer>
-      <grid id="46726" layout-manager="GridLayoutManager" row-count="3" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
-        <margin top="0" left="0" bottom="0" right="0"/>
+      <grid id="46726" layout-manager="GridLayoutManager" row-count="4" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
+        <margin top="10" left="0" bottom="0" right="0"/>
         <constraints>
-          <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
+          <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
         </constraints>
         <properties/>
         <border type="none"/>
@@ -26,36 +26,37 @@
               <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
-              <text value="&amp;Library files"/>
+              <selected value="true"/>
+              <text value="&amp;Make local copy of library (could be stored in VCS) "/>
             </properties>
           </component>
-          <component id="dd25b" class="javax.swing.JCheckBox" binding="ECMAScript3JavaScriptRuntimeCheckBox" default-binding="true">
+          <component id="dd25b" class="javax.swing.JCheckBox" binding="copyJSRuntimeCheckbox" default-binding="true">
             <constraints>
-              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
+              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
             </constraints>
             <properties>
               <selected value="true"/>
-              <text value="EC&amp;MAScript 3 JavaScript runtime file"/>
+              <text value="&amp;Get JavaScript runtime files"/>
             </properties>
           </component>
-          <grid id="11e35" binding="copyIntoPanelPlace" layout-manager="BorderLayout" hgap="0" vgap="0">
+          <grid id="11e35" binding="copyJSIntoPanelPlace" layout-manager="BorderLayout" hgap="0" vgap="0">
             <constraints>
-              <grid row="2" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="3" use-parent-layout="false"/>
+              <grid row="3" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="3" use-parent-layout="false"/>
+            </constraints>
+            <properties/>
+            <border type="none"/>
+            <children/>
+          </grid>
+          <grid id="2bd15" binding="copyHeadersIntoPanelPlace" layout-manager="BorderLayout" hgap="0" vgap="0">
+            <constraints>
+              <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="3" use-parent-layout="false"/>
             </constraints>
             <properties/>
             <border type="none"/>
             <children/>
           </grid>
         </children>
       </grid>
-      <component id="7c0b3" class="com.intellij.ui.TitledSeparator">
-        <constraints>
-          <grid row="1" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="1" indent="0" use-parent-layout="false"/>
-        </constraints>
-        <properties>
-          <text value="Copy Into Project"/>
-        </properties>
-      </component>
       <grid id="d209b" binding="compilerSourcePanelPlace" layout-manager="BorderLayout" hgap="0" vgap="0">
         <constraints>
           <grid row="0" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>

diff --git a/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaScriptLibraryDialog.java b/idea/src/org/jetbrains/jet/plugin/framework/ui/CreateJavaScriptLibraryDialog.java
@@ -12,13 +12,15 @@
 import java.awt.event.ActionListener;
 
 public class CreateJavaScriptLibraryDialog extends DialogWrapper {
-    private final CopyIntoPanel copyIntoPanel;
+    private final CopyIntoPanel copyJSIntoPanel;
+    private final CopyIntoPanel copyLibraryIntoPanel;
 
     private JPanel contentPane;
     private JCheckBox copyLibraryCheckbox;
-    private JCheckBox ECMAScript3JavaScriptRuntimeCheckBox;
+    private JCheckBox copyJSRuntimeCheckbox;
     private JPanel compilerSourcePanelPlace;
-    private JPanel copyIntoPanelPlace;
+    private JPanel copyJSIntoPanelPlace;
+    private JPanel copyHeadersIntoPanelPlace;
 
     public CreateJavaScriptLibraryDialog(@Nullable Project project, @NotNull String title, VirtualFile contextDirectory) {
         super(project);
@@ -30,14 +32,23 @@ public CreateJavaScriptLibraryDialog(@Nullable Project project, @NotNull String
         ChooseCompilerSourcePanel compilerSourcePanel = new ChooseCompilerSourcePanel();
         compilerSourcePanelPlace.add(compilerSourcePanel.getContentPane(), BorderLayout.CENTER);
 
-        copyIntoPanel = new CopyIntoPanel(project, contextDirectory);
-        copyIntoPanel.addValidityListener(new ValidityListener() {
+        copyJSIntoPanel = new CopyIntoPanel(project, FileUIUtils.createRelativePath(project, contextDirectory, "script"), "&Script directory:");
+        copyJSIntoPanel.addValidityListener(new ValidityListener() {
             @Override
             public void validityChanged(boolean isValid) {
                 updateComponents();
             }
         });
-        copyIntoPanelPlace.add(copyIntoPanel.getContentPane(), BorderLayout.CENTER);
+        copyJSIntoPanelPlace.add(copyJSIntoPanel.getContentPane(), BorderLayout.CENTER);
+
+        copyLibraryIntoPanel = new CopyIntoPanel(project, FileUIUtils.createRelativePath(project, contextDirectory, "lib"), "&Lib directory:");
+        copyLibraryIntoPanel.addValidityListener(new ValidityListener() {
+            @Override
+            public void validityChanged(boolean isValid) {
+                updateComponents();
+            }
+        });
+        copyHeadersIntoPanelPlace.add(copyLibraryIntoPanel.getContentPane(), BorderLayout.CENTER);
 
         ActionListener updateComponentsListener = new ActionListener() {
             @Override
@@ -47,29 +58,34 @@ public void actionPerformed(@NotNull ActionEvent e) {
         };
 
         copyLibraryCheckbox.addActionListener(updateComponentsListener);
-        ECMAScript3JavaScriptRuntimeCheckBox.addActionListener(updateComponentsListener);
+        copyJSRuntimeCheckbox.addActionListener(updateComponentsListener);
 
         updateComponents();
     }
 
     @Nullable
-    public String getCopyIntoPath() {
-        return copyIntoPanel.getPath();
+    public String getCopyJsIntoPath() {
+        return copyJSIntoPanel.getPath();
+    }
+
+    @Nullable
+    public String getCopyLibraryIntoPath() {
+        return copyLibraryIntoPanel.getPath();
     }
 
     public boolean isCopyLibraryFiles() {
         return copyLibraryCheckbox.isSelected();
     }
 
-    public boolean isCopyECMA3() {
-        return ECMAScript3JavaScriptRuntimeCheckBox.isSelected();
+    public boolean isCopyJS() {
+        return copyJSRuntimeCheckbox.isSelected();
     }
 
     private void updateComponents() {
-        copyIntoPanel.setEnabled(copyLibraryCheckbox.isSelected() ||
-                                 ECMAScript3JavaScriptRuntimeCheckBox.isSelected());
+        copyLibraryIntoPanel.setEnabled(copyLibraryCheckbox.isSelected());
+        copyJSIntoPanel.setEnabled(copyJSRuntimeCheckbox.isSelected());
 
-        setOKActionEnabled(!copyIntoPanel.hasErrors());
+        setOKActionEnabled(!(copyJSIntoPanel.hasErrors() || copyLibraryIntoPanel.hasErrors()));
     }
 
     @Nullable