From 2043596035e82d5a29b51e2f52368dde44f3d321 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Wed, 23 Sep 2020 19:03:20 +0200
Subject: [PATCH] Use template strings when calculating the CSS `transform`s,
 in the `PDFPageView.cssTransform` method

In my opinion this slightly improves readability, by grouping related properties together.
---
 web/pdf_page_view.js | 27 ++++-----------------------
 1 file changed, 4 insertions(+), 23 deletions(-)

diff --git a/web/pdf_page_view.js b/web/pdf_page_view.js
index 658b06b19..9b6eb3f02 100644
--- a/web/pdf_page_view.js
+++ b/web/pdf_page_view.js
@@ -349,16 +349,7 @@ class PDFPageView {
       scaleX = height / width;
       scaleY = width / height;
     }
-    const cssTransform =
-      "rotate(" +
-      relativeRotation +
-      "deg) " +
-      "scale(" +
-      scaleX +
-      "," +
-      scaleY +
-      ")";
-    target.style.transform = cssTransform;
+    target.style.transform = `rotate(${relativeRotation}deg) scale(${scaleX}, ${scaleY})`;
 
     if (this.textLayer) {
       // Rotating the text layer is more complicated since the divs inside the
@@ -397,19 +388,9 @@ class PDFPageView {
       }
 
       textLayerDiv.style.transform =
-        "rotate(" +
-        textAbsRotation +
-        "deg) " +
-        "scale(" +
-        scale +
-        ", " +
-        scale +
-        ") " +
-        "translate(" +
-        transX +
-        ", " +
-        transY +
-        ")";
+        `rotate(${textAbsRotation}deg) ` +
+        `scale(${scale}) ` +
+        `translate(${transX}, ${transY})`;
       textLayerDiv.style.transformOrigin = "0% 0%";
     }