Design comparison
Solution retrospective
everything, will do the same thing next time. am proud that if you look close at the site and the daily stats everywhere the stats is 1hrs i make it display 1hr without the s just a few more lines of js allow me to do that. I figured it made more sense even if it is not a requirement from frontendmentor
What challenges did you encounter, and how did you overcome them?the json was not loading on the page, found a way around it with google.
What specific areas of your project would you like help with?all is fine
Community feedback
- @JreyIVPosted 7 months ago
Hello Makha,
You did a good job on this challenge! I didn't even think about the 1hr vs 1hrs. That's some good attention to detail, one which I overlooked. Good catch
One thing that I added that you might want to add is setting the weekly button to white on the initial render. You can do that with one line of code by setting the button to active. It's a small thing but it will show the user that they are on the weekly tab by default. Just a little thing that can go a long way.
I don't have much else to say besides that. Really good job!
Marked as helpful1
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