Skip to content
OverviewSolutions
22
Comments
2
Samuel Pletain
@samuelpletain

All comments

  • Kylee Bustamante•90
    @kyleebustamanteSubmitted over 2 years ago

    Could anyone help me with aligning the svgs to the the center in relation to the text to the right of them? As always if you see other areas of improvement feedback is welcome!

    NFT Product Card with CSS Flexbox

    4
    Samuel Pletain•390
    @samuelpletainPosted over 2 years ago

    Congrats on finishing this challenge Kylee! Your code looks clean and your solution looks great on both mobile and desktop views. Just a personal preference: I would add a margin-top when viewing on desktop to push your card off the top a bit. You can use the vertical-align property to adjust the vertical alignment of your svgs. https://developer.mozilla.org/fr/docs/Web/CSS/vertical-align

    Marked as helpful
    0
  • Samuel Pletain•390
    @samuelpletainPosted over 2 years ago

    Good job, it looks really good. Your animation for the hover is great and your code is really clean and organized.

    Here are a couple of things that could improve your app:

    • Your layout is not centered when the width is between 601px and 1199px
    • Your footer positioning is off is you resize the window vertically, overlapping the content if the height of the window is inferior than the size of the card
    • The quotes on the original design are curvy, not straight
    • The disabled animation is a bit too long in my opinion, but that is just personal preference.
    Marked as helpful
    1