Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix newline interpolation of text input values #6935

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

eliranmal
Copy link
Contributor

@eliranmal eliranmal commented Feb 13, 2022

What: Closes #5324

when using ClipboardCopy, and passing an input text which contains newlines as a template string variable, they are omitted from the input value.

fix this in TextInputBase by replacing incoming newlines with whitespace characters, right before setting the input value (when the value is of string type).

Additional issues: N/A

newline characters can be omitted from the input value, fix this by always replacing incoming newlines with whitespace characters.

fixes: patternfly#5324
@patternfly-build
Copy link
Contributor

@nicolethoen nicolethoen requested a review from tlabaj March 19, 2022 03:13
Copy link
Contributor

@tlabaj tlabaj left a comment

Choose a reason for hiding this comment

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

LGTM

@tlabaj tlabaj merged commit c667ef0 into patternfly:main Mar 29, 2022
@patternfly-build
Copy link
Contributor

Your changes have been released in:

  • eslint-plugin-patternfly-react@4.41.13
  • @patternfly/react-catalog-view-extension@4.53.13
  • @patternfly/react-charts@6.55.13
  • @patternfly/react-code-editor@4.43.13
  • @patternfly/react-console@4.53.13
  • @patternfly/react-core@4.202.13
  • @patternfly/react-docs@5.63.13
  • @patternfly/react-icons@4.53.13
  • @patternfly/react-inline-edit-extension@4.47.13
  • demo-app-ts@4.162.13
  • @patternfly/react-integration@4.164.13
  • @patternfly/react-log-viewer@4.47.13
  • @patternfly/react-styles@4.52.13
  • @patternfly/react-table@4.71.13
  • @patternfly/react-tokens@4.54.13
  • @patternfly/react-topology@4.49.13
  • @patternfly/react-virtualized-extension@4.49.13
  • transformer-cjs-imports@4.40.13

Thanks for your contribution! 🎉

@eliranmal eliranmal deleted the 5324-clipboard-copy-multiline-fix branch April 4, 2022 19:25
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.

ClipboardCopy displays multi-line text oddly in the summary section
4 participants