@haquanq
Posted
Hi @7A9Oo,
It's review time!!
- When using
section, article
, you need to choose one not both (what type of content inside it?). Also, everysection, article
need to be provided a heading as it's child. You are using to many wrapping mixed withdiv, article, section
(use it when you need it). - Avoid unnecessary
div
wrapping (div
has no semantic meaning, choose it for styling only). For example, no need to wrapimg
withdiv
, leave theimg
alone. - It is pretty clear that every bold text is a title/heading and you need to wrap it with
section or article
(this is the right time to use them). - Always keep your HTML simple and clean (don't
div
wrapping randomly) and use semantic HTML. - Never use fixed
width, height
on element (unless it really need it), this make your element less responsive, work aroundmin/max-height, min/max-width
instead.
Check out my solution for more references
Have a nice day!!!!
Marked as helpful
@TheAashay
Posted
This isn't my solution but your comment surely helped me. Thanks @haquanq
@haquanq
Posted
@TheAashay that is glad to hear!
@7A9Oo
Posted
@haquanq thanks for your review i apreciate it