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

15 product page #19

Merged
merged 27 commits into from
Jan 17, 2022
Merged

15 product page #19

merged 27 commits into from
Jan 17, 2022

Conversation

keXXpert
Copy link
Collaborator

@keXXpert keXXpert commented Jan 7, 2022

closes #8
closes #15
closes #18
closes #17
closes #16

@keXXpert
Copy link
Collaborator Author

keXXpert commented Jan 7, 2022

Добавил swiper. Лицензия вроде MIT.

@keXXpert
Copy link
Collaborator Author

keXXpert commented Jan 7, 2022

Сделал правильные ссылки на продукт и на предыдущую категорию. Обновил breadcrumbs.

components/FilterForm.tsx Outdated Show resolved Hide resolved
pages/product/[slug].tsx Outdated Show resolved Hide resolved
components/product/ProductImages.tsx Show resolved Hide resolved
@kirill-zhirnov
Copy link
Owner

kirill-zhirnov commented Jan 10, 2022

  • Out of stock показывваем только если выбран вариант. Пока пересечение не выбрана, показывать не надо:

Снимок экрана 2022-01-10 в 12 28 49

  • Тоже самое при выборе варианта

  • Нужен imgPreloader - при загрузке страницы запускаем прелоад для всех больших изображений. Сейчас, если сразу после загрузке начать елозить мышкой по миниатюрам - большая фотография не меняется - кажется, что это баг. С другой стороны, должна же сперва показываться blured фото, но у меня при наведении не показывается. Возможно баг - нужно поковыряться.

  • Цену выводим красной (выделенной) только если это скидка. Если скидки нет, то и выделять нет смысла:

Снимок экрана 2022-01-10 в 12 40 33

  • При установке куки, нужно SameSite передавать, сейчас предупреждение:

Снимок экрана 2022-01-10 в 12 32 31

  • На странице товара добавить canonical тег: <link rel="canonical" href="https://fun-look.ru/product/kigurumi-tigr-zheltyi"> - это абсолютная ссылка на товар, которая должна быть в кеше поисковика, те без ?category и проч параметров.

  • Добавь теги og:type, og:title, og:url, og:image - к основному сайту. При переходе на товар меняем эти параметры. Пропорции изображений для og:image посмотри на фанлуке - скорее всего это квадрат.

  • merge with master and resolve conflicts

  • В корзине нужно также указывать вариант, второй строчкой, text-muted - см fun-look.ru

  • В списке товара ошибка с ценой в schema.org, если у товара нет вариантов:

Снимок экрана 2022-01-10 в 13 35 43

Фильтры пока работают с ошибкой:

  • во первых, отметили галочками неск. полей - ушел запрос на filter-fields - подсчиталось кол-во. Нажали искать - опять ушел запрос на filter-fields, зачем? ведь результат будет такой же.
  • Отфильтровали в родительской категории, затем перешел в дочернюю:

Снимок экрана 2022-01-10 в 13 45 51

@kirill-zhirnov
Copy link
Owner

kirill-zhirnov commented Jan 10, 2022

@kirill-zhirnov kirill-zhirnov merged commit c608f7b into master Jan 17, 2022
@kirill-zhirnov kirill-zhirnov deleted the 15_product_page branch January 17, 2022 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants