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
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
Your design is very similar to mine. The cards are longer than they should be and idk how to fix that tbh.
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.
You could've centered the card by turning the body to a grid and using place-content: center.
The design is good overall but I have a few comments:
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
Your code is semantically organized but I think the buttons should be edited in active not hover state.