Feature/add group block

[Rivka-Development]

No description provided.

[hubbvee]

Code review

Found 4 issues:

1. deferred-hydration.js checks hasAttribute('mobile-only'), but every Liquid caller uses data-mobile-only="true". The check is case/name-exact, so mobileOnly is always false and the "skip defer on desktop" short-circuit is dead code — all instances go through IntersectionObserver on desktop too.

https://github.com/Rivka-Development/Dovetail-2026/blob/eb2d29aec18223e833680cb5d291c53e6e56e28c/assets/deferred-hydration.js#L14-L22

2. search-modal.liquid no longer closes <dialog> (opened L11) or <predictive-search-component> (opened L25). Only </dialog-component> remains. Produces malformed DOM — the trailing <script>/<style> gets parsed inside the dialog scope, likely breaking predictive-search mount and focus trap.

https://github.com/Rivka-Development/Dovetail-2026/blob/eb2d29aec18223e833680cb5d291c53e6e56e28c/snippets/search-modal.liquid#L121-L125

3. button.liquid schema declares padding-block-start, padding-block-end, padding-inline-start, padding-inline-end twice — once as type: "number" (L65-L83) and again as type: "range" (L147-L183). Shopify schemas reject duplicate setting ids; one definition will be silently dropped or the editor will error on the block.

First occurrence (number): https://github.com/Rivka-Development/Dovetail-2026/blob/eb2d29aec18223e833680cb5d291c53e6e56e28c/blocks/button.liquid#L63-L83

Duplicate (range): https://github.com/Rivka-Development/Dovetail-2026/blob/eb2d29aec18223e833680cb5d291c53e6e56e28c/blocks/button.liquid#L145-L184

4. accordion.liquid, review.liquid, and product-recommendations.liquid now render their content inside <template> under <deferred-hydration>. Template content is inert — not rendered, not in SSR output, not indexed by crawlers that don't execute JS, and absent if JS fails. For FAQ/accordion copy and rating stars these are SEO/trust signals, not pure perf concerns. Combined with issue Horizon update to main #1, every desktop visitor also waits on IntersectionObserver for this content.

https://github.com/Rivka-Development/Dovetail-2026/blob/eb2d29aec18223e833680cb5d291c53e6e56e28c/blocks/accordion.liquid#L14-L21

🤖 Generated with Claude Code

_{- If this code review was useful, please react with 👍. Otherwise, react with 👎.}