CMS MADE SIMPLE FORGE

SmartImage

 

[#12480] webp support is partial at best

avatar
Created By: Ludger Merkens (lume)
Date Submitted: Thu Sep 30 05:17:53 -0400 2021

Assigned To: Fernando Morgado (JoMorg)
Version: 1.0
CMSMS Version: 2.2.14
Severity: Minor
Resolution: Fixed
State: Open
Summary:
webp support is partial at best
Detailed Description:
Thanks for starting to support webp in SmartImage. As you just started, this is
more or less a feature request.

While the module can read .webp files now, the output seems to create "icon"
files. Even trying to "force_type" the output does not change anything. But as
those files tend to be bigger than the original file, this kind of beats the
purpose of webp support.

thank your for your work
Ludger Merkens


History

Comments
avatar
Date: 2021-10-21 09:31
Posted By: Ludger Merkens (lume)

class.si_utils.php line 107 in function get_extension($mime_type) should
probably read:

 'image/webp'               => '.webp' 

(instead of  'image/webp'               => '.icon')


      
avatar
Date: 2021-10-21 12:42
Posted By: Ludger Merkens (lume)

class.si_utils.php line 107 in function get_extension($mime_type) should
probably read:

 'image/webp'               => '.webp' 

(instead of  'image/webp'               => '.icon')


      
avatar
Date: 2023-03-03 14:15
Posted By: Fernando Morgado (JoMorg)

Seems to be fixed in SVN, ready for release
      
Updates

Updated: 2023-03-03 14:15
resolution_id: 5 => 7

Updated: 2021-10-21 09:31
description: Thanks for starting to support webp in SmartImage. As you just started, this is more or less a feature request. While the module can read .webp files now, the output seems to create "icon" files. Even trying to "force_type" the output does not change => Thanks for starting to support webp in SmartImage. As you just started, this is more or less a feature request. While the module can read .webp files now, the output seems to create "icon" files. Even trying to "force_type" the output does not change a
resolution_id: => 5