Design comparison
SolutionDesign
Community feedback
- @vandermsPosted over 2 years ago
Nice work!
The only thing I would add is that is a bit dangerous to use
innerHTML
to create textNodes, especially because you are getting the data from a server you have no control. I thinktextContent
orinnerText
were better options.Marked as helpful1@oscarmartinez761Posted over 2 years ago@vanderms Thanks for the feedback! I'll keep it in consideration for future projects
1
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