Projet

Général

Profil

Development #22795

toujours avoir un <div class="table-container"> (montée de version de django-tables2)

Ajouté par Frédéric Péters il y a environ 6 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
26 mars 2018
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 49cc5185 (diff)
Ajouté par Benjamin Dauvergne il y a presque 6 ans

templates: always wrap tables in div.table-container (fixes #22795)

django-tables2 < 1.1 only wrap table if the there are multiple pages.

Révision ed09b385 (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

Revert "templates: always wrap tables in div.table-container (fixes #22795)"

This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.

Historique

#2

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).

#3

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.

#4

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Version cible mis à 0.25
#5

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
#6

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

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

À défaire.

#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Version cible changé de 0.25 à future
#8

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Assigné à mis à Benjamin Dauvergne
#9

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Patch proposed changé de Oui à Non
#10

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Voilà, ça n'est plus nécessaire.

#11

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)

#13

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.

#14

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.
#15

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

Formats disponibles : Atom PDF