Design comparison
SolutionDesign
Solution retrospective
ez
Community feedback
- @frank-itachiPosted over 1 year ago
Hello there ๐. You did a good job!
I have some suggestions about your code that might interest you.
HTML ๐:
- Make sure that the
<img>
elements in your HTML code has an alternate (descriptive) short text. The reason for this is that screen readers canโt translate images into text. So to fix this you can do the following<img src=โโฆโ alt=โshort textโ >
CSS๐จ:
- To set multiple background images you can use the background property and specify multiple paths(urls) in a comma separated base. For instance:
body { background: url(image-1.png), url(image-2.png); }
More information about the background property here
I hope you find it useful! ๐๐ Above all, the solution you submitted is great๐!
Happy
<coding />
๐!1@StarrkeyPosted over 1 year ago@frank-itachi For some reason I tend to forget using alt texts but on real projects I probably should. Thnx so much about the background properties, it was a bit confusing to set it up in easy manner and now your suggestion fixes it
0 - Make sure that the
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