Design comparison
SolutionDesign
Community feedback
- @abhik-bPosted almost 3 years ago
Hello ÉMILIA 👋 Your solution is nearly perfect. Well Done !! I am sharing some of my opinions :
- Just remove the
overflow-y : hidden
from body so that users can scroll in mobile screens . - I guess if you create a
data.js
file where you export aconst
which has the array indata.json
then maybe you won't need jquery. For example :
const data = [ { "title": "Work", "timeframes": { "daily": { "current": 5, "previous": 7 }, "weekly": { "current": 32, "previous": 36 }, "monthly": { "current": 103, "previous": 128 } } }, // ... rest of the data ];
Please keep up this awesome work 👍
Marked as helpful1 - Just remove 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