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

Complete unicode emoji after last colon #13384

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

Conversation

anatawa12
Copy link
Member

@anatawa12 anatawa12 commented Feb 18, 2024

What

Fix #12662

Why

Issue参照。

githubやdiscordなどで :+1: の形式に慣れている人だと息をするように最後の : まで入力してしまい、これが unicode emoji に変換できないのがストレスになるため。

Additional info (optional)

使いまわしたのでfuzzy searchになってますが、完全一致だけでもいいとは思っています。

#13384 (comment) 不都合があったっぽいので完全一致にしました

image

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Feb 18, 2024
Copy link

codecov bot commented Feb 18, 2024

Codecov Report

Attention: Patch coverage is 2.43902% with 120 lines in your changes missing coverage. Please review.

Project coverage is 20.25%. Comparing base (8b4933c) to head (6be45f6).

Files Patch % Lines
...ackages/frontend/src/components/MkAutocomplete.vue 0.00% 61 Missing ⚠️
packages/frontend/src/scripts/autocomplete.ts 3.12% 31 Missing ⚠️
packages/frontend/src/scripts/search-emoji.ts 6.66% 28 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13384       +/-   ##
============================================
- Coverage    40.16%   20.25%   -19.92%     
============================================
  Files         1521      701      -820     
  Lines       188552    98754    -89798     
  Branches      3509     1027     -2482     
============================================
- Hits         75735    19998    -55737     
+ Misses      112246    78233    -34013     
+ Partials       571      523       -48     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@anatawa12 anatawa12 force-pushed the complete-emoji-after-last-colon branch from d207f0c to 94069e7 Compare February 18, 2024 14:06
u1-liquid added a commit to MisskeyIO/misskey that referenced this pull request Mar 15, 2024
Co-authored-by: anatawa12 <anatawa12@icloud.com>
u1-liquid added a commit to MisskeyIO/misskey that referenced this pull request Mar 15, 2024
Co-authored-by: anatawa12 <anatawa12@icloud.com>
@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Mar 24, 2024

こういう事象がおこっているらしい

・【5、6枚目】絵文字の登録名すべてを手打ちして末尾の「:」を手打ちするとなにかの予測が出てくるようになる。(以前は見られなかった挙動)
・【7枚目】リアクションデッキか選択した場合でもなにかの予測が出てくる、画像は:peroro_sama:をクリックして洗濯した場合(以前は見られなかった挙動)

https://misskey.io/notes/9r8o6v1cwxbg0bg5

@anatawa12
Copy link
Member Author

用途的に完全一致で問題ないから完全一致のみ表示するようにするか

@anatawa12
Copy link
Member Author

image

した

@anatawa12 anatawa12 force-pushed the complete-emoji-after-last-colon branch from 188e2a3 to 47c4a6e Compare March 24, 2024 10:02
@anatawa12
Copy link
Member Author

lintはaddし忘れだった(add後に修正してたの忘れてた)すみませn

@anatawa12
Copy link
Member Author

frontend e2eは develop もおちてそう? https://github.com/misskey-dev/misskey/actions/runs/8407815099/job/23023188423

@Sayamame-beans
Copy link
Member

Cypress → #13605 ?

u1-liquid added a commit to MisskeyIO/misskey that referenced this pull request Mar 30, 2024
cheery-picked from anatawa12/misskey@47c4a6e

* fix code quality issues

Co-authored-by: anatawa12 <anatawa12@icloud.com>
u1-liquid added a commit to MisskeyIO/misskey that referenced this pull request Mar 31, 2024
…586)

cheery-picked from anatawa12/misskey@47c4a6e

* fix code quality issues

Co-authored-by: anatawa12 <anatawa12@icloud.com>
@anatawa12
Copy link
Member Author

changelogついかしてなかったので追加しました

@anatawa12
Copy link
Member Author

conflict resolved

@Sayamame-beans
Copy link
Member

conflict解消

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.

:+1::ok_hand:などをunicode絵文字に最後の:を入力してから変換したい
3 participants