Problem/Motivation

The standard profile uses an Image field on Articles. Using Media is the current best practice.

Proposed resolution

Let's remove that and replace it with a Media reference field of the same name that we also add to the Basic Page content type.

This will involve enabling the core Media module as part of the profile install. We might as well enable Media Library too.

In addition to adding the field, let's take this opportunity to configure the display modes and form modes for Article and PAge.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork touchstone-3248737

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

danflanagan8 created an issue. See original summary.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented
Issue summary: View changes

danflanagan8 opened merge request !5

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented
Status: Active » Needs review

I took this opportunity to do some default configuration for form modes and view modes for Article and Basic Page.Here are the highlights:

  1. Use media_library widget on the forms
  2. Set Full View mode to use custom configuration (see #3248576: Configure content types to use custom display settings for Full view mode)
  3. Default View mode and Teaser view mode both render Media as a "Medium" Thumbnail that links to content. The default is to display Media as a rendered entity which can really become a disaster since you end up having to create a new Media view mode for each image style. We want to use the oft misunderstood or overlooked Thumbnail formatter
  4. On Full View Mode render the Media as a "wide" Thumbnail

These defaults use a pretty light touch but should go a long way towards preventing some common problems (particularly 2 and 3).

Allie Micka made their first commit to this issue’s fork.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented
Assigned: danflanagan8 » Unassigned
allie micka’s picture

allie micka
she/her
commented

Sorry to sit on this important one. I had to sit with the idea of adding images to pages (see #3250016) even though I know full well they're needed for those share links. jerdavis helped make this click a by referring to this as a 'representative image,' which you'd have whether or not it appears on the page itself. I liked that so I updated the help text accordingly.

allie micka’s picture

allie micka
she/her
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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