From e6497f31048e8a4ce753702c5ff01cbb3e0a8f90 Mon Sep 17 00:00:00 2001
From: johnny1093 <46250921+jsmorabito@users.noreply.github.com>
Date: Wed, 26 Aug 2026 10:11:10 -0400
Subject: [PATCH] ENG-2181 Uncheck node type filters by default, remove Only
 button

Nodes types now start unchecked; checking one filters to it directly,
so the panel no longer needs to translate an empty selection into an
all-checked display or offer a separate Only shortcut.
---
 .../src/components/NodeTypeFilterMenu.tsx     | 50 +++----------------
 .../src/utils/discourseNodeTypeFilter.ts      | 40 ++-------------
 2 files changed, 10 insertions(+), 80 deletions(-)

diff --git a/apps/obsidian/src/components/NodeTypeFilterMenu.tsx b/apps/obsidian/src/components/NodeTypeFilterMenu.tsx
index e6f9e95a6..e2f7a4cc0 100644
--- a/apps/obsidian/src/components/NodeTypeFilterMenu.tsx
+++ b/apps/obsidian/src/components/NodeTypeFilterMenu.tsx
@@ -6,25 +6,21 @@ import { getAllDiscourseNodeColors } from "~/utils/colorUtils";
 import {
   NODE_TYPE_FILTER_SEARCH_THRESHOLD,
   filterNodeTypesByQuery,
-  fromPanelSelectedIds,
   hasActiveTypeFilter,
-  toPanelSelectedIds,
 } from "~/utils/discourseNodeTypeFilter";
 
 const NodeTypeFilterRow = ({
   color,
   isChecked,
   nodeType,
-  onSelectOnly,
   onToggle,
 }: {
   color: string | undefined;
   isChecked: boolean;
   nodeType: DiscourseNode;
-  onSelectOnly: () => void;
   onToggle: () => void;
 }): ReactElement => (
-  <div className="hover:bg-modifier-hover group flex items-center gap-2 px-3 py-1.5">
+  <div className="hover:bg-modifier-hover flex items-center gap-2 px-3 py-1.5">
     <label className="flex min-w-0 flex-1 cursor-pointer items-center gap-2">
       <input
         type="checkbox"
@@ -40,19 +36,6 @@ const NodeTypeFilterRow = ({
       )}
       <span className="text-normal truncate text-sm">{nodeType.name}</span>
     </label>
-    <button
-      type="button"
-      // Hidden until the row is hovered or focused so the list stays scannable,
-      // but focusable by keyboard rather than pointer-only.
-      className="text-muted hover:text-normal shrink-0 opacity-0 transition-opacity group-focus-within:opacity-100 group-hover:opacity-100"
-      onClick={(event) => {
-        event.stopPropagation();
-        onSelectOnly();
-      }}
-      onMouseDown={(event) => event.preventDefault()}
-    >
-      Only
-    </button>
   </div>
 );
 
