Enhanced Tiled integration. Now in "File->Edit workspace settings" menu,

you have tools to help selecting external tools to bview/edit images and
tmx maps. Still rough.

src/com/gpl/rpg/atcontentstudio/ui/StudioFrame.java

@@ -129,6 +129,8 @@ public class StudioFrame extends JFrame {
 		fileMenu.add(new JMenuItem(actions.closeProject));
 		fileMenu.add(new JMenuItem(actions.deleteProject));
 		fileMenu.add(new JSeparator());
+		fileMenu.add(new JMenuItem(actions.editWorkspaceSettings));
+		fileMenu.add(new JSeparator());
 		fileMenu.add(new JMenuItem(actions.exitATCS));
 		getJMenuBar().add(fileMenu);
 		

src/com/gpl/rpg/atcontentstudio/ui/WorkspaceActions.java

@@ -370,6 +370,15 @@ public class WorkspaceActions {
 		}
 	};
 	
+	public ATCSAction editWorkspaceSettings = new ATCSAction("Edit Workspace Settings", "Change the preferences of this workspace.") {
+		public void actionPerformed(ActionEvent e) {
+			new WorkspaceSettingsEditor(Workspace.activeWorkspace.settings);
+		};
+		public void selectionChanged(ProjectTreeNode selectedNode, TreePath[] selectedPaths) {
+			setEnabled(true);
+		};
+	};
+	
 	List<ATCSAction> actions = new ArrayList<WorkspaceActions.ATCSAction>();
 	
 	public WorkspaceActions() {
@@ -390,6 +399,7 @@ public class WorkspaceActions {
 		actions.add(testWriter);
 //		actions.add(testCommitWriter);
 		actions.add(createWriter);
+		actions.add(editWorkspaceSettings);
 		selectionChanged(null, null);
 	}
 	

src/com/gpl/rpg/atcontentstudio/ui/WorkspaceSettingsEditor.java

@@ -0,0 +1,180 @@
+package com.gpl.rpg.atcontentstudio.ui;
+
+import java.awt.BorderLayout;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
+
+import javax.swing.ButtonGroup;
+import javax.swing.JButton;
+import javax.swing.JDialog;
+import javax.swing.JPanel;
+import javax.swing.JRadioButton;
+import javax.swing.JScrollPane;
+import javax.swing.JTextField;
+
+import com.gpl.rpg.atcontentstudio.ATContentStudio;
+import com.gpl.rpg.atcontentstudio.model.WorkspaceSettings;
+import com.jidesoft.swing.JideBoxLayout;
+
+public class WorkspaceSettingsEditor extends JDialog {
+
+	private static final long serialVersionUID = -1326158719217162879L;
+	
+	WorkspaceSettings settings;
+	
+	JRadioButton useSystemDefaultMapEditorButton, useCustomMapEditorButton;
+	JTextField mapEditorCommandField;
+	
+	JRadioButton useSystemDefaultImageViewerButton, useSystemDefaultImageEditorButton, useCustomImageEditorButton;
+	JTextField imageEditorCommandField;
+	
+	
+	
+	public WorkspaceSettingsEditor(WorkspaceSettings settings) {
+		super(ATContentStudio.frame, "Workspace settings", true);
+		setIconImage(DefaultIcons.getMainIconImage());
+		
+		this.settings = settings;
+		
+		JPanel pane = new JPanel();
+		getContentPane().setLayout(new BorderLayout());
+		getContentPane().add(new JScrollPane(pane), BorderLayout.CENTER);
+		pane.setLayout(new JideBoxLayout(pane, JideBoxLayout.PAGE_AXIS));
+		JPanel buttonPane = new JPanel();
+		buttonPane.setLayout(new JideBoxLayout(buttonPane, JideBoxLayout.LINE_AXIS));
+		getContentPane().add(buttonPane, BorderLayout.SOUTH);
+		
+		
+		pane.add(getExternalToolsPane(), JideBoxLayout.FIX);
+		pane.add(new JPanel(), JideBoxLayout.VARY);
+
+		buttonPane.add(new JPanel(), JideBoxLayout.VARY);
+		JButton ok = new JButton("Ok");
+		ok.addActionListener(new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				pushToModel();
+				dispose();
+			}
+		});
+		buttonPane.add(ok, JideBoxLayout.FIX);
+		JButton reset = new JButton("Reset to defaults");
+		reset.addActionListener(new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				resetDefaults();
+			}
+		});
+		buttonPane.add(reset, JideBoxLayout.FIX);
+		JButton cancel = new JButton("Cancel");
+		cancel.addActionListener(new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				dispose();
+			}
+		});
+		buttonPane.add(cancel, JideBoxLayout.FIX);
+		
+		loadFromModel();
+		pack();
+		setVisible(true);
+		
+	}
+	
+	public JPanel getExternalToolsPane() {
+		CollapsiblePanel pane = new CollapsiblePanel("External tools");
+		pane.setLayout(new JideBoxLayout(pane, JideBoxLayout.PAGE_AXIS));
+		
+		//Tiled
+		CollapsiblePanel tiledPane = new CollapsiblePanel("TMX Map viewer/editor");
+		tiledPane.setLayout(new JideBoxLayout(tiledPane, JideBoxLayout.PAGE_AXIS));
+		ButtonGroup tiledRadioGroup = new ButtonGroup();
+		useSystemDefaultMapEditorButton = new JRadioButton("Use system-default TMX Map editor");
+		tiledRadioGroup.add(useSystemDefaultMapEditorButton);
+		tiledPane.add(useSystemDefaultMapEditorButton, JideBoxLayout.FIX);
+		useCustomMapEditorButton = new JRadioButton("Use custom command to open TMX Map files");
+		tiledRadioGroup.add(useCustomMapEditorButton);
+		tiledPane.add(useCustomMapEditorButton, JideBoxLayout.FIX);
+		mapEditorCommandField = new JTextField();
+		tiledPane.add(mapEditorCommandField, JideBoxLayout.FIX);
+		ActionListener tiledRadioListener = new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				if (useSystemDefaultMapEditorButton.equals(e.getSource())) {
+					mapEditorCommandField.setEnabled(false);
+				} else if (useCustomMapEditorButton.equals(e.getSource())) {
+					mapEditorCommandField.setEnabled(true);
+				}
+			}
+		};
+		useSystemDefaultMapEditorButton.addActionListener(tiledRadioListener);
+		useCustomMapEditorButton.addActionListener(tiledRadioListener);
+		pane.add(tiledPane, JideBoxLayout.FIX);
+		
+		//Images
+		CollapsiblePanel imgPane = new CollapsiblePanel("Image viewer/editor");
+		imgPane.setLayout(new JideBoxLayout(imgPane, JideBoxLayout.PAGE_AXIS));
+		ButtonGroup imgRadioGroup = new ButtonGroup();
+		useSystemDefaultImageViewerButton = new JRadioButton("Use system-default image viewer");
+		imgRadioGroup.add(useSystemDefaultImageViewerButton);
+		imgPane.add(useSystemDefaultImageViewerButton, JideBoxLayout.FIX);
+		useSystemDefaultImageEditorButton = new JRadioButton("Use system-default image editor");
+		imgRadioGroup.add(useSystemDefaultImageEditorButton);
+		imgPane.add(useSystemDefaultImageEditorButton, JideBoxLayout.FIX);
+		useCustomImageEditorButton = new JRadioButton("Use custom command to open images");
+		imgRadioGroup.add(useCustomImageEditorButton);
+		imgPane.add(useCustomImageEditorButton, JideBoxLayout.FIX);
+		imageEditorCommandField = new JTextField();
+		imgPane.add(imageEditorCommandField, JideBoxLayout.FIX);
+		ActionListener imgRadioListener = new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				if (useSystemDefaultMapEditorButton.equals(e.getSource())) {
+					imageEditorCommandField.setEnabled(false);
+				} else if (useSystemDefaultImageViewerButton.equals(e.getSource())) {
+					imageEditorCommandField.setEnabled(false);
+				} else if (useCustomImageEditorButton.equals(e.getSource())) {
+					imageEditorCommandField.setEnabled(true);
+				}
+			}
+		};
+		useSystemDefaultImageViewerButton.addActionListener(imgRadioListener);
+		useSystemDefaultImageEditorButton.addActionListener(imgRadioListener);
+		useCustomImageEditorButton.addActionListener(imgRadioListener);
+		pane.add(imgPane, JideBoxLayout.FIX);
+		
+		pane.expand();
+		return pane;
+	}
+	
+	public void loadFromModel() {
+		//Tiled
+		useSystemDefaultMapEditorButton.setSelected(settings.useSystemDefaultMapEditor.getCurrentValue());
+		useCustomMapEditorButton.setSelected(!settings.useSystemDefaultMapEditor.getCurrentValue());
+		mapEditorCommandField.setText(settings.mapEditorCommand.getCurrentValue());
+		//Images
+		useSystemDefaultImageViewerButton.setSelected(settings.useSystemDefaultImageViewer.getCurrentValue());
+		useSystemDefaultImageEditorButton.setSelected(settings.useSystemDefaultImageEditor.getCurrentValue());
+		useCustomImageEditorButton.setSelected(!(settings.useSystemDefaultImageViewer.getCurrentValue() || settings.useSystemDefaultImageEditor.getCurrentValue()));
+		imageEditorCommandField.setText(settings.imageEditorCommand.getCurrentValue());
+	}
+	
+	public void pushToModel() {
+		//Tiled
+		settings.useSystemDefaultMapEditor.setCurrentValue(useSystemDefaultMapEditorButton.isSelected());
+		settings.mapEditorCommand.setCurrentValue(mapEditorCommandField.getText());
+		//Images
+		settings.useSystemDefaultImageViewer.setCurrentValue(useSystemDefaultImageViewerButton.isSelected());
+		settings.useSystemDefaultImageEditor.setCurrentValue(useSystemDefaultImageEditorButton.isSelected());
+		settings.imageEditorCommand.setCurrentValue(imageEditorCommandField.getText());
+		
+		settings.save();
+	}
+	
+	public void resetDefaults() {
+		settings.resetDefault();
+		settings.save();
+		loadFromModel();
+	}
+
+}

