@graphcommerce/magento-product 3.7.0 → 3.7.4

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.7.2](https://github.com/ho-nl/m2-pwa/compare/@graphcommerce/magento-product@3.7.1...@graphcommerce/magento-product@3.7.2) (2021-12-03)
+
+
+### Bug Fixes
+
+* make sure elements that use scroll positioning are using the right hook ([80ee33f](https://github.com/ho-nl/m2-pwa/commit/80ee33f68ecc223147d63fc37e5b9ba94df4564e))
+
+
+
+
+
 # [3.7.0](https://github.com/ho-nl/m2-pwa/compare/@graphcommerce/magento-product@3.6.4...@graphcommerce/magento-product@3.7.0) (2021-12-01)
 
 

package/components/ProductListFilters/FilterCheckboxType.tsx

@@ -1,5 +1,5 @@
 import { cloneDeep } from '@apollo/client/utilities'
-import { useChipMenuStyles, SvgImage, iconCancelAlt, SvgImageSimple } from '@graphcommerce/next-ui'
+import { useChipMenuStyles, iconCancelAlt, SvgImageSimple } from '@graphcommerce/next-ui'
 import { Chip, ChipProps } from '@material-ui/core'
 import clsx from 'clsx'
 import React from 'react'

package/components/ProductListFilters/FilterEqualType.tsx

@@ -3,7 +3,6 @@ import { FilterEqualTypeInput } from '@graphcommerce/graphql'
 import { ChipMenu, ChipMenuProps, responsiveVal } from '@graphcommerce/next-ui'
 import {
   Checkbox,
-  lighten,
   ListItem,
   ListItemSecondaryAction,
   ListItemText,

package/components/ProductListFilters/index.tsx

@@ -53,7 +53,11 @@ export default function ProductListFilters(props: ProductFiltersProps) {
               />
             )
         }
-        console.log('Filter not recognized', aggregation)
+        console.log(
+          'Filter not recognized',
+          aggregation.attribute_code,
+          filterTypes[aggregation.attribute_code],
+        )
         return null // `FilterMatchTypeInput not ${aggregation.attribute_code}`
       })}
     </>

package/components/ProductListFiltersContainer/index.tsx

@@ -4,11 +4,12 @@ import {
   iconChevronRight,
   responsiveVal,
   SvgImageSimple,
+  useScrollY,
   UseStyles,
 } from '@graphcommerce/next-ui'
 import { makeStyles, Theme } from '@material-ui/core'
 import clsx from 'clsx'
