Audiophile-Ecommerce (React, React-Router, Redux-toolkit, SASS, Grid)

Community feedback

• 

  Stephen Addae•270

  @Stephen-Adom

  Posted over 2 years ago

  Hello David, just checked your site it looks good on desktop as well as mobile. The router link breaks when I refresh the page and also the user has to know which page he/she is on so you need to highlight the nav links as well. For the routing issue, I think you should define your routes in your index.js file and use outlet tag in your app.js file to define your root outlet path for your site pages

  So your app.js file will look like the function below;

  function App() { return ( <div className="App main-layout"> <Wrapper> <Overlay /> <Navbar /> <Outlet /> <Footer /> </Wrapper> </div> ); }

  And your index.js file will be as

  root.render( <React.StrictMode> <Provider store={store}> <PersistGate persistor={persistor}> <BrowserRouter> <Routes> <Route path="/" element={<App />}> <Route index=true element={<Home />} /> <Route path="category/:categoryName" element={<Category />} /> <Route path="product/:productSlug" element={<Product />} /> <Route path="checkout" element={<Checkout />} /> </Route> </Routes> </BrowserRouter> </PersistGate> </Provider> </React.StrictMode> );

  Hopes this helps :)

  12

  Marked as helpful
• 

  Stefan Kremenovic•225

  @kremenovic

  Posted over 2 years ago

  Hey David, I was checking your portfolio and bumped into this project. Looks really good, but I've found a small bug on the site.

  After ordering and if user doesn't close the popup (order confirmation) and goes to the homepage or any other page overlay is gonna stay active > https://prnt.sc/qgaj-84zB18B

  Best,

  Stefan

  01

  Marked as helpful