Skip to content
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

Utiliser le dropdown désactive les modales #1936

Closed
pierre-24 opened this issue Dec 19, 2014 · 3 comments

Comments

@pierre-24
Copy link
Member

commented Dec 19, 2014

Alors, celui là, je l'ai pas trop compris. Tout est dans le titre.

  • Prenons une page avec une boite modale : validation d'un tutoriel. Créer un tuto, demander la validation. Vous pouvez faire apparaître cette boite modale autant de fois que vous voulez (évidement à condition d'appuyer sur annuler à chaque fois)
  • Utilisez n'importe quel élément de dropdown : impossible ensuite de faire réapparaître cette fenêtre. Je n'ai pas d'erreur JS dans la console Chrome.

Marche avec n'importe quel modale (par exemple, signalement sur un forum, publication d'un tuto ... Toussa).

À noter que ce bug est déjà présent sur la prod.

@poulp

This comment has been minimized.

Copy link
Contributor

commented Dec 19, 2014

ça m'arrive aussi, je l'ai remarqué quand j'ai qa la #1595 :

Et en bonus un bug chelou que je ne pourrais expliquer, j'ai aucune erreur sur la console js : Quand je clique sur mon image de profil, ça m'ouvre le menu normal, je le referme en re cliquant sur mon image de profil, et la quand je clique sur le bouton pour proposer une correction la modal ne s'affiche plus.

@pierre-24

This comment has been minimized.

Copy link
Member Author

commented Dec 19, 2014

Bien vu, je me disais aussi que quelqu'un en avais déjà parlé :)

@ghost

This comment has been minimized.

Copy link

commented Feb 9, 2015

Il faudrait envisager de réécrire proprement cette fonction, ou en tout cas de la "nettoyer".

@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 10, 2015

Situphen added a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.