baconfatlabs / capstone-project-2-expresso

0 stars 0 forks source link

Indentation & Code readability #2

Open zannain opened 6 years ago

zannain commented 6 years ago

Avoid chaining multiple block level elements together on one line. It makes it very difficult to read your code when you have a couple nested elements in one line. While it may make your code appear more clean remember that text editors have code folding capability so you can always fold blocks of code to focus on certain sections of code. This is a better approach then nesting multiple elements together. It's okay to nest inline elements such as <span> tags with block level elements but block level elements like <p> and <input> tags should generally have their own line.

baconfatlabs commented 6 years ago

Dear reviewer, the things you have pointed out in your comments are not things I have written as part of this capstone. This was an Express.js course and everything but the express routing components were furnished by Codecademy. Please followup with their developers regarding uncommented code, poor html structure and chaining in React portions. You will find that my portions may not be commented but that is due to my general disdain for much of this course and it's poor delivery.