Umami views should use responsive grid

Found various unformatted lists that are simulating a grid, as well as the legacy views grid style.

Converted all of these in the MR but have not touched the CSS.

The recipes collection looks broken now - the links are green, probably needs a class or two changing.

I'm sure we can remove some now-unnecessary CSS that was styling the unformatted lists but will leave that to the interested reader since CSS isn't my strength.

Gonna check css here

Great initiative!

• I've added 2 more views configs. They should follow same responsive grid
• Removed outdated css grids.
• Fixed css
• Modernized css for promoted view. Article page grid. For me it is easier to manage all together. It can be moved to new issue.

Please review

Nice! Appears to have test failures though :(

Something wrong here with css_class schema. tried with both boolean and false.

views config schema says it should be boolean - did you try installing and then re-exporting the view?

seems that css class should be configured in admin ui and then reexported. probably it will help

All green!

I'm not qualified to review the actual CSS changes here, but the net reduction is great and shows just what an excellent feature responsive grid is!

Rebased, pretty sure this will fail performance tests now for Umami since CSS coverage for added - but we just need to reduce the CSS byte assertions because there'll be less. Having trouble with chromedriver locally at the moment so letting gitlab tell us the new numbers.

Back to green :)

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Seems to have failures in the openTelemetry tests which I believe (correct me if I'm wrong) isn't random.

It's fine to set the ceiling on the OpenTelemetry tests 500k higher than whatever the eventual number is, that way other small CSS/JS changes won't cause this MR to fail.

Finally got around to this one

Did a fresh Umami install without the MR applied to verify column shifts on pages like /articles

Did a fresh Umami install with the MR already applied so the config imported
Visited /articles page to verify the layout is the same and responsive at various breakpoints

Then I did the same for the homepage as I saw that view was updated.
Also did not notice any issue.

Believe this is a good replacement.

nod_ → closed merge request !6970

Since this only impacting umami I'm backporting back to 10.3.x

Committed and pushed 587350c208 to 11.x and f607b858d6 to 11.0.x and 57a0cd753d to 10.4.x and b6026c1215 to 10.3.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.