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.

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

サーバーの異なる同じ絵文字を一緒に表示する #12480

Open
Srgr0 opened this issue Nov 27, 2023 · 5 comments
Open

サーバーの異なる同じ絵文字を一緒に表示する #12480

Srgr0 opened this issue Nov 27, 2023 · 5 comments
Labels
[Feat] Emoji/EmojiPicker Emoji or EmojiPicker related issue ✨Feature This adds/improves/enhances a feature

Comments

@Srgr0
Copy link
Collaborator

Srgr0 commented Nov 27, 2023

Summary

絵文字の情報としてファイルのハッシュを付与しておき、ハッシュが同じ絵文字は一つにまとめて表示する。

Purpose

特に画面の小さいデバイスで表示している場合、同じ絵文字が複数表示されることにより画面内の情報量が少なくなるため。
IMG_0981

@Srgr0 Srgr0 added the ✨Feature This adds/improves/enhances a feature label Nov 27, 2023
@tamaina
Copy link
Contributor

tamaina commented Nov 27, 2023

Related to #8905

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Dec 19, 2023

同じ内容の絵文字を他所からパクるのではなく、自作したうえで登録とかする(もしくは画像に縮小や圧縮などの加工を施す)と、ハッシュは当然合わなくなるのでその場合同一としてまとめる or まとまらないが混在してしまいそう

@Srgr0
Copy link
Collaborator Author

Srgr0 commented Dec 19, 2023

同じ内容の絵文字を他所からパクるのではなく、自作したうえで登録とかする(もしくは画像に縮小や圧縮などの加工を施す)と、ハッシュは当然合わなくなるのでその場合同一としてまとめる or まとまらないが混在してしまいそう

私の考えとしては、一気に全てをまとめるのではなく、とりあえずblobsなど多くのサーバーで共通にインポートされている(ハッシュが同じ)絵文字をカバー出来れば良いのかなと思っています。
色やフォントが微妙に異なる絵文字をまとめる場合はidなどで識別することになるかと思いますが、これには悪用のリスクが存在します。かといって各サーバーの大量の絵文字を手動で紐付けるようにするのも現実的ではないため、こちらは引き続き検討が必要と考えます。

@Srgr0
Copy link
Collaborator Author

Srgr0 commented Dec 21, 2023

例として、以下のノートではリアクションをすべて表示するのに3画面分の長さが必要ですが、happy birthdayの金色の絵文字がひとつにまとまるだけで1画面分節約できます。
まとめないと見にくいほどリアクションがついているノートにおいて、全てが異なる絵文字がついていることは少なく、例のようにサーバーの異なる同じ絵文字がついている場合が多いため、(全く)同じ絵文字をまとめるだけでも効果は大きいのかなと思います。

image
image
image

@KisaragiEffective
Copy link
Collaborator

#14021 が実装されたらその情報を元にマージできるかもしれない

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feat] Emoji/EmojiPicker Emoji or EmojiPicker related issue ✨Feature This adds/improves/enhances a feature
Projects
Status: No status
Development

No branches or pull requests

5 participants