Revert PRs that are breaking dotcom to get the current release out #3408

broccolinisoup · 2023-06-12T21:06:39Z

What is this PR for?

Due to the recent deploy freeze at dotcom, we were unable to release primer/react on a regular schedule and this impacted the scope of the release candidate, therefore made it very challenging for us to address failing tests at dotcom. As a team, we made a decision to lock the main branch on 8th June to prevent more changes coming in and reverting the PRs that we are introducing breaking changes to be able to get the current release out.

Reverted PR List

Release Plan

We have been running integration tests at dotcom using the canary version of this PR's branch and we reached to a point where the jest tests are passing 🎉 https://github.com/github/github/pull/275683 Please note that currently there are still 3 tests are failing but only 1 (I believe) is currently might be related to our changes and we seek advice on how to make it passed, and the licence issue needs to be manually reviewed by the GH open source reviewers which we already requested.

~~Failing test 1 (github-systems): error log and slack ref (Github stuff only link)~~
~~Failing test 2 (github-js): error log and slack ref (GitHub stuff only link) and another slack ref (GitHub stuff only link) - There is a chance that might be from our side which will impact the plan~~
~~Failing test 3 (github-licences): error log and dotcom comment for request (GitHub stuff only link)~~

We plan to merge this PR and once we get the rest of the 3 tests passed, we will cut the release and update the version at dotcom afterwards. Please let me and @colebemis know if you have any concern with the plan or the reverted changes.

Plan forward to re-introduce reverted PRs

We will work with @camertron , @adrianababakanian, @radglob and myself 😅 @broccolinisoup to re-introduce the changes.

We are well aware that this is a disappointing news but we will make sure to get these changes out as soon as possible.

As always, we appreciate your support and understanding ❤️

