ENH: add "show" option to map config

Summary

Add show option to map config.

Version-bumps the map to 0.4.1

Related to https://gitlab.indiscale.com/caosdb/customers/leibniz-zmt/management/-/issues/41

Focus

• The show option in the config defaults to false
• The show option is only being stored locally, when the user toggles the map (with toggle_map or show_map), otherwise the config applies.

Test Environment

Please test manually.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related issues
• Up-to-date CHANGELOG.md (or not necessary)
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md (or not necessary)
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there specifications? Are they satisfied?

For further good practices have a look at our review guidelines.

src/core/js/ext_map.js

@@ -25,7 +25,7 @@
 
 /**
  * @module caosdb_map
- * @version 0.4
+ * @version 0.4.1
  *
  * For displaying a geographical map which shows entities at their associated
  * geolocation.
@@ -34,7 +34,7 @@
  * `conf/ext/json/ext_map.json` and comply with the {@link MapConfig} type
  * which is described below.
  *
- * The current version is 0.4. It is not considered to be stable because
+ * The current version is 0.4.1. It is not considered to be stable because
  * implementation of the graticule still is not satisfactory.
  *
  * Apart from that the specification of the configuration and the
@@ -43,7 +43,7 @@
 var caosdb_map = new function () {
 
     var logger = log.getLogger("caosdb_map");
-    this.version = "0.4";
+    this.version = "0.4.1";
     this.dependencies = ["log", {
         "L": ["latlngGraticule", "Proj"]
     }, "navbar", "caosdb_utils"];
@@ -63,6 +63,9 @@ var caosdb_map = new function () {
      * integrated query generator.
      *
      * @typedef {object} MapConfig
+     * @property {boolean} [show=false] - show the map immediately when it is
+     *     ready. This configuration option is being stored locally for keeping
+     *     the map shown/hidden accross page reloads.
      * @property {string} version - the version of the map
      * @property {string} default_view - the view which is shown when the user
      *     opens the map and has no stored view from prior visits to the map.
@@ -336,6 +339,7 @@ var caosdb_map = new function () {
      */
     this._default_config = {
         "version": this.version,
+        "show": false,
         "datamodel": {
             "lat": "latitude",
             "lng": "longitude",
@@ -843,8 +847,10 @@ var caosdb_map = new function () {
          *
          * The map panel is the HTMLElement which contains the map. Is is
          * hidden or shown when the user clicks on the toggle map button.
+         *
+         * @param {boolean} show - show this panel immediately.
          */
-        this.init_map_panel = function () {
+        this.init_map_panel = function (show) {
             logger.trace("enter init_map_panel");
 
             // remove old
@@ -852,7 +858,9 @@ var caosdb_map = new function () {
 
             let panel = this.create_map_panel();
 
-            $(panel).hide();
+            if(!show) {
+              $(panel).hide();
+            }
             $('nav').first().after(panel);
 
             logger.trace("leave init_map_panel");
@@ -865,6 +873,7 @@ var caosdb_map = new function () {
          */
         this.toggle_map = function () {
             logger.trace("enter toggle_map");
+            sessionStorage["caosdb_map.show"] = !$(".caosdb-f-map-panel").is(":visible");
             $(".caosdb-f-map-panel").toggle(900, () => this
                 ._toggle_cb());
         }
@@ -872,6 +881,7 @@ var caosdb_map = new function () {
 
         this.show_map = function () {
             logger.trace("enter show_map");
+            sessionStorage["caosdb_map.show"] = "true";
             $(".caosdb-f-map-panel").show(900, () => this
                 ._toggle_cb());
         }
@@ -983,14 +993,18 @@ var caosdb_map = new function () {
                     return;
                 }
                 this.config = config;
+                var show_map = config.show;
+                if(sessionStorage["caosdb_map.show"]) {
+                  show_map = JSON.parse(sessionStorage["caosdb_map.show"]);
+                }
                 this.init_select_handler();
                 this.init_view_change_handler();
-                panel = this.init_map_panel();
+                panel = this.init_map_panel(show_map);
 
                 // TODO split in smaller pieces and move callback to separate function
                 this.change_map_view = (view) => {
                     if (this._map) {
-			this._map._container.remove();
+                        this._map._container.remove();
                         this._map.remove();
                     }