@domieee
Posted
👋🏼 Hi Eduardo,
I have looked at your code and I have noticed some things
-
For the web accessibility it is important that the
h1
element is always at the beginning of the page. A web crawler always looks for theh1
element, and uses its content to describe what the page is about. Also you should nest your code into a<main>
-element Take a look here -
Don't use fixed heights or widths. You can give the container for the image a
max-width: 320px;
and awidth: 100%
. Theimg
also getswidth: 100%
. So the container adapts to the browser size, and the image to the container. so everything stays responsive. -
For the
font-size
property you should not specifypx
. Instead you can userem
.px
does not adjust when the brower size is changed. Read more about rem here
Otherwise it looks great and very similar to the design!
Have a nice day and keep coding!
Marked as helpful