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

bug lorsque le button est composé d'une image #2

Closed
sbonaventure opened this Issue Jul 11, 2017 · 6 comments

Comments

Projects
None yet
2 participants
@sbonaventure

sbonaventure commented Jul 11, 2017

Lorsque le bouton est composé de HTML (une image dans mon cas), le script bug :

<button class="js-modal"><img src='[...]' alt='[...]' /><button>

Ce test renvoi false :

hasClass(e.target, MODAL_JS_CLASS)

Et le script ne construit pas la modale

@nico3333fr nico3333fr self-assigned this Jul 11, 2017

@nico3333fr nico3333fr added the bug label Jul 11, 2017

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Jul 11, 2017

Owner

Ah, je crois déjà avoir eu ce cas, je pense qu'il faut que je cherche dans l'événement voir si l'élément où il y a l'event est un enfant de MODAL_JS_CLASS. J'y regarde dès que possible.

Owner

nico3333fr commented Jul 11, 2017

Ah, je crois déjà avoir eu ce cas, je pense qu'il faut que je cherche dans l'événement voir si l'élément où il y a l'event est un enfant de MODAL_JS_CLASS. J'y regarde dès que possible.

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Jul 12, 2017

Owner

Yop, je viens d'envoyer un fix e50ca51

Normalement, c'est bon, mais tu peux confirmer que ça roule de ton coté ?

Owner

nico3333fr commented Jul 12, 2017

Yop, je viens d'envoyer un fix e50ca51

Normalement, c'est bon, mais tu peux confirmer que ça roule de ton coté ?

@sbonaventure

This comment has been minimized.

Show comment
Hide comment
@sbonaventure

sbonaventure Jul 12, 2017

parfait, ça fonctionne !

sbonaventure commented Jul 12, 2017

parfait, ça fonctionne !

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Jul 12, 2017

Owner

Tant mieux ! :)

On a bien mérité ça :
fallon_dance

Merci bien pour avoir signalé le problème ;)

Owner

nico3333fr commented Jul 12, 2017

Tant mieux ! :)

On a bien mérité ça :
fallon_dance

Merci bien pour avoir signalé le problème ;)

@nico3333fr nico3333fr closed this Jul 12, 2017

@sbonaventure

This comment has been minimized.

Show comment
Hide comment
@sbonaventure

sbonaventure commented Jul 12, 2017

merci

@nico3333fr

This comment has been minimized.

Show comment
Hide comment
@nico3333fr

nico3333fr Jul 12, 2017

Owner

Hop, je t'ai mentionné ici du coup https://van11y.net/fr/2017/07/12/fix-pour-la-modale (et sur Twitter, toussa) ;)

Owner

nico3333fr commented Jul 12, 2017

Hop, je t'ai mentionné ici du coup https://van11y.net/fr/2017/07/12/fix-pour-la-modale (et sur Twitter, toussa) ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment