A-Olatunbosun
@A-OlatunbosunAll comments
- @scarlet-azureSubmitted 6 months ago@A-OlatunbosunPosted 6 months ago
you have done a wonderful job @scarlet-azure...
however, it seems the padding main page container is kinda much. this section of the code below, you can reduce the padding to like 40px
.main-page { background-color: hsl(0, 0%, 100%); width: 40%; margin: auto; padding: 60px; border-radius: 20px;
you can also work on responsiveness, so it can be responsive across many devices. how i wish i could provide a link or resources to study media queries, but still you can make research on how to use media queries for making responsive design.
Goodluck scarlet-azure, a nice job you've done here..
0 - @nicosrfSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
.
What challenges did you encounter, and how did you overcome them?.
What specific areas of your project would you like help with?.
- @Sowmisri2306Submitted 6 months agoWhat are you most proud of, and what would you do differently next time?
Good learning
What challenges did you encounter, and how did you overcome them?Good learning
What specific areas of your project would you like help with?Good learning
@A-OlatunbosunPosted 6 months agoNice job @Sowmisri2306..
you may need to check how you link the author's image because the image is not displaying. i guess there is an error in your src link.
<div class="author"> <img src="CH2\assets\images\image-avatar.webp" alt="author"> <b>Greg Hooper</b> </div>
i think it should be this way, since there is no folder named ch2. you can try the code below and see if it works. goodluck
<div class="author"> <img src="./assets/images/image-avatar.webp" alt="author"> <b>Greg Hooper</b> </div>
0 - @benhyhSubmitted 6 months ago@A-OlatunbosunPosted 6 months ago
you did a wonderful job man, but you can improve more on it's responsiveness. it doesn't look good on some screen sizes.
0