agroptima-design-system 1.2.30-beta.0 → 1.2.30

package/.github/workflows/publish-package-to-npmjs.yml

@@ -17,7 +17,7 @@ jobs:
       # Setup .npmrc file to publish to npm
       - uses: actions/setup-node@v6
         with:
-          node-version: '20.x'
+          node-version: 'lts/*'
           registry-url: 'https://registry.npmjs.org'
 
       # Ensure npm 11.5.1 or later is installed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agroptima-design-system",
-  "version": "1.2.30-beta.0",
+  "version": "1.2.30",
   "scripts": {
     "dev": "npm run storybook",
     "storybook": "storybook dev -p 6006 --ci",

package/src/atoms/Button/BaseButton.tsx

@@ -13,8 +13,7 @@ type HtmlButtonProps = ButtonHTMLAttributes<HTMLButtonElement>
 type AnchorProps = AnchorHTMLAttributes<HTMLAnchorElement> & LinkProps
 
 export type BaseButtonProps =
-  | (HtmlButtonProps & CommonProps)
-  | (AnchorProps & CommonProps)
+  (HtmlButtonProps & CommonProps) | (AnchorProps & CommonProps)
 
 const hasHref = (
   props: HtmlButtonProps | AnchorProps,

package/src/atoms/Collapsible/Collapsible.tsx

@@ -27,8 +27,7 @@ export function Collapsible({
   onToggle,
   ...props
 }: CollapsibleProps) {
-  const [expanded, setExpanded] = useState(false)
-  const isOpen = controlledOpen !== undefined ? controlledOpen : expanded
+  const [isOpen, setIsOpen] = useState(controlledOpen ?? false)
 
   const cssClasses = classNames('collapsible', variant, className, {
     open: isOpen,
@@ -46,7 +45,7 @@ export function Collapsible({
       className={cssClasses}
       aria-label={title}
       onToggle={(e) => {
-        if (controlledOpen === undefined) setExpanded(e.currentTarget.open)
+        setIsOpen(e.currentTarget.open)
         onToggle?.(e)
       }}
       {...props}

package/src/atoms/Modal/Modal.tsx

@@ -13,12 +13,7 @@ import {
 } from '.'
 
 export type Variant =
-  | 'info'
-  | 'success'
-  | 'warning'
-  | 'error'
-  | 'discard'
-  | 'details'
+  'info' | 'success' | 'warning' | 'error' | 'discard' | 'details'
 
 export interface ModalProps {
   id: string

package/src/stories/Changelog.mdx

@@ -4,6 +4,14 @@ import { Meta } from "@storybook/addon-docs/blocks";
 
 # Changelog
 
+## 1.2.30
+
+* Fix Webpack icons exports bug
+
+## 1.2.29
+
+* Fix Collapsible styles not updating when closing a default-open collapsible
+
 ## 1.2.28
 
 * Add Link icon

package/tests/Collapsible.spec.tsx

@@ -1,4 +1,5 @@
-import { render } from '@testing-library/react'
+import { render, waitFor } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
 import React from 'react'
 import { Collapsible } from '../src/atoms/Collapsible'
 import { Input } from '../src/atoms/Input'
@@ -24,4 +25,50 @@ describe('Collapsible', () => {
     expect(getByRole('textbox')).toBeInTheDocument()
     expect(getByRole('group')).toHaveClass(`collapsible primary open`)
   })
+
+  it('removes the open styles when a default-open collapsible is closed', async () => {
+    const user = userEvent.setup()
+    const { getByRole } = render(
+      <Collapsible title="My personal data" name="personal-data" open>
+        <Input
+          accessibilityLabel="Fill the form name"
+          id="name_input"
+          label="Name"
+          name="name"
+          type="name"
+          variant="primary"
+        />
+      </Collapsible>,
+    )
+
+    const group = getByRole('group')
+    expect(group).toHaveClass('open')
+
+    await user.click(group.querySelector('summary')!)
+
+    await waitFor(() => expect(group).not.toHaveClass('open'))
+  })
+
+  it('adds the open styles when a closed collapsible is opened', async () => {
+    const user = userEvent.setup()
+    const { getByRole } = render(
+      <Collapsible title="My personal data" name="personal-data">
+        <Input
+          accessibilityLabel="Fill the form name"
+          id="name_input"
+          label="Name"
+          name="name"
+          type="name"
+          variant="primary"
+        />
+      </Collapsible>,
+    )
+
+    const group = getByRole('group')
+    expect(group).not.toHaveClass('open')
+
+    await user.click(group.querySelector('summary')!)
+
+    await waitFor(() => expect(group).toHaveClass('open'))
+  })
 })

package/.claude/settings.local.json

@@ -1,7 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(git rev-list *)"
-    ]
-  }
-}