@perterHUAN
Posted
- Using
flex
combined withrow-gap
(actually, just usinggap
would suffice since there's no wrapping occurring here) on thearticle
element to create spacing between child elements is a good method; elsewhere in the code, you could apply this approach instead of repeatedly usingmargin
. - By setting a padding on the
article
, it eliminates the need to setmargin-left
andmargin-right
over and over again on the child elements. - I think it might be less appropriate to use
header
andfooter
here as they typically refer to headers or footers for the entire page rather than within the context used. - The hover animation seems to be missing; when the mouse hovers over the
card
, thebox-shadow
should shift downwards. - It's worth considering using CSS variables to standardize the spacing values throughout your stylesheet, making maintenance and modification more convenient.
- This is my suggested solution; please note that I am not an expert, and I welcome your feedback and suggestions for discussion and mutual learning.
Marked as helpful