Resolve "PDF export for timetables" #381
No reviewers
Labels
No labels
Security
TeX
auto-update
board
done
board
ready
board
todo
check
delete-eslint-rc-js
check
update-builddeps-package-json
check
update-eslint-rc-js
check
update-gitignore
check
update-merge-request-template
check
update-prettier-ignore
check
update-pyproject-toml
check
update-renovate-json
check
update-tox-ini
part
backend
part
ci
part
docs
part
frontend
part
i18n
part
non-technical
part
packaging
prio
1
prio
2
prio
3
release-mr-5.x
size
large
size
medium
size
small
source
customer
source
customer::fsmw
source
customer::fss
source
customer::teckids
source
downstream
type
breaking
type
bug
type
feature
type
refactoring
workflow
blocked
workflow
confirmed
workflow
current-todo
workflow
discussing
workflow
new-app
workflow
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
aleksis/AlekSIS-App-Chronos!381
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "51-pdf-export-for-timetables"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes #51
added 1 commit
Compare with previous version
added 18 commits
masterCompare with previous version
@lloydmeins Please give us an update on this. Do you need any help or support?
I think it would be good if @hansegucker and I would meet and work on it for some time.
added 1 commit
Compare with previous version
added 106 commits
masterCompare with previous version
added 1 commit
Compare with previous version
assigned to @lloydmeins
unassigned @lloydmeins
assigned to @fph
assigned to @nik and unassigned @fph
requested review from @ZugBahnHof
added 1 commit
Compare with previous version
added 2 commits
Compare with previous version
added 1 commit
Compare with previous version
added 7 commits
Compare with previous version
added 4 commits
Compare with previous version
added 5 commits
08bf4821- Add templates and files for PDF export of timetablesefbf13f3- Export timetable as pdfb393ad2c- Remove import of not used modulescd27bfc9- Remove second page of pdfce2016aa- Use PDF rendering tools from coreCompare with previous version
added 1 commit
4cd826f2- Reformat and fix lintCompare with previous version
??
Can you please elaborate on what you mean exactly. "??" is not a good indicator.
What does
{# Show short weekdays on tablets #}mean in this context?I guess it means that on smaller screen sizes, weekday headers are abbreviated.
If the UI behaviour is different on some devices, please post screenshots showing that!
I think, this is simply old code, from another template, because the pdf won't change on other display sizes
Ah oh, it's the PDF :D
requested review from @hansegucker and removed review request for @ZugBahnHof
Can you please fix the margins here, @lloydmeins?

And it would be nice if the PDF makes use of Roboto as font family.
added 1 commit
759e6fbd- Change font to RobotoCompare with previous version
Blocked by AlekSIS/official/AlekSIS-Core#357
added 11 commits
master85752ec4- Merge branch 'master' into 51-pdf-export-for-timetablesCompare with previous version
added 2 commits
1dd8ab4f- [Print] Show timetable title in page headerCompare with previous version
resolved all threads
Done.
resolved all threads
changed this line in version 16 of the diff
added 1 commit
d518c48e- Reformat timetable_print.htmlCompare with previous version
added 47 commits
masterf839aae1- Merge branch 'master' into 51-pdf-export-for-timetablesCompare with previous version
added 1 commit
241ea05f- Add absolute URL to all static URLs for printing via electron-pdfCompare with previous version
Please remove these superfluous lines.
Unrelated change.
added 6 commits
master2d7c8af0- Merge remote-tracking branch 'origin/master' into 51-pdf-export-for-timetables8a81a899- [Printout] Remove static_prefix for static filesCompare with previous version
added 1 commit
3c824401- Use render_pdf for the substitutions print view, tooCompare with previous version
@lloydmeins This is now longer blocked! 🎉
added 5 commits
masterf5c1befe- Merge branch 'master' into 51-pdf-export-for-timetablesCompare with previous version
changed this line in version 22 of the diff
changed this line in version 22 of the diff
added 1 commit
224de424- Resolve unrelated changes and remove superfluous linesCompare with previous version
added 1 commit
57606698- Reuse proper term for colourCompare with previous version
added 1 commit
4c4e04bc- Remove unnecessary template namesCompare with previous version
resolved all threads
marked this merge request as ready
added 1 commit
b253cd36- Reformat in order for the pipeline to succeedCompare with previous version
approved this merge request
@nik Please review and merge.
Please drop trailing newlines
What is the reason for this change?
Why?
As there is also the possibility to access the pdf version on the smart plan we need to make sure that the gathered plan is the regular one.
changed this line in version 26 of the diff
added 1 commit
2d354396- Remove trailing newlinesCompare with previous version
To prevent an argument overload.
resolved all threads
added 3 commits
master9f9be1fc- Merge branch 'master' into 51-pdf-export-for-timetablesCompare with previous version
marked this merge request as draft
marked this merge request as ready
mentioned in commit
577c6da31d