Raymart Pamplona• 16,140
@pikapikamart
Posted
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