Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Card Preview

msa-sitareā€¢ 70

@msa-sitare

Desktop design screenshot for the Article preview component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@Chirtoaca94

Posted

Excellent work! Looks great in different screen sizes so meets the responsiveness test and it works as expected.

In your html, css, and js files however, I feel like you have too much. I think you have unnecessarily complicated the structure, layout, and code. You could simplify the code and get rid of redundant statements such as your first css line a workaround would be to use * { margin: 0; padding: 0; box-sizing: border-box; }

Try to look for ways to reduce complexity in your code and make it more readable for reviewers of your code. That will help your thinking. Overall, great effort!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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