fix(core): point selections when tooltip is disabled

uniqueId was missing when tooltip was disabled

packages/core/src/marks/mark.js

@@ -951,13 +951,12 @@ export default class Mark {
 
     /**
      * Returns true if this mark instance participates in picking.
-     *
-     * TODO: Check if tooltip is enabled,
-     * TODO: Check if selection (when it's implemented) is enabled
      */
     isPickingParticipant() {
-        // TODO: Should check encoding instead
-        if (this.properties.tooltip === null) {
+        if (
+            this.properties.tooltip === null &&
+            !this.unitView.paramMediator.hasPointSelections()
+        ) {
             // Disabled
             return false;
         }

packages/core/src/view/paramMediator.js

@@ -295,6 +295,29 @@ export default class ParamMediator {
         const fn = this.createExpression(expr);
         return fn();
     }
+
+    /**
+     * Returns true if this ParamMediator has any parameters that are point selections.
+     * Point selections necessitate the use of uniqueIds in the data.
+     *
+     * @returns {boolean}
+     */
+    hasPointSelections() {
+        for (const param of this.#paramConfigs.values()) {
+            if (isSelectionParameter(param)) {
+                const select = param.select;
+                if (isString(select)) {
+                    if (select == "point") {
+                        return true;
+                    }
+                } else if (select.type == "point") {
+                    return true;
+                }
+            }
+        }
+
+        return false;
+    }
 }
 
 /**

packages/core/src/view/paramMediator.test.js

@@ -238,3 +238,23 @@ test("activateExprRefProps", async () => {
         c: 2,
     });
 });
+
+describe("hasPointSelections()", () => {
+    test("false if there are no point selections", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", value: 42 });
+        expect(pm.hasPointSelections()).toBe(false);
+    });
+
+    test("true if there are point selections (1/2)", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", select: "point" });
+        expect(pm.hasPointSelections()).toBe(true);
+    });
+
+    test("true if there are point selections (2/2)", () => {
+        const pm = new ParamMediator();
+        pm.registerParam({ name: "foo", select: { type: "point" } });
+        expect(pm.hasPointSelections()).toBe(true);
+    });
+});