Bug #22795: toujours avoir un <div class="table-container"> (montée de version de django-tables2) - Fargo - Entr'ouvert Projects — Projets d'Entr'ouvert

Bug #22795

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

Added by Frédéric Péters over 1 year ago. Updated over 1 year ago.

Status:

En cours

Priority:

Normal

Assignee:

Benjamin Dauvergne

Target version:

future

Start date:

26 Mar 2018

Due date:

% Done:

100%

Patch proposed:

Planning:

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

Associated revisions

Revision 49cc5185 (diff)
Added by Benjamin Dauvergne over 1 year ago

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

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

History

#1 Updated by Benjamin Dauvergne over 1 year ago

Patch proposed changed from No to Yes
File 0001-templates-always-wrap-tables-in-div.table-container-.patch View added

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