Submitted about 1 year ago
responsive page , please don't hesitate to leave any comment to me
@Islam-Yasser
Design comparison
SolutionDesign
Community feedback
- @AbderrahmaneGuerinikPosted about 1 year ago
Hey👋 Nice work ! but you can improve it
- You should use more semantic HTML like section, main instead of div of class main.
- Use rem unit instead of px for more maintainability and for responsive purposes
- and Also the images in the background are not positioned perfectly like in the design, i've just finished the project and i also struggled width keeping this background images positioned in the right way and stay responsive 😥
Good Luck and Enjoy coding 😃
Marked as helpful1@Islam-YasserPosted about 1 year agoDear @AbderrahmaneGuerinik a big thank u for reading my code and trying to inhance it ❤ !! Yeah background makes me dissappointed so i decide to put it that way 😂 , thank u again my friend wish u happy coding and further tips on my upcoming challenges❤
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord