@JayapreethiN
Submitted
@kinqbert
@JayapreethiN
Submitted
Your overall solution looks goof! But I'd also like to markup some points, you may try to improve.
padding
and margin
properties instead of separate margin-top
, margin-bottom
, margin-right
etc.div
tags. In your code it wasn't necessary to add additonal wraps for image
, next
and para
.next
and para
were, but it could happen because it is written in you language I don't understand :DThat's all issues I could find looking at your code and site. Good job! Love your solution!
Marked as helpful
@murdock33
Submitted
What are you most proud of, and what would you do differently next time?
What challenges did you encounter, and how did you overcome them?
What specific areas of your project would you like help with?
Great job! Your code looks mostly fine to me, but personally I'd suggest some fixes:
Overall, I really like your solution! Code looks pretty clean and good. Great job!
@kunwar91
Submitted
What are you most proud of, and what would you do differently next time?
this was a quick one, but I am proud that I did not have to google for syntax or css props
What challenges did you encounter, and how did you overcome them?
Understanding the Figma images was new to me
What specific areas of your project would you like help with?
NA
Looks fine to me! However, personally, I'd recommend some improvements:
text-align: center;
for the whole card, not to each element of the card (like title, description, etc.), every text inside of it will be alligned to the center.padding-inline: 16px;
instead of padding: 0 16px;
.card-title
, not .card h1
). It doesn't makes any weather in such project, but I believe it would make code more readable in bigger project.Great work! Love your solution!