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

Merge pull request #15185 from calixteman/ink_translation

Browse Source 
[Editor] Ink editor was too much translated after commit
This commit is contained in:
calixteman 2022-07-20 10:34:37 +02:00 committed by GitHub
commit 7a4b72ed11
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/display/editor/ink.js
View File

@ -137,11 +137,11 @@ class InkEditor extends AnnotationEditor {
this.parent.addCommands({
cmd: () => {
this.thickness = thickness;
this.#fitToContent();
this.#fitToContent(/* thicknessChanged = */ true);
},
undo: () => {
this.thickness = savedThickness;
this.#fitToContent();
this.#fitToContent(/* thicknessChanged = */ true);
},
mustExec: true,
type: AnnotationEditorParamsType.INK_THICKNESS,
@ -829,7 +829,9 @@ class InkEditor extends AnnotationEditor {
* @returns {number}
*/
#getPadding() {
return Math.ceil(this.thickness * this.parent.scaleFactor);
return this.#disableEditing
? Math.ceil(this.thickness * this.parent.scaleFactor)
: 0;
}
/**
@ -837,7 +839,7 @@ class InkEditor extends AnnotationEditor {
* the bounding box of the contents.
* @returns {undefined}
*/
#fitToContent() {
#fitToContent(thicknessChanged = false) {
if (this.isEmpty()) {
return;
}
@ -849,8 +851,8 @@ class InkEditor extends AnnotationEditor {
const bbox = this.#getBbox();
const padding = this.#getPadding();
this.#baseWidth = bbox[2] - bbox[0];
this.#baseHeight = bbox[3] - bbox[1];
this.#baseWidth = Math.max(RESIZER_SIZE, bbox[2] - bbox[0]);
this.#baseHeight = Math.max(RESIZER_SIZE, bbox[3] - bbox[1]);
const width = Math.ceil(padding + this.#baseWidth * this.scaleFactor);
const height = Math.ceil(padding + this.#baseHeight * this.scaleFactor);
@ -874,9 +876,12 @@ class InkEditor extends AnnotationEditor {
this.#realHeight = height;
this.setDims(width, height);
const unscaledPadding = thicknessChanged
? 0
: padding / this.scaleFactor / 2;
this.translate(
prevTranslationX - this.translationX,
prevTranslationY - this.translationY
prevTranslationX - this.translationX - unscaledPadding,
prevTranslationY - this.translationY - unscaledPadding
);
}