From e7dcd495062b399e4de2ca6b481943e00bb7e2f9 Mon Sep 17 00:00:00 2001
From: tznind <t.z.nind@dundee.ac.uk>
Date: Mon, 14 Dec 2020 09:25:02 +0000
Subject: [PATCH] Changed GetLinePrefix to use Runes directly (instead of
 strings) This fixes printing in some consoles

---
 Terminal.Gui/Views/TreeView.cs | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/Terminal.Gui/Views/TreeView.cs b/Terminal.Gui/Views/TreeView.cs
index dbdc3dbba..31a03f178 100644
--- a/Terminal.Gui/Views/TreeView.cs
+++ b/Terminal.Gui/Views/TreeView.cs
@@ -597,10 +597,9 @@ namespace Terminal.Gui {
 		/// <param name="availableWidth"></param>
 		public virtual void Draw(ConsoleDriver driver,ColorScheme colorScheme, int y, int availableWidth)
 		{
-
 			// Everything on line before the expansion run and branch text
-			string prefix = GetLinePrefix(driver);
-			var expansion = GetExpandableIcon(driver);
+			Rune[] prefix = GetLinePrefix(driver).ToArray();
+			Rune expansion = GetExpandableIcon(driver);
 			string lineBody = tree.AspectGetter(Model);
 
 			var remainingWidth = availableWidth - (prefix.Length + 1 + lineBody.Length);
@@ -609,7 +608,10 @@ namespace Terminal.Gui {
 
 			driver.SetAttribute(colorScheme.Normal);
 
-			driver.AddStr(prefix + expansion);
+			foreach(Rune r in prefix)
+				driver.AddRune(r);
+
+			driver.AddRune(expansion);
 
 			driver.SetAttribute(tree.SelectedObject == Model ?
 				colorScheme.HotFocus :
@@ -628,26 +630,28 @@ namespace Terminal.Gui {
 		/// </summary>
 		/// <param name="driver"></param>
 		/// <returns></returns>
-		private string GetLinePrefix (ConsoleDriver driver)
+		private IEnumerable<Rune> GetLinePrefix (ConsoleDriver driver)
 		{
 			// If not showing line branches or this is a root object
-			if(!tree.ShowBranchLines)
-				return new string(' ',Depth);
-
-			string prefix = "";
+			if (!tree.ShowBranchLines) {
+				for(int i = 0; i < Depth; i++) {
+					yield return new Rune(' ');
+				}
+			}
 
+			// yield indentations with runes appropriate to the state of the parents
 			foreach(var cur in GetParentBranches().Reverse())
 			{
 				if(cur.IsLast())
-					prefix += " ";
+					yield return new Rune(' ');
 				else
-					prefix += driver.VLine;
+					yield return driver.VLine;
 			}
 
 			if(IsLast())
-				return prefix + driver.LLCorner;
+				yield return driver.LLCorner;
 			else
-				return prefix + driver.LeftTee;
+				yield return driver.LeftTee;
 		}
 
 		/// <summary>