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

feat: 絵文字のポップアップメニューに編集を追加 #15004

Merged
merged 4 commits into from
Nov 20, 2024

Conversation

sakuhanight
Copy link
Contributor

What

絵文字のポップアップメニューに編集を追加しました。

Why

ノート(DM含む)を起点にカテゴリやタグの変更をしたい
#14264

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 20, 2024
@samunohito
Copy link
Member

lintのエラーが出てるので対応お願いしますー
(エディタのコード整形の設定を変えておくか、eslint --fixが走るようにしておくとよいかも…?)

@sakuhanight
Copy link
Contributor Author

修正しました🙏
✅ All checks have passed

os.popupMenu(menuItems, ev.currentTarget ?? ev.target);
}
}

const edit = async (name: string) => {
const emoji = await misskeyApiGet('emoji', {
Copy link
Member

Choose a reason for hiding this comment

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

編集することを考えるとキャッシュされていない方が良いからgetだとアレですね

Copy link
Contributor Author

Choose a reason for hiding this comment

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

だとしたら既存の編集画面の実装もあわせて修正した方が良さそうですね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

勘違いでした

Copy link
Contributor Author

Choose a reason for hiding this comment

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

修正しました

os.popupMenu(menuItems, ev.currentTarget ?? ev.target);
}
}

const edit = async (name: string) => {
Copy link
Member

Choose a reason for hiding this comment

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

他と揃えてfunctionの方が良さそうです

Copy link
Contributor Author

Choose a reason for hiding this comment

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

修正しました

@syuilo syuilo merged commit 4603ab6 into misskey-dev:develop Nov 20, 2024
22 checks passed
@syuilo
Copy link
Member

syuilo commented Nov 20, 2024

👍🏻

HotoRas pushed a commit to HotoRas/misskey-neko that referenced this pull request Nov 24, 2024
* Mod: 絵文字のポップアップメニューに編集を追加

* fix: code styleの修正

* fix: code styleの修正

* fix
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.

3 participants