Problem/Motivation

Alternative thumbnail preview image was working 1.1.0-rc1 version but due to this commit alternative preview is not displaying except content view page.
Introduced in issue#3492560
Prerequisite: Upload alternate thumbnail preview image from widencollective

Steps to reproduce

  • composer require drupal/acquia_dam:1.1.0-rc1
  • drush en acquia_dam
  • Navigate to the path: admin/config/acquia-dam and enter the details.
  • Authenticate the DAM from user's page also.
  • Create media type field (If not exists) on any content type i.e page or article
  • Select Acquia DAM: Image and Acquia DAM: Video and click on save button.
  • Now create one content, click on Add Media and select image or video from the available list.
  • You will notice alternate thumbnail preview image is consistent throughout the application.
  • Now upgrade to version 1.1.0-rc3 or rc2, you will observe that alternate thumbnail preview image is only displaying on content view page.

Proposed resolution

Asset Thumbnail preview should be consistent over the media library pages, Content edit and view page

Issue fork acquia_dam-3499668

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

ankitv18 created an issue. See original summary.

ankitv18 opened merge request !107

ankitv18’s picture

ankitv18 commented
Status: Active » Needs review

Please do review the MR!107

deepakkm’s picture

deepakkm commented
Status: Needs review » Reviewed & tested by the community

verified changes hence RTBC

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented

Wanted to propose updating the target branch for this issue to 1.0.x. Since this change is more of a bug fix, it might be more appropriate to align it with the 1.0.x release. We can then integrate it into the 1.1.x release subsequently.

Please let me know your thoughts on this.

deepakkm’s picture

deepakkm commented

This issue doesn't replicate on branch 1.0.x, if you read the description of the issue this was working until 1.1.0-rc1 but due to certain commit it started failing and hence this was fixed for 1.1.x branch.

japerry made their first commit to this issue’s fork.

  • japerry committed 171fca41 on 1.1.x authored by ankitv18 
    Issue #3499668 by ankitv18: [Regression] Alternative thumbnail image...
japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Fixed

Good catch. Fixed!

Status: Fixed » Closed (fixed)

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