Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Prevent button props from being added to the DOM element #264

Merged
merged 1 commit into from
May 17, 2023

Conversation

kenneth-duverge
Copy link
Collaborator

@kenneth-duverge kenneth-duverge commented May 10, 2023

What this PR does

Cleans up our console!

Screenshots & Recordings

How it does that

Removes the shouldForwardProp attribute as it was exposing other props as DOM attributes.

Testing

@kenneth-duverge kenneth-duverge marked this pull request as ready for review May 10, 2023 17:35
@kenneth-duverge kenneth-duverge requested review from a team as code owners May 10, 2023 17:35
@kenneth-duverge kenneth-duverge temporarily deployed to development May 10, 2023 17:42 — with GitHub Actions Inactive
@kenneth-duverge kenneth-duverge marked this pull request as draft May 10, 2023 18:52
@kenneth-duverge kenneth-duverge force-pushed the stop-prop-forwarding-button branch from 2702b1a to d619b00 Compare May 10, 2023 18:53
@kenneth-duverge kenneth-duverge marked this pull request as ready for review May 10, 2023 19:31
@kenneth-duverge kenneth-duverge force-pushed the stop-prop-forwarding-button branch from d619b00 to 2239780 Compare May 10, 2023 19:49
@kenneth-duverge kenneth-duverge temporarily deployed to development May 10, 2023 19:55 — with GitHub Actions Inactive
@kenneth-duverge kenneth-duverge force-pushed the stop-prop-forwarding-button branch from 2239780 to e40b4f6 Compare May 10, 2023 20:03
@kenneth-duverge kenneth-duverge force-pushed the stop-prop-forwarding-button branch from e40b4f6 to 6a17936 Compare May 10, 2023 20:40
@kenneth-duverge kenneth-duverge temporarily deployed to development May 10, 2023 20:46 — with GitHub Actions Inactive
@juliewongbandue juliewongbandue merged commit 2883e5d into main May 17, 2023
@juliewongbandue juliewongbandue deleted the stop-prop-forwarding-button branch May 17, 2023 13:59
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants