Submitted about 3 years ago
I used html5 and css to complete this challenge
@codebyarsh
Design comparison
SolutionDesign
Solution retrospective
please share share your feedback
Community feedback
- @thinkclear67Posted about 3 years ago
Nice job! My suggestions would be that
- take a look at your mobile(landscape & portrait) and tablet version, because something isn't right and the website is not responsive for mobile-landscape and tablet
- the original design has more white space
- try the mix-blend-mode property for the image instead of opacity
- train yourself to code mobile first and adjust for larger screens with media queries if necessary instead of the other way around. This is much easier and less coding. A plain html website (so without css) is already responsive
- use the rem unit only for font-size. Margins and paddings are best done in em's, in this way margins and paddings act responsive if the default font-size is not 16px. The same goes for height and width, the vw and vh are better options.
Keep on coding ;)
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