Resolve #9695

Includes #10142

What

ノートなどに含まれるemojis（populateEmojiの結果）は（プロキシされたURLではなく）オリジナルのURLを指すように
絵文字は必ずプロキシする
MFMでx3/x4もしくはscale.x/yが2.5以上に指定されていた場合にはオリジナル品質の絵文字を使用するように

Why

populateEmojiの結果がなぜかプロキシだけ通して絵文字品質へ圧縮していない問題も修正されます

Feb 04 '23 15:02 tamaina

ノートなどに含まれるemojis（populateEmojiの結果）は（プロキシされたURLではなく）オリジナルのURLを指すように

リモート直リンクは問題起きない？前にインスタンスのアイコン直リンクしてたらブラウザの警告出たことあったし

Feb 05 '23 05:02 syuilo

リモート直リンクは問題起きない？

このPRではMkCustomEmojiがプロキシを通すようにしている（はず）

Feb 05 '23 06:02 tamaina

MFMでx3/x4もしくはscale.x/yが2.5以上に指定されていた場合にはオリジナル品質の絵文字を使用するように

これあんまり要らない気がした

Feb 05 '23 11:02 syuilo

これあんまり要らない気がした

いいえ、絶対に必要です

Feb 05 '23 13:02 tamaina

ping

Feb 10 '23 13:02 tamaina

MFMでx3/x4もしくはscale.x/yが2.5以上に指定されていた場合にはオリジナル品質の絵文字を使用するように

これ寸法計算したいしやっぱりプロキシの仕様整理しようぜ

Feb 11 '23 03:02 acid-chicken

寸法計算したい

xNオプションになったわけだしそんな厳密にやる必要ない気もする

プロキシの仕様整理

これ以上整理する気ない

Feb 11 '23 04:02 tamaina

xNオプションになったわけだしそんな厳密にやる必要ない気もする

画像サイズ側をちゃんとした寸法で返そうって話

これ以上整理する気ない

今あまり整理された仕様ではなくない？

Feb 11 '23 04:02 acid-chicken

ちゃんとした寸法で

クライアント側での計算量、キャッシュを効かせられないなどの面で不利なのでpx単位でプロキシに要求するのはやめたい

今あまり整理された仕様ではなくない？

個人的にはもう十分整理した気になっている

Feb 11 '23 04:02 tamaina

クライアント側での計算量、キャッシュを効かせられないなどの面で不利なのでpx単位でプロキシに要求するのはやめたい

（キャッシュはできるという前提で）計算量は重なってる scale を掛け合わせるだけでオーダーもへったくれもないし、キャッシュについてはどうせノートで scale かわらんからさして不利どころかオリジナルを問い合わせてキャッシュに乗せる方が容量取る（オリジナルの解像度にもよるが）

個人的にはもう十分整理した気になっている

https://github.com/misskey-dev/misskey/issues/9733#issuecomment-1416369462 などで前にも書いてるけど特に回答もらってない気がする

Feb 11 '23 04:02 acid-chicken

これあんまり要らない気がした

いいえ、絶対に必要です

近い将来カスタム絵文字として使われる画像の大きさを制限する可能性が高いからそれ考えると要らない感

Feb 20 '23 03:02 syuilo

近い将来

いつ

Feb 20 '23 06:02 tamaina

近い将来

Feb 20 '23 07:02 syuilo

重なってる scale を掛け合わせるだけ

端末での表示倍率やMFMば表示されているフォントサイズはまちまちなので、scaleを掛け合わせるだけで必要なpxを得ることはできない
（し、多くの場合普通のサイズで表示されるのだからそこまで複雑な処理を書く必要はない）

どうせノートで scale かわらんからさして不利どころかオリジナルを問い合わせてキャッシュに乗せる方が容量取る（オリジナルの解像度にもよるが）

この部分理解できなかった

Feb 20 '23 07:02 tamaina

Codecov Report

Merging #9794 (28698d8) into develop (0db88a5) will increase coverage by 3.47%. The diff coverage is 59.25%.

@@             Coverage Diff             @@
##           develop    #9794      +/-   ##
===========================================
+ Coverage    74.94%   78.42%   +3.47%     
===========================================
  Files          891      164     -727     
  Lines        87860    20320   -67540     
  Branches      5951      359    -5592     
===========================================
- Hits         65846    15935   -49911     
+ Misses       22014     4385   -17629

Impacted Files	Coverage Δ
packages/frontend/src/components/mfm.ts	`33.33% <34.37%> (+1.32%)`	:arrow_up:
packages/frontend/src/scripts/media-proxy.ts	`18.75% <50.00%> (ø)`
...s/frontend/src/components/global/MkCustomEmoji.vue	`100.00% <100.00%> (ø)`

... and 727 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Feb 22 '23 09:02 codecov[bot]

マージコンフリクト直したけどなんでコンフリクトと言われたかがわからない

Feb 22 '23 09:02 tamaina

conflict resolved

Feb 26 '23 13:02 tamaina

#10142 をマージ

Feb 28 '23 06:02 tamaina

プロキシの仕様整理

https://github.com/misskey-dev/media-proxy/blob/master/SPECIFICATION.md

メディアプロキシの仕様を書き出してみた

Feb 28 '23 12:02 tamaina

scaleで掛け合わせるようにした

Apr 12 '23 01:04 tamaina

今マージする気満々でいる

Apr 12 '23 01:04 tamaina

（一応テストを待ってた

Apr 12 '23 02:04 tamaina

enhance: カスタム絵文字関連の変更

What

Why

Codecov Report