fixed django-tables rendering

This commit is contained in:
Kaibu 2019-11-13 23:44:52 +01:00
parent cead2fc45f
commit aca9501307
5 changed files with 122 additions and 113 deletions

View File

@ -2,31 +2,21 @@
{% load crispy_forms_tags %} {% load crispy_forms_tags %}
{% load i18n %} {% load i18n %}
{% load django_tables2 %} {% load django_tables2 %}
{% load class_tag %}
{% block title %}{% trans 'List' %} - {{ title }}{% endblock %}
{% block content %} {% block content %}
<h3>{{ title }} {% trans 'List' %} <div class="table-container">
{% if table.Meta.model|get_class == 'Category' %} <h3>{{ title }} {% trans 'List' %}
<a href="{% url 'new_category' %}" ><i class="fal fa-plus-circle"></i></a> {% if create_url %}
{% elif table.Meta.model|get_class == 'Keyword' %} <a href="{% url create_url %}">
<a href="{% url 'new_keyword' %}" ><i class="fal fa-plus-circle"></i></a> <button class="btn btn-primary float-right">{% trans 'New' %} <i class="fal fa-plus-circle"></i>
{% elif table.Meta.model|get_class == 'Storage' %} </button>
<a href="{% url 'new_storage' %}" ><i class="fal fa-plus-circle"></i></a> </a>
{% endif %} {% endif %}
</h3> </h3>
<br/>
{% if table.Meta.model|get_class == 'RecipeImport' %}
<a href="{% url 'data_batch_import' %}" class="btn btn-warning">{% trans 'Import all' %}</a>
<br/>
<br/>
{% endif %}
{% if table %}
{% render_table table %} {% render_table table %}
{% endif %} </div>
{% endblock %} {% endblock content %}

View File

