@turboo-sySubmitted 3 months ago
Ahmed_Hazem
@Ahmed-El-bazAll comments
- @Ahmed-El-bazPosted 3 months ago
Very good design you just had to make the parent a little bit bigger
1 - @NeonCodesSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am proud of the final result for the h1 title. Next time, I need to review some of the basics (line breaks in HTML, Box model in CSS for instance)
What challenges did you encounter, and how did you overcome them?I spent a significant amount of time wondering: 1- How to move the second half of the text to the bottom: I though I had to use some complex CSS code or something. After first googling it, I used a tag. 2- Later, after looking at the picture of the final result, I realised there was too much space between the two lines in the title. Again, went thinking it was CSS related. I ended googling it again and found the tag--which, for some mysterious reason, I forgot existed--, and that ended up fixing it. I also googled how to make the image's corners round.
What specific areas of your project would you like help with?I need help with the paragraph in grey: It still doesn't look like the image preview, despite having the font-size indicated in the style-guide.
@Ahmed-El-bazPosted 3 months ago1- You had to put what you call the box in the center of the screen. you could have used the code
.box{ position: absolute; left: 50%; top: 50%; transform: translate( -50% , -50% );}which would have put the box at the center of any screen. 2- you had to make the box visible by giving the body a different color which was written in the Style-guide.md file, so you could have writtenbody{background-color: hsl(212, 45%, 89%);3- you should have made the box a little bit smaller. and lastly i don't see any problem in the pragraph hope that i helped youMarked as helpful0
