Skip to content
OverviewSolutions
6
Comments
3
spymon
@spymon

All comments

  • spymon•90
    @spymonPosted over 3 years ago

    Hi, Albertoflj! 👋

    Great job on this challenge! 🙌 The card component looks good and is responsive. 👏 I developed my version of this challenge recently, I added transition animations for the social media popup with the cubic-bezier.

    If you would like to do something similar, you can compare how I did it here. 😊

    I also suggest taking a look at your solution report and trying to clear up the error that is there. 😉

    Hope that helps. 🙂

    Happy coding! 😁

    0
  • Royer Adames•200
    @royeradamesSubmitted over 3 years ago

    Any good resources for learning CSS grid. This project showed me how flexbox is not the only tool I should depend on for a layout.

    React Sass modules

    2
    spymon•90
    @spymonPosted over 3 years ago

    Hi @Royer Adames! 👋

    To help you with Grid I recommend A Complete Guide to Grid. 😉

    Happy coding! 😊

    0
  • Agata Liberska•3,995
    @AgataLiberskaSubmitted over 3 years ago

    Hi! I have two only questions about this, really:

    • I used a pseudo element to make the background of one of the divs lighter and it works fine, but now I'm wondering if that was the right decision?

    • I added a box-shadow for the hover and focus states, not sure if it's clear enough, what do you think?

    Thanks!

    HTML + CSS grid, BEM naming

    2
    spymon•90
    @spymonPosted over 3 years ago

    Hi, @Agata Liberska, 👋 well done on this challenge, I think it looks really good overall!

    The box-shadow looks clear enough on my screen. The only thing I would add to the CTA button is cursor: pointer; 😉

    Happy coding! 😊

    1