@razanabbas
@f1r3place
Submitted
What are you most proud of, and what would you do differently next time?
I like how little CSS this whole thing took! The repo says I have more HTML than CSS, and I think that's great
Don't really like that I couldn't get the spacing right and as a result there's a lot of empty space in the cards, but oh well
@razanabbas
Posted
Your design is very similar to mine. The cards are longer than they should be and idk how to fix that tbh.
What are you most proud of, and what would you do differently next time?
Finally started using Grid instead of Flexbox for everything.
What challenges did you encounter, and how did you overcome them?
Mostly trying to get everything aligned with grid.
@razanabbas
Posted
great!
@d0t666
Submitted
@razanabbas
Posted
You could've centered the card by turning the body to a grid and using place-content: center.
@PrateekSaini15
Submitted
@razanabbas
Posted
The design is good overall but I have a few comments:
@Sergitxin22
Submitted
@razanabbas
Posted
Does the solution include semantic HTML? Yes but imo the header could be divided into two sections: one for avatar and one for description Is it accessible, and what improvements could be made? Yes Does the layout look good on a range of screen sizes? I honestly do not currently have access to a range of screensizes Is the code well-structured, readable, and reusable? yes Does the solution differ considerably from the design? No
@rakesh265
Submitted
@razanabbas
Posted
Your code is semantically organized but I think the buttons should be edited in active not hover state.
Marked as helpful
@rapadev1
Submitted
@razanabbas
Posted
Great!
