UserModal のはみ出ているアイコン部分をクリックしてもモーダルが閉じないように #4748
Conversation
|
Preview (prod) → https://4748-prod.traq-preview.trapti.tech/ |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4748 +/- ##
==========================================
+ Coverage 11.89% 11.91% +0.01%
==========================================
Files 695 695
Lines 31218 31174 -44
Branches 616 616
==========================================
Hits 3714 3714
+ Misses 27492 27448 -44
Partials 12 12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Takeno-hito
left a comment
Takeno-hito
left a comment
There was a problem hiding this comment.
モーダル開いてるときにアイコンにカーソル合わせると、クリックできそうなカーソルに変わっちゃってそう
| indicatorSize: 10, | ||
| preventModal: false, | ||
| hasNotification: false, | ||
| isInactive: false |
There was a problem hiding this comment.
isInactive のデフォルト値が消えているのはどうしてですか？
| backgroundImage: userIconFileId.value | ||
| ? `url(${buildUserIconPath(userIconFileId.value)})` | ||
| : undefined, | ||
| pointerEvents: props.preventModal ? ('none' as const) : undefined |
There was a problem hiding this comment.
ここの分岐を取り除く代わりに以下の部分で role を用いた判定に変更しているのはどうしてですか？
| v-if="!isMobile" | ||
| :user-id="id" | ||
| prevent-modal | ||
| :prevent-modal="false" |
There was a problem hiding this comment.
prevent-modal のデフォルト値は false なので、この行を取り除くだけでよいと思います．
| :prevent-modal="false" |
close: #4143
めぼしいアイコンを触ってもキチンと動きました。