rm activeSort from isMenuIconDirty for table

RoamJS · Dec 13, 2023 · ee00835 · ee00835
1 parent dd85cb3
commit ee00835
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/components/ResultsView.tsx b/src/components/ResultsView.tsx
@@ -352,16 +352,20 @@ const ResultsView: ResultsViewComponent = ({
   const [isEditColumnSort, setIsEditColumnSort] = useState(false);
   const [isEditColumnFilter, setIsEditColumnFilter] = useState(false);
   const [isEditSearchFilter, setIsEditSearchFilter] = useState(false);
-  const isMenuIconDirty = useMemo(
-    () =>
-      searchFilter || columnFilters.length || random.count || activeSort.length,
-    [searchFilter, columnFilters, random, activeSort]
-  );
+
   const [layout, setLayout] = useState(settings.layout);
   const layoutMode = useMemo(
     () => (Array.isArray(layout.mode) ? layout.mode[0] : layout.mode),
     [layout]
   );
+  const isMenuIconDirty = useMemo(
+    () =>
+      searchFilter ||
+      columnFilters.length ||
+      random.count ||
+      (activeSort.length && layout.mode !== "table"), // indicator is on ResultHeader
+    [searchFilter, columnFilters, random, activeSort, layout.mode]
+  );
   const onViewChange = (view: (typeof views)[number], i: number) => {
     const newViews = views.map((v, j) => (i === j ? view : v));
     setViews(newViews);