Submitted about 3 years ago
I used visual studio and Created This card with html and css only
@Ndayisenga
Design comparison
SolutionDesign
Solution retrospective
Ask me anything I am ready to help !
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, the layout is different. Like what Juveria said, do not rush your website. Make sure to at least provided a site that is somehow close to the design. Still, the layout is good if it does take full viewport.
Some suggestions would be:
- Of course first, refactor, making the layout match or just be made the right way.
( Assuming that the website will be refactored/changed )
- Always have an
main
element to wrap the main content of the webpage. Thesection
element could be replaced with themain
element. - The font style is not applying properly, check if you have added the font-family for this one.
- You don't need to use
h3
on the text after theh1
, that text should only be using ap
tag, since it is just a simple text. - You don't need to use
cursor: pointer
on the card, since it is not really an interactive element. - You are missing the information below the paragraph.
- Needs to have mobile layout and make sure it is responsive.
0 - @Juveria-DalviPosted about 3 years ago
Don't rush to submit code check out if it atleast matches the design
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