Update PRC ActionList implementation to have similar semantics to PVC. #2878
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: 72cd916 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
size-limit report 📦
|
|
Hi @radglob! To provide more context. The PVC ActionList was reviewed by accessibility already and the semantics were changed/updated to work better for users. So, we want to have semantics match to the PVC component. |
Also made some styling changes (ex. no vertical padding on ActionList <ul>) to align with PVC implementation.
radglob
had a problem deploying
to
github-pages
GitHub Actions
Failure
* Limit deploy to has pages * Limit statuses to primer/react * Limit stale to primer/react * Preview cross org forks --------- Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Rez <[email protected]> Co-authored-by: Josh Black <[email protected]>
* duplicate button into drafts * update story title * add visual tests for Button2 * test(vrt): update snapshots --------- Co-authored-by: siddharthkp <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* fix hover borders * Create wicked-knives-sparkle.md * snaps * why so many snaps? * revert? * test(vrt): update snapshots --------- Co-authored-by: langermank <[email protected]>
* Move portal stories to portal directory. * Reorganize portal stories into separate files for default and features. * Generate e2e tests for portal components. * Update generated/components.json * Remove e2e portal tests as they do not play well with Playwright. * Remove unnecessary decorator from story metadata. --------- Co-authored-by: radglob <[email protected]>
* Migrate ActionList to use new slots impl * Remove ActionList slots * Remove unused import * Create lemon-berries-run.md * Update snapshots * Fix slot description logic * Fix style logic * Fix conditional box logic * Fix aria-labelledby logic * Fix flexbox logic
siddharthkp
approved these changes
Apr 24, 2023
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
This was referenced Apr 24, 2023
Merged
This was referenced May 3, 2023
Closed
7 tasks
broccolinisoup
added a commit
that referenced
this pull request
Jun 16, 2023
…3408) * Revert "Refactor(ActionList): ActionList.Item should render content as a button if parent is not interactive. (#3284)" This reverts commit 08d7d5d. * Revert "Address ToggleSwitch a11y feedback (#3251)" This reverts commit 580f165. * Revert "Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" (#3349)" This reverts commit a6b4169. * Revert "fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346)" This reverts commit 6304923. * Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" This reverts commit e865e3e. * Revert "fix(ActionList): update to read from group context if selectionVariant is defined (#3269)" This reverts commit 21ec626. * Revert "Add back check for GroupContext in ActionList.Selection to fix issues introduced to github/github (#3267)" This reverts commit 332a1af. * Revert "Fix create-slots utils path (#3223)" This reverts commit f50997f. * Revert "Allow up to 4 levels of nesting for the NavList (#3343)" This reverts commit 786013e. * Revert "NavList: Fix group dividers (#3328)" This reverts commit 1fd6d32. * Revert "StyledOcticon/Octicon: Update the render func to use `React.forwardRef()` (#3293)" This reverts commit 027d214. * Revert "Rename component StyledOcticon to Octicon (#3148)" This reverts commit ac437bb. * Revert "Update PRC ActionList implementation to have similar semantics to PVC. (#2878)" This reverts commit 87883c3. * Update generated/components.json * Revert "Bump @primer/react-octicons to v19.0.1 (#3266)" This reverts commit 9532977. * Revert "Remove aria-hidden=true from spans with required asterisk (#3320)" This reverts commit 47b7ea0. * Rename component StyledOcticon to Octicon (#3148) * Rename component StyledOcticon to Octicon * Update generated/components.json * Create .changeset/swift-cows-rest.md * Fix snapshot * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * refactor(CircleOcticon): Move files to folder, add storybook and e2e tests. (#3119) * Reorganize CircleOcticon files, add story and e2e tests. * Add index file. * Update generated/components.json * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(Popover): move files into folder, add storybook and e2e tests. (#3135) * Move files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * refactor(ButtonGroup): move files to folder. (#3117) * Reorganize ButtonGroup component and stories. * Update generated/components.json * Add e2e tests. * test(vrt): update snapshots * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(FilteredSearch): move files to folder, add storybook and e2e tests. (#3125) * Move files to folder, add storybook and e2e tests. * Add aria-label to TextInput, fix docs reference. * Update generated/components.json * test(vrt): update snapshots * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Fix generated components. --------- Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> * Refactor(LabelGroup): add files to folder, add storybook and e2e tests. (#3131) * Move files to folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * test(e2e): update ActionMenu e2e test to remove custom contrast rules (#3130) * test(e2e): update ActionMenu e2e test to remove custom contrast rules * test(vrt): update snapshots * test: remove multiple sections test * test(e2e): restore rule exception for ActionMenu --------- Co-authored-by: Josh Black <[email protected]> * NavList SSR: use useIsomorphicLayoutEffect instead of useLayoutEffect (#3127) * use useIsomorphicLayoutEffect * Create .changeset/fast-panthers-refuse.md --------- Co-authored-by: Josh Black <[email protected]> * Bump @primer/primitives to `7.11.5` (#3120) * npm install @primer/[email protected] * Updating`theme-preval` snapshots * Re-run ``` npm run build npm run test -- -u ``` So that /themePreval.test.ts.snap also gets the new timestamp * Update `theme-preval` snapshots docs * Create .changeset/old-falcons-learn.md * test(vrt): update snapshots --------- Co-authored-by: simurai <[email protected]> * Update RelativeTime stories (#3136) * Update RelativeTime stories * Update generated/components.json --------- Co-authored-by: colebemis <[email protected]> * chore(deps-dev): bump @types/jest-axe from 3.5.3 to 3.5.5 (#3141) Bumps [@types/jest-axe](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest-axe) from 3.5.3 to 3.5.5. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest-axe) --- updated-dependencies: - dependency-name: "@types/jest-axe" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump @rollup/plugin-typescript from 11.0.0 to 11.1.0 (#3138) Bumps [@rollup/plugin-typescript](https://github.com/rollup/plugins/tree/HEAD/packages/typescript) from 11.0.0 to 11.1.0. - [Release notes](https://github.com/rollup/plugins/releases) - [Changelog](https://github.com/rollup/plugins/blob/master/packages/typescript/CHANGELOG.md) - [Commits](https://github.com/rollup/plugins/commits/commonjs-v11.1.0/packages/typescript) --- updated-dependencies: - dependency-name: "@rollup/plugin-typescript" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 (#3140) * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 Bumps [eslint-plugin-primer-react](https://github.com/primer/eslint-plugin-primer-react) from 2.0.2 to 2.0.3. - [Release notes](https://github.com/primer/eslint-plugin-primer-react/releases) - [Changelog](https://github.com/primer/eslint-plugin-primer-react/blob/main/CHANGELOG.md) - [Commits](primer/eslint-plugin-primer-react@v2.0.2...v2.0.3) --- updated-dependencies: - dependency-name: eslint-plugin-primer-react dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * chore(eslint): remove unnecessary disable --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 (#3139) * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 Bumps [babel-plugin-styled-components](https://github.com/styled-components/babel-plugin-styled-components) from 2.0.2 to 2.1.1. - [Release notes](https://github.com/styled-components/babel-plugin-styled-components/releases) - [Commits](styled-components/babel-plugin-styled-components@v2.0.2...v2.1.1) --- updated-dependencies: - dependency-name: babel-plugin-styled-components dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * test(snapshots): update snapshots --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(project): update project to use Node.js v18 LTS (#3133) * chore(project): update project to use Node.js v18 LTS * chore: add legacy openssl option for docs builds * chore: update devcontainer image --------- Co-authored-by: Josh Black <[email protected]> * Use SSR-compatible slot implementation in CheckboxGroup/RadioGroup (#3146) * Update slots for checkbox group and radio group * Update useSlot return type * Update exports test * Create .changeset/young-queens-notice.md * Update comment indentation * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * docs(project): move code of conduct under .github (#3145) Co-authored-by: Josh Black <[email protected]> * Add npm workspaces structure to primer/react (#3074) * test(axe): default `color-contrast` check to false * chore: update generate script to not conditionally disable contrast checks * Create adr-015-npm-workspaces.md * docs: update adr * Update adr-015-npm-workspaces.md --------- Co-authored-by: Josh Black <[email protected]> * Update generated/components.json * change styled_octicon key * Update generated/components.json * Update generated/components.json * Update generated/components.json * Octicon folder * Fix package-lock * Fix imports * update path * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> * Allow up to 4 levels of nesting for the NavList (#3343) * allows up to 4 levels of nesting for the NavList * adds changeset * Update src/NavList/NavList.tsx Co-authored-by: Cole Bemis <[email protected]> * updates snapshots --------- Co-authored-by: Cole Bemis <[email protected]> * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346) * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler (#3163) * test: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler Failing test for #3162 * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`ߴs `onSelect` handler * add storybook example: Delayed Menu Close * update docs * docs: changeset * Update changelog --------- Co-authored-by: Siddharth Kshetrapal <[email protected]> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]> * Revert "PageLayout.Content should not use main landmark by default. (#3154)" This reverts commit fed1c57. * Revert "Update `PageLayout` docs around `<main>` usage (#3323)" This reverts commit e1904b2. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: Jon Rohan <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]>
radglob
added a commit
that referenced
this pull request
Jun 16, 2023
…3408) * Revert "Refactor(ActionList): ActionList.Item should render content as a button if parent is not interactive. (#3284)" This reverts commit 08d7d5d. * Revert "Address ToggleSwitch a11y feedback (#3251)" This reverts commit 580f165. * Revert "Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" (#3349)" This reverts commit a6b4169. * Revert "fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346)" This reverts commit 6304923. * Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" This reverts commit e865e3e. * Revert "fix(ActionList): update to read from group context if selectionVariant is defined (#3269)" This reverts commit 21ec626. * Revert "Add back check for GroupContext in ActionList.Selection to fix issues introduced to github/github (#3267)" This reverts commit 332a1af. * Revert "Fix create-slots utils path (#3223)" This reverts commit f50997f. * Revert "Allow up to 4 levels of nesting for the NavList (#3343)" This reverts commit 786013e. * Revert "NavList: Fix group dividers (#3328)" This reverts commit 1fd6d32. * Revert "StyledOcticon/Octicon: Update the render func to use `React.forwardRef()` (#3293)" This reverts commit 027d214. * Revert "Rename component StyledOcticon to Octicon (#3148)" This reverts commit ac437bb. * Revert "Update PRC ActionList implementation to have similar semantics to PVC. (#2878)" This reverts commit 87883c3. * Update generated/components.json * Revert "Bump @primer/react-octicons to v19.0.1 (#3266)" This reverts commit 9532977. * Revert "Remove aria-hidden=true from spans with required asterisk (#3320)" This reverts commit 47b7ea0. * Rename component StyledOcticon to Octicon (#3148) * Rename component StyledOcticon to Octicon * Update generated/components.json * Create .changeset/swift-cows-rest.md * Fix snapshot * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * refactor(CircleOcticon): Move files to folder, add storybook and e2e tests. (#3119) * Reorganize CircleOcticon files, add story and e2e tests. * Add index file. * Update generated/components.json * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(Popover): move files into folder, add storybook and e2e tests. (#3135) * Move files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * refactor(ButtonGroup): move files to folder. (#3117) * Reorganize ButtonGroup component and stories. * Update generated/components.json * Add e2e tests. * test(vrt): update snapshots * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(FilteredSearch): move files to folder, add storybook and e2e tests. (#3125) * Move files to folder, add storybook and e2e tests. * Add aria-label to TextInput, fix docs reference. * Update generated/components.json * test(vrt): update snapshots * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Fix generated components. --------- Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> * Refactor(LabelGroup): add files to folder, add storybook and e2e tests. (#3131) * Move files to folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * test(e2e): update ActionMenu e2e test to remove custom contrast rules (#3130) * test(e2e): update ActionMenu e2e test to remove custom contrast rules * test(vrt): update snapshots * test: remove multiple sections test * test(e2e): restore rule exception for ActionMenu --------- Co-authored-by: Josh Black <[email protected]> * NavList SSR: use useIsomorphicLayoutEffect instead of useLayoutEffect (#3127) * use useIsomorphicLayoutEffect * Create .changeset/fast-panthers-refuse.md --------- Co-authored-by: Josh Black <[email protected]> * Bump @primer/primitives to `7.11.5` (#3120) * npm install @primer/[email protected] * Updating`theme-preval` snapshots * Re-run ``` npm run build npm run test -- -u ``` So that /themePreval.test.ts.snap also gets the new timestamp * Update `theme-preval` snapshots docs * Create .changeset/old-falcons-learn.md * test(vrt): update snapshots --------- Co-authored-by: simurai <[email protected]> * Update RelativeTime stories (#3136) * Update RelativeTime stories * Update generated/components.json --------- Co-authored-by: colebemis <[email protected]> * chore(deps-dev): bump @types/jest-axe from 3.5.3 to 3.5.5 (#3141) Bumps [@types/jest-axe](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest-axe) from 3.5.3 to 3.5.5. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest-axe) --- updated-dependencies: - dependency-name: "@types/jest-axe" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump @rollup/plugin-typescript from 11.0.0 to 11.1.0 (#3138) Bumps [@rollup/plugin-typescript](https://github.com/rollup/plugins/tree/HEAD/packages/typescript) from 11.0.0 to 11.1.0. - [Release notes](https://github.com/rollup/plugins/releases) - [Changelog](https://github.com/rollup/plugins/blob/master/packages/typescript/CHANGELOG.md) - [Commits](https://github.com/rollup/plugins/commits/commonjs-v11.1.0/packages/typescript) --- updated-dependencies: - dependency-name: "@rollup/plugin-typescript" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 (#3140) * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 Bumps [eslint-plugin-primer-react](https://github.com/primer/eslint-plugin-primer-react) from 2.0.2 to 2.0.3. - [Release notes](https://github.com/primer/eslint-plugin-primer-react/releases) - [Changelog](https://github.com/primer/eslint-plugin-primer-react/blob/main/CHANGELOG.md) - [Commits](primer/eslint-plugin-primer-react@v2.0.2...v2.0.3) --- updated-dependencies: - dependency-name: eslint-plugin-primer-react dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * chore(eslint): remove unnecessary disable --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 (#3139) * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 Bumps [babel-plugin-styled-components](https://github.com/styled-components/babel-plugin-styled-components) from 2.0.2 to 2.1.1. - [Release notes](https://github.com/styled-components/babel-plugin-styled-components/releases) - [Commits](styled-components/babel-plugin-styled-components@v2.0.2...v2.1.1) --- updated-dependencies: - dependency-name: babel-plugin-styled-components dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * test(snapshots): update snapshots --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(project): update project to use Node.js v18 LTS (#3133) * chore(project): update project to use Node.js v18 LTS * chore: add legacy openssl option for docs builds * chore: update devcontainer image --------- Co-authored-by: Josh Black <[email protected]> * Use SSR-compatible slot implementation in CheckboxGroup/RadioGroup (#3146) * Update slots for checkbox group and radio group * Update useSlot return type * Update exports test * Create .changeset/young-queens-notice.md * Update comment indentation * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * docs(project): move code of conduct under .github (#3145) Co-authored-by: Josh Black <[email protected]> * Add npm workspaces structure to primer/react (#3074) * test(axe): default `color-contrast` check to false * chore: update generate script to not conditionally disable contrast checks * Create adr-015-npm-workspaces.md * docs: update adr * Update adr-015-npm-workspaces.md --------- Co-authored-by: Josh Black <[email protected]> * Update generated/components.json * change styled_octicon key * Update generated/components.json * Update generated/components.json * Update generated/components.json * Octicon folder * Fix package-lock * Fix imports * update path * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> * Allow up to 4 levels of nesting for the NavList (#3343) * allows up to 4 levels of nesting for the NavList * adds changeset * Update src/NavList/NavList.tsx Co-authored-by: Cole Bemis <[email protected]> * updates snapshots --------- Co-authored-by: Cole Bemis <[email protected]> * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346) * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler (#3163) * test: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler Failing test for #3162 * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`ߴs `onSelect` handler * add storybook example: Delayed Menu Close * update docs * docs: changeset * Update changelog --------- Co-authored-by: Siddharth Kshetrapal <[email protected]> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]> * Revert "PageLayout.Content should not use main landmark by default. (#3154)" This reverts commit fed1c57. * Revert "Update `PageLayout` docs around `<main>` usage (#3323)" This reverts commit e1904b2. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: Jon Rohan <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]>
broccolinisoup
added a commit
that referenced
this pull request
Jun 21, 2023
#2878) * Render ActionList.Group header inside list, use li for semantic HTML reasons. * Update snapshots. * Fix snapshot. No idea why this isn't being generated correctly. * Separate Heading from ActionList.Group. Also made some styling changes (ex. no vertical padding on ActionList <ul>) to align with PVC implementation. * removed usage of actionlist.group. * Fix type errors. * Create modern-coins-destroy.md * Fix ActionList.docs.json to fix components build CI step. * Update generated/components.json * test(vrt): update snapshots * test(vrt): update snapshots * Update snapshots. * Update themePreval snapshot and cache bust file. * Updated snapshots. * Revert padding on ActionList. * test(vrt): update snapshots * Make sure lists are surrounded by a div, fix accessibility issues in Storybook. * Updated snapshots, fix linting error. * Update docs. * Update generated/components.json * Fix linting issues. * Fixing some issues after the previous merge. * Fix ActionMenu feature stories. * Updated snapshots that shouldn't be different from main. * disable axe check * Update changeset to major patch instead of minor. * re-gen test * Add missing selectionVariant to Groups and Descriptions ActionMenu story. * test(vrt): update snapshots * Fix styling of ActionList to match prod. * Fix linting errors in ActionList. * test(vrt): update snapshots * Reduce padding between ActionLists in ActionMenu example. * refactor(DataTable): update optional type signatures (#2987) * refactor(DataTable): update optional type signatures * chore: update docs and add stories to json for DataTable * chore: remove default story * Update generated/components.json --------- Co-authored-by: Josh Black <[email protected]> * Textarea stories (#2988) * checkbox * Update generated/components.json * tests * Update generated/components.json * fix docs * radio stories * Update generated/components.json * fix doc * remove axe * add sb tests * select * add features * e2e * text input * Update generated/components.json * tokens * Update generated/components.json * tokens * textarea * all the things * Update generated/components.json * remove animation test * test(vrt): update snapshots * remove dead files * Update generated/components.json * missing exports --------- Co-authored-by: langermank <[email protected]> * Pagination design updates (#2702) * updates pagination styles to match latest token usage patterns, and updates ARIA markup * adds changeset * Create .changeset/long-bags-double.md --------- Co-authored-by: Cole Bemis <[email protected]> * Make sure ActionList.Heading titles are bolded correctly. * Updated snapshots. * Revert title font weight to semibold. * Update snapshots. * test(vrt): update snapshots * Passing sxProps to ActionList.Heading through ActionList. * More snapshot fiddling. * Formatting. * Use margin and padding to get spacing correct. * Fix aat tests. * Update snapshots. * I think this should get the ActionMenu example back to what is expected. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit a713875. * Revert "test(vrt): update snapshots" This reverts commit 2eae060. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Revert "test(vrt): update snapshots" This reverts commit 1c11a1b. * Revert "test(vrt): update snapshots" This reverts commit a02012b. * Revert "test(vrt): update snapshots" This reverts commit bf59217. * Fix underlinenav. * Updated snapshots. * test(vrt): update snapshots * Dialog v2: Move files, reformat stories and add e2e tests (#2984) * initial * Dialog v2: move files, refaormat stories and add e2e tests * Update generated/components.json * snaps * remove jest disable * Dialogv2 -> Dialog2 * storybook tests and await dialogs * disable animations * disable animations on the first snapshots too * fix page locators * test(vrt): update snapshots * test(Hidden): update export * Revert "test(Hidden): update export" This reverts commit f29edd9. --------- Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: Josh Black <[email protected]> * feat(project): add warning, invariant utilities and dev-expression plugin (#2901) * chore(project): add warning utility and dev-expression plugin * chore: address eslint violations * feat: add invariant and warn helper * chore: flip flag for useMedia warning * chore: add changeset * chore: add minified exception case for invariant * chore: update type signature for invariant * test: update test for warning helper * refactor(hooks): update useControllableState warning usage * test(warning): update test titles with flipped condition --------- Co-authored-by: Josh Black <[email protected]> * chore(deps-dev): bump mdast-util-to-string from 3.1.0 to 3.1.1 (#3018) Bumps [mdast-util-to-string](https://github.com/syntax-tree/mdast-util-to-string) from 3.1.0 to 3.1.1. - [Release notes](https://github.com/syntax-tree/mdast-util-to-string/releases) - [Commits](syntax-tree/mdast-util-to-string@3.1.0...3.1.1) --- updated-dependencies: - dependency-name: mdast-util-to-string dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jscodeshift from 0.13.0 to 0.14.0 (#3015) Bumps [jscodeshift](https://github.com/facebook/jscodeshift) from 0.13.0 to 0.14.0. - [Release notes](https://github.com/facebook/jscodeshift/releases) - [Changelog](https://github.com/facebook/jscodeshift/blob/main/CHANGELOG.md) - [Commits](facebook/jscodeshift@0.13.0...v0.14.0) --- updated-dependencies: - dependency-name: jscodeshift dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(deps-dev): bump @babel/cli from 7.19.3 to 7.21.0 (#3016) Bumps [@babel/cli](https://github.com/babel/babel/tree/HEAD/packages/babel-cli) from 7.19.3 to 7.21.0. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.21.0/packages/babel-cli) --- updated-dependencies: - dependency-name: "@babel/cli" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * Add argTypes for HeadingPlayground ActionList story. * Add aria-busy declaration in relevant story instead of in component. * Update snapshots. * Recalculate autocomplete suggestions if the input data changes while the menu is open (#3009) * Add support for custom emoji suggestions * Allow declaratively setting suggestions as "loading" * Recalculate suggestions on update * Create .changeset/silly-plants-draw.md * Fix bug where page would be unresponsive if no suggestions provided * docs(DataTable): add reference stories for row actions (#2978) * docs(DataTable): add reference stories for row actions * chore: clean-up ts types * fix: add guards for optional id and field * docs: add story ids to DataTable docs json * Update generated/components.json --------- Co-authored-by: Josh Black <[email protected]> * chore(deps): update babel dependencies (#3005) Co-authored-by: Josh Black <[email protected]> * feat(DataTable): add support for alphanumeric, datatable, and custom sort functions (#3001) * chore: check-in work * chore: clean-up work * Update generated/components.json * docs: add custom sorting story --------- Co-authored-by: Josh Black <[email protected]> * Add new usage instructions to changeset. * Move ActionList.Group to versioned deprecated directory. * Fix references in deprecated ActionList.Group. * Fix themePreval snapshot. * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <[email protected]> * Update src/ActionList/ActionList.stories.tsx Co-authored-by: Katie Langerman <[email protected]> * Heading playground only uses heading props. * Use Box instead of styled elements for ActionList.Heading. * Remove presentation role and aria-hidden from ActionList.Heading. * Updated snapshot. * Use restricted as prop instead of headingLevel. * Use slots and contexts to allow developers to declare Heading like a child while still rendering correctly. * Linting. * Add @deprecated tag to ActionList.Group as first step towards deprecating component. * Fix type errors in ActionMenu and ActionList stories. * Formatting. * Updated snapshots. * Updating snapshots with --no-cache flag. * Update docs, fix missing Group export on ActionList. * Terminate comment correctly. * Regenerate ActionMenu e2e tests. * Add padding to fix snapshot diff. * Change padding again. * Remove extra padding. * Add small amount of margin (not padding) to fix snapshot diff. * Less margin. * More margin, actually. * I think this should work. * I think this is the correct margin, something else is off. * . * These snapshot comparisons are confusing me. * Just a little more margin. * Ugh. * test(vrt): update snapshots * Slightly less margin. * More tweaking. * More tweaking. * Disable animations on actionmenu test to see if it helps generate the snapshot correctly. * test(vrt): update snapshots * Don't animate Multiple Sections e2e test. * Format file. * test(vrt): update snapshots * Revert snapshots to match main. * Update snapshots again. * Build snapshots without cache. * Remove lingering references to headingProps. * Update generated/components.json * chore(deps): bump react-intersection-observer from 9.4.1 to 9.4.3 (#3177) Bumps [react-intersection-observer](https://github.com/thebuilder/react-intersection-observer) from 9.4.1 to 9.4.3. - [Release notes](https://github.com/thebuilder/react-intersection-observer/releases) - [Commits](thebuilder/react-intersection-observer@v9.4.1...v9.4.3) --- updated-dependencies: - dependency-name: react-intersection-observer dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump jest-fail-on-console from 3.0.2 to 3.1.1 (#3180) Bumps [jest-fail-on-console](https://github.com/ValentinH/jest-fail-on-console) from 3.0.2 to 3.1.1. - [Release notes](https://github.com/ValentinH/jest-fail-on-console/releases) - [Commits](ValentinH/jest-fail-on-console@v3.0.2...v3.1.1) --- updated-dependencies: - dependency-name: jest-fail-on-console dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump unist-util-find-before from 3.0.0 to 3.0.1 (#3179) Bumps [unist-util-find-before](https://github.com/syntax-tree/unist-util-find-before) from 3.0.0 to 3.0.1. - [Release notes](https://github.com/syntax-tree/unist-util-find-before/releases) - [Commits](syntax-tree/unist-util-find-before@3.0.0...3.0.1) --- updated-dependencies: - dependency-name: unist-util-find-before dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix(PageLayout): update Pane to warn instead of error (#3160) * fix(PageLayout): update Pane to warn instead of error * chore: add changeset * chore: remove invariant * chore: address eslint violations --------- Co-authored-by: Josh Black <[email protected]> * Refactor(PageLayout): use Playwright for interaction tests instead of Storybook. (#3171) * Change PageLayout interactions to use Playwright. * Build snapshots without cache. * Fix PageLayout.test.ts to use Page object because document is not defined. * Pass missing page parameter to isInViewPort function. * test(vrt): update snapshots * Add tabIndex to PullRequestPage test so it is a scrollable region. --------- Co-authored-by: radglob <[email protected]> * Delete docs migration-related code (#3187) * Delete docs_migration.yml * Delete docs-migration.js * Delete props-mdx-to-json.mjs * Docs(Tooltip): Adds `defaultValue` of tooltip direction to documentation (#3175) * docs: adds defaultValue of tooltip direction to documentation * remove the changeset * chore(project): update test-e2e docker image (#3183) * feat(project): add subpath pattern to package.json to restrict internal imports (#3186) * feat(project): add subpath pattern to package.json to restrict internal imports * refactor(project): move useOverflow to internal/hooks * docs(adrs): update adopted status for adr * chore: add changeset * Update generated/components.json --------- Co-authored-by: Josh Black <[email protected]> * Ensure disabled invisible buttons have the right text color. (#3185) * Set disabled invisible button color similarly to other button variants. * Don't set [data-no-visual=true] on invisible button variants. * Update snapshots. * Formatting and linting. * test(vrt): update snapshots * Update generated/components.json * chore(project): update test-e2e docker image (#3183) * Use selector to set disabled invisible button text color correctly. * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> Co-authored-by: Josh Black <[email protected]> * Refactor(Overlay): move files to folder. (#3191) * Move overlay files to directory, move all stories to features as there isn't a clear default. * Update generated/components.json --------- Co-authored-by: radglob <[email protected]> * Refactor(Text): move files to directory, add storybook and e2e tests. (#3189) * Move Text files into directory, create storybook and e2e tests. * Fix Text reference in src/index.ts, update snapshots. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Reduce ❌ for forks (#2283) * Limit deploy to has pages * Limit statuses to primer/react * Limit stale to primer/react * Preview cross org forks --------- Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Rez <[email protected]> Co-authored-by: Josh Black <[email protected]> * Setup snapshots for visual regression for experimental/Button2 (#3181) * duplicate button into drafts * update story title * add visual tests for Button2 * test(vrt): update snapshots --------- Co-authored-by: siddharthkp <[email protected]> * Version Packages (#3106) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Bug fix: `ButtonGroup` borders (#3198) * fix hover borders * Create wicked-knives-sparkle.md * snaps * why so many snaps? * revert? * test(vrt): update snapshots --------- Co-authored-by: langermank <[email protected]> * Refactor(Portal): move stories into Portal directory. (#3188) * Move portal stories to portal directory. * Reorganize portal stories into separate files for default and features. * Generate e2e tests for portal components. * Update generated/components.json * Remove e2e portal tests as they do not play well with Playwright. * Remove unnecessary decorator from story metadata. --------- Co-authored-by: radglob <[email protected]> * Use SSR-compatible slot implementation in ActionList/NavList (#3173) * Migrate ActionList to use new slots impl * Remove ActionList slots * Remove unused import * Create lemon-berries-run.md * Update snapshots * Fix slot description logic * Fix style logic * Fix conditional box logic * Fix aria-labelledby logic * Fix flexbox logic --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: langermank <[email protected]> Co-authored-by: joshblack <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: langermank <[email protected]> Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: Armağan <[email protected]> Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ian Sanders <[email protected]> Co-authored-by: Ederson Lucas <[email protected]> Co-authored-by: Josh Soref <[email protected]> Co-authored-by: Rez <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: siddharthkp <[email protected]> Co-authored-by: GitHub Design Systems Bot <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
7 tasks
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jul 18, 2023
#3389) * Deprecate position prop for PageLayout.Pane, update stories and tests. * Create hungry-spies-remember.md * Fix typo in docs. * Revert PRs that are breaking dotcom to get the current release out (#3408) * Revert "Refactor(ActionList): ActionList.Item should render content as a button if parent is not interactive. (#3284)" This reverts commit 08d7d5d. * Revert "Address ToggleSwitch a11y feedback (#3251)" This reverts commit 580f165. * Revert "Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" (#3349)" This reverts commit a6b4169. * Revert "fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346)" This reverts commit 6304923. * Revert "Refactor FilteredActionList to address a11y violations and use new ActionList. (#3247)" This reverts commit e865e3e. * Revert "fix(ActionList): update to read from group context if selectionVariant is defined (#3269)" This reverts commit 21ec626. * Revert "Add back check for GroupContext in ActionList.Selection to fix issues introduced to github/github (#3267)" This reverts commit 332a1af. * Revert "Fix create-slots utils path (#3223)" This reverts commit f50997f. * Revert "Allow up to 4 levels of nesting for the NavList (#3343)" This reverts commit 786013e. * Revert "NavList: Fix group dividers (#3328)" This reverts commit 1fd6d32. * Revert "StyledOcticon/Octicon: Update the render func to use `React.forwardRef()` (#3293)" This reverts commit 027d214. * Revert "Rename component StyledOcticon to Octicon (#3148)" This reverts commit ac437bb. * Revert "Update PRC ActionList implementation to have similar semantics to PVC. (#2878)" This reverts commit 87883c3. * Update generated/components.json * Revert "Bump @primer/react-octicons to v19.0.1 (#3266)" This reverts commit 9532977. * Revert "Remove aria-hidden=true from spans with required asterisk (#3320)" This reverts commit 47b7ea0. * Rename component StyledOcticon to Octicon (#3148) * Rename component StyledOcticon to Octicon * Update generated/components.json * Create .changeset/swift-cows-rest.md * Fix snapshot * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * refactor(CircleOcticon): Move files to folder, add storybook and e2e tests. (#3119) * Reorganize CircleOcticon files, add story and e2e tests. * Add index file. * Update generated/components.json * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(Popover): move files into folder, add storybook and e2e tests. (#3135) * Move files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * refactor(ButtonGroup): move files to folder. (#3117) * Reorganize ButtonGroup component and stories. * Update generated/components.json * Add e2e tests. * test(vrt): update snapshots * refactor(CircleBadge): move files into folder, add e2e tests. (#3118) * Reorganize CircleBadge, add e2e tests. * Update generated/components.json * test(vrt): update snapshots * Update docs reference. --------- Co-authored-by: radglob <[email protected]> * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Refactor(FilteredSearch): move files to folder, add storybook and e2e tests. (#3125) * Move files to folder, add storybook and e2e tests. * Add aria-label to TextInput, fix docs reference. * Update generated/components.json * test(vrt): update snapshots * replace createRef with useRef (#3124) * replace createRef with useRef * changeset * missed null * refactor(FilterList): add files to folder, add storybook and e2e tests. (#3122) * Reorganize FilterList files into folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * Fix generated components. --------- Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> * Refactor(LabelGroup): add files to folder, add storybook and e2e tests. (#3131) * Move files to folder, add storybook and e2e tests. * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <[email protected]> * test(e2e): update ActionMenu e2e test to remove custom contrast rules (#3130) * test(e2e): update ActionMenu e2e test to remove custom contrast rules * test(vrt): update snapshots * test: remove multiple sections test * test(e2e): restore rule exception for ActionMenu --------- Co-authored-by: Josh Black <[email protected]> * NavList SSR: use useIsomorphicLayoutEffect instead of useLayoutEffect (#3127) * use useIsomorphicLayoutEffect * Create .changeset/fast-panthers-refuse.md --------- Co-authored-by: Josh Black <[email protected]> * Bump @primer/primitives to `7.11.5` (#3120) * npm install @primer/[email protected] * Updating`theme-preval` snapshots * Re-run ``` npm run build npm run test -- -u ``` So that /themePreval.test.ts.snap also gets the new timestamp * Update `theme-preval` snapshots docs * Create .changeset/old-falcons-learn.md * test(vrt): update snapshots --------- Co-authored-by: simurai <[email protected]> * Update RelativeTime stories (#3136) * Update RelativeTime stories * Update generated/components.json --------- Co-authored-by: colebemis <[email protected]> * chore(deps-dev): bump @types/jest-axe from 3.5.3 to 3.5.5 (#3141) Bumps [@types/jest-axe](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest-axe) from 3.5.3 to 3.5.5. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest-axe) --- updated-dependencies: - dependency-name: "@types/jest-axe" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump @rollup/plugin-typescript from 11.0.0 to 11.1.0 (#3138) Bumps [@rollup/plugin-typescript](https://github.com/rollup/plugins/tree/HEAD/packages/typescript) from 11.0.0 to 11.1.0. - [Release notes](https://github.com/rollup/plugins/releases) - [Changelog](https://github.com/rollup/plugins/blob/master/packages/typescript/CHANGELOG.md) - [Commits](https://github.com/rollup/plugins/commits/commonjs-v11.1.0/packages/typescript) --- updated-dependencies: - dependency-name: "@rollup/plugin-typescript" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 (#3140) * chore(deps-dev): bump eslint-plugin-primer-react from 2.0.2 to 2.0.3 Bumps [eslint-plugin-primer-react](https://github.com/primer/eslint-plugin-primer-react) from 2.0.2 to 2.0.3. - [Release notes](https://github.com/primer/eslint-plugin-primer-react/releases) - [Changelog](https://github.com/primer/eslint-plugin-primer-react/blob/main/CHANGELOG.md) - [Commits](primer/eslint-plugin-primer-react@v2.0.2...v2.0.3) --- updated-dependencies: - dependency-name: eslint-plugin-primer-react dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * chore(eslint): remove unnecessary disable --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 (#3139) * chore(deps-dev): bump babel-plugin-styled-components from 2.0.2 to 2.1.1 Bumps [babel-plugin-styled-components](https://github.com/styled-components/babel-plugin-styled-components) from 2.0.2 to 2.1.1. - [Release notes](https://github.com/styled-components/babel-plugin-styled-components/releases) - [Commits](styled-components/babel-plugin-styled-components@v2.0.2...v2.1.1) --- updated-dependencies: - dependency-name: babel-plugin-styled-components dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * test(snapshots): update snapshots --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <[email protected]> * chore(project): update project to use Node.js v18 LTS (#3133) * chore(project): update project to use Node.js v18 LTS * chore: add legacy openssl option for docs builds * chore: update devcontainer image --------- Co-authored-by: Josh Black <[email protected]> * Use SSR-compatible slot implementation in CheckboxGroup/RadioGroup (#3146) * Update slots for checkbox group and radio group * Update useSlot return type * Update exports test * Create .changeset/young-queens-notice.md * Update comment indentation * Add segments to progressbar (#3114) * Allow ProgressBar to either be given child segments or just a progress value. * Update ProgressBar docs. * Fix progress bar tests. * Create dry-weeks-compete.md * Fix errors from CI. * Rename ProgressBar.Segment to ProgressBar.Item. * Pass backgroundColor instead of bg to get away from system props, simplify children check in ProgressBar. * Updated snapshots. * Updated snapshots using --no-cache flag. * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * docs(project): move code of conduct under .github (#3145) Co-authored-by: Josh Black <[email protected]> * Add npm workspaces structure to primer/react (#3074) * test(axe): default `color-contrast` check to false * chore: update generate script to not conditionally disable contrast checks * Create adr-015-npm-workspaces.md * docs: update adr * Update adr-015-npm-workspaces.md --------- Co-authored-by: Josh Black <[email protected]> * Update generated/components.json * change styled_octicon key * Update generated/components.json * Update generated/components.json * Update generated/components.json * Octicon folder * Fix package-lock * Fix imports * update path * Update generated/components.json --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> * Allow up to 4 levels of nesting for the NavList (#3343) * allows up to 4 levels of nesting for the NavList * adds changeset * Update src/NavList/NavList.tsx Co-authored-by: Cole Bemis <[email protected]> * updates snapshots --------- Co-authored-by: Cole Bemis <[email protected]> * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346) * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler (#3163) * test: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`'s `onSelect` handler Failing test for #3162 * fix: prevent closing menu when `event.preventDefault()` is called on `ActionList.Item`ߴs `onSelect` handler * add storybook example: Delayed Menu Close * update docs * docs: changeset * Update changelog --------- Co-authored-by: Siddharth Kshetrapal <[email protected]> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]> * Revert "PageLayout.Content should not use main landmark by default. (#3154)" This reverts commit fed1c57. * Revert "Update `PageLayout` docs around `<main>` usage (#3323)" This reverts commit e1904b2. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: Jon Rohan <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: Jeremy Neal <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]> * Version Packages (#3218) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Update changeset to mention changed component. * Put changed components comment after changeset write up. * Fix some spacing issues, docs and story changes. * Update snapshots, formatting and linting. --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Armağan <[email protected]> Co-authored-by: broccolinisoup <[email protected]> Co-authored-by: Jon Rohan <[email protected]> Co-authored-by: jonrohan <[email protected]> Co-authored-by: radglob <[email protected]> Co-authored-by: Matthew Costabile <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: Siddharth Kshetrapal <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: simurai <[email protected]> Co-authored-by: Cole Bemis <[email protected]> Co-authored-by: colebemis <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <[email protected]> Co-authored-by: Mike Perrotti <[email protected]> Co-authored-by: Gregor Martynus <[email protected]> Co-authored-by: siddharthkp <[email protected]> Co-authored-by: GitHub Design Systems Bot <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes https://github.com/github/accessibility-audits/issues/2939
This PR aligns the semantics (in this case, both the API and the generated markup) of
ActionListand related components to match the implementation in PVC.A significant change is the removal of
ActionList.Group, which allowed sets of items to be labelled with a heading. This allowed somewhat arbitrary nesting, which we no longer want to allow.If an
ActionListneeds a heading, users can pass theheadingPropsprop to theActionList, which will then generate anActionList.Headingbefore the<ul>inside theActionList.Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist
Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.