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

入力フォームでもリアクション選択時に使用するピッカーを使うようにしたい #12337

Merged

Conversation

samunohito
Copy link
Member

@samunohito samunohito commented Nov 15, 2023

What

#12336 の対応です。
この対応により、投稿フォームの絵文字ピッカーがノートにリアクションをつける時のリアクションピッカーと同じものになります(詳細はissueにて)

Why

詳細は #12336 にて

Additional info (optional)

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 Nov 15, 2023
Copy link

codecov bot commented Nov 15, 2023

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Comparison is base (af15f8d) 78.42% compared to head (62a2922) 78.75%.

Files Patch % Lines
packages/frontend/src/scripts/emoji-picker.ts 52.63% 27 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12337      +/-   ##
===========================================
+ Coverage    78.42%   78.75%   +0.33%     
===========================================
  Files          180      954     +774     
  Lines        23857   103809   +79952     
  Branches       495     8353    +7858     
===========================================
+ Hits         18709    81752   +63043     
- Misses        5148    22057   +16909     

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

Copy link
Contributor

github-actions bot commented Nov 15, 2023

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@samunohito samunohito marked this pull request as ready for review November 15, 2023 07:58
@samunohito samunohito changed the title [wip] 入力フォームでもリアクション選択時に使用するピッカーを使うようにしたい 入力フォームでもリアクション選択時に使用するピッカーを使うようにしたい Nov 15, 2023
@@ -124,6 +124,7 @@ import { deepClone } from '@/scripts/clone.js';
import MkRippleEffect from '@/components/MkRippleEffect.vue';
import { miLocalStorage } from '@/local-storage.js';
import { claimAchievement } from '@/scripts/achievements.js';
import { reactionPicker } from '@/scripts/reaction-picker.js';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

リアクションをするわけではないから別のものを作った方が良さそう

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@syuilo
Copy link
Member

syuilo commented Dec 3, 2023

ついでに絵文字ピッカーにリアクションデッキとして設定した絵文字が表示されるのも直すか

@syuilo syuilo self-assigned this Dec 3, 2023
@syuilo syuilo merged commit 5e1d872 into misskey-dev:develop Dec 3, 2023
9 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 3, 2023

👍🏻 👍🏻

camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* 入力フォームでもリアクション選択時に使用するピッカーを使うようにしたい

* erase console.log

* fix CHANGELOG.md

* reaction-picker.ts を戻し、今回の対応を入れた emoji-picker.ts を新たに作成

* fix CHANGELOG.md

* tweak

---------

Co-authored-by: osamu <46447427+sam-osamu@users.noreply.github.com>
Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
@samunohito samunohito deleted the enhance/input_form_reaction_picker branch January 11, 2024 00:15
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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants