Bugfixes (as per https://forum.image.sc/t/shiny-new-script-editor/64160)

Author: tferr

src/main/java/org/scijava/ui/swing/script/EditorPane.java

@@ -157,7 +157,7 @@ public class EditorPane extends RSyntaxTextArea implements DocumentListener {
 	private PlatformService platformService;
 	@Parameter
 	private LogService log;
-	
+
 	/**
 	 * Constructor.
 	 */
@@ -526,7 +526,9 @@ public void open(final File file) throws IOException {
 				string.append(buffer, 0, count);
 			}
 			reader.close();
-			setText(string.toString());
+			SwingUtilities.invokeLater(() -> {
+				setText(string.toString()); // otherwise GUI freezes!??
+			});
 			curFile = file;
 			if (line > getLineCount()) line = getLineCount() - 1;
 			try {

src/main/java/org/scijava/ui/swing/script/TextEditor.java

@@ -2356,7 +2356,11 @@ void setLanguage(final ScriptLanguage language, final boolean addHeader) {
 
 	void updateLanguageMenu(final ScriptLanguage language) {
 		JMenuItem item = languageMenuItems.get(language);
-		if (item == null) item = noneLanguageItem;
+		if (item == null) {
+			// is none
+			item = noneLanguageItem;
+			setIncremental(false);
+		}
 		if (!item.isSelected()) {
 			item.setSelected(true);
 		}
@@ -3494,14 +3498,14 @@ private Reader evalScript(final String filename, Reader reader,
 
 	public void setIncremental(final boolean incremental) {
 
-		if (null == getCurrentLanguage()) {
+		if (incremental && null == getCurrentLanguage()) {
 			error("Select a language first!");
 			return;
 		}
 
 		this.incremental = incremental;
 
-		final JTextArea prompt = this.getTab().getPrompt();
+		final JTextArea prompt = getTab().getPrompt();
 		if (incremental) {
 			getTab().setREPLVisible(true);
 			prompt.addKeyListener(new KeyAdapter() {
@@ -3540,6 +3544,7 @@ public void keyPressed(final KeyEvent ke) {
 							screen.scrollRectToVisible(screen.modelToView(screen.getDocument().getLength()));
 						} catch (final Throwable t) {
 							log.error(t);
+							prompt.requestFocusInWindow();
 						}
 						ke.consume(); // avoid writing the line break
 						return;
@@ -3603,7 +3608,7 @@ public void keyPressed(final KeyEvent ke) {
 			for (final KeyListener kl : prompt.getKeyListeners()) {
 				prompt.removeKeyListener(kl);
 			}
-			getTab().getScreenAndPromptSplit().setDividerLocation(1.0);
+			getTab().setREPLVisible(false);
 		}
 	}
 

src/main/java/org/scijava/ui/swing/script/TextEditorTab.java

@@ -235,7 +235,8 @@ public void actionPerformed(final ActionEvent e) {
 		bottom.add(scroll, bc);
 
 		prompt.setEnabled(false);
-		
+		prompt.setFont(getEditorPane().getFont());
+
 		final JPanel prompt_panel = new JPanel();
 		prompt_panel.setMinimumSize(new Dimension(0, 0));
 		prompt_panel.setVisible(false);
@@ -303,12 +304,6 @@ public void actionPerformed(final ActionEvent e) {
 		});
 
 		screenAndPromptSplit = new JSplitPane(JSplitPane.VERTICAL_SPLIT, bottom, prompt_panel);
-		prompt_panel.addComponentListener(new ComponentAdapter() {
-			@Override
-			public void componentResized(final ComponentEvent e) {
-				if (prompt_panel.getHeight() ==0) incremental.setSelected(false);
-			}
-		});
 
 		// Enable ErrorSrip à la Eclipse. This will keep track of lines with 'Mark All'
 		// occurrences as well as lines associated with ParserNotice.Level.WARNING and
@@ -353,6 +348,7 @@ void setREPLVisible(final boolean visible) {
 				screenAndPromptSplitDividerLocation = getScreenAndPromptSplit().getDividerLocation();
 				getScreenAndPromptSplit().setDividerLocation(1f);
 			}
+			incremental.setSelected(visible);
 		});
 	}
 
@@ -383,6 +379,8 @@ public void restore() {
 			runit.setEnabled(true);
 			killit.setEnabled(false);
 			setExecutor(null);
+			if(incremental.isSelected())
+				prompt.requestFocusInWindow();
 		});
 	}