Submitted over 1 year ago
Responsive preview card page using CSS Flexbox
@klaudiapalubska
Design comparison
SolutionDesign
Community feedback
- @visualdennissPosted over 1 year ago
Hello Klaudia,
Very nice work there! It looks like there is some small issue with border-radius, as it is set to 5% and this seem to result in a sharp cut on the sides. A value like e.g. border-radius: 10px; or in rem/em seems to be work better in this case.
Hope this is helpful!
Marked as helpful0@klaudiapalubskaPosted over 1 year ago@visualdenniss Oh I wondered why this looks like that. Thank u!
0 - @frank-itachiPosted over 1 year ago
Hello there π. You did a good job!
I have some suggestions about your code that might interest you.
HTML π:
- Wrap the page's whole main content in the
<main>
tag. - The heading order is important in the html structure so try to always start your headings and/or titles with an
<h1>
tag and then you can increase by one if you need to use more headings in your html code.
I hope you find it useful! ππ Above all, the solution you submitted is greatπ!
Happy
<coding />
π!Marked as helpful0 - Wrap the page's whole main content in the
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