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

emitの追加によるclosePopupMenuの発火 #4341

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

alter334
Copy link
Contributor

@alter334 alter334 commented Jul 4, 2024

close #3904

上部ツールバー部分において、3点ドットを押すとポップアップメニューが開くが同メニュー内のアイテムがクリックされた際にポップアップが閉じられるようにしました。

ポップアップを親コンポーネントとしたとき、メニュー内アイテムが孫コンポーネントに位置していることからemitを多く用いた実装となっています

@alter334 alter334 requested a review from cp-20 July 4, 2024 07:49
Copy link

github-actions bot commented Jul 4, 2024

Copy link
Contributor

@nokhnaton nokhnaton left a comment

Choose a reason for hiding this comment

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

良さそうです!
emitの階層が深い問題も、孫程度なら全く気にするほどじゃないと思ってるので、問題ないと思います!

@alter334
Copy link
Contributor Author

alter334 commented Jul 4, 2024

レビューありがとうございます
このままマージします

@alter334 alter334 merged commit a75c655 into master Jul 4, 2024
10 checks passed
@alter334 alter334 deleted the feat/hide_popupmenu_onitemclicked branch July 4, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

「チャンネルリンクをコピー」したあと、コンテキストメニューを閉じたい
2 participants