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

Fix PDF timetable breaking when 'print abstracts' option is set #4955

Merged
merged 6 commits into from Jun 21, 2021

Conversation

javfg
Copy link
Contributor

@javfg javfg commented Jun 18, 2021

Changes the layout of the timetable PDF when the advanced option 'Print abstract content of all contributions' is selected.

ReportLab is unable to split table rows when the content is bigger than the page it's placed upon, as described by the discussion linked in the bug report by @bcc.

To overcome this limitation, the timetable will not be contained in a table if the option to print the abstracts is set. The document is slightly different, but still provides the same information. If that option is unset, the previous layout persists.

Closes #4881.

@javfg javfg marked this pull request as ready for review June 18, 2021 15:45
Copy link
Member

@ThiefMaster ThiefMaster left a comment

Choose a reason for hiding this comment

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

I only skimmed the diff (legacy code after all...) but AFAICT it looks good. So if it works fine I think it's good to go

indico/legacy/pdfinterface/conference.py Outdated Show resolved Hide resolved
indico/legacy/pdfinterface/conference.py Outdated Show resolved Hide resolved
@javfg javfg force-pushed the pdfexport-fix branch 2 times, most recently from a2f665a to d22770a Compare June 21, 2021 14:30
@ThiefMaster ThiefMaster merged commit 919665b into indico:master Jun 21, 2021
@ThiefMaster ThiefMaster deleted the pdfexport-fix branch June 21, 2021 19:41
plourenco pushed a commit to plourenco/indico that referenced this pull request Jul 6, 2021
* Fix PDF timetable breaking when abstracts are included
* Widen poster board column so header fits in one line
pferreir pushed a commit to pferreir/indico that referenced this pull request Oct 12, 2021
* Fix PDF timetable breaking when abstracts are included
* Widen poster board column so header fits in one line
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error exporting PDF timetable with a long abstract
2 participants