Max
@korniykomAll comments
- @hariFEDSubmitted 8 months ago@korniykomPosted 3 months ago
- on page should be at least one
main
element. - to render list of buttons use
ul
andli
- put name in
h1
element - put quote in
q
element
0 - on page should be at least one
- @Pablo-Zallio-DevSubmitted 3 months agoWhat are you most proud of, and what would you do differently next time?
In this project I have become more organized and have better analyzed the figma design.
@korniykomPosted 3 months agoThe first thing that catches the eye is the thickness of the border. It should be
1px
. Even though there is no predefined class in Tailwind CSS with that thickness, you should've usedborder-[1px]
.Also, I think you should fix the height to
522px
by usingh-[522px]
.Your custom shadow has an offset of 10px in both directions, which is close to
8px
but still not exactly 8px. Next time, take a closer look at the provided Figma file.Apart from that, your card seems identical to the reference. Keep up the good work!
Marked as helpful0 - @Kevinw14Submitted 3 months agoWhat are you most proud of, and what would you do differently next time?
I don't think I got it close to the design but I am proud of something that does look visually appealing.
What challenges did you encounter, and how did you overcome them?Getting the height to width ratio correct. I used what I thought looked best.
@korniykomPosted 3 months agoThere is a slight problem with the card ratio. You can find the correct one in the provided Figma file. You used w-80, which corresponds to 320px, which is the correct width, and h-[550px], but in the design, 499px was used (really interesting choice). So I suggest changing the height to h-[499px] or h-[500px]. Overall, the card looks very nice, though after you change the height, you may need to adjust the margins of the text as well.
0