search

Maxwell011 / My-Portfolio

This is my portfolio web page built with HTML, CSS, JS which displays my professional skills and languages and projects completed and a form to reach out to me
5 stars 0 forks source link

Peer Review #4

Open Maxwell011 opened 2 years ago

codedbystuart commented 2 years ago

Hi @Maxwell011 , Great work! I have some feedback for you. I see that you have classes box-1, box-2, box-3, box-4 in some elements in your HTML and these happen to have the exact same styling in your CSS. I think it would be better just to give them one class, and then apply the styles instead of chaining them as you did.


.box-2,
.box-3,
.box-4,
.box-5,
.box-6 {
  width: 326px;
  height: 222px;
  border-radius: 8px;
  padding: 16px;
  background-color: #fff;
  margin: 245px 0 0 0;
}```

This makes the short shorter and neat.