Intro page with dropdown menus

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,860

  @catherineisonline

  Posted over 2 years ago

  Looks cool. 😎 Instead of creating a background color div, you can simply apply the color to the body. The logo icon ideally should be inside the navigation or header, not just a separate entity. Same for the separate image in the main tag, you can make the image part of the article because it's part of it. Also, for the desktop, when I click on the "Company" menu it affects the position of the positions of items on the right, might want to fix that part.

  I hope this helps 😊

  12

  Marked as helpful
• 

  Yehan Nilanga•670

  @Yehan20

  Posted over 2 years ago

  Hello there you solution looks awesome and clean , congratulations. For the register and learn more buttons , I think having an <a> tag would be fine because in a real use case it will be some sort of link. instead of the button. and if you have decided to make the header fixed , try to make the nav bar fixed as well so it will better. Also check out on the html report. include and alt tag on the image elements. Good Luck in other challenges as well.

  02

  Marked as helpful
• 

  yishak abrham•2,150

  @yishak621

  Posted over 2 years ago

  i cant scroll down maybe because u set height100vh or something change it to min-height so that the overflow can be set as scroll and also in the mobile version the dropdown links shouldnt have any background color so fix that beside everything is good i love the transitions

  01

  Marked as helpful