GUI: destroy dialogue after we have harvested the widgets

Otherwise we don't get the values Updates #770
shawnlaffan · Nov 1, 2020 · 5b5579a · 5b5579a
1 parent 3ba6567
commit 5b5579a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/Biodiverse/GUI/ParametersTable.pm b/lib/Biodiverse/GUI/ParametersTable.pm
@@ -111,7 +111,7 @@ sub fill {
         $label->set_alignment(0, 0.5);
         #$label->set_text( $label_text );
 
-        my $fill_flags = 'fill';
+        #my $fill_flags = 'fill';
         my $fill_flags = [qw/expand fill/];
         if ($param->{type} =~ 'text') {
             $fill_flags = ['expand', 'fill']

diff --git a/lib/Biodiverse/GUI/RemapGUI.pm b/lib/Biodiverse/GUI/RemapGUI.pm
@@ -246,9 +246,6 @@ sub pre_remap_dlg {
     $dlg->show_all;
     my $response = $dlg->run();
 
-    $dlg->destroy();
-
-
     # The dialog has now finished, process the response and figure out
     # what to return.
     my %results;
@@ -277,6 +274,8 @@ sub pre_remap_dlg {
         %results = (remap_type => "none");
     }
 
+    $dlg->destroy();
+
     return wantarray ? %results : \%results;
 }