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

LEAF-4577 - Print to PDF Image #2590

Merged
merged 2 commits into from
Nov 6, 2024
Merged

LEAF-4577 - Print to PDF Image #2590

merged 2 commits into from
Nov 6, 2024

Conversation

shaneodd
Copy link
Contributor

@shaneodd shaneodd commented Oct 29, 2024

Issue
Images were not allowed in the print to pdf, so for example the image upload would not show when printing a pdf using the browsers print to pdf functionality.

The fix
Adjusted print style sheet to allow images within the request.

** Other Issues Fixed **

  • Skip to content link
  • First section item was pushed over

New
image

Old
image

Testing

  • Upload an image to a form
  • Using the browsers print preview you should now see the image in view to print

aerinkayne
aerinkayne previously approved these changes Oct 30, 2024
jampaul3
jampaul3 previously approved these changes Oct 30, 2024
…to content link and make the first entry load properly.
Pelentan
Pelentan previously approved these changes Nov 4, 2024
@Pelentan Pelentan added the With QA Ticket is to QA. No changes unless pulled back to in progress label Nov 4, 2024
@Pelentan Pelentan changed the base branch from master to dev November 6, 2024 12:41
@Pelentan Pelentan dismissed stale reviews from jampaul3, aerinkayne, and themself November 6, 2024 12:41

The base branch was changed.

@Pelentan Pelentan merged commit c394044 into dev Nov 6, 2024
4 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
With QA Ticket is to QA. No changes unless pulled back to in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants