code review Morning session W3 d4

rachaouldbabaali / Portfolio-setup-and-mobile-first

Setup and mobile first repo portfolio using Figma design to create a UI and Flexbox to place elements in the page.

https://rachaouldbabaali.github.io/Portfolio-setup-and-mobile-first/index.html

1 stars 0 forks source link

code review Morning session W3 d4 #10

Open rachaouldbabaali opened 1 year ago

rachaouldbabaali commented 1 year ago

This review is provided by my morning session team.

kerlos-bahaa commented 1 year ago

the website looks really beautiful I especially like the part with cards when you hover over them they go up a little bit and the fixed header is really a good idea too the only thing that I noticed is that the class could be a little bit more descriptive

Esmatullah12 commented 1 year ago

I think everything is fine in your code it is responsive and looks awesome. keep up the good work👏

musangamfure commented 1 year ago

Hello@rachaouldbabaali,

Thanks for your well-designed portfolio project✔👏👏😎 you did a great job really. but I find something that may worth considering.

You may reduce the width to look like others.

Everything else looks awesome😍👍