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

Allow printing annotations popups #173

Closed
gtache opened this issue Mar 16, 2021 · 7 comments
Closed

Allow printing annotations popups #173

gtache opened this issue Mar 16, 2021 · 7 comments
Labels
enhancement New feature or request Viewer Viewer Module

Comments

@gtache
Copy link
Contributor

gtache commented Mar 16, 2021

So that users can print a document with their comments on it.

@pcorless
Copy link
Owner

Any special requirements for presentation? Guessing a mirror more or less of the swing component?

@gtache
Copy link
Contributor Author

gtache commented Mar 17, 2021

Exactly, it should look more or less the same as the swing components. Only the opened/visible ones would be printed of course.

@pcorless pcorless added the Viewer Viewer Module label Nov 28, 2022
@pcorless pcorless added the enhancement New feature or request label Nov 28, 2022
@pcorless
Copy link
Owner

Couple corner cases:

  • how do we handle popups that have been moved outside of the page margins when printing?
  • do we assume we only print popups that intersect the page component?

@pcorless
Copy link
Owner

Getting there. Should be working by end of week

image

@pcorless
Copy link
Owner

image
Here is some printed popup output. @gtache how important is the "glue" that links the text annotation to the popup? The current code uses transparency which can really increase the print job size. Do you know if the "glue" is preferred and if it can have a solid colour?
image
(Glue)

pcorless pushed a commit that referenced this issue Apr 16, 2023
pcorless pushed a commit that referenced this issue Apr 16, 2023
pcorless pushed a commit that referenced this issue Apr 18, 2023
@pcorless
Copy link
Owner

Add the glue for printed ouput:
image

pcorless pushed a commit that referenced this issue May 8, 2023
# Conflicts:
#	core/core-awt/src/main/java/org/icepdf/core/pobjects/annotations/Annotation.java
#	core/core-awt/src/main/java/org/icepdf/core/pobjects/annotations/MarkupAnnotation.java
#	core/core-awt/src/main/java/org/icepdf/core/pobjects/annotations/PopupAnnotation.java
pcorless pushed a commit that referenced this issue May 8, 2023
pcorless pushed a commit that referenced this issue May 10, 2023
@pcorless
Copy link
Owner

Released as part of 7.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Viewer Viewer Module
Projects
None yet
Development

No branches or pull requests

2 participants