Merge pull request #305 from gui-cs/multi-line

Add checkbox for typing newlines into GetTextDialog

Author: tznind

src/UI/Windows/DialogArgs.cs

@@ -32,6 +32,11 @@ public class DialogArgs
     /// </summary>
     public bool MultiLine { get; set; }
 
+    /// <summary>
+    /// True to make newlines toggleable (e.g. for Label).
+    /// </summary>
+    public bool ToggleableMultiLine { get; set; } = true;
+
     /// <inheritdoc/>
     public override string ToString()
     {

src/UI/Windows/GetTextDialog.cs

@@ -10,8 +10,10 @@ internal class GetTextDialog
     private readonly DialogArgs args;
     private readonly string? initialValue;
     private readonly Window win;
-    private readonly TextView textField;
+    private readonly TextView textView;
     private bool okClicked = false;
+    private static CheckState lastKnownEnableNewlines = CheckState.UnChecked;
+    private bool? multiLineChecked;
 
     public GetTextDialog(DialogArgs args, string? initialValue)
     {
@@ -42,7 +44,7 @@ public GetTextDialog(DialogArgs args, string? initialValue)
 
         this.win.Add(entryLabel);
 
-        this.textField = new TextView()
+        this.textView = new TextView()
         {
             X = 1,
             Y = Pos.Bottom(entryLabel),
@@ -51,19 +53,29 @@ public GetTextDialog(DialogArgs args, string? initialValue)
             Text = this.initialValue ?? string.Empty,
             AllowsTab = false,
         };
-        this.textField.KeyDown += this.TextField_KeyPress;
+        this.textView.KeyDown += this.TextViewKeyPress;
 
         // make it easier for user to replace this text with something else
         // by directly selecting it all so next keypress replaces text
-        this.textField.SelectAll();
+        this.textView.SelectAll();
 
-        this.win.Add(this.textField);
+        this.win.Add(this.textView);
+
+        if (args.MultiLine)
+        {
+            SetupMultiLineForced();
+        }
+        else
+        if (args.ToggleableMultiLine)
+        {
+            SetupMultiLineOptional();
+        }
 
         var btnOk = new Button()
         {
             Text = "Ok",
             X = 0,
-            Y = Pos.Bottom(this.textField),
+            Y = Pos.Bottom(this.textView),
             IsDefault = !this.args.MultiLine,
         };
         btnOk.Accept += (s, e) =>
@@ -75,7 +87,7 @@ public GetTextDialog(DialogArgs args, string? initialValue)
         {
             Text = "Cancel",
             X = Pos.Right(btnOk),
-            Y = Pos.Bottom(this.textField),
+            Y = Pos.Bottom(this.textView),
             IsDefault = false,
         };
         btnCancel.Accept += (s, e) =>
@@ -88,18 +100,59 @@ public GetTextDialog(DialogArgs args, string? initialValue)
         {
             Text = "Clear",
             X = Pos.Right(btnCancel),
-            Y = Pos.Bottom(this.textField),
+            Y = Pos.Bottom(this.textView),
         };
         btnClear.Accept += (s, e) =>
         {
-            this.textField.Text = string.Empty;
+            this.textView.Text = string.Empty;
         };
 
         this.win.Add(btnOk);
         this.win.Add(btnCancel);
         this.win.Add(btnClear);
     }
 
+    private void SetupMultiLineForced()
+    {
+        var cbMultiLine = new CheckBox()
+        {
+            Text = "Enable Newlines",
+            X = Pos.AnchorEnd(),
+            CheckedState = CheckState.Checked,
+            Enabled = false
+        };
+        win.Add(cbMultiLine);
+    }
+
+    private void SetupMultiLineOptional()
+    {
+        // Initial state
+        SetEnableNewlines();
+
+        var cbMultiLine = new CheckBox()
+        {
+            Text = "Enable Newlines",
+            X = Pos.AnchorEnd(),
+            CheckedState = lastKnownEnableNewlines
+        };
+        cbMultiLine.CheckedStateChanging += (s, e) =>
+        {
+            SetEnableNewlines(e.NewValue);
+        };
+        win.Add(cbMultiLine);
+    }
+
+    private void SetEnableNewlines()
+    {
+       SetEnableNewlines(lastKnownEnableNewlines);
+    }
+
+    private void SetEnableNewlines(CheckState newValue)
+    {
+        lastKnownEnableNewlines = newValue;
+        multiLineChecked = textView.AllowsReturn = newValue == CheckState.Checked;
+    }
+
     public string? ResultText { get; set; }
 
     public bool ShowDialog()
@@ -112,16 +165,21 @@ public bool ShowDialog()
     private void Accept()
     {
         this.okClicked = true;
-        this.ResultText = this.textField.Text.ToString();
+        this.ResultText = this.textView.Text.ToString();
         Application.RequestStop();
     }
 
-    private void TextField_KeyPress(object? sender, Key key)
+    private void TextViewKeyPress(object? sender, Key key)
     {
-        if (key == Key.Enter && !this.args.MultiLine)
+        if (key == Key.Enter && !IsMultiLine())
         {
             this.Accept();
             key.Handled = true;
         }
     }
+
+    private bool IsMultiLine()
+    {
+        return this.args.MultiLine || (multiLineChecked ?? false);
+    }
 }