Skip to content

fix: textarea value is being duplicated - #62

Merged
billyvg merged 3 commits into
sentry-v1from
test-inputs-inserted-into-dom
Feb 24, 2023
Merged

fix: textarea value is being duplicated#62
billyvg merged 3 commits into
sentry-v1from
test-inputs-inserted-into-dom

Conversation

@billyvg

@billyvgbillyvg commented Feb 23, 2023

Copy link
Copy Markdown
Member

Fix bug introduced in #43 where we masked both textarea's value and textContent, which meant the text inside of the textarea would get duplicated. We can ignore textContent in this case and set it to an empty string.

@billyvg
billyvgforce-pushed the test-inputs-inserted-into-dom branch from c8d0066 to f7fec59CompareFebruary 23, 2023 01:34
@billyvg
billyvgforce-pushed the test-inputs-inserted-into-dom branch from f7fec59 to 8a487deCompareFebruary 23, 2023 01:34
@billyvgbillyvg changed the title test inputs inserted into domfix: textarea value is being duplicatedFeb 23, 2023
@billyvg
billyvg marked this pull request as ready for review February 23, 2023 19:43
@billyvg
billyvg requested review from Lms24 and mydeaFebruary 23, 2023 19:45

@Lms24Lms24 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@billyvg
billyvg merged commit 00c5d28 into sentry-v1Feb 24, 2023
@billyvg
billyvg deleted the test-inputs-inserted-into-dom branch February 24, 2023 13:47
Sign up for freeto 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.

2 participants

@billyvg@Lms24