Development #22795
toujours avoir un <div class="table-container"> (montée de version de django-tables2)
100%
Description
django_tables2/table.html fait :
{% if table.page %} <div class="table-container"> {% endif %}
mais dans fargo (et dans publik-base-theme), côté styles, on se base sur l'existence du div.table-container.
C'est corrigé dans django-tables 1.2 qui est disponible dans jessie-backports mais le tox.ini bloque encore sur "django-tables2<1.1" (en django 1.8) et je viens de vérifier et de fait avec tox utiliser la 1.2 foire.
Bref, peut-être juste attendre le passage à Django 1.11 et ignorer le bug en attendant.
Fichiers
Révisions associées
Revert "templates: always wrap tables in div.table-container (fixes #22795)"
This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Fichier 0001-templates-always-wrap-tables-in-div.table-container-.patch 0001-templates-always-wrap-tables-in-div.table-container-.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de Nouveau à En cours
Sauf que ça pourra donner deux <div class="table-container"> à partir de la version 1.2 (quand le tableau est vide) (j'ai vérifié ça n'a pas d'incidence avec les règles CSS qu'on a aujourd'hui, mais ce n'est quand même pas terrible).
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
J'étais pour laisser le ticket ouvert et faire un revert quand il sera temps.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à En cours
À défaire.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Voilà, ça n'est plus nécessaire.
Mis à jour par Frédéric Péters il y a presque 4 ans
À rebaser après #41491. (ou sans doute plus simple de juste le refaire)
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch ajouté
Certainement à rebaser encore après #41615.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Vas-y pousse ce commit et je gérerait le rebase de l'autre.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ed09b38596fbbb57902cd527d4f932d9854e5266 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Apr 9 14:36:19 2020 +0200 Revert "templates: always wrap tables in div.table-container (fixes #22795)" This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
templates: always wrap tables in div.table-container (fixes #22795)
django-tables2 < 1.1 only wrap table if the there are multiple pages.