cozy-ui 84.1.1 → 84.1.3

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [84.1.3](https://github.com/cozy/cozy-ui/compare/v84.1.2...v84.1.3) (2023-04-27)
+
+
+### Bug Fixes
+
+* **ActionsMenu actions:** Stop spreading `t`, `f`, `lang` into child ([4fa0cc0](https://github.com/cozy/cozy-ui/commit/4fa0cc0))
+* **ActionsMenuItem:** We used transpiled version of imports, and... ([233ae1f](https://github.com/cozy/cozy-ui/commit/233ae1f))
+* **ListItem:** Stop spreading `gutters` and `ellipsis` to children ([0de5e66](https://github.com/cozy/cozy-ui/commit/0de5e66))
+* **ListItemText:** DisplayName and forwardRef was missing ([a70322c](https://github.com/cozy/cozy-ui/commit/a70322c))
+
+## [84.1.2](https://github.com/cozy/cozy-ui/compare/v84.1.1...v84.1.2) (2023-04-27)
+
+
+### Bug Fixes
+
+* ActionsMenu target wasn't good in global import ([cac334a](https://github.com/cozy/cozy-ui/commit/cac334a))
+
 ## [84.1.1](https://github.com/cozy/cozy-ui/compare/v84.1.0...v84.1.1) (2023-04-27)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-ui",
-  "version": "84.1.1",
+  "version": "84.1.3",
   "description": "Cozy apps UI SDK",
   "main": "./index.js",
   "bin": {

package/react/ActionsMenu/Actions/call.js

@@ -15,7 +15,8 @@ export const call = () => {
       !!phoneNumber && window.open(`tel:${phoneNumber}`, '_self')
     },
     Component: withActionsLocales(
-      forwardRef(({ t, ...props }, ref) => {
+      // eslint-disable-next-line no-unused-vars
+      forwardRef(({ t, f, lang, ...props }, ref) => {
         return (
           <ActionsMenuItem {...props} ref={ref}>
             <ListItemIcon>

package/react/ActionsMenu/Actions/emailTo.js

@@ -15,7 +15,8 @@ export const emailTo = () => {
       !!emailAddress && window.open(`mailto:${emailAddress}`, '_self')
     },
     Component: withActionsLocales(
-      forwardRef(({ t, ...props }, ref) => {
+      // eslint-disable-next-line no-unused-vars
+      forwardRef(({ t, f, lang, ...props }, ref) => {
         return (
           <ActionsMenuItem {...props} ref={ref}>
             <ListItemIcon>

package/react/ActionsMenu/Actions/locales/withActionsLocales.jsx

@@ -8,4 +8,9 @@ export const locales = {
   fr
 }
 
+/**
+ * TODO: We should use something else than withLocales here. Because it's an HOC
+ * so it spreads t, f, lang into child props.
+ * We don't use HOC anymore, so we just want to use new I18n context with new locales
+ */
 export default withLocales(locales)

package/react/ActionsMenu/Actions/modify.js

@@ -26,7 +26,8 @@ export const modify = () => {
       window.open(webLink, '_blank')
     },
     Component: withActionsLocales(
-      forwardRef(({ t, ...props }, ref) => {
+      // eslint-disable-next-line no-unused-vars
+      forwardRef(({ t, f, lang, ...props }, ref) => {
         return (
           <ActionsMenuItem ref={ref} {...props}>
             <ListItemIcon>

package/react/ActionsMenu/Actions/smsTo.js

@@ -15,7 +15,8 @@ export const smsTo = () => {
       !!phoneNumber && window.open(`sms:${phoneNumber}`, '_self')
     },
     Component: withActionsLocales(
-      forwardRef(({ t, ...props }, ref) => {
+      // eslint-disable-next-line no-unused-vars
+      forwardRef(({ t, f, lang, ...props }, ref) => {
         return (
           <ActionsMenuItem {...props} ref={ref}>
             <ListItemIcon>

package/react/ActionsMenu/Actions/viewInContacts.js

@@ -26,7 +26,8 @@ export const viewInContacts = () => {
       window.open(webLink, '_blank')
     },
     Component: withActionsLocales(
-      forwardRef(({ t, ...props }, ref) => {
+      // eslint-disable-next-line no-unused-vars
+      forwardRef(({ t, f, lang, ...props }, ref) => {
         return (
           <ActionsMenuItem ref={ref} {...props}>
             <ListItemIcon>

package/react/ActionsMenu/ActionsMenuItem.jsx

@@ -1,8 +1,8 @@
 import React, { forwardRef } from 'react'
 import PropTypes from 'prop-types'
 
-import MenuItem from 'cozy-ui/transpiled/react/MenuItem'
-import ListItem from 'cozy-ui/transpiled/react/MuiCozyTheme/ListItem'
+import MenuItem from '../MenuItem'
+import ListItem from '../MuiCozyTheme/ListItem'
 
 const ActionsMenuItem = forwardRef(({ isListItem, ...props }, ref) => {
   const Component = isListItem ? ListItem : MenuItem
@@ -10,6 +10,8 @@ const ActionsMenuItem = forwardRef(({ isListItem, ...props }, ref) => {
   return <Component {...props} ref={ref} button ellipsis={false} />
 })
 
+ActionsMenuItem.displayName = 'ActionsMenuItem'
+
 ActionsMenuItem.propTypes = {
   /** Whether the ActionsMenuItem will return a ListItem or MenuItem */
   isListItem: PropTypes.bool

package/react/ActionsMenu/Readme.md

@@ -75,7 +75,10 @@ const customAction = () => ({
         <ListItemIcon>
           <Icon icon={FileIcon} />
         </ListItemIcon>
-        <ListItemText primary="This is a custom action, with a very long text to show how it is displayed" />
+        <ListItemText
+          primary="This is a custom action, with a very long text to show how it is displayed"
+          secondary="This is a custom action, with a very long text to show how it is displayed"
+        />
       </ActionsMenuItem>
     )
   })

package/react/ContactsList/__snapshots__/ContactRow.spec.js.snap

@@ -5,7 +5,6 @@ exports[`ContactRow should match the contact snapshot 1`] = `
   className="MuiListItem-root medium MuiListItem-gutters makeStyles-gutters-1 makeStyles-gutters-11"
   data-testid="contact-listItem"
   disabled={false}
-  gutters="double"
   onClick={[Function]}
   size="medium"
 >

package/react/ListItemText/index.jsx

@@ -1,4 +1,4 @@
-import React, { useEffect, useMemo } from 'react'
+import React, { forwardRef, useEffect, useMemo } from 'react'
 import cx from 'classnames'
 import MUIListItemText from '@material-ui/core/ListItemText'
 import once from 'lodash/once'
@@ -22,7 +22,7 @@ const getTypographyProp = (props, className, ellipsis) => {
       }
 }
 
-const ListItemText = props => {
+const ListItemText = forwardRef((props, ref) => {
   const {
     primaryText,
     secondaryText,
@@ -68,6 +68,7 @@ const ListItemText = props => {
 
   return (
     <MUIListItemText
+      ref={ref}
       primary={primary}
       secondary={secondary}
       primaryTypographyProps={primaryTypographyProps}
@@ -75,7 +76,9 @@ const ListItemText = props => {
       {...rest}
     />
   )
-}
+})
+
+ListItemText.displayName = 'ListItemText'
 
 ListItemText.defaultProps = {
   ellipsis: true

package/react/MuiCozyTheme/ListItem/index.js

@@ -79,25 +79,34 @@ const useOverridenChildren = ({ gutters, ellipsis, ...props }) => {
   })
 }
 
-const ListItem = forwardRef(({ className, ...props }, ref) => {
-  const secondaryActionPaddingRight = useSecondaryAction(props)
-  const guttersValue = useGutters(props)
-
-  const styles = useStyles({ secondaryActionPaddingRight, guttersValue })
-  const overridenChildren = useOverridenChildren(props)
-
-  return (
-    <MuiListItem
-      {...props}
-      ref={ref}
-      classes={merge(props.classes, styles)}
-      className={cx(className, props.size)}
-      disableGutters={props.disableGutters || props.gutters === 'disabled'}
-    >
-      {overridenChildren}
-    </MuiListItem>
-  )
-})
+const ListItem = forwardRef(
+  ({ className, gutters, ellipsis, ...props }, ref) => {
+    const secondaryActionPaddingRight = useSecondaryAction({
+      gutters,
+      ...props
+    })
+    const guttersValue = useGutters({ gutters, ...props })
+
+    const styles = useStyles({ secondaryActionPaddingRight, guttersValue })
+    const overridenChildren = useOverridenChildren({
+      gutters,
+      ellipsis,
+      ...props
+    })
+
+    return (
+      <MuiListItem
+        {...props}
+        ref={ref}
+        classes={merge(props.classes, styles)}
+        className={cx(className, props.size)}
+        disableGutters={props.disableGutters || gutters === 'disabled'}
+      >
+        {overridenChildren}
+      </MuiListItem>
+    )
+  }
+)
 
 ListItem.displayName = 'ListItem'