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.
Files
Associated revisions
Revert "templates: always wrap tables in div.table-container (fixes #22795)"
This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.
History
Updated by Benjamin Dauvergne almost 5 years ago
- File 0001-templates-always-wrap-tables-in-div.table-container-.patch 0001-templates-always-wrap-tables-in-div.table-container-.patch added
- Patch proposed changed from No to Yes
Updated by Frédéric Péters almost 5 years ago
- Status changed from Nouveau to 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).
Updated by Benjamin Dauvergne almost 5 years ago
J'étais pour laisser le ticket ouvert et faire un revert quand il sera temps.
Updated by Benjamin Dauvergne almost 5 years ago
- Status changed from En cours to Résolu (à déployer)
- % Done changed from 0 to 100
Appliqué par commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.
Updated by Benjamin Dauvergne almost 5 years ago
- Status changed from Résolu (à déployer) to En cours
À défaire.
Updated by Benjamin Dauvergne almost 3 years ago
- File 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch added
- Tracker changed from Bug to Development
- Status changed from En cours to Solution proposée
- Patch proposed changed from No to Yes
Voilà, ça n'est plus nécessaire.
Updated by Frédéric Péters almost 3 years ago
À rebaser après #41491. (ou sans doute plus simple de juste le refaire)
Updated by Benjamin Dauvergne almost 3 years ago
- File 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch added
Certainement à rebaser encore après #41615.
Updated by Frédéric Péters almost 3 years ago
- Status changed from Solution proposée to Solution validée
Vas-y pousse ce commit et je gérerait le rebase de l'autre.
Updated by Benjamin Dauvergne almost 3 years ago
- Status changed from Solution validée to 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.
Updated by Frédéric Péters almost 3 years ago
- Status changed from Résolu (à déployer) to 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.