@AgataLiberska
Posted
Hi @tesla-ambassador, it looks great!
I noticed two things: not sure whether <article>
is the best choice for the sections of your card. All three parts form one whole component, and an <article>
is something that makes sense on its own and can be distributed independently. I would probably just use divs here, but there may be a better option.
The second thing is a tiny detail and I don't know this design, so sorry if that's what it was supposed to look like - but all text is centered in column layout, except for the price - was that intentional?
Hope this helps :)
@tesla-ambassador
Posted
@AgataLiberska Thanks for the feedback. I had used divs in the first place but now that I think about it, I agree that I should have stuck with them and the text centered in the column layout except for the price was intentional. I was experimenting hahaha... ( They should add emojis for windows users)