Skip to content
OverviewSolutions
1
Comments
2
P
PetrSidorov
@PetrSidorov

All comments

  • hguevarasaenz•30
    @hguevarasaenzSubmitted 1 day ago
    What are you most proud of, and what would you do differently next time?

    i feel very proud to finish this project. for the next project I want to improve the project solution time.

    What challenges did you encounter, and how did you overcome them?

    the challenges i found in this project were in the "box-shadow" property

    What specific areas of your project would you like help with?

    nothing

    Resolution for "blog-preview-card-main"

    1
    PetrSidorov•50
    @PetrSidorovPosted 1 day ago

    Nice! I have small corrections:

    1. I think id's are not recommended to use in styles, they are meant more for js
    2. I think selectors can be simpler than this, you can just use classes and that's it
    3. It's better to use variables in your styles (not like I did it, lol 😁 but it's considered best practice)
    4. also I recommend installing prettier to have consistent styling But it's all actually pretty small nuances, great job!
    0
  • PetrSidorov•50
    @PetrSidorovPosted 1 day ago

    perfectly written! a couple of small fixes could be: adding semantic HTML (main, etc.) adding one h1 heading centering card vertically adding aria roles

    but the solution overall is nicely crafted 😁

    Marked as helpful
    1