@vladmee
Posted
Good work! Your HTML structure is well-thought-out, and the CSS adheres to best practices. That said, I do have a few observations:
1.Some elements, like the Explore (large) button, don't resize across different devices. It could be beneficial to use media queries or, even better, viewport units (vw) to manage element sizing.
2.@ALI-KHALED-13 has already pointed this out, but the tablet view doesn't quite match the design instructions.
3.The padding around the content appears a bit thin. It might be worth revisiting.
On a positive note, I'm a fan of how you've started your CSS with variables, resets, and utility classes. Your naming conventions are clear and consistent, and the smart use of rem and clamp across all elements is commendable. The use of media queries in em is super impressive.
Overall, it's a solid effort. I'd suggest spending a bit more time refining the responsiveness to polish it up. Keep up the good work!
@khophisnow
Posted
@vladmee thanks for your feedback will see to that as soon as possible .