@@ -102,9 +85,7 @@ const NodeTypeFilterPanel = ({
   return (
     <>
       {/* Clearing is the only thing this control ever does, so it says so and
-          appears only when there is a filter to clear. A "select all" checkbox
-          would sit checked-and-inert whenever no filter is active, since an empty
-          selection and a full one are the same state. */}
+          appears only when there is a filter to clear. */}
       {isFilterActive && (
         <div className="border-modifier-border border-b p-2">
           <button
@@ -141,7 +122,6 @@ const NodeTypeFilterPanel = ({
               color={colorsById.get(nodeType.id)}
               isChecked={selectedIdSet.has(nodeType.id)}
               nodeType={nodeType}
-              onSelectOnly={() => onSelectedIdsChange([nodeType.id])}
               onToggle={() => toggleType(nodeType.id)}
             />
           ))
@@ -166,23 +146,9 @@ export const NodeTypeFilterMenu = ({
   onSelectedNodeTypeIdsChange: (ids: string[]) => void;
   selectedNodeTypeIds: string[];
 }): ReactElement => {
-  const allTypeIds = useMemo(
-    () => nodeTypes.map((nodeType) => nodeType.id),
-    [nodeTypes],
-  );
-
-  const isFilterActive = hasActiveTypeFilter({
-    selectedTypeIds: selectedNodeTypeIds,
-    allTypeIds,
-  });
-
-  const panelSelectedIds = useMemo(
-    () =>
-      toPanelSelectedIds({ selectedTypeIds: selectedNodeTypeIds, allTypeIds }),
-    [allTypeIds, selectedNodeTypeIds],
-  );
+  const isFilterActive = hasActiveTypeFilter(selectedNodeTypeIds);
 
-  const activeFilterCount = isFilterActive ? selectedNodeTypeIds.length : 0;
+  const activeFilterCount = selectedNodeTypeIds.length;
 
   return (
     <SearchDropdown
@@ -208,12 +174,8 @@ export const NodeTypeFilterMenu = ({
       <NodeTypeFilterPanel
         isFilterActive={isFilterActive}
         nodeTypes={nodeTypes}
-        onSelectedIdsChange={(panelIds) =>
-          onSelectedNodeTypeIdsChange(
-            fromPanelSelectedIds({ panelSelectedIds: panelIds, allTypeIds }),
-          )
-        }
-        selectedIds={panelSelectedIds}
+        onSelectedIdsChange={onSelectedNodeTypeIdsChange}
+        selectedIds={selectedNodeTypeIds}
       />
     </SearchDropdown>
   );
diff --git a/apps/obsidian/src/utils/discourseNodeTypeFilter.ts b/apps/obsidian/src/utils/discourseNodeTypeFilter.ts
index 7a54f5006..83f37b4f6 100644
--- a/apps/obsidian/src/utils/discourseNodeTypeFilter.ts
+++ b/apps/obsidian/src/utils/discourseNodeTypeFilter.ts
@@ -2,48 +2,16 @@ import { DiscourseNode } from "~/types";
 
 /**
  * Node type filtering for the search modal. An empty `selectedTypeIds` means no
- * filter, matching `filterCandidatesByNodeTypeIds` in QueryEngine — so "nothing
- * selected" and "every type selected" are the same state and both show all nodes.
+ * filter, matching `filterCandidatesByNodeTypeIds` in QueryEngine — so nothing
+ * checked shows every node, and checking a type narrows results to it.
  * Ported from Roam's advanced search so both apps filter alike.
  */
 
 /** Type count above which the panel adds a search box; the modal is desktop-only. */
 export const NODE_TYPE_FILTER_SEARCH_THRESHOLD = 7;
 
-export const hasActiveTypeFilter = ({
-  selectedTypeIds,
-  allTypeIds,
-}: {
-  selectedTypeIds: string[];
-  allTypeIds: string[];
-}): boolean =>
-  selectedTypeIds.length > 0 && selectedTypeIds.length < allTypeIds.length;
-
-/** Shows no-filter as every row checked, so the panel is never an empty checklist. */
-export const toPanelSelectedIds = ({
-  selectedTypeIds,
-  allTypeIds,
-}: {
-  selectedTypeIds: string[];
-  allTypeIds: string[];
-}): string[] => (selectedTypeIds.length === 0 ? allTypeIds : selectedTypeIds);
-
-/** Collapses both "all checked" and "none checked" back to no filter. */
-export const fromPanelSelectedIds = ({
-  panelSelectedIds,
-  allTypeIds,
-}: {
-  panelSelectedIds: string[];
-  allTypeIds: string[];
-}): string[] => {
-  if (
-    panelSelectedIds.length === 0 ||
-    panelSelectedIds.length === allTypeIds.length
-  ) {
-    return [];
-  }
-  return panelSelectedIds;
-};
+export const hasActiveTypeFilter = (selectedTypeIds: string[]): boolean =>
+  selectedTypeIds.length > 0;
 
 export const filterNodeTypesByQuery = (
   nodeTypes: DiscourseNode[],