Skip to content
Submitted over 3 years ago

Blogr Page Using HTML, CSS, JS (grid and flexbox)

David Oluwatobi Ebenezer•235
@EbvidPro
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Blogr landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Hi guys, It really gave me some tough time, but it was worth it. It's fun learning. I need your feedback on

  1. Maintaining my Code.
  2. Making it shorter instead of repetition of codes.
  3. Advice to become a better web developer.

Thanks in Anticipation.

Community feedback

  • ApplePieGiraffe•30,545
    @ApplePieGiraffePosted over 3 years ago

    Greetings, David Oluwatobi Ebenezer! 👋

    Nice effort on this challenge! 👏

    A few things I think you can do to improve your solution are,

    • Decreasing the space a bit between the submenus in the header of the page.
    • Using absolute positioning for the mobile menu popup so that it doesn't get in the way of the layout of the rest of your site.
    • Making sure the submenus in the desktop layout still appear when you decrease the width of the screen, open/close the mobile menu, and increase the width of the screen again (for some reason, they disappear, then).
    • Wrapping the menu icons for the mobile menu in a <button> element (since it is a type of button and that's better for accessibility).

    Hope these few tips help. 🙂

    Keep coding (and happy coding, too)! 😁

    Marked as helpful
    1
  • David Oluwatobi Ebenezer•235
    @EbvidProPosted over 3 years ago

    Wow. This great. Thanks for taking your time to go through.

    0

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord