Project

General

Profile

Development #22795

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

Added by Frédéric Péters over 2 years ago. Updated 3 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
Start date:
26 Mar 2018
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

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.

0001-templates-always-wrap-tables-in-div.table-container-.patch View (2.06 KB) Benjamin Dauvergne, 26 Mar 2018 10:57 PM

0001-Revert-templates-always-wrap-tables-in-div.table-con.patch View (1.7 KB) Benjamin Dauvergne, 09 Apr 2020 02:37 PM

0001-Revert-templates-always-wrap-tables-in-div.table-con.patch View (1.51 KB) Benjamin Dauvergne, 11 Apr 2020 11:26 AM

Associated revisions

Revision 49cc5185 (diff)
Added by Benjamin Dauvergne over 2 years ago

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

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

Revision ed09b385 (diff)
Added by Benjamin Dauvergne 3 months ago

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

This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.

History

#1 Updated by Benjamin Dauvergne over 2 years ago

#2 Updated by Frédéric Péters over 2 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).

#3 Updated by Benjamin Dauvergne over 2 years ago

J'étais pour laisser le ticket ouvert et faire un revert quand il sera temps.

#4 Updated by Benjamin Dauvergne over 2 years ago

  • Target version set to 0.25

#5 Updated by Benjamin Dauvergne over 2 years ago

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

#6 Updated by Benjamin Dauvergne over 2 years ago

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

À défaire.

#7 Updated by Benjamin Dauvergne over 2 years ago

  • Target version changed from 0.25 to future

#8 Updated by Benjamin Dauvergne over 2 years ago

  • Assignee set to Benjamin Dauvergne

#9 Updated by Benjamin Dauvergne over 2 years ago

  • Patch proposed changed from Yes to No

#10 Updated by Benjamin Dauvergne 3 months ago

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

#11 Updated by Frédéric Péters 3 months ago

À rebaser après #41491. (ou sans doute plus simple de juste le refaire)

#12 Updated by Benjamin Dauvergne 3 months ago

Certainement à rebaser encore après #41615.

#13 Updated by Frédéric Péters 3 months 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.

#14 Updated by Benjamin Dauvergne 3 months 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.

#15 Updated by Frédéric Péters 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF