sunnyegg
@sunnyeggAll comments
- @AeroBWSubmitted 3 days ago
- @devindubinSubmitted 6 days agoWhat are you most proud of, and what would you do differently next time?
I think I got the sizing done much better using measuring tools online.
What challenges did you encounter, and how did you overcome them?Centering all items within the card and getting special spacing between some elements but not others. I was able to do this by using flexbox for the card element but applying data attributes to elements that had unique aspects.
I also used custom properties to set up reusable variables.
What specific areas of your project would you like help with?Be critical! What's a better way I could do something? What did I miss?
@sunnyeggPosted 6 days agoHi! First of all, good job on completing the challenge! There are some things that I noticed when I looked at your site and the code.
-
The website is fine when I open it on Desktop Screen, but when I downsize the screen to Mobile, the card got really tiny. I think you can just remove the
width
and theheight
on yourcard
class. Or you can set the maximum of the card's size usingmax-width
andmax-height
attributes. -
Remove unused codes in your CSS file.
Cheers!
Marked as helpful1 -
- @kirlosbastaSubmitted 7 days ago@sunnyeggPosted 7 days ago
- I learned that to center the card horizontally and vertically, we can just make the container flex and justify & align center. Thank you.
- You can use css variable so you can reuse some of your code. Such as in color. CSS Variable
0 - @velvet-jediSubmitted 7 days ago@sunnyeggPosted 7 days ago
Nice layout and looks like the design on figma. To make it reusable, you could make a class
card
.Marked as helpful1