Style blog/news teaser as cards

Created on 5 December 2024, 17 days ago

Problem/Motivation

We've lost the standard teaser styling in Olivero because we are using different fields, so the content teasers look pretty messy.

We'll ultimately want this for all content types, but we can start with blog and news article.

Proposed resolution

Updated design uses a card for the teaser, with a 1/2/3 column layout based on the content:

πŸ“Œ Task
Status

Active

Component

Olivero

Created by

πŸ‡¦πŸ‡ΊAustralia pameeela

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

Merge Requests

Comments & Activities

  • Issue created by @pameeela
  • πŸ‡¦πŸ‡ΊAustralia pameeela
  • πŸ‡¦πŸ‡ΊAustralia rikki_iki Melbourne
  • πŸ‡¦πŸ‡ΊAustralia rikki_iki Melbourne

    Going to add a new view mode to these (possibly all) content types for the Card display, so need to update a few Recipes. Otherwise the styling is done, I'll push a draft MR anyway as I won't get the config/Recipe updates done today.

  • Merge request !287Resolve #3491807 "Teaser cards" β†’ (Merged) created by rikki_iki
  • Pipeline finished with Failed
    16 days ago
    Total: 650s
    #360869
  • Pipeline finished with Failed
    16 days ago
    Total: 734s
    #360868
  • Pipeline finished with Failed
    9 days ago
    Total: 52s
    #367161
  • Pipeline finished with Failed
    9 days ago
    Total: 52s
    #367162
  • Pipeline finished with Failed
    9 days ago
    Total: 53s
    #367163
  • Pipeline finished with Failed
    9 days ago
    Total: 662s
    #367169
  • πŸ‡¦πŸ‡ΊAustralia rikki_iki Melbourne
  • Pipeline finished with Failed
    9 days ago
    Total: 1576s
    #367170
  • πŸ‡¦πŸ‡ΊAustralia pameeela

    I tested this locally, it looks great! There's an issue with the templates not being detected properly but I tested by copying everything over into the core Olivero theme.

    The only thing that wasn't working as expected was the three-column view, but I'm not sure whether this is because of the other wonkiness (but the one-column works!).

    I also removed the username from the display for now, because that's a whole can of worms that I don't think we have time to properly deal with, for one thing since it links to the profile page which we're not set up for (not to mention the default username 'admin' is.... suboptimal for display purposes). We may want to add a separate author field later on to get around this, just depends how we ultimately hand this.

  • Pipeline finished with Failed
    6 days ago
    Total: 487s
    #369595
  • Pipeline finished with Success
    6 days ago
    Total: 503s
    #369594
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    I think we have to update the machine name of one field, but otherwise this code looks good to me and I would consider it merge-ready.

  • Pipeline finished with Failed
    5 days ago
    Total: 388s
    #370351
  • πŸ‡¦πŸ‡ΊAustralia pameeela

    Manually tested by coping the entire subtheme into the contrib folder, and the three-column layout works as expected. Also fixed the field machine name, so RTBC if green.

  • Pipeline finished with Success
    5 days ago
    Total: 358s
    #370479
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts
  • Pipeline finished with Skipped
    5 days ago
    #370501
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    Glad we're gonna ship looking sharp. :) Merged into 1.x, thanks!

  • πŸ‡¦πŸ‡ΊAustralia pameeela

    This is great but for those wondering why they don't see this improvement yet: πŸ› Add symlink for Drupal CMS Olivero Active

  • πŸ‡¦πŸ‡ΊAustralia pameeela

    pameeela β†’ changed the visibility of the branch pameeela-738525dc-patch-a1f4 to hidden.

Production build 0.71.5 2024