Skip to content

chore(react-swatch-picker-preview): migrate from getNativeElementProps to getIntrinsicElementProps#29534

Merged
bsunderhus merged 1 commit into
microsoft:masterfrom
bsunderhus:react-swatch-picker-preview/chore--migrate-from-getNativeElementProps-to-getIntrinsicElementProps
Oct 16, 2023
Merged

chore(react-swatch-picker-preview): migrate from getNativeElementProps to getIntrinsicElementProps#29534
bsunderhus merged 1 commit into
microsoft:masterfrom
bsunderhus:react-swatch-picker-preview/chore--migrate-from-getNativeElementProps-to-getIntrinsicElementProps

Conversation

@bsunderhus

Copy link
Copy Markdown
Contributor

New Behavior

Follow up on #29310

getNativeElementProps has some type safety issues, allowing erroneous properties to be introduced.

  1. replaces use cases of getNativeElementProps with an equivalent but more restrict method getIntrinsicElementProps

@bsunderhus
bsunderhus requested a review from a team as a code owner October 16, 2023 12:52
@bsunderhus bsunderhus self-assigned this Oct 16, 2023
@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 7dea38da7c75c481d8edf9b102eea8627078179d

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 80e556e:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@size-auditor

size-auditor Bot commented Oct 16, 2023

Copy link
Copy Markdown

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 7dea38da7c75c481d8edf9b102eea8627078179d (build)

@bsunderhus
bsunderhus enabled auto-merge (squash) October 16, 2023 13:35
@bsunderhus
bsunderhus merged commit 4f9b23d into microsoft:master Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants