enhance: keyword in subject should not contains " character

love-linger · love-linger · commit c0244fbe07c8 · 2026-02-28T11:33:02.000+08:00
Signed-off-by: leo &lt;longshuang@msn.cn&gt;
diff --git a/src/Views/CommitSubjectPresenter.cs b/src/Views/CommitSubjectPresenter.cs
@@ -257,14 +257,14 @@ private void GenerateInlineElements()
             else
             {
                 var colonIdx = subject.IndexOf(": ", StringComparison.Ordinal);
-                if (colonIdx > 0 && colonIdx < 32 && colonIdx < subject.Length - 3 && _elements.Intersect(0, colonIdx) == null)
+                if (colonIdx > 0 && colonIdx < 32 && colonIdx < subject.Length - 3 && subject.IndexOf('"', 0, colonIdx) == -1 && _elements.Intersect(0, colonIdx) == null)
                 {
                     _elements.Add(new Models.InlineElement(Models.InlineElementType.Keyword, 0, colonIdx + 1, string.Empty));
                 }
                 else
                 {
                     var hyphenIdx = subject.IndexOf(" - ", StringComparison.Ordinal);
-                    if (hyphenIdx > 0 && hyphenIdx < 32 && hyphenIdx < subject.Length - 4 && _elements.Intersect(0, hyphenIdx) == null)
+                    if (hyphenIdx > 0 && hyphenIdx < 32 && hyphenIdx < subject.Length - 4 && subject.IndexOf('"', 0, hyphenIdx) == -1 && _elements.Intersect(0, hyphenIdx) == null)
                     {
                         _elements.Add(new Models.InlineElement(Models.InlineElementType.Keyword, 0, hyphenIdx, string.Empty));
                     }

Original file line number	Diff line number	Diff line change
`@@ -257,14 +257,14 @@ private void GenerateInlineElements()`
`257`	`257`	`else`
`258`	`258`	`{`
`259`	`259`	`var colonIdx = subject.IndexOf(": ", StringComparison.Ordinal);`
`260`		`- if (colonIdx > 0 && colonIdx < 32 && colonIdx < subject.Length - 3 && _elements.Intersect(0, colonIdx) == null)`
	`260`	`+ if (colonIdx > 0 && colonIdx < 32 && colonIdx < subject.Length - 3 && subject.IndexOf('"', 0, colonIdx) == -1 && _elements.Intersect(0, colonIdx) == null)`
`261`	`261`	`{`
`262`	`262`	`_elements.Add(new Models.InlineElement(Models.InlineElementType.Keyword, 0, colonIdx + 1, string.Empty));`
`263`	`263`	`}`
`264`	`264`	`else`
`265`	`265`	`{`
`266`	`266`	`var hyphenIdx = subject.IndexOf(" - ", StringComparison.Ordinal);`
`267`		`- if (hyphenIdx > 0 && hyphenIdx < 32 && hyphenIdx < subject.Length - 4 && _elements.Intersect(0, hyphenIdx) == null)`
	`267`	`+ if (hyphenIdx > 0 && hyphenIdx < 32 && hyphenIdx < subject.Length - 4 && subject.IndexOf('"', 0, hyphenIdx) == -1 && _elements.Intersect(0, hyphenIdx) == null)`
`268`	`268`	`{`
`269`	`269`	`_elements.Add(new Models.InlineElement(Models.InlineElementType.Keyword, 0, hyphenIdx, string.Empty));`
`270`	`270`	`}`