Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

34 product page improvements #64

Merged
merged 7 commits into from
Feb 17, 2022
Merged

34 product page improvements #64

merged 7 commits into from
Feb 17, 2022

Conversation

keXXpert
Copy link
Collaborator

@keXXpert keXXpert commented Feb 16, 2022

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

@keXXpert
Copy link
Collaborator Author

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

@keXXpert
Copy link
Collaborator Author

keXXpert commented Feb 16, 2022

По swiper:

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

@kirill-zhirnov kirill-zhirnov merged commit 7d5a9d6 into master Feb 17, 2022
@kirill-zhirnov kirill-zhirnov deleted the 34_product_styles branch February 17, 2022 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment