MushroomObserver / mushroom-observer
99%
main: 99%

LAST BUILD BRANCH: njw-4984-gray-thumbnails
DEFAULT BRANCH: main
Repo Added
Files 1821
Badge
Embed â–¾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH nimmo-rss-heading-margin
branch: nimmo-rss-heading-margin
CHANGE BRANCH
x

coverage: 99.248%. Remained the same
30848352419

push

github

nimmolo 
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

Relevant lines Covered
54816 RELEVANT LINES 54404 COVERED LINES
846.05 HITS PER LINE
Source Files on nimmo-rss-heading-margin

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
30848352419 nimmo-rss-heading-margin 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 ... push nimmolo github
99.25
30848358571 nimmo-rss-heading-margin Merge d73704895 into a8388e9d5 Pull #4979 web-flow github
99.25
30847212695 nimmo-rss-heading-margin Use class_names instead of Array#join in Matrix::Box::Title ["rss-name", title_weight].join(" ") always inserts the separator, so the :observation/:name branch (title_weight == "") rendered class="rss-name " with a stray trailing space. class_nam... Pull #4979 nimmolo github
99.25
30842749182 nimmo-rss-heading-margin Fix MatrixBox heading/badge display .rss-heading (title link + IDBadge) is a flex row. Two problems compounded on long name+author combos like "Hemileccinum aenigmaticum": the title link's fixed 7.5rem margin-right was either excessive whitespace... Pull #4979 nimmolo github
99.25
See All Builds (21921)