Submitted about 1 month ago
Product List with Cart (VueJS, Pinia, TypeScript, A11Y)
@tinuola
Design comparison
SolutionDesign
Community feedback
- @christopher-adolphePosted 9 days ago
Hi @tinuola 👋
You did a great job on this challenge. 👍 I had a quick look the code base and here's small feedback:
- You could simplify the interface of the
ProductCard.vue
component by declaring only aproduct
props instead of each field of a product as follows:
defineProps({ product: Product; });
This will enhance readability.
<ProductCard v-for="product in products" :key="product.name" :class="{ selected: product.selected }" :product="product" />
I hope this helps.
Keep it up.
Marked as helpful1@christopher-adolphePosted 8 days ago@tinuola you are welcome.
I'm happy to help 🙂
0 - You could simplify the interface of the
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