Skip to content

Conversation

@PascalVorwerk
Copy link
Collaborator

@PascalVorwerk PascalVorwerk commented Feb 15, 2025

Pull Request

📖 Description

Solving several issues like stated in the issue #3345:

  • Pages now are printed on a new page for each H2 (except the first H2) found.
  • The videos page does not have video's in the printed version, but a link to each video.
  • Hiding certain UI elements that are not relevant when printing a single page like the header, nav menu, footer and aside.
  • Hiding interactive elements like buttons.
  • Whenever a FluentTabs element is used, all the content for each tab will be placed below eachother, instead of only 1 tab showing in the printed version.

🎫 Issues

#3345

👩‍💻 Reviewer Notes

If you find any more issues on pages please let me know, it was really hard to go about each page and find elements that are not correctly printed yet, so i might have missed some!

📑 Test Plan

Run the Demo website locally and try to print the pages in multiple browsers. I have tried Chrome and Microsoft Edge.

✅ Checklist

General

  • I have added tests for my changes. (Not applicable i think?)
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes. (Not applicable i think?)
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

@vnbaaij vnbaaij linked an issue Feb 15, 2025 that may be closed by this pull request
@vnbaaij vnbaaij added this to the v4.11.5 milestone Feb 15, 2025
@vnbaaij
Copy link
Collaborator

vnbaaij commented Feb 15, 2025

This is great. Thanks!

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.

fix: Page printing in templates

3 participants