From 4b8e5ace07bdea2a6aebfa2550a85318402fa971 Mon Sep 17 00:00:00 2001
From: Julian Descottes <julian.descottes@gmail.com>
Date: Sat, 4 Jun 2016 17:03:34 +0200
Subject: [PATCH] Issue #470: remove references to tiled-preview, rename
 preferences

---
 src/js/controller/preview/PreviewController.js   | 10 +++++-----
 .../settings/ApplicationSettingsController.js    | 16 ++++++++--------
 src/js/rendering/frame/CachedFrameRenderer.js    |  2 +-
 src/js/rendering/frame/FrameRenderer.js          |  6 +++---
 src/js/utils/UserSettings.js                     |  4 ++--
 src/templates/settings/application.html          |  6 +++---
 6 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/src/js/controller/preview/PreviewController.js b/src/js/controller/preview/PreviewController.js
index 6efcaeec5..f8cf79b40 100644
--- a/src/js/controller/preview/PreviewController.js
+++ b/src/js/controller/preview/PreviewController.js
@@ -116,8 +116,8 @@
 
   ns.PreviewController.prototype.updateZoom_ = function () {
     var originalSizeEnabled = pskl.UserSettings.get(pskl.UserSettings.ORIGINAL_SIZE_PREVIEW);
-    var tiledPreviewEnabled = pskl.UserSettings.get(pskl.UserSettings.TILED_PREVIEW);
-    var useOriginalSize = originalSizeEnabled || tiledPreviewEnabled;
+    var seamlessModeEnabled = pskl.UserSettings.get(pskl.UserSettings.SEAMLESS_MODE);
+    var useOriginalSize = originalSizeEnabled || seamlessModeEnabled;
 
     var zoom = useOriginalSize ? 1 : this.calculateZoom_();
     this.renderer.setZoom(zoom);
@@ -208,12 +208,12 @@
   };
 
   ns.PreviewController.prototype.updateContainerDimensions_ = function () {
-    var isTiled = pskl.UserSettings.get(pskl.UserSettings.TILED_PREVIEW);
-    this.renderer.setRepeated(isTiled);
+    var isSeamless = pskl.UserSettings.get(pskl.UserSettings.SEAMLESS_MODE);
+    this.renderer.setRepeated(isSeamless);
 
     var height, width;
 
-    if (isTiled) {
+    if (isSeamless) {
       height = PREVIEW_SIZE;
       width = PREVIEW_SIZE;
     } else {
diff --git a/src/js/controller/settings/ApplicationSettingsController.js b/src/js/controller/settings/ApplicationSettingsController.js
index d963e8a7d..50ee21954 100644
--- a/src/js/controller/settings/ApplicationSettingsController.js
+++ b/src/js/controller/settings/ApplicationSettingsController.js
@@ -26,13 +26,13 @@
 
     this.addEventListener(gridSelect, 'change', this.onGridWidthChange_);
 
-    // Tiled preview
-    var tiledPreview = pskl.UserSettings.get(pskl.UserSettings.TILED_PREVIEW);
-    var tiledPreviewCheckbox = document.querySelector('.tiled-preview-checkbox');
-    if (tiledPreview) {
-      tiledPreviewCheckbox.setAttribute('checked', tiledPreview);
+    // Seamless mode
+    var seamlessMode = pskl.UserSettings.get(pskl.UserSettings.SEAMLESS_MODE);
+    var seamlessModeCheckbox = document.querySelector('.seamless-mode-checkbox');
+    if (seamlessMode) {
+      seamlessModeCheckbox.setAttribute('checked', seamlessMode);
     }
-    this.addEventListener(tiledPreviewCheckbox, 'change', this.onTiledPreviewChange_);
+    this.addEventListener(seamlessModeCheckbox, 'change', this.onSeamlessModeChange_);
 
     // Max FPS
     var maxFpsInput = document.querySelector('.max-fps-input');
@@ -55,8 +55,8 @@
     pskl.UserSettings.set(pskl.UserSettings.GRID_WIDTH, width);
   };
 
-  ns.ApplicationSettingsController.prototype.onTiledPreviewChange_ = function (evt) {
-    pskl.UserSettings.set(pskl.UserSettings.TILED_PREVIEW, evt.currentTarget.checked);
+  ns.ApplicationSettingsController.prototype.onSeamlessModeChange_ = function (evt) {
+    pskl.UserSettings.set(pskl.UserSettings.SEAMLESS_MODE, evt.currentTarget.checked);
   };
 
   ns.ApplicationSettingsController.prototype.onBackgroundClick_ = function (evt) {
diff --git a/src/js/rendering/frame/CachedFrameRenderer.js b/src/js/rendering/frame/CachedFrameRenderer.js
index 52dbbb2b8..58fe0004c 100644
--- a/src/js/rendering/frame/CachedFrameRenderer.js
+++ b/src/js/rendering/frame/CachedFrameRenderer.js
@@ -34,7 +34,7 @@
     var serializedFrame = [
       this.getZoom(),
       this.getGridWidth(),
-      pskl.UserSettings.get('TILED_PREVIEW'),
+      pskl.UserSettings.get('SEAMLESS_MODE'),
       offset.x, offset.y,
       size.width, size.height,
       frame.getHash()
diff --git a/src/js/rendering/frame/FrameRenderer.js b/src/js/rendering/frame/FrameRenderer.js
index 9665d3f1b..0ea7723b4 100644
--- a/src/js/rendering/frame/FrameRenderer.js
+++ b/src/js/rendering/frame/FrameRenderer.js
@@ -258,7 +258,7 @@
       this.margin.y - this.offset.y * z
     );
 
-    if (pskl.UserSettings.get('TILED_PREVIEW')) {
+    if (pskl.UserSettings.get('SEAMLESS_MODE')) {
       displayContext.clearRect(-1 * w * z, -1 * h * z, 3 * w * z, 3 * h * z);
     } else {
       displayContext.clearRect(0, 0, w * z, h * z);
@@ -268,14 +268,14 @@
     if (gridWidth > 0) {
       var scaled = pskl.utils.ImageResizer.resizeNearestNeighbour(this.canvas, z, gridWidth);
 
-      if (pskl.UserSettings.get('TILED_PREVIEW')) {
+      if (pskl.UserSettings.get('SEAMLESS_MODE')) {
         this.drawTiledFrames_(displayContext, scaled, w, h, z);
       }
       displayContext.drawImage(scaled, 0, 0);
     } else {
       displayContext.scale(z, z);
 
-      if (pskl.UserSettings.get('TILED_PREVIEW')) {
+      if (pskl.UserSettings.get('SEAMLESS_MODE')) {
         this.drawTiledFrames_(displayContext, this.canvas, w, h, 1);
       }
       displayContext.drawImage(this.canvas, 0, 0);
diff --git a/src/js/utils/UserSettings.js b/src/js/utils/UserSettings.js
index b05c495e9..a72a9faf4 100644
--- a/src/js/utils/UserSettings.js
+++ b/src/js/utils/UserSettings.js
@@ -7,7 +7,7 @@
     DEFAULT_SIZE : 'DEFAULT_SIZE',
     CANVAS_BACKGROUND : 'CANVAS_BACKGROUND',
     SELECTED_PALETTE : 'SELECTED_PALETTE',
-    TILED_PREVIEW : 'TILED_PREVIEW',
+    SEAMLESS_MODE : 'SEAMLESS_MODE',
     ORIGINAL_SIZE_PREVIEW : 'ORIGINAL_SIZE_PREVIEW',
     ONION_SKIN : 'ONION_SKIN',
     LAYER_PREVIEW : 'LAYER_PREVIEW',
@@ -26,7 +26,7 @@
       },
       'CANVAS_BACKGROUND' : 'lowcont-dark-canvas-background',
       'SELECTED_PALETTE' : Constants.CURRENT_COLORS_PALETTE_ID,
-      'TILED_PREVIEW' : false,
+      'SEAMLESS_MODE' : false,
       'ORIGINAL_SIZE_PREVIEW' : false,
       'ONION_SKIN' : false,
       'LAYER_OPACITY' : 0.2,
diff --git a/src/templates/settings/application.html b/src/templates/settings/application.html
index 36a2e2772..504d9d674 100644
--- a/src/templates/settings/application.html
+++ b/src/templates/settings/application.html
@@ -35,7 +35,7 @@
       </div>
 
       <div class="settings-item">
-        <label for="tiled-preview">Layer Opacity</label>
+        <label>Layer Opacity</label>
         <input type="range" class="layer-opacity-input" name="layer-opacity" min="0"  max="1" step="0.05"/>
         <span class="layer-opacity-text"></span>
       </div>
@@ -43,12 +43,12 @@
       <div class="settings-item">
         <label>
           Seamless drawing mode
-          <input type="checkbox" value="1" class="tiled-preview-checkbox" name="tiled-preview-checkbox"/>
+          <input type="checkbox" value="1" class="seamless-mode-checkbox" name="seamless-mode-checkbox"/>
         </label>
       </div>
 
       <div class="settings-item">
-        <label for="tiled-preview">Maximum FPS</label>
+        <label>Maximum FPS</label>
         <input type="text" class="textfield  textfield-small  max-fps-input" autocomplete="off" name="max-fps"/>
       </div>