Design comparison
Solution retrospective
I might have used a lot of code to get the job done with the desktop preview but I had a lot of trouble working on the mobile responsiveness. please advice...
Community feedback
- @grace-snowPosted almost 4 years ago
Thanks for sorting, David. There are quite a few small 'under the hood' changes I can suggest for your code that I hope you find helpful
-
Alt text doesn't need to say 'image of' because it is already announced as an image before the alt is read out
-
You need to place all text inside html tags that are made for presenting text, like paragraphs or headings or list items (not divs or spans on their own, although you can still wrap text in spans that have classes on to help with styling)
-
I would recommend you learn more about css specificity and naming conventions to make your css more maintainable on larger projects. You're using a lot of nesting and ids and element selectors at the moment. I'd expect to see styles done using single classes in the main.
-
I'm not sure you're fully understanding the implications of using position absolute and transform translate as a centering/layout method. Try using methods like flexbox and research what absolute positioning does to the DOM. To be clear, you can use that method sometimes, but I wouldn't expect to see it often and in modern css there is almost always a better way now.
I hope all that's helpful for your learning ☺ Keep going
0 -
- @grace-snowPosted almost 4 years ago
Hi David, your code link is pointing to the wrong challenge on this one
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