Project

General

Profile

Bug #5010

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

Added by Mikaël Ates almost 10 years ago. Updated over 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Jérôme Schneider
Start date:
20 June 2014
Due date:
% Done:

90%

Estimated time:
Patch proposed:
Yes
Planning:

Files

Associated revisions

Revision e8a1fe31 (diff)
Added by Jérôme Schneider over 9 years ago

dossiers: render pause facturation comment expandable

Closes #5010

History

#1

Updated by Jérôme Schneider over 9 years ago

  • Assignee set to Jérôme Schneider
  • Patch proposed set to No
#2

Updated by Jérôme Schneider over 9 years ago

#3

Updated by Benjamin Dauvergne over 9 years ago

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

Updated by Benjamin Dauvergne over 9 years ago

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

Updated by Jérôme Schneider over 9 years ago

  • File 0001-dossiers-render-pause-facturation-comment-expandable.patch added

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

#6

Updated by Jérôme Schneider over 9 years ago

  • File deleted (0001-dossiers-render-pause-facturation-comment-expandable.patch)
#8

Updated by Benjamin Dauvergne over 9 years ago

Ok.

#9

Updated by Jérôme Schneider over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 60 to 100

Appliqué par commit calebasse|commit:e8a1fe3143ad9d92a8b2806c8ef961d7b93f2732.

#10

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 50

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

#11

Updated by Jérôme Schneider over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 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

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to Fermé

Ok

Also available in: Atom PDF