Skip to content
Submitted 8 months ago

Article

NovikovIlya•910
@NovikovIlya
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Article preview component
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

;)

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

;)

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

;)

Community feedback

  • Mitali Shah•320
    @MitaliShahPosted 8 months ago

    @NovikovIlya Great job on completing this project! Here are some suggestions:

    • Use a media query to adjust the width of a card. It exceeds 730px.
    • Change font size and padding for desktop. It seems like there is some extra space between the title and paragraph.
    • The mobile layout active tooltip doesn't match the design.
    • Use relative units for font size such as rem instead of px.

    Hope this helps! Happy coding!

    Marked as helpful
    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord