Time Tracking Dashboard Using React+Sass+css grid

Community feedback

• 

  Prabhash Ranjan•2,540

  @besttlookkPosted over 2 years ago

  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>
    );
  }
  
  

  3. Also before deploying check for all console.log statement and remove it.
  4. 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

  @kirtymeenaPosted over 2 years ago

  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