MushroomObserver / mushroom-observer / 30848352419

99%

main: 99%

push

github

Use Components::Link::Get for the matrix-box title link

Replaces the hand-rolled a(href: url_for(...)) with Link::Get, which
handles the Hash target directly (no manual url_for) and gets the
established CRUD-path-building/attrs conventions other links in the
app already use. Verified the nested Title component render still
lands correctly inside the <a> via link_to's block form -- link_to
doesn't need capture the way button_to does (see Button::CRUDBase's
own comment on that).

Also moves the whitespace() call between the title link and IDBadge
out of the render_id_badge method and up to the call site -- it's not
part of "render the id badge", it's spacing between two siblings that
belongs to whoever is composing them. It's still needed even with the
new gap: 1rem flex spacing (#4979's CSS change) -- gap is layout-only
and has no effect on the accessible text stream, so removing the
whitespace text node would run the title and badge text together for
screen readers ("...betula655576" instead of "...betula 655576").

render_id_badge itself is now a single-call, single-line delegation
with no logic of its own, so it's inlined at its one call site instead
of kept as a separate private method.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

4 of 4 new or added lines in 1 file covered. (100.0%)

54404 of 54816 relevant lines covered (99.25%)

846.05 hits per line