Skip to content

chore(DialogV1): remove sx from deprecated Dialog - #6923

Merged
francinelucca merged 7 commits into
mainfrom
chore/remove-sx-from-deprecated-dialog
Sep 30, 2025
Merged

chore(DialogV1): remove sx from deprecated Dialog#6923
francinelucca merged 7 commits into
mainfrom
chore/remove-sx-from-deprecated-dialog

Conversation

@francinelucca

@francineluccafrancinelucca commented Sep 30, 2025

Copy link
Copy Markdown
Member

Closes#6776

Current sx usage: 0

Changelog

Removed

Remove support for sx from the DialogV1 deprecated component, and associated stories, docs, and tests

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

@changeset-bot

changeset-botBot commented Sep 30, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 848563c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
NameType
@primer/reactMajor
@primer/styled-reactMajor

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

@github-actionsgithub-actionsBot added the staff Author is a staff member label Sep 30, 2025
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks!

@github-actionsgithub-actionsBot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Sep 30, 2025
@github-actions
github-actionsBot requested a deployment to storybook-preview-6923 September 30, 2025 14:10 Abandoned
@github-actions
github-actionsBottemporarily deployed to storybook-preview-6923 September 30, 2025 14:26 Inactive
@francineluccafrancinelucca changed the title chore: remove sx from deprecated Dialogchore(DialogV1): remove sx from deprecated DialogSep 30, 2025
@francinelucca
francinelucca marked this pull request as ready for review September 30, 2025 15:25
@francinelucca
francinelucca requested a review from a team as a code ownerSeptember 30, 2025 15:25

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the sx prop support from the deprecated DialogV1 component as part of the migration away from styled-components. The change is part of a broader effort to eliminate sx prop usage across deprecated components.

  • Removes SxProp and sx prop support from DialogV1 component and its Header subcomponent
  • Replaces BoxWithFallback wrapper components with native HTML elements
  • Updates styled-react package to provide its own DialogV1 wrapper that maintains sx support for backward compatibility

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
FileDescription
packages/react/src/DialogV1/Dialog.tsxRemoves sx prop support and BoxWithFallback wrapper, adds as prop for component polymorphism
packages/styled-react/src/components/DialogV1.tsxNew wrapper component that adds back sx prop support using Box component
packages/styled-react/src/deprecated.tsxUpdates Dialog export to use local DialogV1 wrapper instead of direct import
packages/styled-react/src/tests/primer-react-deprecated.browser.test.tsxUpdates tests to verify sx prop functionality with new wrapper
.changeset/modern-chicken-happen.mdDocuments breaking change for major version bump

Comment threadpackages/styled-react/src/components/DialogV1.tsx Outdated
Comment threadpackages/styled-react/src/components/DialogV1.tsx Outdated
@github-actions
github-actionsBottemporarily deployed to storybook-preview-6923 September 30, 2025 15:31 Inactive
@primer-integration

Copy link
Copy Markdown

👋 Hi from github/github-ui! Your integration PR is ready: https://github.com/github/github-ui/pull/3554

@github-actionsgithub-actionsBot added integration-tests: failing Changes in this PR cause breaking changes in gh/gh integration-tests: passing Changes in this PR do NOT cause breaking changes in gh/gh and removed integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm integration-tests: failing Changes in this PR cause breaking changes in gh/gh labels Sep 30, 2025
@primer-integration

Copy link
Copy Markdown

🟢 ci completed with status success.

@francinelucca
francinelucca added this pull request to the merge queueSep 30, 2025
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Sep 30, 2025
@github-actionsgithub-actionsBot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Sep 30, 2025
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

@github-actionsgithub-actionsBot removed the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Sep 30, 2025
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi, there are new commits since the last successful integration test. We recommend running the integration workflow once more, unless you are sure the new changes do not affect github/github. Thanks!

@github-actionsgithub-actionsBot added the integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm label Sep 30, 2025
@github-actions
github-actionsBottemporarily deployed to storybook-preview-6923 September 30, 2025 17:49 Inactive
@github-actions
github-actionsBottemporarily deployed to storybook-preview-6923 September 30, 2025 18:03 Inactive
@francinelucca
francinelucca added this pull request to the merge queueSep 30, 2025
Merged via the queue into main with commit e5753f0Sep 30, 2025
42 checks passed
@francinelucca
francinelucca deleted the chore/remove-sx-from-deprecated-dialog branch September 30, 2025 19:06
@primerprimerBot mentioned this pull request Sep 30, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: passingChanges in this PR do NOT cause breaking changes in gh/ghintegration-tests: recommendedThis change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpmstaffAuthor is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove support for sx from the Dialog component in @primer/react/deprecated

3 participants

@francinelucca@jonrohan