search

abreham1222 / My-portfolio

1 stars 0 forks source link

peer to peer code review #7

Open MahdiAghaali opened 2 years ago

MahdiAghaali commented 2 years ago

Here we will submit our suggestions and potential improvements to your code.

MahdiAghaali commented 2 years ago

I would suggest avoiding defining a specified amount of pixels for every element. for example, you can remove the width: 80px; attribute from your list of languages used to make it automatically adjust based on the content here are some pictures with width: 80px; : 213213214qes and without : afawdawraw3er2

Alhajideen commented 2 years ago

Hi, Nice layout. I think you can make your cards more responsive just like Mohi suggested in the screenshots. Overall I see a clean code.

Do have a nice day and happy Coding Abreham 😄