fix jump to line command

rstudio · Jan 6, 2012 · 92f8d83 · 92f8d83
1 parent aada669
commit 92f8d83
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/gwt/src/org/rstudio/core/client/widget/TextEntryModalDialog.java b/src/gwt/src/org/rstudio/core/client/widget/TextEntryModalDialog.java
@@ -109,10 +109,9 @@ protected boolean validate(String input)
 
       if (numbersOnly_)
       {
-         setText(getText().trim());
          try
          {
-            Integer.parseInt(getText());
+            Integer.parseInt(input.trim());
          }
          catch (NumberFormatException nfe)
          {

diff --git a/src/gwt/src/org/rstudio/studio/client/common/DefaultGlobalDisplay.java b/src/gwt/src/org/rstudio/studio/client/common/DefaultGlobalDisplay.java
@@ -144,8 +144,8 @@ public void promptForInteger(String title,
             {
                @Override
                public void execute(String input, ProgressIndicator indicator)
-               {
-                  int value = Integer.parseInt(input);
+               {  
+                  int value = Integer.parseInt(input.trim());
                   okOperation.execute(value, indicator);
                }
             },

diff --git a/.../src/org/rstudio/studio/client/workbench/views/source/editors/text/TextEditingTarget.java b/.../src/org/rstudio/studio/client/workbench/views/source/editors/text/TextEditingTarget.java
@@ -1400,6 +1400,8 @@ void onGoToLine()
                @Override
                public void execute(Integer line, ProgressIndicator indicator)
                {
+                  indicator.onCompleted();
+
                   line = Math.max(1, line);
                   line = Math.min(docDisplay_.getRowCount(), line);