Skip to content

Conversation

@YoanKar
Copy link
Contributor

@YoanKar YoanKar commented May 14, 2025

No description provided.

@YoanKar YoanKar requested review from Copilot and dessyordanova May 14, 2025 10:12
@YoanKar YoanKar added the For Review Should be reviewed by a team member. Provide feedback using comments wherever is necessary label May 14, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds documentation for the new Popup Annotation feature and updates related documentation files.

  • Introduces a dedicated documentation file for Popup Annotation with code examples demonstrating how to create and use a Popup Annotation.
  • Updates the annotations overview to include Popup Annotation among the supported types.
  • Updates the FixedContentEditor documentation to detail the new DrawPopupAnnotation method.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
libraries/radpdfprocessing/model/annotations/popup.md New documentation file detailing Popup Annotation properties, usage, and code examples.
libraries/radpdfprocessing/model/annotations/overview.md Updated overview to include a table entry for Popup Annotation.
libraries/radpdfprocessing/editing/fixedcontenteditor.md Updated method list to include a description for DrawPopupAnnotation.

@@ -0,0 +1,83 @@
---
title: Popup
Copy link

Copilot AI May 14, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider updating the title metadata to 'Popup Annotation' to match the page title and ensure consistency across the documentation.

Copilot uses AI. Check for mistakes.
|[Stamp]({%slug radpdfprocessing-model-annotations-stamp%})|Stamp annotations display text or graphics intended to look as if they were stamped on the page with a rubber stamp.|
|[TextMarkup]({%slug radpdfprocessing-model-annotations-text-markup%})| Text markup annotations appear as **Highlights**, **Underlines**, **Strikeouts** or **Squiggly** underlines in the text of a document. When opened, they display a pop-up window containing the text of the associated note.|
|[Popup]({%slug radpdfprocessing-model-annotations-popup%})|A popup annotation is associated with another [markup annotation]({%slug radpdfprocessing-model-annotations-overview%}) and displays its content in a pop-up window for entry and editing. It typically appears as a pop-up note.|
Copy link

Copilot AI May 14, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider clarifying that the popup annotation is initially hidden and only becomes visible when activated to more accurately reflect its behavior.

Copilot uses AI. Check for mistakes.
|**DrawStampAnnotation**|Creates a new [StampAnnotation]({%slug radpdfprocessing-model-annotations-stamp%}) and draws it with a specified size and name.|
|**DrawTextAnnotation**|Creates a new [TextAnnotation]({%slug radpdfprocessing-model-annotations-text%}) and draws it with a specified size and text.|
|**DrawLineAnnotation**|Creates a new [LineAnnotation]({%slug radpdfprocessing-model-annotations-line%}) with starting point the current point of the editor and end point the current point of the editor plus the given distances.|
|**DrawPopupAnnotation**|Creates a new [PopupAnnotation]({%slug radpdfprocessing-model-annotations-popup%}), associates it with a [markup annotation]({%slug radpdfprocessing-model-annotations-overview%}), and draws it with the specified size and properties.|
Copy link

Copilot AI May 14, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider verifying that the description for DrawPopupAnnotation clearly states if a parent annotation is required and whether any default properties are set during its creation.

Copilot uses AI. Check for mistakes.
@dessyordanova dessyordanova merged commit fa1017d into master May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Review Should be reviewed by a team member. Provide feedback using comments wherever is necessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants