Peer to peer review. - Githubissues

shahira-sadat / portfolio-bootstrap

0 stars 0 forks source link

Peer to peer review. #3

Open erickhansx opened 2 years ago

erickhansx commented 2 years ago

Remove fixed width on the about-myself section to stop side to side scrolling.
You can do the bullet-points with the class list-unstyled for the first element and use the bullet-points of the secondary items for the styling of the page.
Maybe use flex-reverse for the cards sections to align the cards for the mobile version.
Optional: Use the accordion class for the about-myself section for practice.

Krishnabot commented 2 years ago

Great Project so far ! Awesome work 👍 Optional suggestions are

Use of flex reverse in cards for the desktop version to alternate images and texts might be a good idea to align items according to the design rather than doing it in HTML element every time.
instead of images for list objects, it's good to use list-unstyled to style individual li elements.