-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
enhance: ドライブに画像ファイルをアップロードするときオリジナル画像を破棄してwebpublicのみ保持するオプション #8216
Conversation
Co-authored-by: tamaina <tamaina@hotmail.co.jp>
クライアントサイド画像圧縮の準備
Safariでwebpを出力するとやたらファイル容量がでかくなる 第二引数(quality)はwebpしか対応していないため
|
CSSのimage-orientationに対応してたら自動回転対応みたいな感じで良いのかもしれない |
2020年4月の時点で自動回転しなかったのはFirefoxだけなので、今は流石に大丈夫だろうというのしかわからないわね。 |
image-orientation対応とcanvas自動回転のタイミングは一致してなさそうだからダメそう |
Firefoxは、たぶん78 (2020年7月) からちゃんとcanvasも回転 Chromeは、たぶん上の通り81から Safariは、canvasサポートかはわからないけどChromeと同じタイミングだと13.1から |
ExifReaderなくしたらコード量が相当減ったので遅延ロードはしないようにした |
misskey.ioに導入されている(?) |
あきうすの別のPRマージしたら一緒にくっついてきた |
#8215 か チェックボックス機能してないなこれ |
p1.a9z.devを除くMisskeyはチェックボックスだけ存在して機能はまったくしていない状態 |
🙏🏻 |
@@ -27,6 +27,7 @@ export async function GenerateVideoThumbnail(path: string): Promise<IImage> { | |||
|
|||
const outPath = `${outDir}/output.png`; | |||
|
|||
// JPEGに変換 (Webpでもいいが、MastodonはWebpをサポートせず表示できなくなる) |
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.
動画のサムネイルがリモートに送られることはなかった気がするわ
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.
misskey/packages/backend/src/core/activitypub/ApRendererService.ts
Lines 155 to 163 in c7f4fd1
@bindThis | |
public renderDocument(file: DriveFile) { | |
return { | |
type: 'Document', | |
mediaType: file.type, | |
url: this.driveFileEntityService.getPublicUrl(file), | |
name: file.comment, | |
}; | |
} |
そうかも
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.
うーん、ffmpegがwebpをサポートしてなかったのが原因か…?
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.
一度ffmpegで原寸のPNGを出してからJPEG変換+リサイズしているのは、ffmpegにいい感じの縮小メソッドがなかった (アス比保持で最大サイズに外接でするのがうまくいかなかった) からだった気がするのだわ。
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.
See #8825
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.
多分ffmpeg周り触るたびに毎回言ってるぐらいの感じなんだろうけど、代案が見つからないのでIssue建てるのを躊躇したみたいな感じなのでは。
移行するなら https://github.com/ffmpegwasm/ffmpeg.wasm とか?
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.
fluent-ffmpegはやめてないはずだわ
#5186
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.
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.
本題のコードの話だけど、これをwebpにする必要ある…?
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.
本題のコードの話だけど、これをwebpにする必要ある…?
他のサムネがWebP化されてたからここは揃えないのかなって思ったのだわ
Resolve #6626
Resolve #7704
Resolve #8167
Resolve #8173
Fix #8558
What
#8176, #8215 のつづき
Why
Additional Info
orientationが付いてる画像をアップするとうまく回転してくれない問題には手を付けていない
ericnograles/browser-image-resizer#34