Responsive landing page

Community feedback

• 

  Matias Ludueña•1,990

  @matiasluduena23

  Posted over 1 year ago

  Hi Anosike Obinna! Nice work!!! Just one recomendation that might make your code more easy to read.

  • try to use semantic html

  You can use your structure like this

  header
    logo
    hamburgerMenu
    nav
       links
    nav
  header
  
  

  • I think that the easy way to make you navigation works is using your position absolute in your ul video. And avoid width in px, use percentage instead

  .navbar {
  position:absolute;
  top: 100px;
  left:0;
  right:0;
  margin-inline: auto; (this center the element with the right and left in 0)
  width: 80% (play with this number )
  

  Good code!

  21

  Marked as helpful
• 

  Yoga Budiman•300

  @ygabdn29

  Posted over 1 year ago

  I'm assuming by navbar transition is when we are looking at the mobile size, then maybe you should try using transition: ?

  11

  Marked as helpful