celestial-chart 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.8.1
+
+### Fixed
+
+- Layers added with `Celestial.add()` could not draw. They reach the map through
+  the global `Celestial` object — `Celestial.container`, `.context`,
+  `.mapProjection` — which is how upstream's examples are written, and which
+  worked upstream because `this` inside `display()` *was* the global object.
+  In this fork the global receives that interface only after the constructor has
+  finished, while the zoom behaviour already triggers a redraw inside it. A
+  layer's `redraw` handler therefore ran against `Celestial.container === null`,
+  and the resulting exception aborted `display()` itself: the map was left
+  without its zoom, rotation and export interface. User layers are now held back
+  until the interface is published, and drawn immediately afterwards. Covered by
+  a browser assertion in `harness/verify.mjs`.
+
 ## 0.8.0 — first release of the fork
 
 Forked from [d3-celestial](https://github.com/ofrohn/d3-celestial) `0.7.35`

package/build/celestial.cjs

@@ -7143,7 +7143,7 @@ function geoZoom() {
 // src/core.js
 var Celestial = {
   // Must match the package version — guarded by test/package.test.mjs.
-  version: "0.8.0",
+  version: "0.8.1",
   container: null,
   data: []
 };
@@ -14485,6 +14485,7 @@ var SkyMap = class {
     var cfg, mapProjection, parentElement, zoom, map2, circle, daylight, starnames = {}, dsonames = {};
     var base = options && options.standalone ? settings : void 0;
     var instance = this;
+    var layersEnabled = false;
     function $(id2) {
       return document.querySelector(parentElement + " #" + id2);
     }
@@ -14972,7 +14973,7 @@ var SkyMap = class {
           }
         });
       }
-      if (Celestial.data.length > 0) {
+      if (layersEnabled && Celestial.data.length > 0) {
         Celestial.data.forEach(function(d) {
           d.redraw();
         });
@@ -15320,6 +15321,7 @@ var SkyMap = class {
       animate();
     };
     load();
+    layersEnabled = true;
   }
 };
 ["constellations", "constellation"].forEach(function(name_) {
@@ -15342,6 +15344,7 @@ Celestial.display = function(config) {
     if (key_ === "constructor" || key_ === "constellations" || key_ === "constellation") continue;
     Object.defineProperty(Celestial, key_, descriptors[key_]);
   }
+  instance.redraw();
   return instance;
 };
 

package/build/celestial.js

@@ -7123,7 +7123,7 @@
   // src/core.js
   var Celestial = {
     // Must match the package version — guarded by test/package.test.mjs.
-    version: "0.8.0",
+    version: "0.8.1",
     container: null,
     data: []
   };
@@ -14465,6 +14465,7 @@
       var cfg, mapProjection, parentElement, zoom, map2, circle, daylight, starnames = {}, dsonames = {};
       var base = options && options.standalone ? settings : void 0;
       var instance = this;
+      var layersEnabled = false;
       function $(id2) {
         return document.querySelector(parentElement + " #" + id2);
       }
@@ -14952,7 +14953,7 @@
             }
           });
         }
-        if (Celestial.data.length > 0) {
+        if (layersEnabled && Celestial.data.length > 0) {
           Celestial.data.forEach(function(d) {
             d.redraw();
           });
@@ -15300,6 +15301,7 @@
         animate();
       };
       load();
+      layersEnabled = true;
     }
   };
   ["constellations", "constellation"].forEach(function(name_) {
@@ -15322,6 +15324,7 @@
       if (key_ === "constructor" || key_ === "constellations" || key_ === "constellation") continue;
       Object.defineProperty(Celestial, key_, descriptors[key_]);
     }
+    instance.redraw();
     return instance;
   };