Projet

Général

Profil

Bug #18887

Popup non affiché si aucun élement @form@ n'existe

Ajouté par Josué Kouka il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
21 septembre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

La popup n'est pas affichée si aucune n'élement form n'existe, meme quand un selecteur différent est passé.

Fichiers

0001-display-popup-for-element-different-selector-form-18.patch (1,09 ko) 0001-display-popup-for-element-different-selector-form-18.patch Josué Kouka, 21 septembre 2017 17:18
0001-display-popup-for-element-different-selector-form-18.patch (1,09 ko) 0001-display-popup-for-element-different-selector-form-18.patch Josué Kouka, 21 septembre 2017 17:22
0001-display-popup-for-element-different-selector-form-18.patch (1,09 ko) 0001-display-popup-for-element-different-selector-form-18.patch Josué Kouka, 21 septembre 2017 19:05
0001-display-popup-for-selector-different-from-form-18887.patch (1,09 ko) 0001-display-popup-for-selector-different-from-form-18887.patch Josué Kouka, 21 septembre 2017 19:06
0001-js-allow-non-form-elements-as-popup-main-widget-1888.patch (1,52 ko) 0001-js-allow-non-form-elements-as-popup-main-widget-1888.patch Josué Kouka, 21 septembre 2017 22:14
0001-js-allow-non-form-elements-as-popup-main-widget-1888.patch (3,61 ko) 0001-js-allow-non-form-elements-as-popup-main-widget-1888.patch Frédéric Péters, 22 septembre 2017 11:04

Demandes liées

Lié à Chrono - Development #12550: exception aux récurrencesFermé12 juillet 201629 août 2017

Actions

Révisions associées

Révision 5e06b6cb (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

js: allow non-<form> elements as popup main widget (#18887)

Historique

#2

Mis à jour par Josué Kouka il y a plus de 6 ans

#4

Mis à jour par Frédéric Péters il y a plus de 6 ans

Comme ça modifie le comportement, tu as trouvé l'origine de la situation présente et regarder si ça affectait quelque chose ?

#5

Mis à jour par Josué Kouka il y a plus de 6 ans

Frédéric Péters a écrit :

Comme ça modifie le comportement, tu as trouvé l'origine de la situation présente

Oui, meme si un autre selector est définie, on recherche toujours un from comme sous element.

et regarder si ça affectait quelque chose ?

A coup de grep git grep 'rel="popup"' | grep 'data-selector' dans les différentes briques.

ps: message de commit modifié.

#7

Mis à jour par Frédéric Péters il y a plus de 6 ans

A coup de grep git grep 'rel="popup"' | grep 'data-selector' dans les différentes briques.

Et la conclusion, de la recherche des origines et des usages ?

                    /* if the form doesn't have an @action attribute, set it to URL */
                    if (! $form.attr('action')) {
                        $form.attr('action', url);
                    }

Dans la suite, et donc si $form n'est plus un formulaire, ça n'a plus vraiment de sens. De manière générale, il y a beaucoup dans la suite qui n'a pas de sens si $form n'est pas un <form>; je me dis qu'il faudrait être clair et arrêter l'affaire façon "if not <form>: return;", avant tous les endroits qui parlent de attr('action'), buttons, etc.

ps: message de commit modifié.

js: allow non-<form> elements as popup main widget.

#8

Mis à jour par Josué Kouka il y a plus de 6 ans

Frédéric Péters a écrit :

A coup de grep git grep 'rel="popup"' | grep 'data-selector' dans les différentes briques.

Et la conclusion, de la recherche des origines et des usages ?

[...]

data-selector n'est utilisé nulle part.

Dans la suite, et donc si $form n'est plus un formulaire, ça n'a plus vraiment de sens. De manière générale, il y a beaucoup dans la suite qui n'a pas de sens si $form n'est pas un <form>; je me dis qu'il faudrait être clair et arrêter l'affaire façon "if not <form>: return;", avant tous les endroits qui parlent de attr('action'), buttons, etc.

OK

#9

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Assigné à changé de Josué Kouka à Frédéric Péters

J'ai regardé plus avant le code actuel et suis d'avis qu'il faut reprendre davantage plutôt que perdre quelque chose ici; je vais faire le boulot.

#10

Mis à jour par Frédéric Péters il y a plus de 6 ans

Garder toutes les fonctionnalités, surtout la possibilité d'avoir des boutons, comme ça il y aura quand même moyen d'ajouter :

   <div class="buttons">
           <a class="cancel" href="...">Fermer</a>
   </div>

dans une boite de dialogue qui n'est pas un formulaire.

J'ai testé pas mal dans authentic (mais découvert en passant qu'il n'y avait plus d'utilisation d'ajaxform (?)) et en faisant des variations sur la nouvelle boite chrono (manager_time_period_exception_list.html).

#11

Mis à jour par Josué Kouka il y a plus de 6 ans

Ack

#12

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Frédéric Péters a écrit :

J'ai testé pas mal dans authentic (mais découvert en passant qu'il n'y avait plus d'utilisation d'ajaxform (?)) et en faisant des variations sur la nouvelle boite chrono (manager_time_period_exception_list.html).

Oui top de soucis sur les pages de création, éditions des utilisateurs, maintenant en plus on a 2 vues différentes, détail et édition.

#13

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Sujet changé de Popup non affiché si aucune n'élement @form@ n'existe à Popup non affiché si aucun élement @form@ n'existe
  • Statut changé de En cours à Résolu (à déployer)
commit 5e06b6cb9096ee369b585c598dcc0a31d25e9984
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Sep 22 10:48:37 2017 +0200

    js: allow non-<form> elements as popup main widget (#18887)
#14

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

Formats disponibles : Atom PDF