Design comparison
Community feedback
- @ClickClickKatePosted about 1 month ago
I think this looks really good. I used flex instead of grid as I haven't got to grid yet. I also tried to do <picture> to swap images but couldn't get it to work so did a swap with ID's instead. It was good seeing it in your as it means it does work with local files and I just must have got the link wrong. HA HA One question (for my own learning) - any reason you used span instead of p for a couple of those items?
0@firdaus3Posted about 1 month agoFor the "Perfume" label, I decided to use a
span
because it’s just a short piece of content, not a full paragraph. As for the button, to be honest, I’m not sure why I used aspan
there either, it’s probably not necessary! Plain text would have worked just as well. Haha! Thanks for pointing that out. I just noticed it now. 😄0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord