Responsive 3-column preview card component
Design comparison
Solution retrospective
Feedback appreciated! First challenge submission. Really just trying to figure out how to submit a solution, so some things might be a little all over the place.
In this particular challenge I didn't feel the need to recreate the design 1-to-1 as I felt like it was more about nailing the visual elements than centering the content on the page.
Community feedback
- Account deleted
It doesn't seem to have been centered vertically, add this to
html
and it'll take care of it;display: flex; justify-content: center; align-items: center; height:100vh;
You should also check out the report.
Marked as helpful0@topieskelinenPosted about 3 years ago@thulanigamtee thank you! That was one of the things I referred to with "not recreating it 1-to1", but I will definitely put that little snippet into my toolbox.
Edit: and yes, the report. Will have a look! Accessible design is really not a strong point for me as of yet, will work on it in the future!
0 - @Shahab-MalikkPosted about 3 years ago
This comment was deleted about 3 years ago
0@topieskelinenPosted about 3 years ago@shahabmalik996 good point. If it was a real site I would definitely work on responsiveness more, but given it's such a simple task I didn't feel the need to necessarily put in the time. Arguably it would be a good practice, though – thanks for the comment!
0@Shahab-MalikkPosted about 3 years ago@topieskelinen it's not just a task you are going to put it on your github which can effect later. Keep Coding
Marked as helpful1@topieskelinenPosted about 3 years ago@shahabmalik996 very good point, did not even think of that. Thanks.
0
Please log in to post a comment
Log in with GitHubJoin 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