coopjakob
commented
4 years ago - ProductCard.spec.js Bra att ni har både Unit och E2E tester Detta test känns dock ganska centralt för ert projekt och är just nu tom.
- Grid.vue Finns en funktion som heter selectedCards() som är väldigt svårtläst. satsa på läsbarhet och maintainability.
- ProductImage.vue
Annorlunda struktur med annoteringar
*InfoCard.vue
ni har en svg icon för en pointer som man tryckt in i templaten,
tror inte ni återanvänder den änsålänge så den kanske inte behöver vara en egen icon-component,
men den tar kanske onödigt mycket plats i templaten. kunde istället vara
.. typ. - ProductCard.vue 'const cart = this.$store.state.cart' bör använda vuex Getter istället