diff --git a/.changeset/cool-clubs-think.md b/.changeset/cool-clubs-think.md new file mode 100644 index 00000000000..f025ef30476 --- /dev/null +++ b/.changeset/cool-clubs-think.md @@ -0,0 +1,5 @@ +--- +'@primer/react': major +--- + +Removes Box usage and sx prop from NavList and ActionList diff --git a/.changeset/gold-geckos-send.md b/.changeset/gold-geckos-send.md new file mode 100644 index 00000000000..5e3441f4a4a --- /dev/null +++ b/.changeset/gold-geckos-send.md @@ -0,0 +1,5 @@ +--- +'@primer/react': major +--- + +Removes sx prop from PageLayout and subcomponents diff --git a/.changeset/light-schools-wish.md b/.changeset/light-schools-wish.md new file mode 100644 index 00000000000..7368a2e3133 --- /dev/null +++ b/.changeset/light-schools-wish.md @@ -0,0 +1,5 @@ +--- +'@primer/react': major +--- + +Removes `Box` component usage and `sx` prop from the `Link` component, Storybook stories, and a .figma.tsx file diff --git a/.changeset/lovely-plums-buy.md b/.changeset/lovely-plums-buy.md new file mode 100644 index 00000000000..d97dc239e2d --- /dev/null +++ b/.changeset/lovely-plums-buy.md @@ -0,0 +1,5 @@ +--- +"@primer/react": major +--- + +Remove support for styled-components/sx from ActionList component and sub-components diff --git a/.changeset/lucky-wasps-nail.md b/.changeset/lucky-wasps-nail.md new file mode 100644 index 00000000000..37174b1f61f --- /dev/null +++ b/.changeset/lucky-wasps-nail.md @@ -0,0 +1,6 @@ +--- +"@primer/react": major +"@primer/styled-react": patch +--- + +Remove support for `sx` from the `TabNav` component diff --git a/.changeset/pre.json b/.changeset/pre.json index fa447ffe3d3..29a6d5e10b7 100644 --- a/.changeset/pre.json +++ b/.changeset/pre.json @@ -12,10 +12,14 @@ "@primer/styled-react": "0.1.0" }, "changesets": [ + "beige-plums-obey", + "beige-shrimps-sleep", + "better-coins-dream", "big-oranges-marry", "bright-dolls-dance", "bright-parents-flow", "busy-bananas-smash", + "chilly-dingos-build", "chubby-colts-nail", "cold-teams-buy", "cruel-papers-film", @@ -24,20 +28,30 @@ "deprecate-textinputwithtokens", "dirty-chairs-invent", "eighty-queens-tap", + "fair-views-laugh", "few-mails-sing", + "fifty-walls-yawn", "five-squids-sin", + "fresh-points-cover", "gentle-rockets-go", + "gentle-sheep-feel", + "giant-impalas-occur", "gold-dingos-sneeze", "great-hats-serve", "hot-bears-cry", "hot-melons-jump", + "hot-mirrors-feel", "icy-pianos-say", "lemon-jokes-camp", + "light-colts-burn", "little-peas-mix", "lovely-vans-burn", "metal-badgers-joke", "metal-cups-peel", + "metal-deer-refuse", "metal-lamps-appear", + "mighty-lizards-lick", + "nasty-seals-carry", "new-dragons-melt", "new-shrimps-appear", "nine-cobras-talk", @@ -45,25 +59,43 @@ "pretty-women-sing", "proud-chairs-study", "rich-walls-fold", + "rotten-carpets-raise", + "rude-cycles-scream", "salty-geese-own", "seven-coats-hang", "seven-forks-join", + "sharp-beans-run", + "sharp-papayas-destroy", + "shiny-deer-lose", + "short-cloths-deny", + "shy-flies-marry", + "silent-wasps-warn", "silly-parks-give", "slick-months-cry", + "smooth-singers-thank", "soft-islands-warn", "spotty-colts-hear", + "stale-avocados-enjoy", "stale-wasps-smell", "strange-knives-study", "strange-peas-kiss", "strong-falcons-design", "strong-lions-tan", + "tame-ducks-ring", + "tasty-cheetahs-pay", + "ten-jobs-attack", "thin-oranges-lick", "three-hounds-refuse", + "three-vans-join", "tricky-eggs-cross", + "twelve-sloths-flow", "twenty-baboons-pull", "unlucky-hotels-shake", + "upset-flowers-yawn", "wet-crews-drum", + "wet-mammals-feel", "wet-terms-argue", + "wicked-starfishes-shout", "witty-chicken-leave", "young-dots-heal", "young-timers-accept" diff --git a/.changeset/pretty-emus-cover.md b/.changeset/pretty-emus-cover.md new file mode 100644 index 00000000000..d92a66b9daf --- /dev/null +++ b/.changeset/pretty-emus-cover.md @@ -0,0 +1,5 @@ +--- +"@primer/react": patch +--- + +update FormControl ValidationIcon position diff --git a/.changeset/sweet-islands-sleep.md b/.changeset/sweet-islands-sleep.md new file mode 100644 index 00000000000..25ca2791643 --- /dev/null +++ b/.changeset/sweet-islands-sleep.md @@ -0,0 +1,5 @@ +--- +'@primer/react': patch +--- + +Replace StateLabel color with new draft token diff --git a/.changeset/sweet-results-smell.md b/.changeset/sweet-results-smell.md new file mode 100644 index 00000000000..63155480051 --- /dev/null +++ b/.changeset/sweet-results-smell.md @@ -0,0 +1,5 @@ +--- +'@primer/react': major +--- + +Remove support for `sx` from the Dialog component and sub-components diff --git a/.changeset/yummy-years-greet.md b/.changeset/yummy-years-greet.md new file mode 100644 index 00000000000..3e2db59c156 --- /dev/null +++ b/.changeset/yummy-years-greet.md @@ -0,0 +1,5 @@ +--- +'@primer/react': major +--- + +Removes usage of Box component from other components. diff --git a/.github/workflows/graphql-test.yml b/.github/workflows/graphql-test.yml new file mode 100644 index 00000000000..f1969b8a870 --- /dev/null +++ b/.github/workflows/graphql-test.yml @@ -0,0 +1,80 @@ +name: Test GraphQL Queries +on: + workflow_dispatch: + +jobs: + test-query: + runs-on: ubuntu-latest + steps: + - name: Test GraphQL Query + run: | + echo "Testing GraphQL query against GitHub API..." + + # Test query to get repository information + QUERY='query { + repository(owner: "primer", name: "react") { + name + description + stargazerCount + forkCount + defaultBranchRef { + name + } + languages(first: 5) { + nodes { + name + color + } + } + } + }' + + # Make the GraphQL API call with detailed error logging + echo "Making GraphQL API call..." + HTTP_STATUS=$(curl -w "%{http_code}" -s -X POST \ + -H "Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}" \ + -H "Content-Type: application/json" \ + -d "{\"query\": \"$(echo $QUERY | tr '\n' ' ' | sed 's/"/\\"/g')\"}" \ + -o response.json \ + https://api.github.com/graphql) + + RESPONSE=$(cat response.json) + + echo "HTTP Status Code: $HTTP_STATUS" + echo "Full GraphQL Response:" + echo "$RESPONSE" | jq '.' || echo "Raw response (invalid JSON): $RESPONSE" + + # Check HTTP status first + if [ "$HTTP_STATUS" != "200" ]; then + echo "❌ HTTP request failed with status code: $HTTP_STATUS" + echo "Full response body:" + echo "$RESPONSE" + exit 1 + fi + + # Check if the response contains GraphQL errors + if echo "$RESPONSE" | jq -e '.errors' > /dev/null 2>&1; then + echo "❌ GraphQL query returned errors:" + echo "$RESPONSE" | jq '.errors' + echo "Full error details:" + echo "$RESPONSE" | jq '.errors[] | {message: .message, type: .type, path: .path, locations: .locations}' || true + exit 1 + fi + + # Check if the query was successful and has expected data + if echo "$RESPONSE" | jq -e '.data.repository.name' > /dev/null 2>&1; then + echo "✅ GraphQL query executed successfully!" + echo "Repository name: $(echo "$RESPONSE" | jq -r '.data.repository.name')" + echo "Stars: $(echo "$RESPONSE" | jq -r '.data.repository.stargazerCount')" + echo "Default branch: $(echo "$RESPONSE" | jq -r '.data.repository.defaultBranchRef.name')" + else + echo "❌ GraphQL query failed - no valid data returned" + echo "Expected data.repository.name but got:" + echo "$RESPONSE" | jq '.data // "No data field found"' + exit 1 + fi + + # Cleanup + rm -f response.json + env: + GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} diff --git a/.github/workflows/lock-release.yml b/.github/workflows/lock-release.yml index 9de1283379f..9a588d0590c 100644 --- a/.github/workflows/lock-release.yml +++ b/.github/workflows/lock-release.yml @@ -2,7 +2,9 @@ name: Toggle Release Lock on: release: - types: [released] + types: + - released + - prereleased workflow_dispatch: inputs: action: @@ -78,7 +80,7 @@ jobs: GITHUB_TOKEN: ${{ steps.app-token.outputs.token }} - name: Update all PRs that are toggled merge when ready run: | - PR_NUMBERS=$(gh pr list -L 100 -R primer/react --state open --json number,baseRefName,autoMergeRequest,mergeStateStatus,reviewDecision -q '.[] | select(.autoMergeRequest != null) | select(.baseRefName == "main") | select(.mergeStateStatus == "BEHIND") | select(.reviewDecision == "APPROVED") | .number') + PR_NUMBERS=$(gh pr list -L 100 -R primer/react --state open --json number,baseRefName,autoMergeRequest,reviewDecision -q '.[] | select(.autoMergeRequest != null) | select(.baseRefName == "main") | select(.reviewDecision == "APPROVED") | .number') if [ -n "$PR_NUMBERS" ]; then echo "Updating $PR_NUMBERS" echo "$PR_NUMBERS" | xargs -I {} gh pr update-branch -R primer/react {} diff --git a/e2e/components/Dialog.test.ts b/e2e/components/Dialog.test.ts index 84365d50f31..f622e770b35 100644 --- a/e2e/components/Dialog.test.ts +++ b/e2e/components/Dialog.test.ts @@ -27,18 +27,6 @@ const stories = [ title: 'Position sidesheet', id: 'components-dialog-features--side-sheet', }, - { - title: 'Dev: With Css', - id: 'components-dialog-dev--with-css', - }, - { - title: 'Dev: With Sx', - id: 'components-dialog-dev--with-sx', - }, - { - title: 'Dev: With Sx And Css', - id: 'components-dialog-dev--with-sx-and-css', - }, ] as const test.describe('Dialog', () => { diff --git a/examples/codesandbox/package.json b/examples/codesandbox/package.json index 72d87534498..c72b18fef1e 100644 --- a/examples/codesandbox/package.json +++ b/examples/codesandbox/package.json @@ -17,7 +17,7 @@ "@types/react": "^18.3.11", "@types/react-dom": "^18.3.0", "@vitejs/plugin-react": "^4.3.3", - "@primer/react": "38.0.0-rc.3", + "@primer/react": "38.0.0-rc.4", "styled-components": "5.x", "typescript": "^5.9.2", "vite": "^7.1.5" diff --git a/examples/nextjs/package.json b/examples/nextjs/package.json index 9270d7027d8..f87a3f877a3 100644 --- a/examples/nextjs/package.json +++ b/examples/nextjs/package.json @@ -10,7 +10,7 @@ "type-check": "tsc --noEmit" }, "dependencies": { - "@primer/react": "38.0.0-rc.3", + "@primer/react": "38.0.0-rc.4", "next": "^15.2.3", "react": "18.3.1", "react-dom": "18.3.1", diff --git a/examples/theming/package.json b/examples/theming/package.json index 736f21bada8..09a278253f7 100644 --- a/examples/theming/package.json +++ b/examples/theming/package.json @@ -11,7 +11,7 @@ }, "dependencies": { "@primer/octicons-react": "^19.14.0", - "@primer/react": "38.0.0-rc.3", + "@primer/react": "38.0.0-rc.4", "clsx": "^2.1.1", "next": "^15.2.3", "react": "18.3.1", diff --git a/package-lock.json b/package-lock.json index 1bf95e6e2bf..3a6b3f394d1 100644 --- a/package-lock.json +++ b/package-lock.json @@ -31,7 +31,7 @@ "@primer/stylelint-config": "13.3.0", "@size-limit/preset-big-lib": "11.2.0", "@vitest/browser": "^3.2.4", - "@vitest/eslint-plugin": "^1.3.12", + "@vitest/eslint-plugin": "^1.3.13", "eslint": "^9.35.0", "eslint-import-resolver-typescript": "3.7.0", "eslint-plugin-clsx": "^0.0.10", @@ -75,7 +75,7 @@ "react-dom": "^18.3.1" }, "devDependencies": { - "@primer/react": "38.0.0-rc.2", + "@primer/react": "38.0.0-rc.4", "@types/react": "^18.3.11", "@types/react-dom": "^18.3.0", "@vitejs/plugin-react": "^4.3.3", @@ -88,7 +88,7 @@ "name": "example-nextjs", "version": "0.0.0", "dependencies": { - "@primer/react": "38.0.0-rc.2", + "@primer/react": "38.0.0-rc.4", "next": "^15.2.3", "react": "18.3.1", "react-dom": "18.3.1", @@ -104,7 +104,7 @@ "version": "0.0.0", "dependencies": { "@primer/octicons-react": "^19.14.0", - "@primer/react": "38.0.0-rc.2", + "@primer/react": "38.0.0-rc.4", "clsx": "^2.1.1", "next": "^15.2.3", "react": "18.3.1", @@ -8742,9 +8742,9 @@ } }, "node_modules/@vitest/eslint-plugin": { - "version": "1.3.12", - "resolved": "https://registry.npmjs.org/@vitest/eslint-plugin/-/eslint-plugin-1.3.12.tgz", - "integrity": "sha512-cSEyUYGj8j8SLqKrzN7BlfsJ3wG67eRT25819PXuyoSBogLXiyagdKx4MHWHV1zv+EEuyMXsEKkBEKzXpxyBrg==", + "version": "1.3.13", + "resolved": "https://registry.npmjs.org/@vitest/eslint-plugin/-/eslint-plugin-1.3.13.tgz", + "integrity": "sha512-QfzXd1+lCY3dIqPHOZlagA2bJYoWC5yAU3adv8Gks0rHAL6FpyXKYBiyMCuU6mRrbKUMphGqwDQobinOvYgJig==", "dev": true, "license": "MIT", "dependencies": { @@ -23266,7 +23266,9 @@ } }, "node_modules/tar-fs": { - "version": "3.1.0", + "version": "3.1.1", + "resolved": "https://registry.npmjs.org/tar-fs/-/tar-fs-3.1.1.tgz", + "integrity": "sha512-LZA0oaPOc2fVo82Txf3gw+AkEd38szODlptMYejQUhndHMLQ9M059uXR+AfS7DNo0NpINvSqDsvyaCrBVkptWg==", "dev": true, "license": "MIT", "dependencies": { @@ -25576,13 +25578,13 @@ }, "packages/mcp": { "name": "@primer/mcp", - "version": "0.0.4-rc.0", + "version": "0.1.0-rc.1", "dependencies": { "@babel/runtime": "^7.28.4", "@modelcontextprotocol/sdk": "^1.12.0", "@primer/octicons": "^19.15.5", "@primer/primitives": "10.x || 11.x", - "@primer/react": "^38.0.0-rc.0", + "@primer/react": "^38.0.0-rc.3", "cheerio": "^1.0.0", "turndown": "^7.2.0", "zod": "^3.23.8" @@ -25839,7 +25841,7 @@ }, "packages/react": { "name": "@primer/react", - "version": "38.0.0-rc.2", + "version": "38.0.0-rc.4", "license": "MIT", "dependencies": { "@github/mini-throttle": "^2.1.1", @@ -26000,7 +26002,9 @@ "license": "BSD-3-Clause" }, "packages/react/node_modules/@primer/primitives": { - "version": "11.1.0", + "version": "11.2.0", + "resolved": "https://registry.npmjs.org/@primer/primitives/-/primitives-11.2.0.tgz", + "integrity": "sha512-JDX0kFjPqQ2gEv7ryDzS1tk9ZM8cylCet1VzImaK9Mv8/jj+/I5Ox7Z1UdM8N35y0aUetJzB3HqodtMnQ2oPBg==", "license": "MIT" }, "packages/react/node_modules/@types/react-test-renderer": { @@ -26413,11 +26417,11 @@ }, "packages/styled-react": { "name": "@primer/styled-react", - "version": "1.0.0-rc.2", + "version": "1.0.0-rc.4", "devDependencies": { "@babel/preset-react": "^7.27.1", "@babel/preset-typescript": "^7.27.1", - "@primer/react": "^38.0.0-rc.2", + "@primer/react": "^38.0.0-rc.4", "@rollup/plugin-babel": "^6.0.4", "@types/react": "18.3.11", "@types/react-dom": "18.3.1", @@ -26433,7 +26437,7 @@ "typescript": "^5.9.2" }, "peerDependencies": { - "@primer/react": "38.0.0-rc.2", + "@primer/react": "38.0.0-rc.4", "@types/react": "18.x || 19.x", "@types/react-dom": "18.x || 19.x", "@types/react-is": "18.x || 19.x", diff --git a/package.json b/package.json index a950950c6d0..0a77520c34f 100644 --- a/package.json +++ b/package.json @@ -59,7 +59,7 @@ "@primer/stylelint-config": "13.3.0", "@size-limit/preset-big-lib": "11.2.0", "@vitest/browser": "^3.2.4", - "@vitest/eslint-plugin": "^1.3.12", + "@vitest/eslint-plugin": "^1.3.13", "eslint": "^9.35.0", "eslint-import-resolver-typescript": "3.7.0", "eslint-plugin-clsx": "^0.0.10", diff --git a/packages/react/CHANGELOG.md b/packages/react/CHANGELOG.md index d9837b20d88..98280f07b3a 100644 --- a/packages/react/CHANGELOG.md +++ b/packages/react/CHANGELOG.md @@ -1,5 +1,71 @@ # @primer/react +## 38.0.0-rc.4 + +### Major Changes + +- [#6857](https://github.com/primer/react/pull/6857) [`b9b558e`](https://github.com/primer/react/commit/b9b558efe1033150b05adc88d28234db5dc82f95) Thanks [@pksjce](https://github.com/pksjce)! - Remove PointerBox from @primer-react + +- [#6708](https://github.com/primer/react/pull/6708) [`b7b8a36`](https://github.com/primer/react/commit/b7b8a36d14b3c5b6f9c289423691fe9285eae786) Thanks [@liuliu-dev](https://github.com/liuliu-dev)! - Remove sx prop support from the Spinner component. + +- [#6833](https://github.com/primer/react/pull/6833) [`5ad1e93`](https://github.com/primer/react/commit/5ad1e939c1ef494f7680b0753a4d7c6296e088d6) Thanks [@joshblack](https://github.com/joshblack)! - Remove sx support from Radio + +- [#6841](https://github.com/primer/react/pull/6841) [`3e02e2c`](https://github.com/primer/react/commit/3e02e2c235a3633b646b8f2a493f3a23e17187bf) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` from `deprecated/Pagehead` + +- [#6692](https://github.com/primer/react/pull/6692) [`bb8d648`](https://github.com/primer/react/commit/bb8d648a42ec84e9baefae00fa87ede2be449e64) Thanks [@liuliu-dev](https://github.com/liuliu-dev)! - Remove sx Props and BoxWithFallBack from Timeline component. + +- [#6668](https://github.com/primer/react/pull/6668) [`acb6233`](https://github.com/primer/react/commit/acb6233fa0cf715d660ae222ebca9853115cb707) Thanks [@liuliu-dev](https://github.com/liuliu-dev)! - Update UnderlineNav component to no longer support sx and remove Box usage from it. + +- [#6835](https://github.com/primer/react/pull/6835) [`961c1c4`](https://github.com/primer/react/commit/961c1c48d942b9d8d81289a5d09d78f4d985f5da) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` from `ButtonGroup` + +- [#6834](https://github.com/primer/react/pull/6834) [`fa70b5e`](https://github.com/primer/react/commit/fa70b5eaae4c68ccac2bbfa44546ee85594a054a) Thanks [@joshblack](https://github.com/joshblack)! - Remove the `sx` prop from `Flash` + +- [#6876](https://github.com/primer/react/pull/6876) [`2102252`](https://github.com/primer/react/commit/2102252ef34899bb37c52ca4df193ba5cca41a94) Thanks [@llastflowers](https://github.com/llastflowers)! - Update FilteredActionList and FilteredActionListLoaders components to no longer support sx + +- [#6840](https://github.com/primer/react/pull/6840) [`aba8050`](https://github.com/primer/react/commit/aba8050ee9d324fa53a7b81d30b4f8dc7186c70a) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` prop from `deprecated/FilteredSearch` + +- [#6685](https://github.com/primer/react/pull/6685) [`00261c9`](https://github.com/primer/react/commit/00261c934fe88a74ef310c3134fe202226156b4b) Thanks [@liuliu-dev](https://github.com/liuliu-dev)! - Remove the sx prop from RadioGroup and Truncate. + +- [#6878](https://github.com/primer/react/pull/6878) [`7081dd3`](https://github.com/primer/react/commit/7081dd3b45f7dae08f7a39038be95dbf20dd8a27) Thanks [@llastflowers](https://github.com/llastflowers)! - Update BaseStyles component to no longer support sx + +- [#6679](https://github.com/primer/react/pull/6679) [`6158135`](https://github.com/primer/react/commit/6158135e42be9486bb7de35fbedf96e412b8a1d7) Thanks [@llastflowers](https://github.com/llastflowers)! - Update CounterLabel component to no longer support sx + +- [#6827](https://github.com/primer/react/pull/6827) [`6843040`](https://github.com/primer/react/commit/6843040ad9b9810cb14f6186e53b01e83bed60f8) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for the sx prop from RelativeTime + +- [#6813](https://github.com/primer/react/pull/6813) [`a1a4ad0`](https://github.com/primer/react/commit/a1a4ad00b87b489f6341aa818a9b8b8662fe2a82) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` prop for `InlineMessage` + +- [#6866](https://github.com/primer/react/pull/6866) [`3237a4e`](https://github.com/primer/react/commit/3237a4e549779a2b3144b1f48974e533d28cca2a) Thanks [@francinelucca](https://github.com/francinelucca)! - chore(LinkButton): remove sx from LinkButton + +- [#6655](https://github.com/primer/react/pull/6655) [`092185f`](https://github.com/primer/react/commit/092185f7b9d6c70296e629e03790676502b701e5) Thanks [@llastflowers](https://github.com/llastflowers)! - Update CheckboxGroup component to no longer support sx + +- [#6654](https://github.com/primer/react/pull/6654) [`513cc3f`](https://github.com/primer/react/commit/513cc3feed5723632e54f07cae9d4397e3783fdc) Thanks [@llastflowers](https://github.com/llastflowers)! - Update Checkbox component to no longer support sx + +- [#6837](https://github.com/primer/react/pull/6837) [`134f96e`](https://github.com/primer/react/commit/134f96e81476f829f2b0f0b44fa73cc1604983d5) Thanks [@TylerJDev](https://github.com/TylerJDev)! - Update ProgressBar to no longer support sx + +- [#6839](https://github.com/primer/react/pull/6839) [`32febac`](https://github.com/primer/react/commit/32febac10a887cce418abf6f4419cfb100a8f292) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` from `Header` + +- [#6871](https://github.com/primer/react/pull/6871) [`44b3d73`](https://github.com/primer/react/commit/44b3d732157048059681f3fdcb86c5d77a59aec5) Thanks [@joshblack](https://github.com/joshblack)! - Remove support for `sx` from `PageHeader` + +- [#6863](https://github.com/primer/react/pull/6863) [`9614c0e`](https://github.com/primer/react/commit/9614c0eae7d77bdf879df474c44d613b673e6d4e) Thanks [@pksjce](https://github.com/pksjce)! - Remove Banner from experimental + +### Minor Changes + +- [#6456](https://github.com/primer/react/pull/6456) [`fbd3ac7`](https://github.com/primer/react/commit/fbd3ac75a67b89635dcd28879a1b4a93cef70289) Thanks [@TylerJDev](https://github.com/TylerJDev)! - TreeView: Add trailing actions prop `secondaryActions` + +- [#6855](https://github.com/primer/react/pull/6855) [`356a129`](https://github.com/primer/react/commit/356a129d2a1a0c02d9ca59280e7711e501c0d20d) Thanks [@cheshire137](https://github.com/cheshire137)! - Add buttonLabelOn and buttonLabelOff to ToggleSwitch + +### Patch Changes + +- [#6829](https://github.com/primer/react/pull/6829) [`b568765`](https://github.com/primer/react/commit/b568765159a22fefbb2e521947a6c99109cb6f19) Thanks [@francinelucca](https://github.com/francinelucca)! - chore: fix ActionBar gap issue + +- [#6879](https://github.com/primer/react/pull/6879) [`c8fc6b8`](https://github.com/primer/react/commit/c8fc6b81d61834ab143a578b379d7dcc49e17d62) Thanks [@francinelucca](https://github.com/francinelucca)! - chore(FilteredActionList): fix className override + +- [#6896](https://github.com/primer/react/pull/6896) [`5c29b01`](https://github.com/primer/react/commit/5c29b010db51e85c387963aee45ee97f12aae7ef) Thanks [@liuliu-dev](https://github.com/liuliu-dev)! - Fix typing issues in PageHeader exports. + +- [#6881](https://github.com/primer/react/pull/6881) [`8d52362`](https://github.com/primer/react/commit/8d523624366dda9cbb5c200560ffcf9b91f57655) Thanks [@langermank](https://github.com/langermank)! - Implement forced colors for progress bar + +- [#6854](https://github.com/primer/react/pull/6854) [`dd8eeed`](https://github.com/primer/react/commit/dd8eeeddb39a7afbdd38d3df8f0568f176e5d4de) Thanks [@pksjce](https://github.com/pksjce)! - Breadcrumbs: Fix esc button not being able to focus on the menubuttonRef + ## 38.0.0-rc.3 ### Major Changes diff --git a/packages/react/package.json b/packages/react/package.json index a9232579168..5c3f1e4008c 100644 --- a/packages/react/package.json +++ b/packages/react/package.json @@ -1,7 +1,7 @@ { "name": "@primer/react", "type": "module", - "version": "38.0.0-rc.3", + "version": "38.0.0-rc.4", "description": "An implementation of GitHub's Primer Design System using React", "main": "./dist/index.js", "module": "./dist/index.js", diff --git a/packages/react/src/ActionList/ActionList.docs.json b/packages/react/src/ActionList/ActionList.docs.json index 6afbe71c766..ccaf7865466 100644 --- a/packages/react/src/ActionList/ActionList.docs.json +++ b/packages/react/src/ActionList/ActionList.docs.json @@ -36,11 +36,6 @@ "type": "AriaRole", "defaultValue": "", "description": "ARIA role describing the function of the list. `listbox` and `menu` are a common values." - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ], "subcomponents": [ @@ -113,11 +108,6 @@ "required": false, "description": "id to attach to the root element of the Item", "defaultValue": "" - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] }, @@ -144,11 +134,6 @@ "required": false, "description": "", "defaultValue": "" - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] }, @@ -183,11 +168,6 @@ "required": false, "description": "Text describing why the item is inactive. This may be used when an item's usual functionality\nis unavailable due to a system error such as a database outage.", "defaultValue": "" - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ], "passthrough": { @@ -204,11 +184,6 @@ "defaultValue": "", "required": true, "description": "Icon (or similar) positioned before item text." - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] }, @@ -221,11 +196,6 @@ "defaultValue": "", "required": true, "description": "Visual positioned after item text." - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] }, @@ -282,11 +252,6 @@ "defaultValue": "'inline'", "description": "`inline` descriptions are positioned beside primary text. `block` descriptions are positioned below primary text." }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true - }, { "name": "className", "type": "string | undefined", @@ -337,11 +302,6 @@ "defaultValue": "h3", "required": false, "description": "The level of the heading and it is only required (enforce by runtime warning) for lists. (i.e. not required for ActionMenu or listbox roles)" - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] }, @@ -385,22 +345,6 @@ "type": "AriaRole", "defaultValue": "", "description": "ARIA role describing the function of the list inside the group. `listbox` and `menu` are a common values." - }, - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true - } - ] - }, - { - "filePath": "/Users/mperrotti/work-dir/react/packages/react/src/ActionList/Divider.tsx", - "name": "ActionList.Divider", - "props": [ - { - "name": "sx", - "type": "SystemStyleObject", - "deprecated": true } ] } diff --git a/packages/react/src/ActionList/Description.tsx b/packages/react/src/ActionList/Description.tsx index c4624c6f061..cb1e7bd425d 100644 --- a/packages/react/src/ActionList/Description.tsx +++ b/packages/react/src/ActionList/Description.tsx @@ -1,6 +1,5 @@ import React from 'react' import Truncate from '../Truncate' -import type {SxProp} from '../sx' import {ItemContext} from './shared' import classes from './ActionList.module.css' import {BoxWithFallback} from '../internal/components/BoxWithFallback' @@ -20,11 +19,10 @@ export type ActionListDescriptionProps = { * Whether the inline description should truncate the text on overflow. */ truncate?: boolean -} & SxProp +} export const Description: React.FC> = ({ variant = 'inline', - sx, className, truncate, ...props @@ -47,7 +45,6 @@ export const Description: React.FC> = ({sx, className}) => { +export const Divider: React.FC> = ({className}) => { return (