-
Notifications
You must be signed in to change notification settings - Fork 5
883478 How to change cursor and add tool-tip when mouse over the uri annotation in PdfViewer #36
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
Conversation
…annotation in PdfViewer
…annotation in PdfViewer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested changes
docView.Load(stream); | ||
annotationPageIndex = 0; | ||
annotationBounds = uriAnnotation.Bounds; | ||
//annotationBounds = convertor.ConvertToPixels(annotationBounds, PdfGraphicsUnit.Point); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Remove the commented lines
- Remove unwanted namespaces
- Add comments wherever possible as it is a public repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have addressed the provided feedbacks.
…annotation in PdfViewer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested changes
@@ -0,0 +1,9 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<packages> | |||
<package id="Syncfusion.Compression.Base" version="26.2.5" targetFramework="net48" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why we specifically mention the version in the package? Can this be "*"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have addressed the provided feedback
…annotation in PdfViewer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
How to change cursor and add tool-tip when mouse over the uri annotation in PdfViewer