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

Merged
christian.merten merged 7 commits from MK/multi_page_tables into main 10 months ago
Collaborator

closes #128

closes #128
christian.merten reviewed 10 months 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"""

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 %}

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]

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?

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:
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 10 months 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 10 months ago
christian.merten merged commit 308d9f3114 into main 10 months ago
christian.merten deleted branch MK/multi_page_tables 10 months ago

Reviewers

christian.merten approved these changes 10 months 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.