search

jossi30 / Portfolio

Apache License 2.0
1 stars 0 forks source link

peer-to-peer reviews week2 #3

Open jossi30 opened 1 year ago

jossi30 commented 1 year ago

please review my latest commits

MAleemH commented 1 year ago

I think, you should change nested divs class names i.e., flex-item-1, flex-item-2 ... so that your css will be applied successfully. Thanks

EmekaEmmanuel commented 1 year ago

On line 75 change the classname to something more clear than placeholder

Tobby8629 commented 1 year ago

on line 23-26 the div worked fine but would have been nicer if you use an appropriate semantic tag for example h1 tag would work very fine on line 23