Skip to content
OverviewSolutions
5
Comments
5
davidosi93
@davidosi93

All comments

  • davidosi93•50
    @davidosi93Posted 2 months ago

    Hi, nice work you done. I would just recommend you to use a padding of about 30 px at the right-content.

    0
  • davidosi93•50
    @davidosi93Posted 2 months ago

    It looks very good. Nothing to mention.

    0
  • davidosi93•50
    @davidosi93Posted 2 months ago

    Hi, I see that your picture is not showing. Try these src="assets/images/avatar-jessica.jpeg"

    I would also recommend you to use flexbox inside you body tag to show the card in the center of the screen.

    Otherwise you did a great job and I realy like the green background-color, when I hover the links.

    0
  • davidosi93•50
    @davidosi93Posted 2 months ago

    Hi, nice work you done. At least you haven´t forgot the border of the card like me. :)

    I just saw one little problem with the responsive viewer that on a screen size large about 650px 1000px (typical tablet screen size) the card is very tight with a width of about only 300px. I guess you can use the whole width. I can recommend you to use the chrome extension "responsive viewer". There you can see different screen sizes at one page.

    0
  • bukola2005•20
    @bukola2005Submitted 4 months ago
    What are you most proud of, and what would you do differently next time?

    I am proud the my webpage is fully responsive , and what I will do differently next time is use sass/scss

    What specific areas of your project would you like help with?

    I will like help in understanding when i hosted my webpage on GitHub URL ,the image on the webpage is not displaying

    Desktop-first solution using Flexbox and Fully Responsive

    2
    davidosi93•50
    @davidosi93Posted 4 months ago

    Hi, I would recommend you to increase the border radius. How about 2 or 3 rem. And the last thing ist only to put the whole layout at the center of the page with flexbox.

    I hope it helps you.

    Best regards

    0