Produits Entr'ouvert » Combo

Quelques retours

Ajouter un role=button au titre

    $cell_title.attr('tabindex', '0');
+   $cell_title.attr('role', 'button');

Tester si la cell est `.foldable` directement en début de fonction pour éviter de stocker des vars pour rien, éviter une imbrication et exit le plus tôt possible

  function prepare_foldable(cell) {
+    if (!$cell.is('.foldable')) {
+      return;
+    }
…
    var $cell_title = $cell.find('> div > h2:first-child');
-   if ($cell.is('.foldable')) {
…
        return false;
      });
-   }

J'aurais eu tendance à faire une function pour

    if ($cell.is('.folded')) {
      $cell_title.attr('aria-expanded', 'false');
    } else {
      $cell_title.attr('aria-expanded', 'true');
    }

et ainsi éviter de dupliquer du code.

Enfin je ne suis pas sûr de l'intérêt de mettre un .keydown sur l'image sachant qu'elle n'est pas focusable.