search

astrajingga / template-sirclo-valentino

Apache License 2.0
0 stars 0 forks source link

CODE REVIEW (HOMEPAGE) #51

Open muhammadanova opened 2 years ago

muhammadanova commented 2 years ago

Issue 1 : image

Untuk penulisan className nya bisa diperbaiki biar ga terlalu sulit saat maintenance, saat perbaikan nama className nya akan impact juga di file scss, oleh karena itu jangan sampai kelupaan cek jg dengan file scss nya

Sample 1 productImageContainerClassName: "products__item--image-container" ini bisa menjadi productImageContainerClassName: "products__item--imageContainer

  1. products ini sebagai Block item Element imageContainer Modifier
  2. Saat maintenance kita bisa langsung cari di scss file imageContainer karena apabla image-container khawatir container masih nested dari image dan effort saat dicari

Issue 2 : image

Untuk widget diatas diperuntukan untuk apa kalo boleh tau? boleh dicoba input widget image full container di pos home advertisement 2, khawatir ini thumbor setting nya ga mendukung untuk image, dan perlu diperhatikan size nya kalo nanti ada perbaikan

Issue 3 : Apakah staging nya update? karena saya liat di staging ada Product Terkait, tp dari sisi code nya tidak ada

Issue 4 : image Untuk import Router bisa diubah dengan menggunakan useRouter hooks, dan penggunaan router push nya bisa cek docs nya ada disini https://nextjs.org/docs/api-reference/next/router