fix for null scales

Author: mbostock

src/legends.js

@@ -1,5 +1,5 @@
 import {rgb} from "d3";
-import {isObject} from "./options.js";
+import {isScaleOptions} from "./options.js";
 import {normalizeScale} from "./scales.js";
 import {legendRamp} from "./legends/ramp.js";
 import {legendSwatches, legendSymbols} from "./legends/swatches.js";
@@ -13,17 +13,17 @@ const legendRegistry = new Map([
 export function legend(options = {}) {
   for (const [key, value] of legendRegistry) {
     const scale = options[key];
-    if (isObject(scale)) { // e.g., ignore {color: "red"}
+    if (isScaleOptions(scale)) { // e.g., ignore {color: "red"}
       let hint;
       // For symbol legends, pass a hint to the symbol scale.
       if (key === "symbol") {
-        const {fill, stroke = fill === undefined && isObject(options.color) ? "color" : undefined} = options;
+        const {fill, stroke = fill === undefined && isScaleOptions(options.color) ? "color" : undefined} = options;
         hint = {fill, stroke};
       }
       return value(
         normalizeScale(key, scale, hint),
         legendOptions(scale, options),
-        key => isObject(options[key]) ? normalizeScale(key, options[key]) : null
+        key => isScaleOptions(options[key]) ? normalizeScale(key, options[key]) : null
       );
     }
   }

src/options.js

@@ -74,7 +74,16 @@ export function arrayify(data, type) {
 
 // Disambiguates an options object (e.g., {y: "x2"}) from a primitive value.
 export function isObject(option) {
-  return option && option.toString === objectToString;
+  return option ? option.toString === objectToString : false;
+}
+
+// Disambiguates a scale options object (e.g., {color: {type: "linear"}}) from
+// some other option (e.g., {color: "red"}). When creating standalone legends,
+// this is used to test whether a scale is defined; this should be consistent
+// with inferScaleType when there are no channels associated with the scale, and
+// if this returns true, then normalizeScale must return non-null.
+export function isScaleOptions(option) {
+  return isObject(option) && (option.type !== undefined || option.domain !== undefined);
 }
 
 // Disambiguates an options object (e.g., {y: "x2"}) from a channel value

test/legends/legends-test.js

@@ -5,3 +5,11 @@ it("Plot.legend({color: {type:'identity'}}) returns undefined", () => {
   const l = Plot.legend({color: {type: "identity"}});
   assert.strictEqual(l, undefined);
 });
+
+it("Plot.legend({}) throws an error", () => {
+  assert.throws(() => Plot.legend({}), /unknown legend type/);
+});
+
+it("Plot.legend({color: {}}) throws an error", () => {
+  assert.throws(() => Plot.legend({color: {}}), /unknown legend type/);
+});