Development #18181
Édition en popup et suppression directement depuis les listes
0%
Description
Avec #18179 dans Gadjo on peut désormais avoir rel="popup" sur les liens de modification et une icône supprimer dans le listing.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-manager-use-popups-to-edit-and-add-delete-icons-in-l.patch 0001-manager-use-popups-to-edit-and-add-delete-icons-in-l.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
Ça va marcher le rel=popup avec href=# ?
Ça ne va rien provoquer et c'est le comportement attendu (le <a> est là pour faciliter les styles, pas pour être de vrais liens); cela étant c'est un peu par chance que ça ne donnera rien, je vais ajouter dans Gadjo du vrai code pour proprement ignorer.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Lié à Development #18184: Totalement ignorer le rel="popup" sur les liens avec # comme destination ajouté
Mis à jour par Thomas Noël il y a plus de 6 ans
En fait je trouve qu'on pourrait ne pas mettre de <a> quand on sait que user_can_manage est faux. Parce que ça dessine un lien qui va rien faire... Je suis pas sûr que l'usager comprenne.
Mais bon, détail, ça peut être poussé ainsi.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Mais bon, détail, ça peut être poussé ainsi.
Tu peux faire un ticket pour parler du sujet différent :) (comme je le notais, ça simplifiait un peu les CSS d'avoir <a> dans tous les cas, mais c'est bien sûr amendable).
Mis à jour par Thomas Noël il y a plus de 6 ans
#18185 fait, et ack pour le patch attaché ici, donc
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit be59a60b254beca26bca4c854b799c95f5083fc7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Aug 27 09:52:05 2017 +0200
manager: use popups to edit and add delete icons in listings (#18181)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: use popups to edit and add delete icons in listings (#18181)