@R3ygoski
Posted
Hello @elsayedelbauomy, congratulations on your project, it almost resembles the proposed design.
I have some tips regarding your HTML, specifically about semantics. Firstly, I'd like to mention that semantics are crucial for improving the accessibility of your page.
Starting with <div class="top">, it could be a <header>, as this part serves as the introductory content of your page.
Also, concerning the four <div class="box">, it would be more appropriate to use them as <article>, as the content within them is self-explanatory.
Lastly, instead of using <span>, you could have used <p>.
Now, a suggestion regarding CSS. In your .box selector, you could add a box-shadow, which would add a shadow to it, creating more contrast with the background and making it resemble more of a card.
Again, congratulations on your project, keep practicing and improving. If you have any doubts about what I said, please comment below and I'll try to help as best as possible.
