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 almost 5 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Benjamin Dauvergne
Target version:
future
Start date:
26 March 2018
Due date:
% Done:

100%

Estimated time:
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.

Files

0001-templates-always-wrap-tables-in-div.table-container-.patch (2.06 KB) 0001-templates-always-wrap-tables-in-div.table-container-.patch Benjamin Dauvergne, 26 March 2018 10:57 PM
0001-Revert-templates-always-wrap-tables-in-div.table-con.patch (1.7 KB) 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch Benjamin Dauvergne, 09 April 2020 02:37 PM
0001-Revert-templates-always-wrap-tables-in-div.table-con.patch (1.51 KB) 0001-Revert-templates-always-wrap-tables-in-div.table-con.patch Benjamin Dauvergne, 11 April 2020 11:26 AM

Associated revisions

Revision 49cc5185 (diff)
Added by Benjamin Dauvergne almost 5 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 almost 3 years ago

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

This reverts commit 49cc5185b27914b7ade7f4e5cfe1627b90e1496e.

History

#2

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

#3

Updated by Benjamin Dauvergne almost 5 years ago

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

#4

Updated by Benjamin Dauvergne almost 5 years ago

  • Target version set to 0.25
#5

Updated by Benjamin Dauvergne almost 5 years ago

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

Updated by Benjamin Dauvergne almost 5 years ago

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

À défaire.

#7

Updated by Benjamin Dauvergne almost 5 years ago

  • Target version changed from 0.25 to future
#8

Updated by Benjamin Dauvergne almost 5 years ago

  • Assignee set to Benjamin Dauvergne
#9

Updated by Benjamin Dauvergne almost 5 years ago

  • Patch proposed changed from Yes to No
#10

Updated by Benjamin Dauvergne almost 3 years ago

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

#11

Updated by Frédéric Péters almost 3 years ago

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

#13

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.

#14

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

Updated by Frédéric Péters almost 3 years ago

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

Also available in: Atom PDF