Projet

Général

Profil

Development #61022

cell fiche : dynamic_fields sur le parent/group directement

Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
24 janvier 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Relecture après coup de dynamic_fields #58800.

Les attributs data des champs conditionnels se posent sur le field et va ensuite effectué un hide/show sur le ".parent().parent()".
Ce qui est plutôt bad niveau performance, complique le code côté html et JS, mais surtout ne permet pas des conditions imbriquées.

je propose de pouvoir masquer / afficher directement un groupe de champs et ainsi au sein de ce groupe pouvoir gérer des conditions internes au groupe (comme le champ field_empty_display_mode_select) avec data attribut egalement plutôt qu'avec un custom code inutile.


Fichiers


Demandes liées

Lié à Combo - Development #58800: "contenu d'une fiche", séparer contenu & mode d'affichageFermé20 novembre 2021

Actions

Révisions associées

Révision d6cfee4e (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

card-cell: use data-dynamic attr directly on fields group/parent (#61022)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

  • Lié à Development #58800: "contenu d'une fiche", séparer contenu & mode d'affichage ajouté
#2

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

  • Fichier 0001-scss-use-css-grid-to-provide-alternate-form-title-mo.patch ajouté
  • Statut changé de Nouveau à Solution proposée
  • Patch proposed changé de Non à Oui
#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

  • Fichier 0001-scss-use-css-grid-to-provide-alternate-form-title-mo.patch supprimé
#5

Mis à jour par Lauréline Guérin il y a environ 2 ans

(tu peux pousser ta branche stp ?)

#7

Mis à jour par Lauréline Guérin il y a environ 2 ans

Tu supprimes pas:

    const field_content_select = this.grid_cell_form.field_content;
    $(field_content_select).on('change', function() {
      if ($(this).hasClass('field-hidden') || $(this).val() == 'label-and-value') {
        $(_self.grid_cell_form.field_display_mode).removeClass('field-hidden').parent().parent().hide();
      } else {
        $(_self.grid_cell_form.field_display_mode).addClass('field-hidden').parent().parent().show();
      }
    });

?

#8

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

Pas encore possible. (il me semble)
Dans ce cas il faudrait que le script JS gère "!=" ou alors j'ai zappé un truc.
(Et j'avais prévu de faire ça dans un second temps).

#9

Mis à jour par Lauréline Guérin il y a environ 2 ans

Ca doit pouvoir se gérer avec un value-in: data-dynamic-display-value-in="label|value"

#10

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a environ 2 ans

ok, value-in permet de lister plusieurs valeurs et fait l'affaire pour le moment, une option "value-not" viendra surement, mais plus tard.

#11

Mis à jour par Lauréline Guérin il y a environ 2 ans

  • Statut changé de Solution proposée à Solution validée
#12

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit d6cfee4ec3931a9601fec9ea748a918ca016aa2a
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Mon Jan 24 16:57:54 2022 +0100

    card-cell: use data-dynamic attr directly on fields group/parent (#61022)
#13

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#14

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF