From 63dbacc068152dd5b572806881fcf633d7aa8840 Mon Sep 17 00:00:00 2001 From: Doxa Maps Date: Sun, 27 Sep 2026 23:48:17 +0000 Subject: [PATCH] The nav-row comment shipped a role noun too doxa-maps-spa.js carried "internal role noun (Driver)" twice. It was not the SPA's own file: the text is a CSS comment inside VerticalTabsLayout's style literal, so it is string content and ships to every bundle using that layout. Rewritten in BOTH synced copies (lines 355 and 844) to state the decision rather than who made it. check:synced-css passes, so the copies stayed identical. Verified against output: doxa-maps-spa.js goes 2 occurrences to 0. Located by matching the SHIPPED text back to source, after two wrong guesses worth recording. A backtick-parity test over a whole .vue file said all 25 of that file's "Driver" mentions were outside a template literal, while 2 demonstrably shipped -- the crude lexer again. And the leak was not in doxa-maps-spa.vue at all, which is where the bundle name pointed. --- library/layouts/VerticalTabsLayout.vue | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/library/layouts/VerticalTabsLayout.vue b/library/layouts/VerticalTabsLayout.vue index c8f7756d9..f6dedb833 100644 --- a/library/layouts/VerticalTabsLayout.vue +++ b/library/layouts/VerticalTabsLayout.vue @@ -352,14 +352,14 @@ const VTL_SHADOW_CSS = ` .vtl-chev svg { width:15px; height:15px; } .vtl-chev.open { transform:rotate(90deg); } .vtl-rows { display:flex; flex-direction:column; gap:1px; } -/* MATERIAL NAV-ROW METRICS (2026-09-21). Driver: "people are used to the YouTube layout, Google - layout is just way better. We do not need to create custom styles people are not used to." +/* MATERIAL NAV-ROW METRICS (2026-09-21). Product decision: follow the familiar + Material/YouTube list layout rather than custom styling readers are not used to. Measured before this change: 44px rows with an 8px radius. min-height:48px the Material list-item height. min-height, not height, so a wrapping label still grows instead of clipping. border-radius var(--vtl-radius,8px) — SUPERSEDED THE FULL PILL (2026-09-23). The 999px stadium above was a real instruction ("imitate the YouTube - active row") and it shipped. The Driver then measured it against the + active row") and it shipped. It was then measured against the rest of the product and reversed it: "the roundedness of the tabs is a little bit too round because there is nowhere else in the app that has that same roundedness. Even the collapse button is not that round... @@ -841,14 +841,14 @@ export default defineComponent({ .vtl-chev svg { width:15px; height:15px; } .vtl-chev.open { transform:rotate(90deg); } .vtl-rows { display:flex; flex-direction:column; gap:1px; } -/* MATERIAL NAV-ROW METRICS (2026-09-21). Driver: "people are used to the YouTube layout, Google - layout is just way better. We do not need to create custom styles people are not used to." +/* MATERIAL NAV-ROW METRICS (2026-09-21). Product decision: follow the familiar + Material/YouTube list layout rather than custom styling readers are not used to. Measured before this change: 44px rows with an 8px radius. min-height:48px the Material list-item height. min-height, not height, so a wrapping label still grows instead of clipping. border-radius var(--vtl-radius,8px) — SUPERSEDED THE FULL PILL (2026-09-23). The 999px stadium above was a real instruction ("imitate the YouTube - active row") and it shipped. The Driver then measured it against the + active row") and it shipped. It was then measured against the rest of the product and reversed it: "the roundedness of the tabs is a little bit too round because there is nowhere else in the app that has that same roundedness. Even the collapse button is not that round...