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

Support triple-click selections for smartCopy #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benwiley4000
Copy link

A triple-click selection typically results in a newline character being added to the end of the selection string, which was incompatible with how we were comparing the copied string to our known text (===). By trimming the whitespace from the selection we can ensure a correct match.

Pull Request Checklist

  • Added/updated the following to reflect my change:
    • [I can add one if you think it's needed] Unit tests
    • [N/A?] Documentation
  • Tested against the different major versions of React
    • [N/A? Not a react-related issue] v15.5.x
    • [N/A?] v16.x.x
  • Unit tests have passed.
  • Code linting has passed.

A triple-click selection typically results in a newline character being added to the end of the selection string, which was incompatible with how we were comparing the copied string to our known text (===). By trimming the whitespace from the selection we can ensure a correct match.
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.

1 participant