Responsive space tourisum website

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…

  • The explore circle have a hover state, which means it's clickable which leads to something. Since this project have multiple pages, at least one page should be used and let user explore from the main page. You could make this part a link...

  <div class="circle">
          <p>Explore</p>
   </div>
  
  Into:
    <a href="./destination.html" class="circle">Explore</a>
  

  • The hamburger menu is not functional. At least it should show a menu list

  Other than that...

  • The HTML structure is organized and proper usage of semantic tags
  • Great job in using Sass for styling with proper components and usage of properties
  • The project is responsive and have utilized the details based on the original design
  • Well use of the data in every page
  • The hover state of selections are well done
  • The alignments and spacing is also done well

  Above all, The project is awesome! Keep it up!

  Marked as helpful

  0

  ---

  

  T.Praveen•870

  @gwtpraveenPosted over 2 years ago

  @RioCantre thanks man i really appreciate it.

  • i forgot to add functionality to hamburger icon. i will fix that.

  0
• 

  Divine Obeten•2,415

  @Deevyn9Posted over 2 years ago

  This was represented beautifully, it would be great if it had more functionality

  0