Submitted over 3 years ago

HTML, CSS, Vanilla JS

Maria Glynou• 290

@Msgl

Preview site View code

This is a solution for...

Article preview component

HTML
CSS
JS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Any feedback would be greatly appreciated!

Community feedback

ApplePieGiraffe• 30,565

@ApplePieGiraffe

Posted over 3 years ago

Hey, Maria Glynou! 👋

Nice work on this challenge! 👍 The card component looks great and responds nicely! 👏

I only suggest using flexbox (rather than a combination of auto and explicit margins) to center the card component in the viewport so that it remains nicely vertically centered on all screen sizes.

Also, you should be able to clear up that error on your solution report by adding a <title> tag to the inline SVG on the page. 😉

Keep coding (and happy coding, too)! 😁

Maria Glynou• 290

@Msgl

Posted over 3 years ago

Thanks a lot for the feedback, ApplePieGirrafe! I fixed the SVG error and I'll check out the flexbox alternative!

Web Frog• 1,840

@MasterDev333

Posted over 3 years ago

Great work @Maria. Your solution is spot on. Everything is fully responsive and close to design. It would be great if you add box-shadow for the card. Happy coding~ :)

Maria Glynou• 290

@Msgl

Posted over 3 years ago

@MasterDev333 Thank you very much for the feedback!

Abhik• 4,840

@abhik-b

Posted over 3 years ago

Hi Maria, Your solution is very responsive and it looks amazing everywhere...

Good Job , Keep it up 💯

Maria Glynou• 290

@Msgl

Posted over 3 years ago

@abhik-b thank you very much!

Please log in to post a comment

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