Project

General

Profile

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
Target version:
Start date:
26 Mar 2018
Due date:
% Done:

100%

Patch proposed:
No
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

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

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

#3 Updated by Benjamin Dauvergne over 1 year ago

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

#4 Updated by Benjamin Dauvergne over 1 year ago

  • Target version set to 0.25

#5 Updated by Benjamin Dauvergne over 1 year ago

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

#6 Updated by Benjamin Dauvergne over 1 year ago

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

À défaire.

#7 Updated by Benjamin Dauvergne over 1 year ago

  • Target version changed from 0.25 to future

#8 Updated by Benjamin Dauvergne over 1 year ago

  • Assignee set to Benjamin Dauvergne

#9 Updated by Benjamin Dauvergne over 1 year ago

  • Patch proposed changed from Yes to No

Also available in: Atom PDF