Skip to content
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.

Sign up for GitHub

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

Jump to bottom

アニメーション画像の表示に色々問題がある #4728

Closed
tamaina opened this issue Apr 17, 2019 · 8 comments
Closed

アニメーション画像の表示に色々問題がある #4728

tamaina opened this issue Apr 17, 2019 · 8 comments
Labels
🐛Bug Unexpected behavior

Comments

@tamaina
Copy link
Member

tamaina commented Apr 17, 2019 •
edited

summaly

APNG

webpublicはアニメーションしなくなる → #4108

GIF

サムネイルが作られない

📌 Environment

misskey v11.1.3
@tamaina tamaina added the ⚠️bug? This might be a bug label Apr 17, 2019
@tamaina tamaina changed the title v11でアニメーション画像のサムネイルが表示されない アニメーション画像のサムネイルが表示されない Apr 17, 2019
@tamaina tamaina changed the title アニメーション画像のサムネイルが表示されない アニメーション画像の表示に色々問題がある Apr 17, 2019
@syuilo syuilo added 🐛Bug Unexpected behavior and removed ⚠️bug? This might be a bug labels May 10, 2019
syuilo added a commit that referenced this issue May 15, 2019
@syuilo
GIFのサムネイルが生成されないのを修正
3d8bbed 
#4728
@syuilo
Copy link
Member

syuilo commented May 15, 2019

FYI
sindresorhus/file-type#212

@tamaina
Copy link
Member Author

tamaina commented May 15, 2019

一応こういうのもあるっぽい
https://github.com/davidmz/apng-js

@mei23
Copy link
Contributor

mei23 commented May 15, 2019

アニメーションを再構成したりリサイズするのは大変なので、現実的なのはこのあたり

  • PNGだったら無条件でメタデータ除去もリサイズもしない (GIFと同じ動作)
  • APNGだったら無条件でメタデータ除去もリサイズもしない
    (APNG検出ルーチンを追加する必要がある)
  • APNGだったらメタデータ除去をするがリサイズはしない
    (APNG検出ルーチンとメタデータだけを除去するルーチンを追加する必要がある)

@mei23
Copy link
Contributor

mei23 commented Jun 14, 2019

Related #4108

@mei23
Copy link
Contributor

mei23 commented Jun 14, 2019

file-typeはAPNGに対応していた
https://github.com/sindresorhus/file-type/releases/tag/v12.0.0

ただ、今までAPNGもimage/pngだったのがimage/apng扱いになりそうなので
ちょっと注意が必要そう

@tamaina
Copy link
Member Author

tamaina commented Jun 14, 2019

file-typeをv12にアップデートしてgifと同じ扱いにすれば解決できそう
#4108 に統合します
（余談: image/apngっていうMIME、どうなの…）

@tamaina tamaina closed this as completed Jun 14, 2019
@acid-chicken
Copy link
Member

image/vnd.mozilla.apng?

@tamaina
Copy link
Member Author

tamaina commented Jun 14, 2019

IANA的にはimage/vnd.mozilla.apngらしい
https://www.iana.org/assignments/media-types/image/vnd.mozilla.apng

@mei23 mei23 mentioned this issue Jul 3, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛Bug Unexpected behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@syuilo @tamaina @acid-chicken @mei23