Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(print): table layout instead of flex layout for content nodes #2654

Merged
merged 4 commits into from
May 11, 2022

Conversation

usu
Copy link
Member

@usu usu commented Apr 24, 2022

Use table layout for ColumnLayout and for Storyboard instead of flex layout

For Storyboard, this fixes the issue with inconsistent column widths. For ColumnLayout no change at the moment, but probably easier to generate consistent results with PagedJS & Chrome.

@usu usu changed the title chore (print): table layout instead of flex layout for content nodes chore(print): table layout instead of flex layout for content nodes Apr 24, 2022
Copy link
Member

@carlobeltrame carlobeltrame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht visuell sehr gut aus, aber jetzt können lange Storyboards gar nicht mehr über mehrere Seiten umbrechen. Ich habe als User jetzt gar keine Möglichkeit mehr, lange Inhalte in einem Storyboard zu erfassen (nicht mal in mehreren Sections wie es in eCamp2 möglich war). Ich sehe das als eine Regression, was findest du?

@usu
Copy link
Member Author

usu commented May 8, 2022

Sieht visuell sehr gut aus, aber jetzt können lange Storyboards gar nicht mehr über mehrere Seiten umbrechen.

Sections brechen bei mir in Tabellen-Layout um (siehe angehängtes PDF). Mit schönem Layout natürlich nur mit Chrome schön, mit PagedJS noch distorted.
Harry-Potter-Lager (10).pdf

Hat bei dir nicht funktioniert?

@carlobeltrame
Copy link
Member

Bei mir funktionierts nicht ganz so gut, nein. Ich teste mit dem GRGR-Lager, und drucke jeweils das ganze Lager mit den Standardeinstellungen. Die Issues sind zum Beispiel beim Block 2.6 sichtbar.

In der Preview wird die Tabelle nicht umgebrochen:
Screenshot from 2022-05-10 12-07-23

Das Puppeteer-PDF sieht dann ganz anders aus, was mich schon mal besorgt. Der Block 2.6 bricht zwar um, aber das Problem mit den verschwundenen Spalten ist immer noch da (siehe im PDF die Seite 9):
GRGR-2654.pdf

@usu usu added the deploy! Creates a feature branch deployment for this PR label May 10, 2022
@usu
Copy link
Member Author

usu commented May 10, 2022

In der Preview wird die Tabelle nicht umgebrochen:

Preview funktioniert mit PagedJS und dort funktioniert das Layout sowohl mit flex als auch mit table nicht. Die Chance, dass table gefixt wird ist aber grösser (laut den Entwicklern wird dort aktiv daran gearbeitet).

Das Puppeteer-PDF sieht dann ganz anders aus, was mich schon mal besorgt

Tailwind-Issue, gefixt mit #2652. In diesem Druck ist Tailwind-CSS nicht geladen.

Ich habe devel gemerged damit beide #2652 und #2653 drin sind. GRGR-Lager sieht bei mir gedruckt so aus wie angehängt:
GRGR (2).pdf

Copy link
Member

@carlobeltrame carlobeltrame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht gut aus jetzt, merci!

@carlobeltrame carlobeltrame merged commit 7c7ba53 into ecamp:devel May 11, 2022
@usu usu deleted the feat/print-nuxt-content-nodes branch November 6, 2022 05:52
@usu usu mentioned this pull request Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy! Creates a feature branch deployment for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants