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

enhance(frontend): 絵文字のオートコンプリートのアルゴリズムの改善 #261

Merged
merged 1 commit into from
Nov 23, 2023

Conversation

u1-liquid
Copy link
Member

@u1-liquid u1-liquid commented Nov 23, 2023

What

  • 実際は同じ絵文字なら重複してサジェストに出ないように
  • エイリアスではない絵文字>前方一致>部分一致>あいまい検索順で表示されるようになるように

Why

期待通りの検索結果になっていない場合が多いため

Additional info (optional)

ref: misskey-dev#12365
cc: @syuilo

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

エイリアスではない絵文字>前方一致>部分一致>あいまい検索順で表示されるようになるはず
Copy link

sonarcloud bot commented Nov 23, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@u1-liquid u1-liquid merged commit 70d81e4 into MisskeyIO:io Nov 23, 2023
16 checks passed
@u1-liquid u1-liquid deleted the eh-autocomplete-emoji branch November 23, 2023 21:53
syuilo pushed a commit to misskey-dev/misskey that referenced this pull request Nov 24, 2023
* 実際は同じ絵文字なら重複してサジェストに出ないように
* エイリアスではない絵文字>前方一致>部分一致>あいまい検索順で表示されるようになるように
anatawa12 pushed a commit to anatawa12/misskey that referenced this pull request Nov 27, 2023
* 実際は同じ絵文字なら重複してサジェストに出ないように
* エイリアスではない絵文字>前方一致>部分一致>あいまい検索順で表示されるようになるように
camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* 実際は同じ絵文字なら重複してサジェストに出ないように
* エイリアスではない絵文字>前方一致>部分一致>あいまい検索順で表示されるようになるように
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant