モーダル内でmousedownしてモーダル外でmouseupしたときはモーダルを閉じないように

[mehm8128]

close #3536
discussionタグついてるけど使いにくいと感じるときがあるので直しました
考慮漏れがないかあんまり自信ないかもです

[github-actions]

Preview (prod) → https://4119-prod.traq-preview.trapti.tech/
Preview (dev) → https://4119-dev.traq-preview.trapti.tech/

[nokhnaton]

mouseupイベントで閉じるようにするだけでもいいのかなと思いました
正直mousedownのタイミングで閉じたいことって無いような気がしてます。
どっちかって言うと、mousedownした直後にやっぱり閉じたくないなみたいに思う時はあるので、私はmouseupイベントだけで閉じる方が嬉しいです

[mehm8128]

それは僕も同じ意見で、この変更でもmouseupで閉じるようにして僕の環境では確認できたのですが、なっていなさそうですか？
mousedownを考えずにmouseupイベントで閉じる、だけにすると現状と同じような挙動になってしまうため、モーダル内でmousedownしたときはmouseupでemitしないような実装にしてあります

[nokhnaton]

あ、ごめんなさい。適当にコード読んでタイトルだけ見てコメントしてました。
良さげな挙動っぽいです

[nokhnaton]

モーダル外でmousedown->モーダル内でmosueupでモーダルが閉じるのだけが気になりました。
個人的にはこのパターンだと閉じないでほしいかなって思います

[mehm8128]

確かに
直しました

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0ef6320) 86.35% compared to head (80e8814) 86.35%.
Report is 58 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4119   +/-   ##
=======================================
  Coverage   86.35%   86.35%           
=======================================
  Files          66       66           
  Lines        4719     4719           
  Branches      564      564           
=======================================
  Hits         4075     4075           
  Misses        638      638           
  Partials        6        6           

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

[ras0q]

動作確認しました、よさそうです！