@DomeniqueRosaSubmitted 2 months ago
Ahmed Elmaghraby
@A-ElmaghrabyAll comments
- @A-ElmaghrabyPosted 2 months ago
Good job! I noticed small thing, you need to add alt attributes to images and aria-label for better accessibility.
0 - P@vercthuSubmitted 2 months agoWhat challenges did you encounter, and how did you overcome them?
I was having a hard time styling the table on the Nutrition section, so I looked up Mozilla MDN and was able to learn that, to style a table border, I need to first add
border-collapse: collapse;on the table css. - P@AnonymousCoder323Submitted 2 months ago
- @hikkenoace1Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
Proud of getting better at structuring html elements and using divs
@A-ElmaghrabyPosted 2 months agoHi, congrats on completing your challenge I looked at your code and it's good I would like to recommend you to add comments inside the head for google fonts and custom css links for better readability
0 - @goriola2010Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I was used to styling an image with the width and height but i was able to discover styling with other tags such as the margin and the padding
What challenges did you encounter, and how did you overcome them?i encountered challenges like how to move the image and all but i encountered them by continous trying
What specific areas of your project would you like help with?I would like help with the measurements of the styling
@A-ElmaghrabyPosted 2 months agoHey there!
Great work on your challenge the design is coming along nicely!
I noticed you’re using inline styles in your code. While that works, I’d recommend switching to an external stylesheet instead. It makes your code cleaner, easier to manage, and keeps all your styling in one place. Plus, it’s a good habit as you work on bigger projects!
Keep up the great work—you’re doing awesome!
0
