feat(members/pdf): improve handling of long tables #135

Merged
christian.merten merged 7 commits from MK/multi_page_tables into main 1 year ago
Collaborator

closes #128

closes #128
christian.merten reviewed 1 year ago
christian.merten left a comment
Owner

Sieht insgesamt gut, danke! In der TN-Liste auf dem Seminarbericht gibt es noch einen seltsamen Offset Bug, siehe screenshot:

Sieht insgesamt gut, danke! In der TN-Liste auf dem Seminarbericht gibt es noch einen seltsamen Offset Bug, siehe screenshot:
@ -367,0 +367,4 @@
@property
def address_multiline(self):
"""Returning the whole address"""
Owner

Kannst du den doc string hier noch anpassen, sodass da nicht der gleiche steht wie in address?

Kannst du den doc string hier noch anpassen, sodass da nicht der gleiche steht wie in `address`?
christian.merten marked this conversation as resolved
@ -43,3 +43,4 @@
{% for p in people %}
{{ forloop.counter }} & {{ p.name|esc_all }} & {{ p.qualities|esc_all }} & {{ p.comments|esc_all }} \\
{% endfor %}
{% for p in people %}
Owner

Ich glaube du hast hier vergessen deinen test code zu entfernen :D

Ich glaube du hast hier vergessen deinen test code zu entfernen :D
christian.merten marked this conversation as resolved
@ -122,38 +123,38 @@
\section{Teilnehmer*innenliste}
\begin{table}[H]
Owner

Für mein Verständnis: Das Entfernen von \begin{table} ist was den Bug fixed. Der Rest ist stylistisch, oder?

Für mein Verständnis: Das Entfernen von `\begin{table}` ist was den Bug fixed. Der Rest ist stylistisch, oder?
Poster
Collaborator

Exakt, das entfernen von der table environment ermöglicht, dass die Tabellen über mehrere Seiten gehen, der Teil mit \endhead und \endfoot sorgt dafür, dass auf einer möglichen neuen Seite wieder die Spaltennamen angezeigt werden.

Exakt, das entfernen von der table environment ermöglicht, dass die Tabellen über mehrere Seiten gehen, der Teil mit `\endhead` und `\endfoot` sorgt dafür, dass auf einer möglichen neuen Seite wieder die Spaltennamen angezeigt werden.
christian.merten marked this conversation as resolved
Poster
Collaborator

Sieht insgesamt gut, danke! In der TN-Liste auf dem Seminarbericht gibt es noch einen seltsamen Offset Bug, siehe screenshot:

das kann ich nicht ohne weiteres reproduzieren. Was hast du denn an Texten eingetragen?

> Sieht insgesamt gut, danke! In der TN-Liste auf dem Seminarbericht gibt es noch einen seltsamen Offset Bug, siehe screenshot: das kann ich nicht ohne weiteres reproduzieren. Was hast du denn an Texten eingetragen?
Poster
Collaborator

habe jetzt einen anderen Spaltentyp versucht, hoffe der offset geht dadurch weg?

habe jetzt einen anderen Spaltentyp versucht, hoffe der offset geht dadurch weg?
Owner

habe jetzt einen anderen Spaltentyp versucht, hoffe der offset geht dadurch weg?

jetzt sieht es so aus bei mir:

> habe jetzt einen anderen Spaltentyp versucht, hoffe der offset geht dadurch weg? jetzt sieht es so aus bei mir:
Owner
Das kommt jetzt auch auf `testing`: https://dev.jdav-hd.merten.dev/de/kompassmembers/freizeit/4/download/ljp_report_costs_and_participants
Poster
Collaborator
> Das kommt jetzt auch auf `testing`: > > https://dev.jdav-hd.merten.dev/de/kompassmembers/freizeit/4/download/ljp_report_costs_and_participants sollte gefixed sein.
christian.merten approved these changes 1 year ago
christian.merten left a comment
Owner

Top, danke!

Top, danke!
christian.merten changed title from Mehrseitige Tabellen in PDFs to feat(members/pdf): improve handling of long tables 1 year ago
christian.merten merged commit 308d9f3114 into main 1 year ago
christian.merten deleted branch MK/multi_page_tables 1 year ago

Reviewers

christian.merten approved these changes 1 year ago
The pull request has been merged as 308d9f3114.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: digitales/kompass#135
Loading…
There is no content yet.