@Alex-Archer-I
Posted
Hi!
Your work is quite neat, congrats with finishing it =)
I can give a couple of suggestions.
First of all, a few semantic tips. There should be only one h1
element on the page. And it should be inside main
tag, not header
. Top level header
is for page navigation, site logo and stuff. Here all content should be inside main
(and you actually doesn't need a wrapper for it).
And I want to say, that if you used grid that you didn't have to wrap two middle elements in additional wrapper. It's cool if you wanted to practice flex more, I just suggest you a bit more convenient way =)
Hope that helps. Keep doing cool stuff =)
Marked as helpful
@HeshamAliAhma
Posted
thank you for helping me@Alex-Archer-I