Peer to peer review 4th nov, 2021

[williamrolando88]

Hi Cris, 👋 👋

Great job so far, you accomplished to make responsive the 5 template, which at first glance it looks pretty difficult to render. So you have done it great to this point. 🏆

What I am going to suggest is just my opinion from my point of view, some advice that could come in handy in the future.

• Here I suggest you to put all <a></a> elements inside an unnumbered list, this will help you to organize your code. Like the example code below:

  <ul>
  <li>
  <a></a>
  </li>
  </ul>

• I would help you in the responsive web design of your page, to avoid using px for constraining your dimension, you could use relative dimensions like vw, vh, %, rem, etc. You could find more about this in this link.
• In the following the code, it will help you to speed up the code writing if you use functions like "repeat" to for example in this case, generate the columns:

Continue with your great job Chris 👋 💪

[TheLuisAbarca]

Hi Cris,

I was going to remark this part, that @williamrolando88 have also noticed:

• You have to remember using that standard, when you have some elements like that is better use an < ul > tag in order to mantain that guidance, is a good practice.

https://github.com/CRClothier/My-Portfolio-Website/blob/c182008404f4185e82051289b64889b570c793fe/index.html#L28-L33

• I think is a better idea, that for your template you use at the end, outside the main tag, you could create a < footer > tag in order to put in use that good web developing practice.

https://github.com/CRClothier/My-Portfolio-Website/blob/c182008404f4185e82051289b64889b570c793fe/index.html#L186-L193

Keep doing the great job that you are doing until now. :)