From 1705f12a400ac8202961a4af0193f7af8870e57f Mon Sep 17 00:00:00 2001
From: Manuel Trezza <5673677+mtrezza@users.noreply.github.com>
Date: Thu, 11 Sep 2025 01:45:33 +0200
Subject: [PATCH 1/5] feat

---
 .../BrowserFilter/FilterRow.react.js          | 30 +++++++++++++++++--
 .../PushAudiencesSelector.react.js            |  2 +-
 src/lib/Filters.js                            | 13 +++++---
 src/lib/queryFromFilters.js                   |  2 +-
 4 files changed, 39 insertions(+), 8 deletions(-)

diff --git a/src/components/BrowserFilter/FilterRow.react.js b/src/components/BrowserFilter/FilterRow.react.js
index cf86451037..71c0e2ef32 100644
--- a/src/components/BrowserFilter/FilterRow.react.js
+++ b/src/components/BrowserFilter/FilterRow.react.js
@@ -28,8 +28,33 @@ function compareValue(
   onKeyDown,
   active,
   parentContentId,
-  setFocus
+  setFocus,
+  currentConstraint
 ) {
+  if (currentConstraint === 'containedIn') {
+    return (
+      <input
+        type="text"
+        value={Array.isArray(value) ? JSON.stringify(value) : value || ''}
+        placeholder="[1, 2, 3]"
+        onChange={e => {
+          try {
+            const parsed = JSON.parse(e.target.value);
+            if (Array.isArray(parsed)) {
+              onChangeCompareTo(parsed);
+            } else {
+              onChangeCompareTo(e.target.value);
+            }
+          } catch {
+            onChangeCompareTo(e.target.value);
+          }
+        }}
+        onKeyDown={onKeyDown}
+        ref={setFocus}
+      />
+    );
+  }
+
   switch (info.type) {
     case null:
       return null;
@@ -223,7 +248,8 @@ const FilterRow = ({
         onKeyDown,
         active,
         parentContentId,
-        setFocus
+        setFocus,
+        currentConstraint
       )}
       <button type="button" className={styles.remove} onClick={onDeleteRow}>
         <Icon name="minus-solid" width={14} height={14} fill="rgba(0,0,0,0.4)" />
diff --git a/src/components/PushAudiencesSelector/PushAudiencesSelector.react.js b/src/components/PushAudiencesSelector/PushAudiencesSelector.react.js
index 970bde709a..7eb2761dff 100644
--- a/src/components/PushAudiencesSelector/PushAudiencesSelector.react.js
+++ b/src/components/PushAudiencesSelector/PushAudiencesSelector.react.js
@@ -24,7 +24,7 @@ const PushAudiencesOptions = ({ current, onChange, onEditAudience, schema, audie
             fromJS({
               field: 'deviceType',
               constraint: 'containedIn',
-              array: query.deviceType['$in'],
+              compareTo: query.deviceType['$in'],
             })
           )
           : query;
diff --git a/src/lib/Filters.js b/src/lib/Filters.js
index 6947c0556b..4c8ddf7f2f 100644
--- a/src/lib/Filters.js
+++ b/src/lib/Filters.js
@@ -175,13 +175,17 @@ export const Constraints = {
     field: null,
     comparable: false,
   },
+  containedIn: {
+    name: 'contained in',
+    comparable: true,
+  },
 };
 
 export const FieldConstraints = {
-  Pointer: ['exists', 'dne', 'eq', 'neq', 'starts', 'unique'],
-  Boolean: ['exists', 'dne', 'eq', 'neq', 'unique'],
-  Number: ['exists', 'dne', 'eq', 'neq', 'lt', 'lte', 'gt', 'gte', 'unique'],
-  String: ['exists', 'dne', 'eq', 'neq', 'starts', 'ends', 'stringContainsString', 'unique'],
+  Pointer: ['exists', 'dne', 'eq', 'neq', 'starts', 'containedIn', 'unique'],
+  Boolean: ['exists', 'dne', 'eq', 'neq', 'containedIn', 'unique'],
+  Number: ['exists', 'dne', 'eq', 'neq', 'lt', 'lte', 'gt', 'gte', 'containedIn', 'unique'],
+  String: ['exists', 'dne', 'eq', 'neq', 'starts', 'ends', 'stringContainsString', 'containedIn', 'unique'],
   Date: [
     'exists',
     'dne',
@@ -189,6 +193,7 @@ export const FieldConstraints = {
     'onOrBefore',
     'after',
     'onOrAfter',
+    'containedIn',
     'unique',
   ],
   Object: [
diff --git a/src/lib/queryFromFilters.js b/src/lib/queryFromFilters.js
index ea4c49a591..24d5c729de 100644
--- a/src/lib/queryFromFilters.js
+++ b/src/lib/queryFromFilters.js
@@ -166,7 +166,7 @@ function addConstraint(query, filter) {
       query.notEqualTo(filter.get('field'), filter.get('compareTo'));
       break;
     case 'containedIn':
-      query.containedIn(filter.get('field'), filter.get('array'));
+      query.containedIn(filter.get('field'), filter.get('compareTo'));
       break;
     case 'stringContainsString':
       query.matches(filter.get('field'), filter.get('compareTo'), 'i');

From 4080349f0ae75cd7bc2ec991d896488b12327899 Mon Sep 17 00:00:00 2001
From: Manuel Trezza <5673677+mtrezza@users.noreply.github.com>
Date: Thu, 11 Sep 2025 01:57:26 +0200
Subject: [PATCH 2/5] view link

---
 src/dashboard/Data/Views/Views.react.js | 79 +++++++++++--------------
 1 file changed, 34 insertions(+), 45 deletions(-)

diff --git a/src/dashboard/Data/Views/Views.react.js b/src/dashboard/Data/Views/Views.react.js
index c2b63df30b..186e32ced9 100644
--- a/src/dashboard/Data/Views/Views.react.js
+++ b/src/dashboard/Data/Views/Views.react.js
@@ -572,8 +572,8 @@ class Views extends TableView {
                   // Remove focus after action to follow UX best practices
                   e.currentTarget.blur();
                 }}
-                aria-label={`Open all pointers in ${name} column in new tabs`}
-                title="Open all pointers in new tabs"
+                aria-label={`Filter to show all pointers from ${name} column`}
+                title="Filter to show all pointers from this column"
               >
                 <Icon
                   name="right-outline"
@@ -870,59 +870,48 @@ class Views extends TableView {
       .map(row => row[columnName])
       .filter(value => value && value.__type === 'Pointer' && value.className && value.objectId);
 
-    // Open each unique pointer in a new tab
-    const uniquePointers = new Map();
+    if (pointers.length === 0) {
+      this.showNote('No pointers found in this column', true);
+      return;
+    }
+
+    // Group pointers by target class
+    const pointersByClass = new Map();
     pointers.forEach(pointer => {
-      // Use a more collision-proof key format with explicit separators
-      const key = `className:${pointer.className}|objectId:${pointer.objectId}`;
-      if (!uniquePointers.has(key)) {
-        uniquePointers.set(key, pointer);
+      if (!pointersByClass.has(pointer.className)) {
+        pointersByClass.set(pointer.className, new Set());
       }
+      pointersByClass.get(pointer.className).add(pointer.objectId);
     });
 
-    if (uniquePointers.size === 0) {
-      this.showNote('No pointers found in this column', true);
+    // If multiple target classes, show error
+    if (pointersByClass.size > 1) {
+      const classNames = Array.from(pointersByClass.keys()).join(', ');
+      this.showNote(`Cannot filter pointers from multiple classes: ${classNames}. Please use this feature on columns with pointers to a single class.`, true);
       return;
     }
 
-    const pointersArray = Array.from(uniquePointers.values());
-
-    // Confirm for large numbers of tabs to prevent overwhelming the user
-    if (pointersArray.length > 10) {
-      const confirmMessage = `This will open ${pointersArray.length} new tabs. This might overwhelm your browser. Continue?`;
-      if (!confirm(confirmMessage)) {
-        return;
-      }
-    }
+    // Get the single target class and unique object IDs
+    const targetClassName = Array.from(pointersByClass.keys())[0];
+    const uniqueObjectIds = Array.from(pointersByClass.get(targetClassName));
 
-    // Open all tabs immediately to maintain user activation context
-    let errorCount = 0;
+    // Navigate to the target class with containedIn filter
+    const filters = JSON.stringify([{
+      field: 'objectId',
+      constraint: 'containedIn',
+      compareTo: uniqueObjectIds
+    }]);
 
-    pointersArray.forEach((pointer) => {
-      try {
-        const filters = JSON.stringify([{ field: 'objectId', constraint: 'eq', compareTo: pointer.objectId }]);
-        const url = generatePath(
-          this.context,
-          `browser/${pointer.className}?filters=${encodeURIComponent(filters)}`,
-          true
-        );
-        window.open(url, '_blank', 'noopener,noreferrer');
-        // Note: window.open with security attributes may return null even when successful,
-        // so we assume success unless an exception is thrown
-      } catch (error) {
-        console.error('Failed to open tab for pointer:', pointer, error);
-        errorCount++;
-      }
-    });
+    const path = generatePath(
+      this.context,
+      `browser/${targetClassName}?filters=${encodeURIComponent(filters)}`,
+      true
+    );
 
-    // Show result notification
-    if (errorCount === 0) {
-      this.showNote(`Opened ${pointersArray.length} pointer${pointersArray.length > 1 ? 's' : ''} in new tab${pointersArray.length > 1 ? 's' : ''}`, false);
-    } else if (errorCount < pointersArray.length) {
-      this.showNote(`Opened ${pointersArray.length - errorCount} of ${pointersArray.length} tabs. ${errorCount} failed to open.`, true);
-    } else {
-      this.showNote('Unable to open tabs. Please allow popups for this site and try again.', true);
-    }
+    window.open(path, '_blank', 'noopener,noreferrer');
+    
+    // Show success notification
+    this.showNote(`Applied filter to show ${uniqueObjectIds.length} pointer${uniqueObjectIds.length > 1 ? 's' : ''} from ${targetClassName}`, false);
   }
 
   showNote(message, isError) {

From d5e2cf90b86ad2bd48d88409c62125384f66aeff Mon Sep 17 00:00:00 2001
From: Manuel Trezza <5673677+mtrezza@users.noreply.github.com>
Date: Thu, 11 Sep 2025 01:59:13 +0200
Subject: [PATCH 3/5] lint

---
 src/dashboard/Data/Views/Views.react.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/dashboard/Data/Views/Views.react.js b/src/dashboard/Data/Views/Views.react.js
index 186e32ced9..eb71a0c866 100644
--- a/src/dashboard/Data/Views/Views.react.js
+++ b/src/dashboard/Data/Views/Views.react.js
@@ -909,7 +909,7 @@ class Views extends TableView {
     );
 
     window.open(path, '_blank', 'noopener,noreferrer');
-    
+
     // Show success notification
     this.showNote(`Applied filter to show ${uniqueObjectIds.length} pointer${uniqueObjectIds.length > 1 ? 's' : ''} from ${targetClassName}`, false);
   }

From 4cce65a97b59f8dfbf7336613f6323b09dfe63dc Mon Sep 17 00:00:00 2001
From: Manuel Trezza <5673677+mtrezza@users.noreply.github.com>
Date: Thu, 11 Sep 2025 02:13:00 +0200
Subject: [PATCH 4/5] default

---
 src/components/Filter/Filter.react.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/components/Filter/Filter.react.js b/src/components/Filter/Filter.react.js
index 67592444ec..9d0947114a 100644
--- a/src/components/Filter/Filter.react.js
+++ b/src/components/Filter/Filter.react.js
@@ -62,7 +62,8 @@ function changeConstraint(schema, currentClassName, filters, index, newConstrain
     field: field,
     constraint: newConstraint,
     compareTo:
-      compareType && prevCompareTo ? prevCompareTo : Filters.DefaultComparisons[compareType],
+      compareType && prevCompareTo ? prevCompareTo : 
+      newConstraint === 'containedIn' ? [] : Filters.DefaultComparisons[compareType],
   });
   return filters.set(index, newFilter);
 }

From 5d193da7d13832d36c3405dd354af8d2f14d6957 Mon Sep 17 00:00:00 2001
From: Manuel Trezza <5673677+mtrezza@users.noreply.github.com>
Date: Thu, 11 Sep 2025 02:14:23 +0200
Subject: [PATCH 5/5] lint

---
 src/components/Filter/Filter.react.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/components/Filter/Filter.react.js b/src/components/Filter/Filter.react.js
index 9d0947114a..2bea58a078 100644
--- a/src/components/Filter/Filter.react.js
+++ b/src/components/Filter/Filter.react.js
@@ -62,8 +62,7 @@ function changeConstraint(schema, currentClassName, filters, index, newConstrain
     field: field,
     constraint: newConstraint,
     compareTo:
-      compareType && prevCompareTo ? prevCompareTo : 
-      newConstraint === 'containedIn' ? [] : Filters.DefaultComparisons[compareType],
+      compareType && prevCompareTo ? prevCompareTo : newConstraint === 'containedIn' ? [] : Filters.DefaultComparisons[compareType],
   });
   return filters.set(index, newFilter);
 }