Peer to Peer review - Githubissues

carlosmuerto commented 2 years ago

great job, please fix these issues

[ ] Add life demo link at README
[ ] Add the presentation video to the README
[ ] Delete the white space between the top bar and the start of the page
[ ] When hovering the featured person there is a red border, this moves all the content inside, this can be fixed by adding a transparent border of the same size to the element that has the :hover.
[ ] at the about page, the headline section does not have a title.
[ ] the headline section does not have a margin/padding at the top in firefox, the problem is that the class="advert-txt" means something in firefox, change for something anything

limavfabio commented 2 years ago

Your website is looking great Kumilachew! But I think it would be better if you improved some aspects: The two images on the about page break on mobile version, I suggest you add a width so they keep the right aspect ratio

cmolinan commented 2 years ago

Great job Kumilachew. Only a few details to improve. -At aprox 1200 px screen, the text beside the Logo is too close to the left margin -I think It could be better and easy to include a link to Program option of the Menu

Kumilachew-g / capstone-project-one

Peer to Peer review #3