@lab1210
Submitted
davidosi93
@davidosi93
All comments
- davidosi93• 50
@davidosi93
Posted
Hi, nice work you done. I would just recommend you to use a padding of about 30 px at the right-content.
0 - Allan Sancley• 60
@AllanSancley
Submitted
davidosi93• 50@davidosi93
Posted
It looks very good. Nothing to mention.
0 - E-m-e-k-a• 30
@E-m-e-k-a
Submitted
davidosi93• 50@davidosi93
Posted
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 - DONUZZ• 130
@mrdonuzz
Submitted
davidosi93• 50@davidosi93
Posted
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
@bukola2005
Submitted
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
davidosi93• 50@davidosi93
Posted
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
