Mart• 120
@UnexplainableNo47
Submitted
RyZe
@RyZeDZ
All comments
- RyZe• 60
@RyZeDZ
Posted
Good effort! You missed some stuff in the design like the colors in the Nutrition section and the instructions section doesn't look same as the design but still good job!
Marked as helpful
0 - Simon Capillon• 100
@simon7195
Submitted
What are you most proud of, and what would you do differently next time?
I think I've done a good job of organizing my CSS and that my solution fits in well with the proposed design. I'll try to stick to this technique for future CSS files.
What challenges did you encounter, and how did you overcome them?
I got quite tangled up in choosing my technique, as I wanted the CSS file to be as clean and readable as possible, which wasted a lot of time.
What specific areas of your project would you like help with?
All clear!
RyZe• 60@RyZeDZ
Posted
Great job! Even better than my solution :D
0 - Satyam-Pandey677• 220
@Satyam-Pandey677
Submitted
What are you most proud of, and what would you do differently next time?
I completed this challenge in just 30 hour and next time I try to write more readable code for better understandable for others
What challenges did you encounter, and how did you overcome them?
none
What specific areas of your project would you like help with?
none
RyZe• 60@RyZeDZ
Posted
Hey there! I checked out your code.
Cool stuff:
- Big thumbs up for using semantic HTML. That makes things work smoother for everyone.
- Accessibility seems on point, which is awesome!
Room for improvement:
- Centering the text inside the QR code card might make it easier to read, especially on smaller screens.
- The layout seems a bit finicky on different screen sizes. Using some CSS magic can fix that.
- The code itself could be tidier. Think code organization and readability - it'll make future tweaks a breeze.
- There seems to be a bit of a gap between the design you had in mind and what the code shows. Nailed down design details can help bridge that gap.
Overall, the card is on a good track!
0
