Removed circular referencing which was causing the main frame to be recreated again and fixed some minor UI resizing issues

Author: sparkhi

csv-validator-ui/src/main/scala/uk/gov/nationalarchives/csv/validator/ui/CsvValidatorUi.scala

@@ -40,7 +40,6 @@ import scala.util.Using
  * @author Adam Retter <[email protected]>
  */
 object CsvValidatorUi extends SimpleSwingApplication {
-
   override def startup(args: Array[String]) : Unit = {
     try {
       UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName)
@@ -52,10 +51,11 @@ object CsvValidatorUi extends SimpleSwingApplication {
   }
 
   def top = new SJXFrame {
+
     title = "CSV Validator"
     peer.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE)
     contents = {
-      val settings = new SettingsPanel
+      val settings = new SettingsPanel(this)
 
       //handle resizing the main window, when resizing the settings panel
       settings.settingsGroup.reactions += SJXTaskPane.onViewStateChanged {
@@ -64,10 +64,9 @@ object CsvValidatorUi extends SimpleSwingApplication {
         } else {
           new Dimension(this.size.getWidth.toInt, (this.size.getHeight + settings.size.getHeight).toInt)
         }
-        this.size = newSize
+        this.preferredSize = newSize
         this.pack()
       }
-
       new ContentPanel(settings)
     }
   }
@@ -408,7 +407,7 @@ object CsvValidatorUi extends SimpleSwingApplication {
    * us to break up the code easily, hopefully
    * making it more understandable.
    */
-  private class SettingsPanel extends SJXTaskPaneContainer {
+  private class SettingsPanel(parentFrame: SJXFrame) extends SJXTaskPaneContainer {
 
     private lazy val CHARACTER_ENCODINGS =
       if(Charset.defaultCharset.name == "UTF-8") {
@@ -455,7 +454,7 @@ object CsvValidatorUi extends SimpleSwingApplication {
 
     private val spTblPathSubstitutions = new ScrollPane(tblPathSubstitutions)
     private val btnAddPathSubstitution = new Button("Add Path Substitution...")
-    btnAddPathSubstitution.reactions += onClick(addToTableDialog(top, "Add Path Substitution...", tblPathSubstitutions, tblPathSubstitutions.addRow))
+    btnAddPathSubstitution.reactions += onClick(addToTableDialog(parentFrame, "Add Path Substitution...", tblPathSubstitutions, tblPathSubstitutions.addRow))
 
     private val settingsGroupLayout = new GridBagPanel {
       private val c = new Constraints

csv-validator-ui/src/main/scala/uk/gov/nationalarchives/csv/validator/ui/ScalaSwingHelpers.scala

@@ -26,7 +26,7 @@ object ScalaSwingHelpers {
   /**
    * Opens a FileChooser and sets the path of the chosen file as the text of a Text Component
    *
-   * @param fileChooser
+   * @param fileChooser FileChooser which is Used to open file dialogs
    * @param output A text component which displays the absolute path of the chosen file
    * @param locateOver A component over which the FileChooser dialog should be located
    */
@@ -37,7 +37,7 @@ object ScalaSwingHelpers {
   /**
    * Opens a FileChooser and sends the result to a function
    *
-   * @param fileChooser
+   * @param fileChooser FileChooser which is Used to open file dialogs
    * @param result A function which takes the chosen file
    * @param locateOver A component over which the FileChooser dialog should be located
    */
@@ -67,10 +67,10 @@ object ScalaSwingHelpers {
 
     val btnOk = new Button("Ok")
 
-    val optionLayout = new GridBagPanel {
+    val optionLayout: GridBagPanel = new GridBagPanel {
       val c = new Constraints
 
-      for(colIdx <- (0 to table.model.getColumnCount - 1)) {
+      for(colIdx <- 0 to table.model.getColumnCount - 1) {
         c.gridx = 0
         c.gridy = colIdx
         c.anchor = Anchor.LineStart
@@ -93,6 +93,7 @@ object ScalaSwingHelpers {
     dialog.modal = true
     dialog.title = title
     dialog.contents = optionLayout
+    dialog.setLocationRelativeTo(owner)
 
     btnOk.reactions += onClick(result({
       val textValues = for{
@@ -130,7 +131,7 @@ object ScalaSwingHelpers {
       action
   }
 
-  def PropertyChangeListener(f: PropertyChangeEvent => Unit) = new PropertyChangeListener {
+  def PropertyChangeListener(f: PropertyChangeEvent => Unit): PropertyChangeListener = new PropertyChangeListener {
     def propertyChange(e: PropertyChangeEvent) : Unit = {
       f(e)
     }