Design comparison
SolutionDesign
Community feedback
- @besttlookkPosted over 2 years ago
Hi, Below are some points I like to make: 1.Center the card w.r.t the screen. Use below code:
body{ min-height:100vh; display: flex; align-items:center; justify-content:center; }
- Border radius is too much decrease to either 5px or 8px.
- Also overall size of card is small for large screen.
- Try using external css file and link that file inside HTML file.
- Wrap main content inside "main tag" to improve accessibility. 6; While adding image , always add "alt" attribute to "img" tag.
Good Luck, #happyCoding
Marked as helpful1
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