pptx-react-viewer 4.52.0 → 4.52.1

package/CHANGELOG.md

@@ -7,6 +7,17 @@ A release listed with no entries carried no Conventional Commit in this package'
 scope: scripts/release-plan.mjs releases whenever any published file changes, not only on
 conventional commits.
 
+## [4.52.1](https://github.com/ChristopherVR/ooxml/releases/tag/pptx-react-viewer@4.52.1) - 2026-10-25
+
+### Bug Fixes
+
+- **pptx:** Preview colour drags as one undo step ([#203](https://github.com/ChristopherVR/ooxml/issues/203)) ([27cc512](https://github.com/ChristopherVR/ooxml/commit/27cc5126cee1c351069e8eb7c2dafe4a0d7806a1))
+- **pptx:** Save hyperlinks to their current targets ([#204](https://github.com/ChristopherVR/ooxml/issues/204)) ([371fe79](https://github.com/ChristopherVR/ooxml/commit/371fe799d88e4158fb7ed895a1e32b0c9b67682e))
+
+### Refactor
+
+- **ui:** Run pptx find and replace on one shared session ([#202](https://github.com/ChristopherVR/ooxml/issues/202)) ([61ff51e](https://github.com/ChristopherVR/ooxml/commit/61ff51e2710edad14212132b475652b27bb05d43))
+
 ## [4.52.0](https://github.com/ChristopherVR/ooxml/releases/tag/pptx-react-viewer@4.52.0) - 2026-10-25
 
 ### Features

package/dist/{chunk-6GAWXARM.js → chunk-2AM5YYWI.js}

@@ -731,7 +731,11 @@ function useEditorHistory(input) {
   React17.useEffect(() => {
     onDirtyRef.current = onDirty;
   }, [onDirty]);
+  const untrackedDepthRef = React17.useRef(0);
   const markDirty = React17.useCallback(() => {
+    if (untrackedDepthRef.current > 0) {
+      return;
+    }
     setEditCommitNonce((previous) => previous + 1);
     onDirtyRef.current?.();
   }, []);
@@ -748,6 +752,14 @@ function useEditorHistory(input) {
     },
     []
   );
+  const withoutUndoStep = React17.useCallback((apply) => {
+    untrackedDepthRef.current += 1;
+    try {
+      apply();
+    } finally {
+      untrackedDepthRef.current -= 1;
+    }
+  }, []);
   const handleUndo = React17.useCallback(() => {
     if (!history.getSnapshot().canUndo) {
       return;
@@ -906,6 +918,7 @@ function useEditorHistory(input) {
     resetHistory,
     forgetHistory: history.forget,
     markDirty,
+    withoutUndoStep,
     buildHistorySnapshot
   };
 }
@@ -3204,109 +3217,62 @@ function usePresentationAnnotations(input) {
 function useFindReplace({
   slides,
   mode,
-  onSetActiveSlideIndex,
-  onSetSelectedElementId,
+  onShowMatch,
   onUpdateSlides,
   onMarkDirty,
   keyboard
 }) {
   const [findReplaceOpen, setFindReplaceOpen] = React17.useState(false);
-  const [findQuery, setFindQuery] = React17.useState("");
-  const [replaceQuery, setReplaceQuery] = React17.useState("");
-  const [findMatchCase, setFindMatchCase] = React17.useState(false);
-  const [findIncludeNotes, setFindIncludeNotes] = React17.useState(false);
-  const [findIncludeChartTitles, setFindIncludeChartTitles] = React17.useState(true);
-  const [findIncludeChartData, setFindIncludeChartData] = React17.useState(false);
-  const [findResults, setFindResults] = React17.useState([]);
-  const [findResultIndex, setFindResultIndex] = React17.useState(-1);
-  const pendingRefindRef = React17.useRef(false);
-  React17.useEffect(() => {
-    if (findReplaceOpen) {
-      return;
-    }
-    setFindResults([]);
-    setFindResultIndex(-1);
-    pendingRefindRef.current = false;
-  }, [findReplaceOpen]);
-  const performFind = React17.useCallback(() => {
-    const results = pptx.findInSlides(slides, findQuery, {
-      matchCase: findMatchCase,
-      includeNotes: findIncludeNotes,
-      includeChartTitles: findIncludeChartTitles,
-      includeChartData: findIncludeChartData
-    });
-    if (!findQuery) {
-      setFindResults([]);
-      setFindResultIndex(-1);
-      return;
-    }
-    setFindResults(results);
-    setFindResultIndex(results.length > 0 ? 0 : -1);
-    if (results.length > 0) {
-      onSetActiveSlideIndex(results[0].slideIndex);
-      onSetSelectedElementId(results[0].elementId);
-    }
-  }, [
-    slides,
-    findQuery,
-    findMatchCase,
-    findIncludeNotes,
-    findIncludeChartTitles,
-    findIncludeChartData,
-    onSetActiveSlideIndex,
-    onSetSelectedElementId
-  ]);
-  const navigateFindResult = React17.useCallback(
-    (direction) => {
-      if (findResults.length === 0) {
-        return;
-      }
-      const nextIdx = (findResultIndex + direction + findResults.length) % findResults.length;
-      setFindResultIndex(nextIdx);
-      const match = findResults[nextIdx];
-      if (match) {
-        onSetActiveSlideIndex(match.slideIndex);
-        onSetSelectedElementId(match.elementId);
-      }
-    },
-    [findResults, findResultIndex, onSetActiveSlideIndex, onSetSelectedElementId]
-  );
-  const applyReplacements = React17.useCallback(
-    (toReplace) => {
-      if (toReplace.length === 0) {
-        return;
-      }
-      if (toReplace.some((match) => match.readOnly)) {
-        return;
+  const [session, setSession] = React17.useState(pptx.INITIAL_FIND_REPLACE_SESSION);
+  const latest = React17.useRef({ slides, onShowMatch, onUpdateSlides, onMarkDirty });
+  latest.current = { slides, onShowMatch, onUpdateSlides, onMarkDirty };
+  const latestSession = React17.useRef(session);
+  const [actions] = React17.useState(
+    () => pptx.findReplaceSessionActions(
+      {
+        get: () => latestSession.current,
+        set: (next) => {
+          latestSession.current = next;
+          setSession(next);
+        }
+      },
+      {
+        slides: () => latest.current.slides,
+        commit: (next) => {
+          latest.current.onUpdateSlides(() => next);
+          latest.current.onMarkDirty();
+        },
+        navigate: ({ slideIndex, elementId }) => latest.current.onShowMatch(slideIndex, elementId)
       }
-      onUpdateSlides(
-        (prevSlides) => pptx.applyFindReplacements(prevSlides, toReplace, replaceQuery).slides
-      );
-      onMarkDirty();
-    },
-    [replaceQuery, onUpdateSlides, onMarkDirty]
+    )
   );
-  const handleReplace = React17.useCallback(() => {
-    if (findResults.length === 0 || findResultIndex < 0) {
-      return;
-    }
-    applyReplacements([findResults[findResultIndex]]);
-    pendingRefindRef.current = true;
-  }, [findResults, findResultIndex, applyReplacements]);
-  const handleReplaceAll = React17.useCallback(() => {
-    if (findResults.length === 0) {
-      return;
-    }
-    applyReplacements(findResults);
-    pendingRefindRef.current = true;
-  }, [findResults, applyReplacements]);
   React17.useEffect(() => {
-    if (!pendingRefindRef.current) {
-      return;
+    if (!findReplaceOpen) {
+      actions.close();
     }
-    pendingRefindRef.current = false;
-    performFind();
-  }, [performFind]);
+  }, [findReplaceOpen, actions]);
+  React17.useEffect(() => {
+    actions.slidesChanged();
+  }, [slides, actions]);
+  const handlers = React17.useMemo(
+    () => ({
+      setFindQuery: (query) => actions.search({ query }),
+      setReplaceQuery: actions.setReplacement,
+      setFindMatchCase: (matchCase) => actions.search({ options: { matchCase } }),
+      setFindIncludeNotes: (includeNotes) => actions.search({ options: { includeNotes } }),
+      setFindIncludeChartTitles: (includeChartTitles) => actions.search({ options: { includeChartTitles } }),
+      setFindIncludeChartData: (includeChartData) => actions.search({ options: { includeChartData } }),
+      performFind: () => actions.search(),
+      navigateFindResult: (direction) => direction > 0 ? actions.next() : actions.previous(),
+      handleReplace: () => {
+        actions.replaceCurrent();
+      },
+      handleReplaceAll: () => {
+        actions.replaceAll();
+      }
+    }),
+    [actions]
+  );
   React17.useEffect(() => {
     const handler = (event) => {
       const { action } = pptx.mapCustomizedEditorKey(
@@ -3329,24 +3295,15 @@ function useFindReplace({
   return {
     findReplaceOpen,
     setFindReplaceOpen,
-    findQuery,
-    setFindQuery,
-    replaceQuery,
-    setReplaceQuery,
-    findMatchCase,
-    setFindMatchCase,
-    findIncludeNotes,
-    setFindIncludeNotes,
-    findIncludeChartTitles,
-    findIncludeChartData,
-    setFindIncludeChartTitles,
-    setFindIncludeChartData,
-    findResults,
-    findResultIndex,
-    performFind,
-    navigateFindResult,
-    handleReplace,
-    handleReplaceAll
+    findQuery: session.query,
+    replaceQuery: session.replacement,
+    findMatchCase: session.options.matchCase,
+    findIncludeNotes: session.options.includeNotes,
+    findIncludeChartTitles: session.options.includeChartTitles,
+    findIncludeChartData: session.options.includeChartData,
+    findResults: session.matches,
+    findResultIndex: session.current,
+    ...handlers
   };
 }
 function useModalDismissDrag(onClose, threshold = 120) {
@@ -4006,6 +3963,35 @@ function useInlineEditingState(onTextChange) {
     registerInlineEditReader
   };
 }
+function useSlideChangeSelection({
+  setActiveSlideIndex,
+  setSelectedElementId,
+  setSelectedElementIds
+}) {
+  const slideSelectionRef = React17.useRef(null);
+  const showSlideSelection = React17.useCallback(
+    (slideIndex, elementId) => {
+      slideSelectionRef.current = slideIndex;
+      setActiveSlideIndex(slideIndex);
+      setSelectedElementId(elementId);
+      setSelectedElementIds([]);
+    },
+    [setActiveSlideIndex, setSelectedElementId, setSelectedElementIds]
+  );
+  return { slideSelectionRef, showSlideSelection };
+}
+function useClearSelectionOnSlideChange(state, activeSlideIndex) {
+  React17.useEffect(() => {
+    const kept = state.slideSelectionRef.current === activeSlideIndex;
+    state.slideSelectionRef.current = null;
+    state.setInlineEditingElementId(null);
+    if (kept) {
+      return;
+    }
+    state.setSelectedElementId(null);
+    state.setSelectedElementIds([]);
+  }, [activeSlideIndex]);
+}
 function useSlideState() {
   const [renderedSlides, setRenderedSlides] = React17.useState([]);
   const slidesRef = React17.useRef(renderedSlides);
@@ -4045,6 +4031,11 @@ function useViewerCoreState(_input) {
   const [activeSlideIndex, setActiveSlideIndex] = React17.useState(0);
   const [selectedElementId, setSelectedElementId] = React17.useState(null);
   const [selectedElementIds, setSelectedElementIds] = React17.useState([]);
+  const slideChangeSelection = useSlideChangeSelection({
+    setActiveSlideIndex,
+    setSelectedElementId,
+    setSelectedElementIds
+  });
   const [isDirty, setIsDirty] = React17.useState(false);
   const autosaveDraftSinkRef = React17.useRef(null);
   const publishInlineText = React17.useCallback(
@@ -4148,6 +4139,7 @@ function useViewerCoreState(_input) {
     setSelectedElementId,
     selectedElementIds,
     setSelectedElementIds,
+    ...slideChangeSelection,
     isDirty,
     setIsDirty,
     editTemplateMode,
@@ -9801,8 +9793,7 @@ function useEditorOperations(input) {
   const findReplace = useFindReplace({
     slides,
     mode,
-    onSetActiveSlideIndex: state.setActiveSlideIndex,
-    onSetSelectedElementId: state.setSelectedElementId,
+    onShowMatch: state.showSlideSelection,
     onUpdateSlides: ops.updateSlides,
     onMarkDirty: history.markDirty,
     keyboard
@@ -12876,11 +12867,7 @@ function useViewerBuildingBlocksState(input) {
   });
   const readOnlyRec = useReadOnlyRecommendationState(content);
   const compatToastsState = useCompatibilityToastsState();
-  React17.useEffect(() => {
-    state.setSelectedElementId(null);
-    state.setSelectedElementIds([]);
-    state.setInlineEditingElementId(null);
-  }, [activeSlideIndex]);
+  useClearSelectionOnSlideChange(state, activeSlideIndex);
   const editorOps = useEditorOperations({
     state,
     history,
@@ -18763,6 +18750,7 @@ exports.useBroadcastFollower = useBroadcastFollower;
 exports.useCanvasInteractions = useCanvasInteractions;
 exports.useChartFormatContext = useChartFormatContext;
 exports.useChartPartSelection = useChartPartSelection;
+exports.useClearSelectionOnSlideChange = useClearSelectionOnSlideChange;
 exports.useClipboardHandlers = useClipboardHandlers;
 exports.useCollaboration = useCollaboration;
 exports.useCollaborationDocumentSync = useCollaborationDocumentSync;