FranciscoDavidCampuzanoMelgarejo
@FranciscoDavidCampuzanoMelgarejoAll comments
- @astnioSubmitted 3 months ago@FranciscoDavidCampuzanoMelgarejoPosted 3 months ago
It's pretty good. Good use of HTML semantic tags and well organized. Here is a YouTube video from Kevin Powell about this same problem. It will be more helpful to you.
https://www.youtube.com/watch?v=B2WL6KkqhLQ&t=2209s
0 - @RavikantMundaSubmitted 3 months ago@FranciscoDavidCampuzanoMelgarejoPosted 3 months ago
Good job. The only bad thing I see is that you are not using the appropriate colors and font weights. For desktop, the right margin is larger than the left (remember to use margin: auto). It gives me the feeling that everything is very close together (improve spacing). The spacing within the table is not all that good.
0 - @AamnbabaSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I am proud of to complete this project 3x faster than my previous 2 projects. I didn't face any problem coding layout and responsive model. It gave me much confidence.
What challenges did you encounter, and how did you overcome them?In absence of Figma Design file, it was tough for me to adjust the sizes of my containers.
What specific areas of your project would you like help with?I would like to have help about the more layout models like grid.
@FranciscoDavidCampuzanoMelgarejoPosted 3 months agoPretty good. But, just like you used 'gap' on the container, you could have used 'gap' on the <div> of the personal information (more direct than using margin) and you could have done the same with the link list, that way the last element does not add margin.
Overall, all good
0 - @phtm-haruSubmitted 3 months ago@FranciscoDavidCampuzanoMelgarejoPosted 3 months ago
Pretty solid. Although you could use fewer lines of code, but overall pretty good. Keep it up
1 - @Hamza-MusaSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
got the basics down
@FranciscoDavidCampuzanoMelgarejoPosted 3 months agoThe html code is not finished and, practically you did not add CSS code to style your page. Try at least put all the content inside a container and center it inside the <body> tag.
0