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

fix(frontend): 絵文字ピッカーのカスタム絵文字検索で絵文字名に大文字が含まれているものを検索できない件の修正 #11321

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

senamih
Copy link

@senamih senamih commented Jul 19, 2023

What

  1. 絵文字ピッカーのカスタム絵文字検索において、
    まずは検索文字列と絵文字名orエイリアスのそれぞれそのまま変換せずに比較検索を試みて、
    見つからなければ小文字同士に変換して比較検索する。

  2. 検索ロジックにおいて、検索処理毎に絵文字リストを走査するコードになっていたため、
    一度の絵文字リスト走査だけで検索を完了させるコードへの変更・軽量化。

上記メイン修正1において検索機能が重くなるのを避けるために、上記2を含ませていただきました。

Why

#11308 絵文字ピッカー上の検索機能で、絵文字名に大文字が含まれているものを検索できない
上記不具合に対する修正。
検索文字列を小文字に変換してから検索されていることで絵文字名内の大文字がヒットしなくなっていた。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

備考

先にPR作成させていただきました。お手数おかけしますがご確認をお願い致します。

#11318
storybook落ちの件判明しましたらこちらでも対応します。

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jul 19, 2023
@github-actions github-actions bot requested review from EbiseLutica and tamaina July 19, 2023 02:59
@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Merging #11321 (879dee8) into develop (b0b0ecb) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop   #11321      +/-   ##
===========================================
+ Coverage    77.70%   77.73%   +0.03%     
===========================================
  Files          917      919       +2     
  Lines        92568    92728     +160     
  Branches      7743     7743              
===========================================
+ Hits         71926    72086     +160     
  Misses       20642    20642              
Impacted Files Coverage Δ
packages/frontend/src/components/MkEmojiPicker.vue 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

@tamaina tamaina requested a review from syuilo July 19, 2023 04:02
@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

ありがとうございます 🙏🏻

まずは検索文字列と絵文字名orエイリアスのそれぞれそのまま変換せずに比較検索を試みて、
見つからなければ小文字同士に変換して比較検索する。

大文字小文字を区別したいことがないかも?

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

絵文字名に大文字を使えること自体がバグに近かった記憶

@senamih
Copy link
Author

senamih commented Jul 20, 2023

ご確認ありがとうございます。
現状 絵文字名に大文字が使える状態であり、区別して登録できてしまうことにより発生する現象になりますので、
例えば運用側で大文字を使わないように通知したり、絵文字名に大文字を使えないようにする等の対応でも問題ないと思います。
issue側 #11308 にも似たような事を書かせていただいておりますが、
今回、あるユーザがカスタム絵文字を一部大文字で登録したことで、
絵文字ピッカーの検索に(何故か)出てこず利用される機会を損失してしまっているのを見たので、
PRさせていただいております。

最終的にどのような仕様が良いのかについてはご検討いただければ幸いです。
宜しくお願い致します。

@tamaina
Copy link
Contributor

tamaina commented Jul 21, 2023

しゅいろ氏曰く、カスタム絵文字サジェストにおいては大文字小文字が区別されるのはバグだそうです

#6489

@senamih
Copy link
Author

senamih commented Jul 21, 2023

@tamaina
そんな前の方に…!既にIssueあったのに検索しきれておらずすみません。

@syuilo
3年前程、上記Issueが既にしゅいろさんによって起こされていたようです。
引き続き最終仕様についてご判断いただきたく思います。
お時間ある際に、ご検討宜しくお願い致します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
Development

Successfully merging this pull request may close these issues.

3 participants