Space_Tourism

Community feedback

• 

  IryDev•1,580

  @IryDevPosted about 1 year ago

  Hey @ahmedyasserdev, good job on completing this challenge😀

  I have some advice in order to improve your solution :

  • For the navbar component if you don't want to repeat your self you can initialize an object with all the links you need and then use the map function with react :

  REACT (example):

  const links = [
      {
        name: "home",
        to: "/",
      },
      {
        name: "destination",
        to: "/destination",
      },
      {
        name: "crew",
        to: "/crew",
      },
  
      {
        name: "technology",
        to: "/technology",
      },
    ];
  
  {links.map((link, index) => (
                <li key={index}>
                  <NavLink onClick={updateMenu} to={link.to}>
                    <span>0{index}</span> {link.name}
                  </NavLink>
                </li>
              ))}
  

  I hope you'll find this helpful😄

  0

  ---

  

  Ahmed Yasser•460

  @ahmedyasserdevPosted about 1 year ago

  Hey @IryDev , thank you it was very helpful , I really appreciate it

  0

  

  IryDev•1,580

  @IryDevPosted about 1 year ago

  @ahmedyasserdev no problem I'm glad you found this helpful

  Marked as helpful

  0