@ali-reza-2531
Latest comments
- @zzdevs
excellent!
- @tris6What are you most proud of, and what would you do differently next time?
I'm really happy with how I pre-empted cascading layout properties to all cards, but using a separate class for each color setting. The separation between layout and skin means I could minimize code. Taking that minute to look at the design overall, and think about what to cascade and how to apply classes, really pays off.
I didn't pre-empt needing a div for person image and name, so had to go back and wrap each markup individually.
I learned that the gap property can be used for flex children, not just grid cells.
I had to Google background image properties to get the quotes like the design.
What challenges did you encounter, and how did you overcome them?I spent a lot time fine-tuning vertical margin between text elements. Until, I finally realized each section is just evenly spaced. A quick space-around property for the card class, and problem solved.
What specific areas of your project would you like help with?Nothing seemed ambiguous for this project.
@zzdevsGood work!
- @slnwrld@zzdevs
it looks better than mine
- P@TheWraithDevWhat are you most proud of, and what would you do differently next time?
I did an entire mobile first approach trying to ensure everything was consistent and focused on accessibility aswell as a responsive layout.
I also focused on the CSS to make sure it was structured and appropriate
What challenges did you encounter, and how did you overcome them?None - my biggest area I ran into was a bug with the scaling of my layout but it was a typo that caused it
What specific areas of your project would you like help with?Responsiveness & Accessibility Best practice guidance where possible as I am always wanting to improve
@zzdevsyou can use the clamp function in css to scale up the font size in higher widths for a better user experience.
- @oMarcelosantos@zzdevs
Excellent result in my opinion. You could have used the nutrition section to use flex or grid but only to make it more similar to the original, I don't think it's so necessary.
Marked as helpful - @RieneeWhat are you most proud of, and what would you do differently next time?
I'm absolutely thrilled about this project. It took me less than two days in building this project and I am specifically proud of it's responsiveness.
What challenges did you encounter, and how did you overcome them?To be honest, this has got to be one of the easiest project I have worked on, thus far.
What specific areas of your project would you like help with?All feedback is welcome. Thank you in advance!
@zzdevsIt's really good, the only thing I would do would be to apply padding to the card so that the texts do not exceed the width of the links.
