Skip to content
OverviewSolutions
21
Comments
6
Jack
@Jack-WebDev

All comments

  • Jack•490
    @Jack-WebDevPosted over 1 year ago

    Hey 7deniz

    Well done on finishing your project.

    In this pull request I made a few changes.

    Check out the recommendations I made for your HTML, mostly the naming On the CSS I noticed you didn't add any hover effects, so check out the recommendations on that too.

    I hope you enjoyed this project and will continue to code. Feel free to ask for help ;)

    Marked as helpful
    0
  • Jack•490
    @Jack-WebDevPosted over 1 year ago

    Hey Daisy Well done on completing the project.

    Check out the pull request I made on the updates I recommend ;)

    I hope you continue coding and enjoy the journey!

    PS: I use pull requests because I'm a fan of open source and this is my way of encouraging people to do it too.

    Marked as helpful
    0
  • Jack•490
    @Jack-WebDevPosted over 2 years ago

    Your project is working great, congratulations.

    But also consider doing media queries because on my phone it's looking horrible🤣🤣🤣🤣

    0
  • Jack•490
    @Jack-WebDevPosted over 2 years ago

    Hello👋🏾 Firstly...I liked your nav menu transition. Perhaps you could adjust the width. And I noticed your hero background isn't working (on mobile), maybe there was an issue with deployment.

    Overall, not bad👏🏾

    Marked as helpful
    0
  • Jack•490
    @Jack-WebDevPosted over 2 years ago

    Hey😁😁 The background image on mobile is not working. I just went to your github and it seems your css folder wasn't deployed

    Marked as helpful
    0
  • Jack•490
    @Jack-WebDevPosted over 2 years ago

    Hii, the mobile design is not working. Try playing around with some media queries

    Marked as helpful
    0