[MGNLDAM-807] Assets editor-subapp fails to display GIF assets Created: 04/Nov/19  Updated: 17/Mar/20  Resolved: 19/Dec/19

Status: Closed
Project: Magnolia DAM Module
Component/s: None
Affects Version/s: None
Fix Version/s: 3.0

Type: Bug Priority: Neutral
Reporter: Christoph Meier Assignee: Rishab Dhar
Resolution: Obsolete Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: Text File MGNLDAM-807-Add-.gif-MediaType-to-supported-image-as.patch    
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: Migrate apps to new UI framework

 Description   

The subapp of the new assets-app to edit (or upload) single assets does not display the image IF the asset is a GIF image.
My first assumption was, that this issued depends on the image size, but it appears to depend on the mime type (failing for GIF images).

(If the asset is an image, its preview (on the bottom of the action bar is always shown, also for GIFs.))

For testing reasons, I have uploaded the following list of assets: 

name file size image shown in detail-subapp
the-damned_big-size.gif 52KB
the-damned_big-size.jpg 111KB
the-damned_big-size.png 122KB
the-damned_small-size.gif 564B
the-damned_small-size.jpg 643KB
the-damned_small-size.png 710B
another-big-GIF.gif 300KB
animated.gif 72KB

The assets used for the tests can be found in dropbox

 

P.S.
I have just checked on https://demo.magnolia-cms.com/ - there, with the old version of the DAM app, GIFs are displayed in the detail-subapp.



 Comments   
Comment by Rishab Dhar [ 12/Dec/19 ]

The issue is fixed:

https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/1241/commits/d03c46e69c5bbd2797e8c8a5c2a153b9041ae80a

Currently there is an open PR with this fix:

https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/1241/overview

A patch for the fix has also been added to the ticket to test the fix until the PR is merged.

Generated at Mon Feb 12 05:03:28 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.