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

Do not modify data.rect in AnnotationElement_createContainer, since that will corrupt the annotation position on subsequent calls

Browse Source 
Fixes 6804; this regressed in PR 6714.
This commit is contained in:
Jonas Jenwald 2015-12-27 12:46:20 +01:00
parent cba8a87f84
commit 3c7088dc44
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/display/annotation_layer.js
View File

@ -109,7 +109,9 @@ var AnnotationElement = (function AnnotationElementClosure() {
container.setAttribute('data-annotation-id', data.id);
data.rect = Util.normalizeRect([
// Do *not* modify `data.rect`, since that will corrupt the annotation
// position on subsequent calls to `_createContainer` (see issue 6804).
var rect = Util.normalizeRect([
data.rect[0],
page.view[3] - data.rect[1] + page.view[1],
data.rect[2],
@ -119,7 +121,7 @@ var AnnotationElement = (function AnnotationElementClosure() {
CustomStyle.setProp('transform', container,
'matrix(' + viewport.transform.join(',') + ')');
CustomStyle.setProp('transformOrigin', container,
-data.rect[0] + 'px ' + -data.rect[1] + 'px');
-rect[0] + 'px ' + -rect[1] + 'px');
if (data.borderStyle.width > 0) {
container.style.borderWidth = data.borderStyle.width + 'px';
@ -174,8 +176,8 @@ var AnnotationElement = (function AnnotationElementClosure() {
}
}
container.style.left = data.rect[0] + 'px';
container.style.top = data.rect[1] + 'px';
container.style.left = rect[0] + 'px';
container.style.top = rect[1] + 'px';
container.style.width = width + 'px';
container.style.height = height + 'px';