Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Check the relevant parameters inside of the mustBeViewedWhenEditing method

Browse Source 
Similar to the `mustBeViewed` method, we can check the relevant parameters within the `mustBeViewedWhenEditing` method itself since that (in my opinion) slightly helps readability of the code in the `src/core/document.js` file.
This commit is contained in:
Jonas Jenwald 2024-07-05 11:34:46 +02:00
parent 9065ee465b
commit 5f744904ac
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/core/annotation.js
View File

@ -777,8 +777,8 @@ class Annotation {
return this.printable; return this.printable;
} }
mustBeViewedWhenEditing() { mustBeViewedWhenEditing(isEditing, modifiedIds = null) {
return !this.data.isEditable; return isEditing ? !this.data.isEditable : !modifiedIds?.has(this.data.id);
} }
/** /**

3
src/core/document.js
View File

@ -582,8 +582,7 @@ class Page {
intentAny || intentAny ||
(intentDisplay && (intentDisplay &&
annotation.mustBeViewed(annotationStorage, renderForms) && annotation.mustBeViewed(annotationStorage, renderForms) &&
((isEditing && annotation.mustBeViewedWhenEditing()) || annotation.mustBeViewedWhenEditing(isEditing, modifiedIds)) ||
(!isEditing && !modifiedIds?.has(annotation.data.id)))) ||
(intentPrint && annotation.mustBePrinted(annotationStorage)) (intentPrint && annotation.mustBePrinted(annotationStorage))
) { ) {
opListPromises.push( opListPromises.push(