-import { m, useMotionTemplate, useTransform, useViewportScroll } from 'framer-motion'
+import { m, useTransform } from 'framer-motion'
 import React, { PropsWithChildren, useEffect, useRef, useState } from 'react'
 
 const useStyles = makeStyles(
@@ -91,7 +92,7 @@ export type ProductListFiltersContainerProps = PropsWithChildren<UseStyles<typeo
 export default function ProductListFiltersContainer(props: ProductListFiltersContainerProps) {
   const { children } = props
   const classes = useStyles(props)
-  const { scrollY } = useViewportScroll()
+  const scrollY = useScrollY()
 
   const [isSticky, setIsSticky] = useState<boolean>(false)
   const [startPosition, setStartPosition] = useState<number>(100)
@@ -148,7 +149,7 @@ export default function ProductListFiltersContainer(props: ProductListFiltersCon
           >
             {children}
           </Scroller>
-          <m.div className={classes.shadow} style={{ opacity }}></m.div>
+          <m.div className={classes.shadow} style={{ opacity }} />
         </div>
         <ScrollerButton direction='right' className={classes.sliderNext}>
           <SvgImageSimple src={iconChevronRight} />

package/components/ProductListItem/index.tsx

@@ -1,9 +1,9 @@
+import { usePageRouter } from '@graphcommerce/framer-next-pages'
 import { Image, ImageProps } from '@graphcommerce/image'
 import { responsiveVal, UseStyles } from '@graphcommerce/next-ui'
-import { ButtonBase, Link as MuiLink, makeStyles, Theme, Typography } from '@material-ui/core'
+import { ButtonBase, makeStyles, Theme, Typography } from '@material-ui/core'
 import clsx from 'clsx'
 import PageLink from 'next/link'
-import { useRouter } from 'next/router'
 import React, { PropsWithChildren, useCallback } from 'react'
 import { ProductListItemFragment } from '../../Api/ProductListItem.gql'
 import { useProductLink } from '../../hooks/useProductLink'
@@ -157,7 +157,7 @@ export default function ProductListItem(props: ProductListItemProps) {
   const classes = useProductListItemStyles(props)
   const productLink = useProductLink(props)
   const discount = Math.floor(price_range.minimum_price.discount?.percent_off ?? 0)
-  const { locale } = useRouter()
+  const { locale } = usePageRouter()
 
   // eslint-disable-next-line react-hooks/exhaustive-deps
   const format = useCallback(

package/components/ProductListItems/ProductListItemsBase.tsx

@@ -1,4 +1,3 @@
-import { Maybe } from '@graphcommerce/graphql'
 import { RenderType, UseStyles, responsiveVal } from '@graphcommerce/next-ui'
 import { Theme, makeStyles } from '@material-ui/core'
 import clsx from 'clsx'

package/components/ProductListItems/ProductListParamsProvider.tsx

@@ -1,4 +1,4 @@
-import { PropsWithChildren, useState, useEffect } from 'react'
+import { PropsWithChildren, useState, useEffect, useMemo } from 'react'
 import { productListParamsContext } from '../../context/productListParamsContext'
 import { ProductListParams } from './filterTypes'
 
@@ -8,13 +8,11 @@ export default function ProductListParamsProvider({
 }: PropsWithChildren<{ value: ProductListParams }>) {
   const [params, setParams] = useState<ProductListParams>(value)
 
-  useEffect(() => {
-    setParams(value)
-  }, [value])
+  useEffect(() => setParams(value), [value])
+
+  const prov = useMemo(() => ({ params, setParams }), [params])
 
   return (
-    <productListParamsContext.Provider value={{ params, setParams }}>
-      {children}
-    </productListParamsContext.Provider>
+    <productListParamsContext.Provider value={prov}>{children}</productListParamsContext.Provider>
   )
 }

package/components/ProductListSort/index.tsx

@@ -53,7 +53,7 @@ export default function ProductListSort(props: ProductListSortProps) {
             key={option?.value ?? ''}
             dense
             selected={option?.value === currentSort}
-            component={(chipProps) => (
+            component={React.memo((chipProps) => (
               <ProductListLink
                 {...chipProps}
                 {...linkParams}
@@ -61,7 +61,7 @@ export default function ProductListSort(props: ProductListSortProps) {
                 underline='none'
                 link={{ scroll: false, replace: true }}
               />
-            )}
+            ))}
           >
             <ListItemText secondary>{option?.label}</ListItemText>
           </ListItem>

package/components/ProductPageDescription/index.tsx

@@ -41,23 +41,22 @@ export default function ProductPageDescription(props: ProductPageDescriptionProp
   const { description, name, right } = props
 
   return (
-    <>
-      <ColumnTwoWithTop
-        top={
-          <Typography variant='h1' component='h2'>
-            {name}
-          </Typography>
-        }
-        left={
-          description && (
-            <div
-              className={classes.description}
-              dangerouslySetInnerHTML={{ __html: description.html }}
-            />
-          )
-        }
-        right={right}
-      />
-    </>
+    <ColumnTwoWithTop
+      top={
+        <Typography variant='h1' component='h2'>
+          {name}
+        </Typography>
+      }
+      left={
+        description && (
+          <div
+            className={classes.description}
+            // eslint-disable-next-line react/no-danger
+            dangerouslySetInnerHTML={{ __html: description.html }}
+          />
+        )
+      }
+      right={right}
+    />
   )
 }

package/components/ProductShortDescription/index.tsx

@@ -1,19 +1,17 @@
 import { UseStyles } from '@graphcommerce/next-ui'
-import { makeStyles, Theme, Typography } from '@material-ui/core'
+import { makeStyles, Typography } from '@material-ui/core'
 import React from 'react'
 import { ProductShortDescriptionFragment } from './ProductShortDescription.gql'
 
 const useStyles = makeStyles(
-  (theme: Theme) => ({
+  {
     root: {
       '& > p': {
         marginTop: 0,
       },
     },
-  }),
-  {
-    name: 'ProductShortDescription',
   },
+  { name: 'ProductShortDescription' },
 )
 
 type ProductShortDescriptionProps = ProductShortDescriptionFragment & UseStyles<typeof useStyles>

package/components/ProductSidebarDelivery/index.tsx

@@ -1,4 +1,4 @@
-import { SvgImage, responsiveVal, iconOrderBefore, SvgImageSimple } from '@graphcommerce/next-ui'
+import { responsiveVal, iconOrderBefore, SvgImageSimple } from '@graphcommerce/next-ui'
 import { darken, lighten, makeStyles, Theme, Typography } from '@material-ui/core'
 import React from 'react'
 

package/hooks/useProductListLinkReplace.ts

@@ -1,4 +1,4 @@
-import { default as Router } from 'next/router'
+import { useRouter } from 'next/router'
 import { ProductListParams } from '../components/ProductListItems/filterTypes'
 import { createProductListLink } from './useProductListLink'
 import { useProductListParamsContext } from './useProductListParamsContext'
@@ -11,9 +11,10 @@ type UseProductLinkPushProps = {
 
 export function useProductListLinkReplace(props?: UseProductLinkPushProps) {
   const { setParams } = useProductListParamsContext()
+  const router = useRouter()
 
   return (params: ProductListParams) => {
-    const queryUrl = Router.query.url ?? []
+    const queryUrl = router.query.url ?? []
     const comingFromURLWithoutFilters = !queryUrl.includes('q')
 
     setParams(params)
@@ -23,10 +24,10 @@ export function useProductListLinkReplace(props?: UseProductLinkPushProps) {
     // push the first filter, so the new route (on browser back) will be e.g. /women/fruit instead of /women
     if (comingFromURLWithoutFilters) {
       // eslint-disable-next-line @typescript-eslint/no-floating-promises
-      Router.push(path, path, props)
+      router.push(path, path, props)
     } else {
       // eslint-disable-next-line @typescript-eslint/no-floating-promises
-      Router.replace(path, path, props)
+      router.replace(path, path, props)
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphcommerce/magento-product",
-  "version": "3.7.0",
+  "version": "3.7.4",
   "sideEffects": false,
   "prettier": "@graphcommerce/prettier-config-pwa",
   "browserslist": [
@@ -14,19 +14,19 @@
   },
   "devDependencies": {
     "@graphcommerce/browserslist-config-pwa": "^3.0.2",
-    "@graphcommerce/eslint-config-pwa": "^3.1.5",
+    "@graphcommerce/eslint-config-pwa": "^3.1.6",
     "@graphcommerce/prettier-config-pwa": "^3.0.4",
     "@graphcommerce/typescript-config-pwa": "^3.1.1",
     "@playwright/test": "^1.16.2"
   },
   "dependencies": {
     "@apollo/client": "^3.4.16",
-    "@graphcommerce/framer-scroller": "^1.0.4",
-    "@graphcommerce/graphql": "^2.105.5",
-    "@graphcommerce/image": "^2.105.4",
-    "@graphcommerce/magento-cart": "^3.8.0",
-    "@graphcommerce/magento-store": "^3.3.5",
-    "@graphcommerce/next-ui": "^3.19.0",
+    "@graphcommerce/framer-scroller": "^1.1.2",
+    "@graphcommerce/graphql": "^2.105.6",
+    "@graphcommerce/image": "^2.105.5",
+    "@graphcommerce/magento-cart": "^3.8.4",
+    "@graphcommerce/magento-store": "^3.3.9",
+    "@graphcommerce/next-ui": "^3.20.2",
     "@lingui/macro": "^3.12.1",
     "@material-ui/core": "^4.12.3",
     "@material-ui/lab": "^4.0.0-alpha.60",
@@ -38,5 +38,5 @@
     "schema-dts": "^1.0.0",
     "type-fest": "^2.5.1"
   },
-  "gitHead": "f079c0b5b96dd1a048ca4d3c1d07916353c6bc33"
+  "gitHead": "688a7042901bf36f8ab8bbf8e1886570a5350b9e"
 }