From fced9fc3cac285242d03955bb7636319bb48acc4 Mon Sep 17 00:00:00 2001
From: Maxim Kropotov <maxim.kropotov@jetbrains.com>
Date: Wed, 6 Sep 2017 12:37:42 +0300
Subject: [PATCH] WEB-28608: reorder 'name' and 'type' inputs in TS change
 signature

use order they go in code
---
 .../ChangeSignatureDialogBase.java            | 20 +++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/platform/lang-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureDialogBase.java b/platform/lang-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureDialogBase.java
index 9b76f822d3d3..e42117611ae7 100644
--- a/platform/lang-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureDialogBase.java
+++ b/platform/lang-impl/src/com/intellij/refactoring/changeSignature/ChangeSignatureDialogBase.java
@@ -215,6 +215,9 @@ public abstract class ChangeSignatureDialogBase<ParamInfo extends ParameterInfo,
     return myParametersList == null ? myParametersTable : myParametersList.getTable();
   }
 
+  public boolean placeReturnTypeBeforeName() {
+    return true;
+  }
 
   @Override
   protected JComponent createNorthPanel() {
@@ -270,11 +273,20 @@ public abstract class ChangeSignatureDialogBase<ParamInfo extends ParameterInfo,
       typePanel.add(typeLabel, BorderLayout.NORTH);
       IJSwingUtilities.adjustComponentsOnMac(typeLabel, myReturnTypeField);
       typePanel.add(myReturnTypeField, BorderLayout.SOUTH);
-      panel.add(typePanel, gbc);
-      gbc.gridx++;
+      if (placeReturnTypeBeforeName()) {
+        panel.add(typePanel, gbc);
+        gbc.gridx++;
+        panel.add(myNamePanel, gbc);
+      }
+      else {
+        panel.add(myNamePanel, gbc);
+        gbc.gridx++;
+        panel.add(typePanel, gbc);
+      }
+    }
+    else {
+      panel.add(myNamePanel, gbc);
     }
-
-    panel.add(myNamePanel, gbc);
 
     return panel;
   }