Projet

Général

Profil

Bug #5010

Masquer le champs commentaire de la pause facturation, afficher avec un bouton et indiquer quand il y a un commentaire.

Ajouté par Mikaël Ates il y a presque 10 ans. Mis à jour il y a plus de 9 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Jérôme Schneider
Début:
20 juin 2014
Echéance:
% réalisé:

90%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision e8a1fe31 (diff)
Ajouté par Jérôme Schneider il y a plus de 9 ans

dossiers: render pause facturation comment expandable

Closes #5010

Historique

#1

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Assigné à mis à Jérôme Schneider
  • Patch proposed mis à Non
#2

Mis à jour par Jérôme Schneider il y a plus de 9 ans

#3

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

J'aurai plutôt utilisé jQuery.toggleClass() et beaucoup de CSS pour en faire un truc plus générique utilisable partout, par exemple:

Pour la structure HTML:

  <div class="js-expandable">
    <label class="js-click-to-expand">Infos d'assuré</label>
    <div class="js-to-expand">
      ....
    </div>
  </div>

Pour la CSS:

.js-expandable .js-click-to-expand:after { content: " [-]" }
.js-expendable.js-expanded .js-click-to-expand:after { content: " [+]" }

.js-expandable .js-to-expand { display: none; }
.js-expandable.js-expanded .js-to-expand { display: block; }

Et pour le JS:

$(document).on('click', '.js-click-to-expand', function (event) {
   $(event.target).parents('.js-expandable').toggleClass('js-expanded');
});

Je n'ai pas testé mais l'esprit est là.

#4

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

On doit aussi pouvoir faire ça sans JS du tout avec des URLs fragment:

  <div id="coin" class="expandable">
    <label class="to-hide">
      <a href="#coin">label [-]</a>
    </label>
    <label class="to-show">
      <a href="#">label [+]</a>
    </label>
    <div class="to-show">
    </div>
  </div>

CSS:

.expandable .to-show { display: none }
.expandable:target .to-show { display: block }
.expandable:target .to-hide { display: none }

Et Fred doit pouvoir pondre une animation CSS pour avoir un bel effet accordéon.

#5

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Fichier 0001-dossiers-render-pause-facturation-comment-expandable.patch ajouté

Et voilà un patch qui tient compte de la remarque initiale. Pour l'effet accordéon ça attendra je pense.

#6

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Fichier 0001-dossiers-render-pause-facturation-comment-expandable.patch supprimé
#8

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

Ok.

#9

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 60 à 100

Appliqué par commit calebasse|commit:e8a1fe3143ad9d92a8b2806c8ef961d7b93f2732.

#10

Mis à jour par Mikaël Ates il y a plus de 9 ans

  • Statut changé de Résolu (à déployer) à En cours
  • % réalisé changé de 100 à 50

Lorsque le commentaire est déroulé, il ne se "renroule" pas en cliquant sur 'Commentaire de la pause facturation [-]'.

#11

Mis à jour par Jérôme Schneider il y a plus de 9 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 50 à 90

C'était un bug avec Firefox. Je l'ai corrigé avec :

commit 91140baf5cb33f4f02a4bfbb1afa11bd5fe274e6
Author: Jérôme Schneider <jschneider@entrouvert.com>
Date:   Mon Aug 18 19:22:03 2014 +0200

    fixes a toggle bug specific to Firefox
#12

Mis à jour par Mikaël Ates il y a plus de 9 ans

  • Statut changé de Résolu (à déployer) à Fermé

Ok

Formats disponibles : Atom PDF