Alex• 3,130
@Alex-Archer-I
Posted
Hi! Well done, neat work =)
I can give a couple of suggestions of how you can improve it:
- Try not to use
br
to force line break. It is often case in real projects when content could change time to time and to be dynamically rendered. So, it is better to allow thep
tag to align it's content by itself. - Almost the same about
height
of the container. The better approach to not hardcode it. - Don't forget about
main
tag - it is important semantic tag. Every page should contain one. - Use
rem
values at least forfont-size
. It's a relative values which depends on user's settings. By default1rem = 16px
.
Hope that helps. Good luck and keep coding =)
Marked as helpful
1