Time tracking dashboard using HTML, SASS, FLEXBOX, GRID LAYOUT, and JS
Design comparison
Solution retrospective
If you'd like feedback on my code, please add your questions below. Thanks!!
Community feedback
- @MathisHumbertPosted almost 3 years ago
Hi! Nice work, everything seems to work! This looks great and responsive, good job!
I think you could improve your app using more JS and less html. Because you are hardcoding every cart in your html.
I recommend to map over the data array that you have and return html that you will set as innerHtml to your container. If you are new to this go check this video: https://www.youtube.com/watch?v=80KX6aD9R7M.
Then call call it using 'DOMContentLoaded' event to your window.
Marked as helpful0 - @GabrielLaminasPosted almost 3 years ago
Thanks for your feedback it's so important for me. I am going to watch this video.
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