Merge branch 'feat/use-motion-presence-hook' into feat/add-motion-to-…

…drawer

* feat/use-motion-presence-hook:
  fix: wrong merge
  fix: remove ref from dependencies
  fix: generate new API documentation
  fix: revert changes to CODEOWNERS
  fix: remove duplicated code due to a merge conflict

.github/CODEOWNERS

@@ -239,7 +239,6 @@ packages/react-components/react-jsx-runtime @microsoft/teams-prg
 packages/react-components/react-toast @microsoft/teams-prg
 packages/react-components/react-search-preview @microsoft/cxe-coastal
 packages/react-components/react-colorpicker-compat @microsoft/cxe-red @sopranopillow
-packages/react-components/react-motion-preview @microsoft/cxe-prg @marcosmoura
 packages/react-components/react-nav-preview @microsoft/cxe-red @mltejera
 packages/react-components/react-motion-preview @microsoft/cxe-prg @marcosmoura
 # <%= NX-CODEOWNER-PLACEHOLDER %>

packages/react-components/react-motion-preview/etc/react-motion-preview.api.md

@@ -29,7 +29,7 @@ export type MotionState<Element extends HTMLElement = HTMLElement> = {
 };
 
 // @public (undocumented)
-export type MotionType = 'unmounted' | 'entering' | 'entered' | 'idle' | 'exiting' | 'exited';
+export type MotionType = 'entering' | 'entered' | 'idle' | 'exiting' | 'exited' | 'unmounted';
 
 // @public
 export function useMotion<Element extends HTMLElement>(shorthand: MotionShorthand<Element>, options?: MotionOptions): MotionState<Element>;

packages/react-components/react-motion-preview/src/hooks/useMotion.ts

@@ -182,7 +182,9 @@ function useMotionPresence<Element extends HTMLElement>(
       active,
       canRender: type !== 'unmounted',
     }),
-    [active, ref, type],
+    // No need to add ref to the deps array as it is stable
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [active, type],
   );
 }