Peer-to-Peer code review

[IsaMarvin]

Hi there 👋

• This is a very nice project you have here! 💯

• One of these practices is DRY (Don't Repeat Yourself). That means if you find yourself repeating 🔁 a specific line of code in multiple places, you have to find a way to only use it once 1️⃣. This way when you update ♻️ this line, you won't have to remember where else did you write that to update the other copies.

• In our case

you can easily use a .font class in multiple places or simply put the font-family property in the body selector and all other children elements going to inherit it by default.

• (your code)

css .menu { / ...code / font-family: Arial, Helvetica, sans-serif; / ...code / }

.w1-title { / ...code / font-family: Arial sans-serif; / ...code / }

• suggestion

css .font { font-family: inter, sans-serif; }

Note: Also, you can do the same thing with the display: flex declaration.

[ogiogiovictor]

I think the code is great, Good job Mike however, you can apply the pseudo selector :root for classes that are the same so you don't repeat same css style. I am also during that currently in mine code base and i think its very nice.

[masangana]

very good job. Very impressed. It would surely be better if you could name your classes in more telling ways. For example work1-title instead of w1-title And I love your work. You challenge me 👍