search

YukaMurata / modal

0 stars 1 forks source link

アニメーション中にクリック時の処理が実行されている。 #9

Closed liberta793 closed 7 years ago

liberta793 commented 7 years ago

アニメーション中にクリック時処理が実行されており、予期せぬ動作を招く可能性があります。 そのためアニメーション中はクリック時の処理が実行されないようにしてください。

YukaMurata commented 7 years ago

遅くなって申し訳ございません。 e0d64f3 上記、対応いたしました。 お手隙の際に確認よろしくお願いいたします。

liberta793 commented 7 years ago

この実装の考え方

みたいな感じでお願いします。

YukaMurata commented 7 years ago

お疲れ様です。 上記、対応しました。 ご確認よろしくお願いいたします。

liberta793 commented 7 years ago

私も知らなかったですがaddを利用すると、fadeOutの中の処理がすぐに実行されてしまうみたいです。 そのため、セレクタ毎にフェードアウト処理を記述してください。 $('.modal[data-modal="'+ modaldata + '"], .overlay')のように複数のセレクタを一括で指定するのもありですが、セレクタ毎に処理を分ける必要になることを考慮して、あらかじめ分けといた方が良いかもしれないです。

あと、bindEventとaddEventを分ける理由はないと思うので、どちらか一つにまとめてください。

YukaMurata commented 7 years ago

承知しました。 対応いたします。

YukaMurata commented 7 years ago

お疲れ様です。 セレクタごとに分けてfadeさせました。 ご確認よろしくお願いいたします。 c7285c6 9c9a386

liberta793 commented 7 years ago

確認できました!