ユーザーモーダルのアイコンのはみ出している部分をクリックすると閉じないように修正した

[Nattuki]

close #4143

[github-actions]

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9e8f0d6) 86.35% compared to head (6dbae70) 86.35%.
Report is 4 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4190   +/-   ##
=======================================
  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.

[mehm8128]

ありがとうございます！
グループのオーナー・メンバー追加(UsersSelectorコンポーネント)で、アイコンの部分をクリックしても追加できていたのができなくなってしまっているのでどうにかしたいですね:thinking:
元々preventModalのときに@click.stopされるのをpointer-events: noneで防いでいたのがなくなってしまったから起きてそうです

[mehm8128]

機能的には要件満たせているのですが、やっぱりisUserModalで分けるのあんまりよくなさそうですね...
そもそもボタンとして扱うコンポーネントと画像として扱うコンポーネントが一緒になっているのがよくないので、今あるUserIconコンポーネントは画像として扱うもの(ユーザーモーダルで表示するものなど、isClickableがfalseになるようなもの)で直接使うようにして、その他ボタンとして扱う用のUserIconButtonコンポーネント(UserIconコンポーネントをbuttonタグで囲ったもの)を別で作るのがよさそうな気がします

現在のボタンと画像が一緒になっちゃっているUserIconの実装だと、a11y的にも色々とよくないので、変更箇所は多くなりますがコード的にだけでなく、機能的にもかなり意味のあるPRになると思うのでお願いしたいです！