OSS/build_webstatic
SHA256
0
0
Fork 0
Code Activity

refactor: Remove unnecessary substring operation from navigation labels

Browse Source
This commit is contained in:
gc-dev-afjd
2026-07-13 09:43:22 +02:00
parent 086eecd190
commit dd8edf0f20
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Builder.cs
+2 -6
View File
@@ -269,8 +269,6 @@ public static class Builder
if (page.Meta.Order == 0 || page.Meta.Order == 1) continue; if (page.Meta.Order == 0 || page.Meta.Order == 1) continue;
var title = page.Meta.LocalizedTitle(lang, Path.GetFileNameWithoutExtension(page.Path)); var title = page.Meta.LocalizedTitle(lang, Path.GetFileNameWithoutExtension(page.Path));
var navLabel = page.Meta.Eyebrow ?? title; var navLabel = page.Meta.Eyebrow ?? title;
if (navLabel.Contains(" — "))
navLabel = navLabel.Substring(navLabel.IndexOf(" — ") + 3);
var slug = page.Href.Trim('/'); var slug = page.Href.Trim('/');
var href = string.IsNullOrEmpty(slug) ? "index.html" : $"{slug}.html"; var href = string.IsNullOrEmpty(slug) ? "index.html" : $"{slug}.html";
sb.AppendLine($" <a class=\"nav-gcbok__link\" href=\"{href}\">{HtmlEncode(navLabel)}</a>"); sb.AppendLine($" <a class=\"nav-gcbok__link\" href=\"{href}\">{HtmlEncode(navLabel)}</a>");
@@ -290,8 +288,6 @@ public static class Builder
foreach (var page in navPages) foreach (var page in navPages)
{ {
var navLabel = page.Meta.Eyebrow ?? page.Meta.LocalizedTitle(lang, page.Meta.Title ?? ""); var navLabel = page.Meta.Eyebrow ?? page.Meta.LocalizedTitle(lang, page.Meta.Title ?? "");
if (navLabel.Contains(" — "))
navLabel = navLabel.Substring(navLabel.IndexOf(" — ") + 3);
var slug = page.Href.Trim('/'); var slug = page.Href.Trim('/');
var href = string.IsNullOrEmpty(slug) ? "index.html" : $"{slug}.html"; var href = string.IsNullOrEmpty(slug) ? "index.html" : $"{slug}.html";
var idx = page.Meta.Order?.ToString() ?? "·"; var idx = page.Meta.Order?.ToString() ?? "·";
@@ -341,7 +337,7 @@ public static class Builder
{ {
var prev = navPages[currentIndex - 1]; var prev = navPages[currentIndex - 1];
var prevLabel = (prev.Meta.Eyebrow ?? prev.Meta.LocalizedTitle(lang, prev.Meta.Title ?? "")); var prevLabel = (prev.Meta.Eyebrow ?? prev.Meta.LocalizedTitle(lang, prev.Meta.Title ?? ""));
if (prevLabel.Contains(" — ")) prevLabel = prevLabel.Substring(prevLabel.IndexOf(" — ") + 3);
sb.AppendLine($" <a class=\"page-nav__link\" href=\"{prev.Href.Trim('/')}.html\">"); sb.AppendLine($" <a class=\"page-nav__link\" href=\"{prev.Href.Trim('/')}.html\">");
sb.AppendLine($" <span class=\"page-nav__label\">{I18n.Get(lang, "nav.prev")}</span>"); sb.AppendLine($" <span class=\"page-nav__label\">{I18n.Get(lang, "nav.prev")}</span>");
sb.AppendLine($" <span class=\"page-nav__title\">{HtmlEncode(prevLabel)}</span>"); sb.AppendLine($" <span class=\"page-nav__title\">{HtmlEncode(prevLabel)}</span>");
@@ -353,7 +349,7 @@ public static class Builder
{ {
var next = navPages[currentIndex + 1]; var next = navPages[currentIndex + 1];
var nextLabel = (next.Meta.Eyebrow ?? next.Meta.LocalizedTitle(lang, next.Meta.Title ?? "")); var nextLabel = (next.Meta.Eyebrow ?? next.Meta.LocalizedTitle(lang, next.Meta.Title ?? ""));
if (nextLabel.Contains(" — ")) nextLabel = nextLabel.Substring(nextLabel.IndexOf(" — ") + 3);
sb.AppendLine($" <a class=\"page-nav__link page-nav__link--next\" href=\"{next.Href.Trim('/')}.html\">"); sb.AppendLine($" <a class=\"page-nav__link page-nav__link--next\" href=\"{next.Href.Trim('/')}.html\">");
sb.AppendLine($" <span class=\"page-nav__label\">{I18n.Get(lang, "nav.next")}</span>"); sb.AppendLine($" <span class=\"page-nav__label\">{I18n.Get(lang, "nav.next")}</span>");
sb.AppendLine($" <span class=\"page-nav__title\">{HtmlEncode(nextLabel)}</span>"); sb.AppendLine($" <span class=\"page-nav__title\">{HtmlEncode(nextLabel)}</span>");