@pikapikamart
Posted
Hey, awesome work on this one. Layout in desktop is much wider, maybe scaling it down would be really great, it is somehow responsive and the mobile layout looks great as well.
Some suggestions would be:
- Always have a
main
element that will wrap the main content of your webpage. This helps users to properly navigate your site. The.container
selector could have usedmain
element instead ofdiv
. - Always have an
h1
on a webpage. For this, the text:
Reliable, efficient delivery
Powered by Technology
Should have been the h1
, you only need 1 h1
per page. So only use 1 h1
to wrap both the text, you can either use max-width
on the h1
to make the text wraps on another row, or use a span
to wrap the second text and have a display: block
on it.
- When using heading tag, make sure that if you use
h4
,h1, h2, h3
are present before theh4
. Always use heading tag incrementally by 1 level.
Aside from those, great work again on this one.