Skip to content
OverviewSolutions
2
Comments
2
Varghab Shib
@Varghab

All comments

  • Bright Anua•170
    @prolixtrexSubmitted over 1 year ago

    I had a little bit of challenge building the hamburger menu for the mobile view especially on making it close when the use clicks outside of it and dimming the background of the page when the hamburger menu opens. Although I was able to work around it and get it to work well but I think there maybe be a better way of doing it.

    intro section built with html, css and javascript

    2
    Varghab Shib•50
    @VarghabPosted over 1 year ago

    You did great work 👏, however I have a few suggestions -

    1. Don't use both width and height properties for your images. Doing this will shrink your images.
    2. Use an additional wrapper div for your hero section and give flex property to that wrapper.
    3. Lastly, the site is not responsive for tablets. Try to make it responsive for tablets.
    1
  • Varghab Shib•50
    @VarghabPosted over 1 year ago

    There are a few suggestions from my side -

    📌 Decrease the transition duration for the Features and Company links on the navbar and also for the hamburger menu button.

    📌 For the mobile view, clients' icons are not centered, centering would make it look better.

    I have also completed the same project, feel free to review my project. Any advice or suggestions are welcome.

    0