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

絵文字ピッカーでloveで検索しても🥰が出てこなくなった #10921

Closed
tamaina opened this issue May 30, 2023 · 14 comments
Closed
Labels
⚠️bug? This might be a bug

Comments

@tamaina
Copy link
Contributor

tamaina commented May 30, 2023

💡 Summary

バンドル削減で辞書を消した?

🥰 Expected Behavior

今まではloveで検索すると🥰が出てきていた

🤬 Actual Behavior

loveで検索すると🥰が出てこない

📝 Steps to Reproduce

📌 Environment

Misskey version: 13.13.0-beta.4 (develop)

@tamaina tamaina added the ⚠️bug? This might be a bug label May 30, 2023
@syuilo
Copy link
Member

syuilo commented May 30, 2023

各絵文字のkeyword情報を消した

  • サイズがデカい
  • 英語固定なのが微妙
  • 固有の絵文字名では検索できるから消しても極端には困らなそう

@EbiseLutica
Copy link
Contributor

慣れの問題ではあるけど地味に使いづらさ出てきそう

@syuilo
Copy link
Member

syuilo commented May 30, 2023

使いづらいのは確実

@CyberRex0
Copy link
Contributor

これぐらいは消さなくても…

@EbiseLutica
Copy link
Contributor

ロードにかかる時間より絵文字探すのにかかる時間のほうが増えてお釈迦になるくらいならここは消しちゃだめかも。

@syuilo
Copy link
Member

syuilo commented May 30, 2023

dynamic importするのも考えたけど色々なところで同期的に参照されてるから困難だった

@syuilo
Copy link
Member

syuilo commented May 30, 2023

  • Misskeyの読み込みが少し速くなるメリットを優先する
  • 絵文字名では検索できるからkeywordsは消す

@tamaina
Copy link
Contributor Author

tamaina commented May 30, 2023

以前も(キャッシュがあれば)そこまで遅いと感じたことはなかった。利便性をとってほしい。

@CyberRex0
Copy link
Contributor

ちなみに絵文字の辞書を残した場合、今の最大限軽量化した状態からどのぐらい元に戻る?

@tamaina
Copy link
Contributor Author

tamaina commented May 30, 2023

(ちなみに: 今回の諸々の削減でトータル300kbぐらい減ったっぽい)
(スクリプトサイズは転送速度だけでなくRAM消費についても絡みそうなことは覚えておきたいかも)

@syuilo
Copy link
Member

syuilo commented May 30, 2023

キーワード部分だけdynamic importするような仕組みにすれば初回読み込みを高速化したまま多言語対応もできそう
けど初回検索時にちょっと待たないといけなくなりそう

@tamaina
Copy link
Contributor Author

tamaina commented Jun 1, 2023

これすぐ直す気ないのか

@syuilo
Copy link
Member

syuilo commented Jun 1, 2023

一応意図した変更

@tamaina
Copy link
Contributor Author

tamaina commented Jun 1, 2023

めちゃくちゃ不便だから戻して

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

No branches or pull requests

4 participants