From c6f8dd4c5408af66da083ce449b45f4b262f91f0 Mon Sep 17 00:00:00 2001 From: Maria Hutt Date: Fri, 13 Jun 2025 13:46:41 -0700 Subject: [PATCH] refactor(many): replace `render` with `element` --- .../pages/dynamic-routes/DynamicRoutes.tsx | 18 +++++------ .../src/pages/dynamic-tabs/DynamicTabs.tsx | 2 +- .../src/pages/muiltiple-tabs/MultipleTabs.tsx | 20 +++++------- .../src/pages/nested-outlet/NestedOutlet.tsx | 2 +- .../src/pages/nested-outlet/NestedOutlet2.tsx | 2 +- .../base/src/pages/outlet-ref/OutletRef.tsx | 4 +-- .../test/base/src/pages/refs/Refs.tsx | 4 +-- .../base/src/pages/replace-action/Replace.tsx | 2 +- .../test/base/src/pages/routing/OtherPage.tsx | 4 +-- .../test/base/src/pages/routing/PropsTest.tsx | 2 +- .../test/base/src/pages/routing/Routing.tsx | 31 ++++++------------- .../test/base/src/pages/routing/Tabs.tsx | 10 +++--- 12 files changed, 41 insertions(+), 60 deletions(-) diff --git a/packages/react-router/test/base/src/pages/dynamic-routes/DynamicRoutes.tsx b/packages/react-router/test/base/src/pages/dynamic-routes/DynamicRoutes.tsx index 50f5ce736b..a5c4c37fdb 100644 --- a/packages/react-router/test/base/src/pages/dynamic-routes/DynamicRoutes.tsx +++ b/packages/react-router/test/base/src/pages/dynamic-routes/DynamicRoutes.tsx @@ -11,14 +11,6 @@ import { Route, Navigate } from 'react-router'; import { Link } from 'react-router-dom'; const DynamicRoutes: React.FC = () => { - const [routes, setRoutes] = useState([ - } - />, - ]); - const addRoute = () => { const newRoute = ( } /> @@ -26,12 +18,20 @@ const DynamicRoutes: React.FC = () => { setRoutes([...routes, newRoute]); }; + const [routes, setRoutes] = useState([ + } + />, + ]); + return ( {routes} {/* } /> */} } /> - } /> + } /> ); }; diff --git a/packages/react-router/test/base/src/pages/dynamic-tabs/DynamicTabs.tsx b/packages/react-router/test/base/src/pages/dynamic-tabs/DynamicTabs.tsx index df174cc0ea..ac353fa718 100644 --- a/packages/react-router/test/base/src/pages/dynamic-tabs/DynamicTabs.tsx +++ b/packages/react-router/test/base/src/pages/dynamic-tabs/DynamicTabs.tsx @@ -40,7 +40,7 @@ const DynamicTabs: React.FC = () => { - + {render2ndTabRoute()} } /> } /> diff --git a/packages/react-router/test/base/src/pages/muiltiple-tabs/MultipleTabs.tsx b/packages/react-router/test/base/src/pages/muiltiple-tabs/MultipleTabs.tsx index ce612126c9..60b7696e26 100644 --- a/packages/react-router/test/base/src/pages/muiltiple-tabs/MultipleTabs.tsx +++ b/packages/react-router/test/base/src/pages/muiltiple-tabs/MultipleTabs.tsx @@ -27,15 +27,11 @@ const MultipleTabs: React.FC = () => { { - return ; - }} + element={} /> { - return ; - }} + element={} /> } /> @@ -62,11 +58,11 @@ const Tab1: React.FC = () => { } + element={} /> {/* } /> */} - } /> - } /> + } /> + } /> ); @@ -88,11 +84,11 @@ const Tab2: React.FC = () => { } + element={} /> {/* } /> */} - } /> - } /> + } /> + } /> ); diff --git a/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet.tsx b/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet.tsx index bac2799458..78b16f6873 100644 --- a/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet.tsx +++ b/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet.tsx @@ -48,7 +48,7 @@ const SecondPage: React.FC = () => { } + element={} /> } /> diff --git a/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet2.tsx b/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet2.tsx index 1b101ae46e..7ce1b03de7 100644 --- a/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet2.tsx +++ b/packages/react-router/test/base/src/pages/nested-outlet/NestedOutlet2.tsx @@ -126,7 +126,7 @@ const NestedOutlet2: React.FC = () => ( } /> } + element={} /> ); diff --git a/packages/react-router/test/base/src/pages/outlet-ref/OutletRef.tsx b/packages/react-router/test/base/src/pages/outlet-ref/OutletRef.tsx index 9aa3e9e0ba..e8ac23b65c 100644 --- a/packages/react-router/test/base/src/pages/outlet-ref/OutletRef.tsx +++ b/packages/react-router/test/base/src/pages/outlet-ref/OutletRef.tsx @@ -22,9 +22,7 @@ export const OutletRef: React.FC = () => { { - return
; - }} + element={
} /> ); diff --git a/packages/react-router/test/base/src/pages/refs/Refs.tsx b/packages/react-router/test/base/src/pages/refs/Refs.tsx index 4385fe8281..d8225612bb 100644 --- a/packages/react-router/test/base/src/pages/refs/Refs.tsx +++ b/packages/react-router/test/base/src/pages/refs/Refs.tsx @@ -14,9 +14,9 @@ interface RefsProps {} const Refs: React.FC = () => { return ( - {/* } /> */} + {/* } /> */} } /> - } /> + } /> ); }; diff --git a/packages/react-router/test/base/src/pages/replace-action/Replace.tsx b/packages/react-router/test/base/src/pages/replace-action/Replace.tsx index 69a52e3f51..ccf3e874bf 100644 --- a/packages/react-router/test/base/src/pages/replace-action/Replace.tsx +++ b/packages/react-router/test/base/src/pages/replace-action/Replace.tsx @@ -20,7 +20,7 @@ const ReplaceAction: React.FC = () => { } /> } /> } /> - } /> + } /> ); }; diff --git a/packages/react-router/test/base/src/pages/routing/OtherPage.tsx b/packages/react-router/test/base/src/pages/routing/OtherPage.tsx index 6f1138329f..3370774c08 100644 --- a/packages/react-router/test/base/src/pages/routing/OtherPage.tsx +++ b/packages/react-router/test/base/src/pages/routing/OtherPage.tsx @@ -25,7 +25,7 @@ const OtherPage: React.FC = () => { return ( // - // ( + // @@ -39,7 +39,7 @@ const OtherPage: React.FC = () => { Go to tab3 - // )}> + // }> // ); }; diff --git a/packages/react-router/test/base/src/pages/routing/PropsTest.tsx b/packages/react-router/test/base/src/pages/routing/PropsTest.tsx index d8cc386a1b..b8808a7312 100644 --- a/packages/react-router/test/base/src/pages/routing/PropsTest.tsx +++ b/packages/react-router/test/base/src/pages/routing/PropsTest.tsx @@ -21,7 +21,7 @@ const PropsTest: React.FC = () => { } + element={} /> ); diff --git a/packages/react-router/test/base/src/pages/routing/Routing.tsx b/packages/react-router/test/base/src/pages/routing/Routing.tsx index ee13845b88..6ba605a854 100644 --- a/packages/react-router/test/base/src/pages/routing/Routing.tsx +++ b/packages/react-router/test/base/src/pages/routing/Routing.tsx @@ -20,38 +20,27 @@ const Routing: React.FC = () => { - } /> + } /> {/* } /> */} - } /> + } /> } /> - {/* { - return ( - - } /> - - ); - }} /> */} - {/* { - return ( - - } /> - - ); - }} /> */} + {/* } />} /> */} + {/* } />} /> */} } /> } /> - } /> - } /> + } /> + } /> ( + path="*" + element={
Not found
- )} + } /> - {/* } /> */} + {/* } /> */} ); diff --git a/packages/react-router/test/base/src/pages/routing/Tabs.tsx b/packages/react-router/test/base/src/pages/routing/Tabs.tsx index 115f706904..8db7b77efc 100644 --- a/packages/react-router/test/base/src/pages/routing/Tabs.tsx +++ b/packages/react-router/test/base/src/pages/routing/Tabs.tsx @@ -16,21 +16,19 @@ const Tabs: React.FC = () => { } /> } /> - {/* { - return
- }} /> */} + {/* } /> */} } /> } /> } /> } + element={} /> } + element={} /> - {/* } /> */} + {/* } /> */}