…s a button if parent is not interactive. (#3284)" This reverts commit 08d7d5d.

changeset-bot · 2023-06-12T21:06:42Z

⚠️ No Changeset found

Latest commit: ed03efe

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

github-actions · 2023-06-12T21:10:26Z

size-limit report 📦

Path	Size
dist/browser.esm.js	101.05 KB (-0.81% 🔽)
dist/browser.umd.js	101.58 KB (-0.77% 🔽)

This reverts commit 580f165.

…s and use new ActionList. (#3247)" (#3349)" This reverts commit a6b4169.

@gr2m

…lled on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346)" This reverts commit 6304923.

…e new ActionList. (#3247)" This reverts commit e865e3e.

…onVariant is defined (#3269)" This reverts commit 21ec626.

…x issues introduced to github/github (#3267)" This reverts commit 332a1af.

This reverts commit f50997f.

This reverts commit 786013e.

This reverts commit 1fd6d32.

…orwardRef()` (#3293)" This reverts commit 027d214.

This reverts commit ac437bb.

…s to PVC. (#2878)" This reverts commit 87883c3.

* allows up to 4 levels of nesting for the NavList * adds changeset * Update src/NavList/NavList.tsx Co-authored-by: Cole Bemis <colebemis@github.com> * updates snapshots --------- Co-authored-by: Cole Bemis <colebemis@github.com>

@gr2m

…`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 <siddharthkp@github.com> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com>

adrianababakanian · 2023-06-15T13:38:57Z

👋 Thanks for the information @broccolinisoup ! Please let me know how I can help re-introduce the changes from my PRs that couldn't be included, so that they can make it to the next release! Happy to help with whatever tweaks are needed.

camertron · 2023-06-15T20:35:53Z

Oh no 😱 Please let me know what I can do to help as well ❤️

…3154)" This reverts commit fed1c57.

This reverts commit e1904b2.

broccolinisoup · 2023-06-16T04:49:46Z

I am merging this PR as we got all checks ✅ on the latest integration commit

@gr2m

…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 <radglob@users.noreply.github.com> * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * Fix generated components. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> * 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 <radglob@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * NavList SSR: use useIsomorphicLayoutEffect instead of useLayoutEffect (#3127) * use useIsomorphicLayoutEffect * Create .changeset/fast-panthers-refuse.md --------- Co-authored-by: Josh Black <joshblack@github.com> * Bump @primer/primitives to `7.11.5` (#3120) * npm install @primer/primitives@7.11.5 * 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 <simurai@users.noreply.github.com> * Update RelativeTime stories (#3136) * Update RelativeTime stories * Update generated/components.json --------- Co-authored-by: colebemis <colebemis@users.noreply.github.com> * 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] <support@github.com> 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] <support@github.com> 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] <support@github.com> * chore(eslint): remove unnecessary disable --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * 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] <support@github.com> * test(snapshots): update snapshots --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> 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 <joshblack@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: jonrohan <jonrohan@users.noreply.github.com> Co-authored-by: Jeremy Neal <radglob@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <josepmartins@github.com> * 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 <colebemis@github.com> * updates snapshots --------- Co-authored-by: Cole Bemis <colebemis@github.com> * 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 <siddharthkp@github.com> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: Jon Rohan <yes@jonrohan.codes> Co-authored-by: jonrohan <jonrohan@users.noreply.github.com> Co-authored-by: Jeremy Neal <radglob@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <josepmartins@github.com> Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com>

@gr2m

#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 <radglob@users.noreply.github.com> * Update generated/components.json * test(vrt): update snapshots --------- Co-authored-by: radglob <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * 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 <radglob@users.noreply.github.com> * Fix generated components. --------- Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> * 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 <radglob@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * NavList SSR: use useIsomorphicLayoutEffect instead of useLayoutEffect (#3127) * use useIsomorphicLayoutEffect * Create .changeset/fast-panthers-refuse.md --------- Co-authored-by: Josh Black <joshblack@github.com> * Bump @primer/primitives to `7.11.5` (#3120) * npm install @primer/primitives@7.11.5 * 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 <simurai@users.noreply.github.com> * Update RelativeTime stories (#3136) * Update RelativeTime stories * Update generated/components.json --------- Co-authored-by: colebemis <colebemis@users.noreply.github.com> * 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] <support@github.com> 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] <support@github.com> 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] <support@github.com> * chore(eslint): remove unnecessary disable --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * 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] <support@github.com> * test(snapshots): update snapshots --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> 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 <joshblack@users.noreply.github.com> * 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 <joshblack@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: jonrohan <jonrohan@users.noreply.github.com> Co-authored-by: Jeremy Neal <radglob@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <josepmartins@github.com> * 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 <colebemis@github.com> * updates snapshots --------- Co-authored-by: Cole Bemis <colebemis@github.com> * 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 <siddharthkp@github.com> * Update generated/components.json --------- Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: Jon Rohan <yes@jonrohan.codes> Co-authored-by: jonrohan <jonrohan@users.noreply.github.com> Co-authored-by: Jeremy Neal <radglob@github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <josepmartins@github.com> Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> * 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] <support@github.com> Co-authored-by: Armağan <broccolinisoup@github.com> Co-authored-by: broccolinisoup <broccolinisoup@users.noreply.github.com> Co-authored-by: Jon Rohan <yes@jonrohan.codes> Co-authored-by: jonrohan <jonrohan@users.noreply.github.com> Co-authored-by: radglob <radglob@users.noreply.github.com> Co-authored-by: Matthew Costabile <mattcosta7@github.com> Co-authored-by: Josh Black <joshblack@github.com> Co-authored-by: Josh Black <joshblack@users.noreply.github.com> Co-authored-by: Siddharth Kshetrapal <siddharthkp@github.com> Co-authored-by: simurai <simurai@github.com> Co-authored-by: simurai <simurai@users.noreply.github.com> Co-authored-by: Cole Bemis <colebemis@github.com> Co-authored-by: colebemis <colebemis@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Josep Martins <josepmartins@github.com> Co-authored-by: Mike Perrotti <mperrotti@github.com> Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com> Co-authored-by: siddharthkp <siddharthkp@users.noreply.github.com> Co-authored-by: GitHub Design Systems Bot <30705008+primer-css@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Revert "Refactor(ActionList): ActionList.Item should render content a…

53ec7b1

…s a button if parent is not interactive. (#3284)" This reverts commit 08d7d5d.

broccolinisoup temporarily deployed to github-pages June 12, 2023 21:14 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 12, 2023 21:14 Inactive

broccolinisoup and others added 13 commits June 13, 2023 07:58

Revert "Address ToggleSwitch a11y feedback (#3251)"

bbc86d0

This reverts commit 580f165.

Revert "Revert "Refactor FilteredActionList to address a11y violation…

78b01ad

…s and use new ActionList. (#3247)" (#3349)" This reverts commit a6b4169.

Revert "fix: prevent closing menu when event.preventDefault() is ca…

c6b9e25

…lled on `ActionList.Item`'s `onSelect` handler by @gr2m (#3346)" This reverts commit 6304923.

Revert "Refactor FilteredActionList to address a11y violations and us…

780602f

…e new ActionList. (#3247)" This reverts commit e865e3e.

Revert "fix(ActionList): update to read from group context if selecti…

741c8a7

…onVariant is defined (#3269)" This reverts commit 21ec626.

Revert "Add back check for GroupContext in ActionList.Selection to fi…

700dffd

…x issues introduced to github/github (#3267)" This reverts commit 332a1af.

Revert "Fix create-slots utils path (#3223)"

bec5827

This reverts commit f50997f.

Revert "Allow up to 4 levels of nesting for the NavList (#3343)"

f316432

This reverts commit 786013e.

Revert "NavList: Fix group dividers (#3328)"

ae5a587

This reverts commit 1fd6d32.

Revert "StyledOcticon/Octicon: Update the render func to use `React.f…

4de8bd1

…orwardRef()` (#3293)" This reverts commit 027d214.

Revert "Rename component StyledOcticon to Octicon (#3148)"

8ced0af

This reverts commit ac437bb.

Revert "Update PRC ActionList implementation to have similar semantic…

ffe13a7

…s to PVC. (#2878)" This reverts commit 87883c3.

Update generated/components.json

e60671c

github-actions bot temporarily deployed to storybook-preview-3408 June 12, 2023 22:36 Inactive

primer bot temporarily deployed to github-pages June 12, 2023 22:37 Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 12, 2023 22:38 Inactive

broccolinisoup force-pushed the revert-3284-refactor-actionlist-item-as-button branch from 4461c63 to e60671c Compare June 13, 2023 03:26

broccolinisoup temporarily deployed to github-pages June 13, 2023 03:34 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 13, 2023 03:34 Inactive

broccolinisoup force-pushed the revert-3284-refactor-actionlist-item-as-button branch from 4393680 to e60671c Compare June 13, 2023 05:12

github-actions bot temporarily deployed to storybook-preview-3408 June 13, 2023 05:16 Inactive

broccolinisoup temporarily deployed to github-pages June 13, 2023 05:17 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 13, 2023 05:17 Inactive

broccolinisoup temporarily deployed to github-pages June 13, 2023 05:49 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 13, 2023 05:49 Inactive

Allow up to 4 levels of nesting for the NavList (#3343)

080f4ce

* allows up to 4 levels of nesting for the NavList * adds changeset * Update src/NavList/NavList.tsx Co-authored-by: Cole Bemis <colebemis@github.com> * updates snapshots --------- Co-authored-by: Cole Bemis <colebemis@github.com>

broccolinisoup temporarily deployed to github-pages June 15, 2023 03:57 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 15, 2023 03:57 Inactive

broccolinisoup temporarily deployed to github-pages June 15, 2023 06:41 — with GitHub Actions Inactive

github-actions bot temporarily deployed to storybook-preview-3408 June 15, 2023 06:41 Inactive

colebemis self-requested a review June 15, 2023 15:41

colebemis approved these changes Jun 15, 2023

View reviewed changes

broccolinisoup marked this pull request as ready for review June 16, 2023 02:15

broccolinisoup requested a review from a team June 16, 2023 02:15

broccolinisoup added 2 commits June 16, 2023 13:49

Revert "PageLayout.Content should not use main landmark by default. (#…

8c2732c

…3154)" This reverts commit fed1c57.

Revert "Update PageLayout docs around <main> usage (#3323)"

ed03efe

This reverts commit e1904b2.

broccolinisoup deployed to github-pages June 16, 2023 03:55 — with GitHub Actions View deployment

github-actions bot deployed to storybook-preview-3408 June 16, 2023 03:56 View deployment

broccolinisoup added this pull request to the merge queue Jun 16, 2023

Merged via the queue into main with commit d90d9be Jun 16, 2023
30 checks passed

broccolinisoup deleted the revert-3284-refactor-actionlist-item-as-button branch June 16, 2023 05:04

broccolinisoup mentioned this pull request Jun 16, 2023

Release tracking #3218

Merged

TylerJDev mentioned this pull request Aug 29, 2023

Update PageLayout docs #3684

Merged

7 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Revert PRs that are breaking dotcom to get the current release out #3408

Revert PRs that are breaking dotcom to get the current release out #3408

broccolinisoup commented Jun 12, 2023 •

edited

Loading

changeset-bot bot commented Jun 12, 2023 •

edited

Loading

github-actions bot commented Jun 12, 2023 •

edited

Loading

adrianababakanian commented Jun 15, 2023

camertron commented Jun 15, 2023

broccolinisoup commented Jun 16, 2023

Revert PRs that are breaking dotcom to get the current release out #3408

Revert PRs that are breaking dotcom to get the current release out #3408

Conversation

broccolinisoup commented Jun 12, 2023 • edited Loading

What is this PR for?

Reverted PR List

Release Plan

Plan forward to re-introduce reverted PRs

changeset-bot bot commented Jun 12, 2023 • edited Loading

⚠️ No Changeset found

github-actions bot commented Jun 12, 2023 • edited Loading

size-limit report 📦

adrianababakanian commented Jun 15, 2023

camertron commented Jun 15, 2023

broccolinisoup commented Jun 16, 2023

broccolinisoup commented Jun 12, 2023 •

edited

Loading

changeset-bot bot commented Jun 12, 2023 •

edited

Loading

github-actions bot commented Jun 12, 2023 •

edited

Loading