Merge pull request #11097 from Snuffleupagus/textLayer-measure-width

[TextLayer] Only measure the width of the text, in `_layoutText`, for multi-char text divs
This commit is contained in:
Tim van der Meij 2019-08-25 16:08:51 +02:00 committed by GitHub
commit 184d416639
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -541,12 +541,14 @@ var renderTextLayer = (function renderTextLayerClosure() {
this._layoutTextLastFontFamily = fontFamily;
}
let width = this._layoutTextCtx.measureText(textDiv.textContent).width;
let transform = '';
if (textDivProperties.canvasWidth !== 0 && width > 0) {
textDivProperties.scale = textDivProperties.canvasWidth / width;
transform = `scaleX(${textDivProperties.scale})`;
if (textDivProperties.canvasWidth !== 0) {
// Only measure the width for multi-char text divs, see `appendText`.
const { width, } = this._layoutTextCtx.measureText(textDiv.textContent);
if (width > 0) {
textDivProperties.scale = textDivProperties.canvasWidth / width;
transform = `scaleX(${textDivProperties.scale})`;
}
}
if (textDivProperties.angle !== 0) {
transform = `rotate(${textDivProperties.angle}deg) ${transform}`;
@ -634,11 +636,8 @@ var renderTextLayer = (function renderTextLayerClosure() {
transformBuf.length = 0;
paddingBuf.length = 0;
if (divProps.angle !== 0) {
transformBuf.push(`rotate(${divProps.angle}deg)`);
}
if (divProps.scale !== 1) {
transformBuf.push(`scaleX(${divProps.scale})`);
if (divProps.originalTransform) {
transformBuf.push(divProps.originalTransform);
}
if (divProps.paddingTop > 0) {
paddingBuf.push(`${divProps.paddingTop}px`);