src/com/gpl/rpg/atcontentstudio/ui/map/TMXMapEditor.java

@@ -99,6 +99,7 @@ import com.gpl.rpg.atcontentstudio.ui.Editor;
 import com.gpl.rpg.atcontentstudio.ui.FieldUpdateListener;
 import com.gpl.rpg.atcontentstudio.ui.IntegerBasedCheckBox;
 import com.gpl.rpg.atcontentstudio.ui.ScrollablePanel;
+import com.gpl.rpg.atcontentstudio.utils.DesktopIntegration;
 import com.jidesoft.swing.JideBoxLayout;
 import com.jidesoft.swing.JideTabbedPane;
 
@@ -1608,11 +1609,7 @@ public class TMXMapEditor extends Editor {
 			gdeIcon.addActionListener(new ActionListener() {
 				@Override
 				public void actionPerformed(ActionEvent e) {
-					try {
-						Runtime.getRuntime().exec(new String[]{"tiled",map.tmxFile.getAbsolutePath()});
-					} catch (IOException e1) {
-						e1.printStackTrace();
-					}
+					DesktopIntegration.openTmxMap(map.tmxFile);
 				}
 			});
 			

src/com/gpl/rpg/atcontentstudio/ui/sprites/SpritesheetEditor.java

@@ -4,6 +4,8 @@ import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Component;
 import java.awt.Image;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
 import java.awt.event.KeyAdapter;
 import java.awt.event.KeyEvent;
 import java.awt.event.MouseAdapter;
@@ -17,6 +19,7 @@ import java.util.Map;
 import javax.swing.BorderFactory;
 import javax.swing.DefaultListCellRenderer;
 import javax.swing.ImageIcon;
+import javax.swing.JButton;
 import javax.swing.JCheckBox;
 import javax.swing.JComboBox;
 import javax.swing.JComponent;
@@ -41,8 +44,10 @@ import com.gpl.rpg.atcontentstudio.model.GameDataElement;
 import com.gpl.rpg.atcontentstudio.model.ProjectTreeNode;
 import com.gpl.rpg.atcontentstudio.model.maps.TMXMap;
 import com.gpl.rpg.atcontentstudio.model.sprites.Spritesheet;
+import com.gpl.rpg.atcontentstudio.ui.DefaultIcons;
 import com.gpl.rpg.atcontentstudio.ui.Editor;
 import com.gpl.rpg.atcontentstudio.ui.FieldUpdateListener;
+import com.gpl.rpg.atcontentstudio.utils.DesktopIntegration;
 import com.jidesoft.swing.JideBoxLayout;
 import com.jidesoft.swing.JideTabbedPane;
 
@@ -84,6 +89,14 @@ public class SpritesheetEditor extends Editor {
 		pane.setLayout(new JideBoxLayout(pane, JideBoxLayout.PAGE_AXIS, 6));
 
 		add(getWarningLabel(), JideBoxLayout.FIX);
+		JButton openImage = new JButton(new ImageIcon(DefaultIcons.getTileLayerImage()));
+		openImage.addActionListener(new ActionListener() {
+			@Override
+			public void actionPerformed(ActionEvent e) {
+				DesktopIntegration.openImage(((Spritesheet)target).spritesheetFile);
+			}
+		});
+		pane.add(openImage, JideBoxLayout.FIX);
 		addLabelField(pane, "Spritesheet ID: ", sheet.id);
 		addLabelField(pane, "File: ", sheet.spritesheetFile.getAbsolutePath());
 		widthField = addIntegerField(pane, "Sprite width (px): ", sheet.spriteWidth, false, true, listener);