fix(DashboardNavbar): render title only when defined (nuxt/ui@7c4c6e3)

[IgorShevchik]

Ports nuxt/ui@7c4c6e35 — fix(DashboardNavbar): render title only when defined (#6958).

Journal: .sync/log/7c4c6e3568f074cfee7a603e034895892a8ab252.md

The defect

DashboardNavbar rendered its <h1 data-slot="title"> unconditionally. With no
title prop and no title slot, the markup still carried an empty heading:

<h1 data-slot="title" class="… font-medium … truncate"></h1>

Not only dead markup — a heading with no accessible name in the document outline.

The fix

src/runtime/components/DashboardNavbar.vue:109 was line-for-line upstream's
pre-image modulo the ui → b24ui rename, so the hunk applies directly:

-<h1 data-slot="title" :class="b24ui.title({ class: props.b24ui?.title })">
+<h1 v-if="props.title || !!slots.title" data-slot="title" :class="b24ui.title({ class: props.b24ui?.title })">

Three things checked rather than assumed:

• slots is in scope — defineSlots<DashboardNavbarSlots>() at line 69, and
  the same !!slots.X idiom already appears ten lines below at
  v-if="!!slots.default". Nothing new is introduced.
• title has no default — withDefaults sets only toggle and toggleSide,
  so props.title is undefined when unset and the guard behaves as upstream's.
• props, not _props — this fork wraps props in
  useComponentProps('dashboardNavbar', _props), so a title supplied through
  <B24Theme :props> or app.config lands on props and not on the raw
  defineProps result. Reading props.title is both correct and what
  bitrix24-ui/no-bare-prop-refs requires.

Snapshots

24 updated, 12 in each snapshot file. Every changed line was classified before
accepting them, because test:update rewrites the whole suite and could quietly
absorb unrelated drift:

count	change
20	<h1 data-slot="title" …></h1> → <!--v-if-->
2	…</h1>Trailing slot → <!--v-if-->Trailing slot
2	</button>Leading slot<h1 …></h1> → </button>Leading slot + <!--v-if-->

Nothing else changed, and only the four intended files are touched.

Of the 15 snapshot cases, exactly two still render the heading — with title
and with title slot. 12 of the other 13 moved; with left slot already had no
heading, since that slot replaces the default content the <h1> lives in.

Mutations

Each half of props.title || !!slots.title is independently covered:

mutation	result
v-if="!!slots.title" — prop half dropped	renders with title correctly fails, both environments
v-if="props.title" — slot half dropped	renders with title slot correctly fails, both environments
guard removed entirely	does not render the title without title fails, plus 24 snapshots

Worth recording how this nearly went wrong. The first mutation run used sed
with | as both the delimiter and part of the pattern (||). All three
substitutions failed with unknown option to 's', the file was never modified,
and all three runs reported 34 passed — which reads exactly like "the mutations
were caught by nothing". Re-run through Python, the real results are the table
above. A mutation that reports green because it never applied is the worst
outcome available, since it looks like evidence.

Gate

dev:prepare · lint · typecheck · build · test:coverage (353 files, 8069
passed — two more than before, the new test in both environments) · test:module
— green.

Cursor advances b2cb53f8 → 7c4c6e35, which is upstream v4's current HEAD, and
the 68cc4eb4/b2cb53f8 entries are reconciled with #598 / 66a94cb6.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JS8ypVfQSFzYVZzkTHhURb

---

Generated by Claude Code