search

kirill-zhirnov / boundless-nextjs-ecommerce-template

Next.js E-Commerce template. Strong TypeScript, Ready to use, For high-performance e-commerce websites.
https://boundless-commerce.com
MIT License
114 stars 50 forks source link

34 product page improvements #64

Closed keXXpert closed 2 years ago

keXXpert commented 2 years ago

closes #34 closes #23 closes #57 closes #39 closes #41

keXXpert commented 2 years ago

Похоже, что изображения для photoswipe загружаются через lazyLoad. По крайней мере на development.

keXXpert commented 2 years ago

По swiper:

  1. Вынести css в динамически подгружаемый компонент не дает. Ругается, что нужно в _app.tsx. Есть подозрение, что это из-за наименования самих файлов (не прямого импорта файла), потому что в соседнем файле спокойно импортит css от photoswipe. Там есть обходные "костыльные" пути типа локальных cssModules. Но что-то мне кажется, это совсем криво смотреться будет. Да и еще не факт, что будет работать.
  2. Обертку можно сделать (например в универсальный imageSlider просто передавать компонент самого слайда и настройки), но, в целом, думаю, не должно быть проблем два раза заимпортить динамически компонент. Т.е. получится еще один чанк для карусели. Предлагаю к этому вопросу вернуться, когда карусель будет, чтобы было понятно, что именно в этой обертке должно быть.