Remove the view-snap component in favor of concrete classes in the views-view-unformatted template

Created on 14 August 2025, 1 day ago

Problem/Motivation

The view-snap component is a shim to add some wrapping classes around the rows rendered by a view. The problem is, it takes a string for its `rows` prop, but Views probably isn't passing it a string. Nothing is ever easy when Views is around.

This has the unfortunate effect of completely breaking Experience Builder when you are trying to use or render a view.

Proposed resolution

This component shouldn't have to exist. We should replace it with the same set of wrappers in the generic views-view-unformatted.html.twig template. But we only want it to happen if there's a particular tag on the view to opt-in to the display behavior this affords us (let's say we have a preprocess function that adds a `snap` flag if that tag exists on the view, which then influences the display of the template).

πŸ› Bug report
Status

Active

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024