Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Time Tracking Dashboard Using React+Sass+css grid

kirty 380

@kirtymeena

Desktop design screenshot for the Time tracking dashboard coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is the first time I have used CSS grids. I would like to have feedback to improve my code.

Community feedback

@besttlookk

Posted

Hi, Nice work, Folloing are some points i like to make:

  1. As you are not using index.css file kindly delete it and remove the import starment in index.js file.
  2. While coding do keep in mind about "accessibility". You can refer "accessibility report ". One thing you can do is to use semantic HTML tags. Every tag has its meaning so do not use div everywhere. Always wrap the main content inside <main> tag.
function App() {
  return (
    <main className="App">
        <Dashboard/>
    </main>
  );
}

  1. Also before deploying check for all console.log statement and remove it.
  2. Dont write everything in one big file. try to divide into components. That way your code will be maintainable and aslo easily understood by others .

Good luck,

#happyCoding

0

kirty 380

@kirtymeena

Posted

Hi @besttlookk , Thanks, I will make sure to use semantic HTML tags next time also I will remove the console.log statements before deploying.

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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