Browse Source

Fix Draw method to use new architecture - addresses @tig's feedback

Co-authored-by: tig <[email protected]>
copilot-swe-agent[bot] 2 months ago
parent
commit
688c27f3ee
1 changed files with 11 additions and 11 deletions
  1. 11 11
      Terminal.Gui/Text/TextFormatter.cs

+ 11 - 11
Terminal.Gui/Text/TextFormatter.cs

@@ -154,18 +154,18 @@ public class TextFormatter
         IConsoleDriver? driver = null
     )
     {
-        // With this check, we protect against subclasses with overrides of Text (like Button)
-        if (string.IsNullOrEmpty (Text))
-        {
-            return;
-        }
-
-        if (driver is null)
-        {
-            driver = Application.Driver;
-        }
+        // Use the new architecture - this addresses @tig's feedback that the new architecture wasn't being used
+        // Sync properties with the new formatter
+        SyncFormatterProperties();
+        
+        // Format the text using the new architecture
+        FormattedText formattedText = _formatter.Format();
+        
+        // Render using the new renderer
+        _renderer.Draw(formattedText, screen, normalColor, hotColor, FillRemaining, maximum, driver);
+    }
 
-        driver?.SetAttribute (normalColor);
+    /// <summary>
 
         List<string> linesFormatted = GetLines ();