Chilufya• 30
@cmmutale
Submitted
Christian Maldonado
@chrismaldona2
All comments
@chrismaldona2
Posted
Great CSS! I like the way you think and manage class selectors. I would recommend using clip-path: circle(50%); instead of border-radius: 100px; to round the photo, as it offers more precision and flexibility.
0- Mohammad Razim• 40
@yours-truly68
Submitted
@chrismaldona2
Posted
Pretty good. I reviewed the code, and I’d like to offer a few recommendations. For the card size, I suggest using width: 90% and max-width: 384px, for example, this way, the card will maintain the proper size without needing to rely on media queries.
Also, if your goal is to center the card, instead of using margin: 5/2rem, you could apply min-height: 100vh; display: grid; place-items: center; to the element that contains the card. Alternatively, you can use min-height: 100vh; display: flex; on that container and margin: auto; on the proper card.
0 - Tgimba1• 10
@Tgimba1
Submitted
What are you most proud of, and what would you do differently next time?
I am proud to have succesfully applied my knowledge in CSS Flexbox to live and I am looking to imorove my knowledge and understanding of figma.
What challenges did you encounter, and how did you overcome them?
Because I am not used to designing using figma files, I was a little bit slowed down.
What specific areas of your project would you like help with?
I think I am good with this project because it was meant for starters.
@chrismaldona2
Posted
To enhance fidelity with the original design, I would recommend adding more padding to the bottom of the card. Additionally, I find the subtitle to be a bit too large.
0
