- Fixed TfrmToolWindow.FormDeactivate to check CanClose first

- Fixed TfrmDrawSettings and TfrmHueSettings to be shown non-modal (fixes #49)

Client/Tools/UfrmDrawSettings.lfm

@@ -7,13 +7,14 @@ inherited frmDrawSettings: TfrmDrawSettings
   Caption = 'Draw settings'
   ClientHeight = 180
   ClientWidth = 242
+  OnCreate = FormCreate
   object rbTileList: TRadioButton[0]
     AnchorSideLeft.Control = Owner
     AnchorSideTop.Control = Owner
     Left = 8
-    Height = 19
+    Height = 22
     Top = 8
-    Width = 125
+    Width = 146
     BorderSpacing.Left = 8
     BorderSpacing.Top = 8
     BorderSpacing.Bottom = 4
@@ -27,9 +28,9 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideTop.Control = rbTileList
     AnchorSideTop.Side = asrBottom
     Left = 8
-    Height = 19
-    Top = 31
-    Width = 184
+    Height = 22
+    Top = 34
+    Width = 213
     BorderSpacing.Top = 4
     Caption = 'Use tiles from the random pool'
     TabOrder = 1
@@ -43,18 +44,18 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideRight.Side = asrBottom
     Left = 8
     Height = 49
-    Top = 123
+    Top = 132
     Width = 226
     Anchors = [akTop, akLeft, akRight]
     BorderSpacing.Around = 8
     Caption = 'Hue (Statics only)'
-    ClientHeight = 31
+    ClientHeight = 45
     ClientWidth = 222
     TabOrder = 2
     object pbHue: TPaintBox
       Cursor = crHandPoint
       Left = 4
-      Height = 27
+      Height = 41
       Top = 0
       Width = 214
       Align = alClient
@@ -70,9 +71,9 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideTop.Control = cbForceAltitude
     AnchorSideTop.Side = asrBottom
     Left = 8
-    Height = 19
-    Top = 93
-    Width = 135
+    Height = 22
+    Top = 102
+    Width = 149
     BorderSpacing.Top = 12
     Caption = 'Add Random Altitude'
     TabOrder = 3
@@ -83,8 +84,8 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideRight.Control = Owner
     AnchorSideRight.Side = asrBottom
     Left = 184
-    Height = 25
-    Top = 90
+    Height = 21
+    Top = 103
     Width = 50
     Anchors = [akTop, akRight]
     BorderSpacing.Right = 8
@@ -95,9 +96,9 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideTop.Control = rbRandom
     AnchorSideTop.Side = asrBottom
     Left = 8
-    Height = 19
-    Top = 62
-    Width = 95
+    Height = 22
+    Top = 68
+    Width = 111
     BorderSpacing.Top = 12
     Caption = 'Force altitude:'
     TabOrder = 5
@@ -108,8 +109,8 @@ inherited frmDrawSettings: TfrmDrawSettings
     AnchorSideRight.Control = Owner
     AnchorSideRight.Side = asrBottom
     Left = 184
-    Height = 25
-    Top = 59
+    Height = 21
+    Top = 69
     Width = 50
     Anchors = [akTop, akRight]
     BorderSpacing.Right = 8
@@ -117,4 +118,6 @@ inherited frmDrawSettings: TfrmDrawSettings
     MinValue = -128
     TabOrder = 6
   end
+  inherited tmClose: TTimer[7]
+  end
 end

Client/Tools/UfrmDrawSettings.pas

@@ -46,12 +46,15 @@ type
     rbTileList: TRadioButton;
     seForceAltitude: TSpinEdit;
     seRandomHeight: TSpinEdit;
+    procedure FormCreate(Sender: TObject);
     procedure pbHueClick(Sender: TObject);
     procedure pbHuePaint(Sender: TObject);
     procedure seForceAltitudeChange(Sender: TObject);
     procedure seRandomHeightChange(Sender: TObject);
-  public
-    { public declarations }
+  private
+    FCanClose: Boolean;
+    function CanClose: Boolean; override;
+    procedure OnHueClose(Sender: TObject; var ACloseAction: TCloseAction);
   end; 
 
 var
@@ -65,14 +68,17 @@ uses
 { TfrmDrawSettings }
 
 procedure TfrmDrawSettings.pbHueClick(Sender: TObject);
-var
-  msg: TLMessage;
 begin
   frmHueSettings.Left := Mouse.CursorPos.x - 8;
   frmHueSettings.Top := Mouse.CursorPos.y - 8;
-  frmHueSettings.ShowModal;
-  pbHue.Repaint;
-  MouseLeave(msg);
+  frmHueSettings.OnClose := @OnHueClose;
+  frmHueSettings.Show;
+  FCanClose := False;
+end;
+
+procedure TfrmDrawSettings.FormCreate(Sender: TObject);
+begin
+  FCanClose := True;
 end;
 
 procedure TfrmDrawSettings.pbHuePaint(Sender: TObject);
@@ -100,7 +106,21 @@ begin
   cbRandomHeight.Checked := True;
 end;
 
-//TODO : canclose ---> hue settings
+function TfrmDrawSettings.CanClose: Boolean;
+begin
+  Result := FCanClose and inherited CanClose;
+end;
+
+procedure TfrmDrawSettings.OnHueClose(Sender: TObject;
+  var ACloseAction: TCloseAction);
+var
+  msg: TLMessage;
+begin
+  FCanClose := True;
+  frmHueSettings.OnClose := nil;
+  pbHue.Repaint;
+  MouseLeave(msg);
+end;
 
 initialization
   {$I UfrmDrawSettings.lrs}

Client/Tools/UfrmToolWindow.pas

@@ -59,7 +59,8 @@ implementation
 
 procedure TfrmToolWindow.FormDeactivate(Sender: TObject);
 begin
-  Close;
+  if CanClose then
+    Close;
 end;
 
 procedure TfrmToolWindow.FormClose(Sender: TObject;

Client/UfrmMain.pas

@@ -1084,7 +1084,7 @@ begin
   acDraw.Checked := True;
   tbDrawTile.Down := True;
   mnuDraw.Checked := True;
-  frmDrawSettings.ShowModal;
+  frmDrawSettings.Show;
   ProcessToolState;
 end;