Browse Source

Don't scale single-char text divs.

This change makes scrolling noticeably smoother on files with many
single-char text divs, such as the one in #1045. The trade-off is that
the visual appearance of text selection in such documents is slightly
worse, because more text divs overlap.

This change also uses `scaleX(N)` instead of `scale(N, 1)`. This might
be marginally more efficient in terms of JS string concatenation.
Nicholas Nethercote 11 years ago
parent
commit
cd61aad24f
  1. 28
      web/text_layer_builder.js

28
web/text_layer_builder.js

@ -83,14 +83,21 @@ var TextLayerBuilder = (function TextLayerBuilderClosure() {
var width = ctx.measureText(textDiv.textContent).width; var width = ctx.measureText(textDiv.textContent).width;
if (width > 0) { if (width > 0) {
textLayerFrag.appendChild(textDiv); textLayerFrag.appendChild(textDiv);
// Dataset values come of type string. var transform;
var textScale = textDiv.dataset.canvasWidth / width; if (textDiv.dataset.canvasWidth !== undefined) {
// Dataset values come of type string.
var textScale = textDiv.dataset.canvasWidth / width;
transform = 'scaleX(' + textScale + ')';
} else {
transform = '';
}
var rotation = textDiv.dataset.angle; var rotation = textDiv.dataset.angle;
var transform = 'scale(' + textScale + ', 1)';
if (rotation) { if (rotation) {
transform = 'rotate(' + rotation + 'deg) ' + transform; transform = 'rotate(' + rotation + 'deg) ' + transform;
} }
CustomStyle.setProp('transform' , textDiv, transform); if (transform) {
CustomStyle.setProp('transform' , textDiv, transform);
}
} }
} }
@ -165,10 +172,15 @@ var TextLayerBuilder = (function TextLayerBuilderClosure() {
if (angle !== 0) { if (angle !== 0) {
textDiv.dataset.angle = angle * (180 / Math.PI); textDiv.dataset.angle = angle * (180 / Math.PI);
} }
if (style.vertical) { // We don't bother scaling single-char text divs, because it has very
textDiv.dataset.canvasWidth = geom.height * this.viewport.scale; // little effect on text highlighting. This makes scrolling on docs with
} else { // lots of such divs a lot faster.
textDiv.dataset.canvasWidth = geom.width * this.viewport.scale; if (textDiv.textContent.length > 1) {
if (style.vertical) {
textDiv.dataset.canvasWidth = geom.height * this.viewport.scale;
} else {
textDiv.dataset.canvasWidth = geom.width * this.viewport.scale;
}
} }
}, },

Loading…
Cancel
Save