Easy bank landing page

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted about 2 years ago

  Looks great. You also might need to add cursor: pointer for your hamburger icon 🙌🏻

  2
• 

  Eugenia•950

  @JaneMorozPosted about 2 years ago

  Hey! Your solution to this challenge is great! 👍

  The only thing I've noticed:

  • You need to wrap the main content of the page into the <main> tag. It will solve all these landmark issues and improve accessibility.

  Keep it up! And good luck 🍀

  1
• 

  Bizzare•1,320

  @godmayhem7Posted about 2 years ago

  hey @PChaparro your code was absolutely amazing you did well using multiple frame works👏👏, but there was just something you didnt do well i checked out your solution on mobile view i found out that some of your elements was over your button and it made your website look kinda weird, i am sure that you must have forgotten to debug your code judging from the appearance of your website. i always advise people to at least try and debug their codes after committing their files to their repository this helps out in a lot of ways. this is a helpful tool to us as a developer, the rest of your code was okay 👌👌, hope this feedback was helpful 👍👍

  1

  ---

  

  Pedro Chaparro•260

  @PChaparroPosted about 2 years ago

  @godmayhem7 Thanks for your feedback, I appreciate it. ¿Can you tell me the device screen size and the page section (Navbar, hero...) in what you found the problem so I can fix it? I checked the responsive layout with the browser tool and a real device before submit this solution but maybe I missed that one problem 😟.

  0