Migrate home toolbar button to be a reset button that also resets lay…

…er visibility
NCEAS · May 1, 2024 · c79cd42 · c79cd42
1 parent 5d616f9
commit c79cd42
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/js/models/maps/assets/MapAsset.js b/src/js/models/maps/assets/MapAsset.js
@@ -351,6 +351,7 @@ define([
             }
           }
 
+          this.set("originalVisibility", this.get("visible"));
           this.setListeners();
         } catch (e) {
           console.log("Error initializing a MapAsset model", e);
@@ -393,7 +394,6 @@ define([
        * @since 2.27.0
        */
       handleError: function () {
-        this.set("originalVisibility", this.get("visible"));
         this.set("visible", false);
         this.stopListening(this, "change:visible");
       },

diff --git a/src/js/views/maps/ToolbarView.js b/src/js/views/maps/ToolbarView.js
@@ -171,10 +171,15 @@ define(
             },
           },
           {
-            label: 'Home',
-            icon: 'home',
+            label: 'Reset',
+            icon: 'rotate-left',
             action: function (view, model) {
               model.flyHome();
+
+              // Reset the visibility of all layers.
+              for (const layer of model.getAllLayers()) {
+                layer.set("visible", layer.get('originalVisibility'));
+              }
             },
             isVisible(model) {
               return model.get("showHomeButton");