Skip to content
OverviewSolutions
11
Comments
4
Oladele Bidemi
@iamRed-One

All comments

  • Oladele Bidemi•230
    @iamRed-OnePosted over 1 year ago

    Hi there Tommy, Congrats 🎉 on completing the project

    i have some suggestions for you

    • instead of using sm:hidden for you image which was a great btw use the picture element. it has wide ranges as well that can help in future project learn more
    <picture>
<source media="(min-width: 45rem)" srcset="./images/image-product-desktop.jpg" />
<img src="./images/image-product-mobile.jpg" alt="product image" />
</picture>
    • Your button active state isn't quite close, mine wasn't exact but close enough, i suggest hovering of the color in vs code a palette normally shows up you can drag to get particular color of choice.

    Overall great solution ♥

    0
  • Oladele Bidemi•230
    @iamRed-OnePosted over 1 year ago

    Hi there Congratulations 🎉 on finishing the project.

    I can't help but notice this errors in the design

    • the mobile navigation does not close when you switch to a new page
    • the content is too close to the edges
    • The crew images Is overlapping the header title

    Is there a way you can fix that

    Overall i love your react representation of this project🤩.

    Awesome work sir 👌

    0
  • Oladele Bidemi•230
    @iamRed-OnePosted over 1 year ago

    good work bro/sis congrats on finishing the project we had the same idea on the border bottom on the header navigation 😊 those little animations is sweet

    1
  • Oladele Bidemi•230
    @iamRed-OnePosted over 1 year ago

    i love this bro just followed you in linkedin

    1