BasedDevNGMI
@BasedDevNgmiAll comments
- @ShivangamSoniSubmitted 4 months ago@BasedDevNgmiPosted 4 months ago
Well done, HTML and CSS look absolutely strong and clear. I have no feedback to be honest.
1 - @ownedbyanonymousSubmitted 4 months agoWhat 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
@BasedDevNgmiPosted 4 months ago- 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 - @tiagocostarebeloSubmitted 4 months ago@BasedDevNgmiPosted 4 months ago
Nah man, this looks proper. Only small thing is using better titles for the alt text on images.
1 - @git-prakash-rajSubmitted 4 months ago@BasedDevNgmiPosted 4 months ago
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