-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
絵文字のオートコンプリート強化の対応 #12365
絵文字のオートコンプリート強化の対応 #12365
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12365 +/- ##
===========================================
- Coverage 78.77% 78.77% -0.01%
===========================================
Files 947 947
Lines 102778 102778
Branches 8285 8284 -1
===========================================
- Hits 80968 80966 -2
- Misses 21810 21812 +2 ☔ View full report in Codecov by Sentry. |
このPRによるapi.jsonの差分 差分はこちら |
const queryChars = [...query]; | ||
const hitEmojis = new Map<string, EmojiScore>(); | ||
|
||
emojiDb.forEach(x => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for of が使えない理由ってあるかしら?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
全く無いですね…コピペもとがsomeで、それをforEachにしただけでした
特にこだわりはない+周りを見たらfor ofだったので合わせました
👍👍👍 |
* 前方一致・部分一致でなくても近似値でヒットするように * fix CHANGELOG.md * for of に変更 --------- Co-authored-by: osamu <[email protected]>
What
#12364 の対応です。
この対応により、絵文字をオートコンプリートが強化されます。
Why
絵文字名が長い時など、オートコンプリートの絞り込み機能が弱いと入力が面倒なため
Additional info (optional)
実際に動作させて確認
Checklist