-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add WebP support #9879
Add WebP support #9879
Conversation
I think that we can't see this file type in Safari and Firefox which are currently released. |
@mayaeh the coming version of Firefox 56 will likely include WebP support. |
Update to this: Firefox 65 is now out and does support WebP. This includes Firefox for Android and Desktop, though not iOS. See https://www.mozilla.org/en-US/firefox/65.0/releasenotes/ |
@@ -25,10 +25,10 @@ class MediaAttachment < ApplicationRecord | |||
|
|||
enum type: [:image, :gifv, :video, :unknown] | |||
|
|||
IMAGE_FILE_EXTENSIONS = ['.jpg', '.jpeg', '.png', '.gif'].freeze | |||
IMAGE_FILE_EXTENSIONS = ['.jpg', '.jpeg', '.png', '.gif', '.webp'].freeze |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about this: Keep the support but remove the changes to the tooltip. Then it's future-proof without encouraging users to upload images that still aren't supported by Safari/iOS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reverts commit 5092d17.
* Add WebP support * Remove the changes to the tooltip refs: mastodon#9879 (review)
* Add WebP support * Remove the changes to the tooltip refs: mastodon#9879 (review)
Resolves #8165