@begli-amanov
Posted
Hi @ShinRaw11, a really nice work here. Especially I found the Idea to work with flexbox
and use gap
property with it very interesting. I have even implemented it in my own solution from this challenge. There are however few suggestions I could give:
-
Most important thing I guess is the multiple use of
<h1>
heading. It usually used as a page hero section (content description). It should be used just once per page. You could read here a bit about it. -
It might be not the best stuff to use fixed heights on the elements and let them expand naturally. You could stumble on overflow issue. It this case it is not so important, but it might once. I would suggest to avoid them unless you really need.
Nevertheless it is really good job. I've learned few stuff from your solution so my big appreciation for that! Keep up the good work and have fun on your journey!
Marked as helpful