Skip to content

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Jul 2, 2021

#34634 added partial support for webp by adding webp to the Legal Image Extensions (File Types) and Legal MIME Types

This PR completes the support for webp by adding webp to the Allowed Extensions

Also adds thumbnail and icon for webp

Makes sure the default values in the xml matches the base.sql

This PR does not add the ability to edit a webp image as I tried to do with #33481 as cropperjs is not compatible with firefox for windows. As a result the edit icon is not displayed for webp images

After the PR

media

# added partial support for webp by adding webp to the Legal Image Extensions (File Types) and Legal MIME Types

This PR completes the support for webp by adding webp to the Allowed Extensions

Also adds thumbnail and icon for webp

Makes sure the default values in the xml matches the base.sql

This PR does __not__ add the ability to edit a webp image as I tried to do with # as cropperjs is not compatible with firefox for windows. As a result the edit icon is not displayed for webp images
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 2, 2021
@ChristineWk
Copy link

Check 1st Issue-nr. pls. Maybe you meant this: #31048 ?


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34690.

@brianteeman
Copy link
Contributor Author

updated the original post

@dgrammatiko
Copy link
Contributor

I have tested this item ✅ successfully on acd772a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34690.

1 similar comment
@Quy
Copy link
Contributor

Quy commented Jul 3, 2021

I have tested this item ✅ successfully on acd772a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34690.

@Quy
Copy link
Contributor

Quy commented Jul 3, 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34690.

@Quy Quy removed the NPM Resource Changed This Pull Request can't be tested by Patchtester label Jul 3, 2021
@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jul 3, 2021
@Quy Quy added this to the Joomla 4.0 milestone Jul 3, 2021
@wilsonge wilsonge merged commit 7a58d05 into joomla:4.0-dev Jul 3, 2021
@wilsonge
Copy link
Contributor

wilsonge commented Jul 3, 2021

Thanks!

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester and removed RTC This Pull Request is Ready To Commit labels Jul 3, 2021
@brianteeman
Copy link
Contributor Author

woohoo

@brianteeman brianteeman deleted the post-media branch July 3, 2021 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants