MushroomObserver / mushroom-observer / 29340960310

99%

main: 99%

Pull #4798

github

Add Components::Row, sweep all direct .row call sites

Same pattern as Components::Container/Collapsible/Navbar: prop
:element, Symbol, default: :div; prop :attributes, _Hash(Symbol,
_Any?), :**; dispatch via send(@element, ...). Unlike Container,
there's no symbol->class mapping -- .row is untouched Bootstrap, this
just centralizes the one hardcoded "row" string that was previously
duplicated (concatenated with whatever extra classes each site
needed) across ~88 call sites in ~65 files.

Converted every direct div(class: "row"...)/ul(class: "row"...) call
site to Row(...)/Row(element: :ul, ...), preserving extra utility/
alignment classes and other HTML attrs (id, data) via the same
class_names merge Container uses. Includes the one row-offcanvas
pairing (layouts/application.rb) and the list-unstyled matrix-table
rows (several observation/collection_number/herbarium_record/
sequence/external_link new+edit pairs, occurrences/form.rb,
components/matrix/table.rb).

Left alone (indirect: a raw string passed into another component's
own class-string prop, not a direct tag wrap): upload_gallery.rb's
controls_wrap_class: "carousel-control-wrap row" passed into
Components::Carousel.

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

94 of 94 new or added lines in 65 files covered. (100.0%)

52225 of 52690 relevant lines covered (99.12%)

756.64 hits per line