@ -1,100 +1,117 @@
{% load django_tables2 %} {% load crispy_forms_tags %}
{% load i18n %} {% load i18n %}
{% block table-wrapper %} {% load django_tables2 %}
<div class="row"> {% block content %}
<div class="table-responsive">
{% block table %}
<table {% if table.attrs %} <!-- style='font-size: 24px' removed because smaller looked better -->
{{ table.attrs.as_html }}{% else %}class="table table-bordered"{% endif %}> <div class="table-container">
{% block table.thead %} {% if create_url %}
{% if table.show_header %} <a href="{% url create_url %}">
<thead> <button class="btn btn-primary float-right">{% trans 'New' %} <i class="fal fa-plus-circle"></i>
<tr> </button>
{% for column in table.columns %} </a>
{% endif %}
</h3>
{% block table %}
<table {% render_attrs table.attrs class="table" %}>
{% block table.thead %}
{% if table.show_header %}
<thead {{ table.attrs.thead.as_html }}>
<tr>
{% for column in table.columns %}
<th {{ column.attrs.th.as_html }}>
{% if column.orderable %} {% if column.orderable %}
<th {{ column.attrs.th.as_html }}><a <a href="{% querystring table.prefixed_order_by_field=column.order_by_alias.next %}">{{ column.header }}</a>
href="{% querystring table.prefixed_order_by_field=column.order_by_alias.next %}">{{ column.header }}</a>
</th>
{% else %} {% else %}
<th {{ column.attrs.th.as_html }}>{{ column.header }}</th> {{ column.header }}
{% endif %} {% endif %}
</th>
{% endfor %}
</tr>
</thead>
{% endif %}
{% endblock table.thead %}
{% block table.tbody %}
<tbody {{ table.attrs.tbody.as_html }}>
{% for row in table.paginated_rows %}
{% block table.tbody.row %}
<tr {{ row.attrs.as_html }}>
{% for column, cell in row.items %}
<td {{ column.attrs.td.as_html }}>
{% if column.localize == None %}{{ cell }}{% else %}{% if column.localize %}
{{ cell|localize }}{% else %}{{ cell|unlocalize }}
{% endif %}{% endif %}</td>
{% endfor %} {% endfor %}
</tr> </tr>
</thead> {% endblock table.tbody.row %}
{% endif %} {% empty %}
{% endblock table.thead %} {% if table.empty_text %}
{% block table.tbody %} {% block table.tbody.empty_text %}
<tbody> <tr>
{% for row in table.page.object_list|default:table.rows %} {# support pagination #} <td colspan="{{ table.columns|length }}">{{ table.empty_text }}</td>
{% block table.tbody.row %}
<tr {{ row.attrs.as_html }}>
{% for column, cell in row.items %}
<td {{ column.attrs.td.as_html }}>
{% if column.localize == None %}{{ cell }}{% else %}{% if column.localize %}
{{ cell|localize }}{% else %}{{ cell|unlocalize }}
{% endif %}{% endif %}</td>
{% endfor %}
</tr> </tr>
{% endblock table.tbody.row %} {% endblock table.tbody.empty_text %}
{% empty %}
{% if table.empty_text %}
{% block table.tbody.empty_text %}
<tr>
<td colspan="{{ table.columns|length }}">{{ table.empty_text }}</td>
</tr>
{% endblock table.tbody.empty_text %}
{% endif %}
{% endfor %}
</tbody>
{% endblock table.tbody %}
{% block table.tfoot %}
{% if table.has_footer %}
<tfoot>
<tr>
{% for column in table.columns %}
<td {{ column.attrs.tf.as_html }}>{{ column.footer }}</td>
{% endfor %}
</tr>
</tfoot>
{% endif %} {% endif %}
{% endblock table.tfoot %} {% endfor %}
</table> </tbody>
{% endblock table %} {% endblock table.tbody %}
{% block table.tfoot %}
{% if table.has_footer %}
<tfoot {{ table.attrs.tfoot.as_html }}>
<tr>
{% for column in table.columns %}
<td {{ column.attrs.tf.as_html }}>{{ column.footer }}</td>
{% endfor %}
</tr>
</tfoot>
{% endif %}
{% endblock table.tfoot %}
</table>
{% endblock table %}
{% block pagination %}
{% if table.page and table.paginator.num_pages > 1 %} {% if table.page and table.paginator.num_pages > 1 %}
{% with current_page=table.page.number page_count=table.paginator.num_pages rows_per_page=table.page.object_list|length total_rows=table.page.paginator.count %} <nav aria-label="Table navigation">
{% block pagination %} <ul class="pagination justify-content-center">
<nav> {% if table.page.has_previous %}
<ul class="pagination justify-content-center"> {% block pagination.previous %}
{% block pagination.allpages %} <li class="previous page-item">
<li class="page-item"> <a href="{% querystring table.prefixed_page_field=table.page.previous_page_number %}"
class="page-link">
<span aria-hidden="true">&laquo;</span>
{% trans 'previous' %}
</a>
</li>
{% endblock pagination.previous %}
{% endif %}
{% if table.page.has_previous or table.page.has_next %}
{% block pagination.range %}
{% for p in table.page|table_page_range:table.paginator %}
<li class="page-item{% if table.page.number == p %} active{% endif %}">
<a class="page-link"
{% if p != '...' %}href="{% querystring table.prefixed_page_field=p %}"{% endif %}>
{{ p }}
</a>
</li> </li>
{% for page in table.paginator.page_range %} {% endfor %}
{% with range_start=current_page|add:"-3" range_end=current_page|add:"3" page_count_minus_5=page_count|add:"-5" page_count_minus_1=page_count|add:"-1" %} {% endblock pagination.range %}
{% if page == current_page %} {% endif %}
<li class="page-item active"> {% if table.page.has_next %}
<a class="page-link" href="#">{{ page }}</a> {% block pagination.next %}
<li class="next page-item">
</li> <a href="{% querystring table.prefixed_page_field=table.page.next_page_number %}"
{% elif page == 1 or page >= range_start and page <= range_end or page == page_count %} class="page-link">
<li class="page-item"> {% trans 'next' %}
<a class="page-link" <span aria-hidden="true">&raquo;</span>
href="{% querystring table.prefixed_page_field=page %}">{{ page }}</a> </a>
</li> </li>
{% endif %} {% endblock pagination.next %}
{% if page == 1 and current_page > 5 or pagina == page_count_minus_1 and current_page <= page_count_minus_5 %} {% endif %}
<li class="page-item"></li> </ul>
{% endif %} </nav>
{% endwith %}
{% endfor %}
{% endblock pagination.allpages %}
</ul>
</nav>
{% endblock pagination %}
{% endwith %}
{% endif %} {% endif %}
{% endblock pagination %}
</div>
</div> </div>
{% endblock table-wrapper %} {% endblock content %}

View File

@ -34,7 +34,7 @@
<br/> <br/>
{% if recipes %} {% if user.is_authenticated and recipes %}
{% render_table recipes %} {% render_table recipes %}
{% else %} {% else %}
<div class="alert alert-danger" role="alert"> <div class="alert alert-danger" role="alert">

View File

@ -1,6 +1,7 @@
from django.contrib.auth.decorators import login_required from django.contrib.auth.decorators import login_required
from django.db.models.functions import Lower from django.db.models.functions import Lower
from django.shortcuts import render from django.shortcuts import render
from django.urls import reverse_lazy
from django_tables2 import RequestConfig from django_tables2 import RequestConfig
from django.utils.translation import gettext as _ from django.utils.translation import gettext as _
@ -13,7 +14,7 @@ def keyword(request):
table = KeywordTable(Keyword.objects.all()) table = KeywordTable(Keyword.objects.all())
RequestConfig(request, paginate={'per_page': 25}).configure(table) RequestConfig(request, paginate={'per_page': 25}).configure(table)
return render(request, 'generic/list_template.html', {'title': _("Keyword"), 'table': table}) return render(request, 'generic/list_template.html', {'title': _("Keyword"), 'table': table, 'create_url': 'new_keyword'})
@login_required @login_required
@ -27,6 +28,7 @@ def sync_log(request):
@login_required @login_required
def recipe_import(request): def recipe_import(request):
table = RecipeImportTable(RecipeImport.objects.all()) table = RecipeImportTable(RecipeImport.objects.all())
RequestConfig(request, paginate={'per_page': 25}).configure(table) RequestConfig(request, paginate={'per_page': 25}).configure(table)
return render(request, 'generic/list_template.html', {'title': _("Import"), 'table': table}) return render(request, 'generic/list_template.html', {'title': _("Import"), 'table': table})
@ -37,4 +39,4 @@ def storage(request):
table = StorageTable(Storage.objects.all()) table = StorageTable(Storage.objects.all())
RequestConfig(request, paginate={'per_page': 25}).configure(table) RequestConfig(request, paginate={'per_page': 25}).configure(table)
return render(request, 'generic/list_template.html', {'title': _("Storage Backend"), 'table': table}) return render(request, 'generic/list_template.html', {'title': _("Storage Backend"), 'table': table, 'create_url': 'new_storage'})

View File

@ -27,7 +27,7 @@ LOGIN_REDIRECT_URL = "index"
LOGOUT_REDIRECT_URL = "index" LOGOUT_REDIRECT_URL = "index"
CRISPY_TEMPLATE_PACK = 'bootstrap4' CRISPY_TEMPLATE_PACK = 'bootstrap4'
DJANGO_TABLES2_TEMPLATE = 'django_tables2/bootstrap4.html' DJANGO_TABLES2_TEMPLATE = 'cookbook/templates/generic/table_template.html'
MESSAGE_TAGS = { MESSAGE_TAGS = {
messages.ERROR: 'danger' messages.ERROR: 'danger'