fixed a markup interpolation bug with bools

Author: McJones

CHANGELOG.md

@@ -10,6 +10,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Updated
 
+- Fixed a bug where interpolated boolean values inside markup were being incorrectly determined as a string and not a bool.
+
 ### Removed
 
 ## [3.0.1] 2025-06-12

YarnSpinner.Tests/MarkupTests.cs

@@ -1083,6 +1083,13 @@ public void TestMarkupMultiplePropertyParsing()
         [InlineData("[a p={$someValue}]s[/a]", MarkupValueType.String, "$someValue")]
         [InlineData("[p=-1 /]", MarkupValueType.Integer, "-1")]
         [InlineData("[p=-1.1 /]", MarkupValueType.Float, "-1.1")]
+        [InlineData("[p={$someValue}]s[/p]", MarkupValueType.String, "$someValue")]
+        [InlineData("[p=True]s[/p]", MarkupValueType.Bool, "True")]
+        [InlineData("[p=true]s[/p]", MarkupValueType.Bool, "True")]
+        [InlineData("[p=False]s[/p]", MarkupValueType.Bool, "False")]
+        [InlineData(@"[p=""string""]s[/p]", MarkupValueType.String, "string")]
+        [InlineData("[p=string]s[/p]", MarkupValueType.String, "string")]
+        [InlineData(@"[p=""str\""ing""]s[/p]", MarkupValueType.String, @"str""ing")]
         public void TestMarkupPropertyParsing(string input, MarkupValueType expectedType, string expectedValueAsString)
         {
             var lineParser = new LineParser();

YarnSpinner/YarnSpinner.Markup/LineParser.cs

@@ -641,9 +641,9 @@ internal List<LexerToken> LexMarkup(string input)
                             // ok so now we have accumulated all the value characters
                             // which we now want to do some quick checks on, is it true or false
                             var value = input.Substring(token.Start, currentPosition + 1 - token.Start);
-                            if (value == "true" || value == "false")
+                            if (value == "true" || value == "True" || value == "false" || value == "False")
                             {
-                                // This was the word 'true' or 'false', so this
+                                // This was the word '[T]rue' or '[F]alse', so this
                                 // is a boolean value and not an undelimited
                                 // string
                                 token.Type = LexerTokenTypes.BooleanValue;
@@ -967,13 +967,15 @@ bool TryFloatFromToken(LexerToken token, out float value)
             }
             bool TryBoolFromToken(LexerToken token, out bool value)
             {
+                // at this point the lexer has determined that it is either a true or false value
+                // so we are safe to do the ignore case check
                 var valueString = OG.Substring(token.Start, token.Range);
-                if (valueString == "true")
+                if (valueString.Equals("true", StringComparison.OrdinalIgnoreCase))
                 {
                     value = true;
                     return true;
                 }
-                if (valueString == "false")
+                if (valueString.Equals("false", StringComparison.OrdinalIgnoreCase))
                 {
                     value = false;
                     return true;