Shivangam Soni• 970
@ShivangamSoni
Submitted
BasedDevNGMI
@BasedDevNgmi
All comments
- BasedDevNGMI• 40
@BasedDevNgmi
Posted
Well done, HTML and CSS look absolutely strong and clear. I have no feedback to be honest.
1 - Pownedbyanonymous• 160
@ownedbyanonymous
Submitted
What are you most proud of, and what would you do differently next time?
Honestly, l am not sure l am proud of anything in particular other than the fact that l am making progress in my learning journey.
What challenges did you encounter, and how did you overcome them?
The main issue was trying to build without any Figma files. I later used an AI tool that translates an image into a Figma file. It was helpful but not accurate in its conversion.
What specific areas of your project would you like help with?
Code quality: how best l can improve on the use of HTML semantic tags, media queries and Flexbox
BasedDevNGMI• 40@BasedDevNgmi
Posted
- Where are your paddings in the links?
- For SEO you could add a meta description
- Good alt text
- Nicely structured HTML
- For external links, consider using target="_blank" with rel="noopener noreferrer" to open links in a new tab and ensure securit
0 - Tiago Costa Rebelo• 180
@tiagocostarebelo
Submitted
BasedDevNGMI• 40@BasedDevNgmi
Posted
Nah man, this looks proper. Only small thing is using better titles for the alt text on images.
1 - zexez• 10
@git-prakash-raj
Submitted
BasedDevNGMI• 40@BasedDevNgmi
Posted
The padding around the copy/text should be more than what you have on the image. Also the padding of the white box seems to be too little.
Try going for <main> and <section> instead of going for all <div>'s
1
