Responsive landing page

Community feedback

• 

  Matias Ludueña•1,990

  @matiasluduena23Posted 12 months 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!

  Marked as helpful

  2

  ---

  

  Anosike Obinna•400

  @Smartlify08Posted 12 months ago

  @matiasluduena23 Thanks for the feedback. I'll try doing that in other projects too

  0
• 

  Yoga Budiman•300

  @ygabdn29Posted 12 months ago

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

  Marked as helpful

  1

  ---

  

  Anosike Obinna•400

  @Smartlify08Posted 12 months ago

  @ygabdn29 Okay I'll try doing that

  1