asnganje/My-Portfolio

peer-to-peer code reviews

Opened this issue · 0 comments

Peter1907 commented

Good use of HTML semantics

  • It would be better if you gave the sections some descriptive names (eg. class="works" rather than class="grid-container")
  • And it would look better if you align the first section (class="grid-container") to the center.
  • You can use more meaningful commit messages rather than "modified CSS" for example ... It would make it easier for you & other code reviewers to track the changes you made.

Other than that the project looks professional and well-built, Good Job!