Problem/Motivation

After #2934962: Ship local audio and video media types in Standard the standard profile adds two new Media types as soon as the media module is installed: Audio (local) and Video (local).

In #2895382: Hide label, thumbnail, etc. for default display of media file and image references we improved the default display of the File and Image media types to show better defaults:

Default File display:

Default Image display:

We should do the same with the Audio and Video types, which currently are not consistent with that:

Current default Audio display:

Current default Video display:

Proposed resolution

Improve the defaults for the Audio and Video displays by hiding some unnecessary fields from the shipped configuration.

Proposed default for Audio:

Proposed default for Video:

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2935318-2.patch3.16 KBmarcoscano
#2 video_default_after.png62.79 KBmarcoscano
#2 audio_default_after.png9.08 KBmarcoscano
video_default.png49.43 KBmarcoscano
audio_default.png16.48 KBmarcoscano
image_default.png906.41 KBmarcoscano
file_default.png8.63 KBmarcoscano
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcoscano created an issue. See original summary.

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
CreditAttribution: marcoscano for Acquia commented
Assigned: marcoscano » Unassigned
Issue summary: View changes
Status: Active » Needs review
FileSize
audio_default_after.png9.08 KB
video_default_after.png62.79 KB
2935318-2.patch3.16 KB

Here we go.

Added screenshots after the patch is applied to the IS.

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented

I love this patch! I was going to file an image issue for exactly this.

chr.fritsch’s picture

chr.fritsch
he/him
Primary language German
Location 🇩🇪🇪🇺🌍
CreditAttribution: chr.fritsch at Thunder commented
Status: Needs review » Reviewed & tested by the community

Tested the patch locally and can confirm that it does what it promises. Audio and Video views are much cleaner now.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Issue tags: +Usability

YES PLEASE

  • xjm committed 6abe25e on 8.5.x 
    Issue #2935318 by marcoscano: Improve default display of Audio and Video...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.5.x. Thanks!

Status: Fixed » Closed (fixed)

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