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

Hotfix line endings #26

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

andmetoo
Copy link

🔨The ends of lines don't work correctly in the created layer, textarea element is necessary for correct work.

@sudodoki
Copy link
Owner

Can you provide jsfiddle/jsbin with example of what exactly didn't work for you? Not sure what's your exact problem is, but release https://github.com/sudodoki/copy-to-clipboard/releases/tag/1.1.1 did work with newlines correctly (although there was an issue with content that needed escaping - see some closed issues for more info).

@andmetoo
Copy link
Author

andmetoo commented May 23, 2016

Yea you can check jsfiddle
Just past copied data to any spreadsheets, like google spreadsheets.
test in master branch

@sudodoki
Copy link
Owner

Will try to get to this on weekend, need to figure out if there are other workarounds and test usual test cases.

@sudodoki
Copy link
Owner

Thanks for bringing this up.

@andmetoo
Copy link
Author

There are no problems, can help at any time with any questions

@shvaikalesh
Copy link
Collaborator

@sudodoki if this PR passes the tests, we can merge it, make couple fixes and release new patch (3.0.5). With textarea we will drop couple variables and make code a little better.

@sudodoki
Copy link
Owner

@shvaikalesh need to verify we are not going to run into issues with multiline text and final newline in chrome/safari vs firefox in textarea as during writing e2e test it turned out there are inconsistencies (might be only with pasting, and not setting value through js). Don't think I will have time to do that now, although this change might be desirable.

@andmetoo
Copy link
Author

andmetoo commented Jul 10, 2016

hello old fix really has a big problem if the copy 40000 rows. I found a solution which is suitable for all scenarios, this method work in all cases for themselves now. I'm sorry for error.

@shvaikalesh
Copy link
Collaborator

innerHTML is not cool. Please, see closed issues for details.

@andmetoo
Copy link
Author

andmetoo commented Jul 10, 2016

yea this is sucks, can we use method depending on the situation?

@SEAPUNK
Copy link

SEAPUNK commented Sep 2, 2016

Is there any progress on this? This problem is biting me in the ass too. I'm willing to help out, if needed.

@SEAPUNK
Copy link

SEAPUNK commented Oct 18, 2016

Bump; any updates?

@sudodoki
Copy link
Owner

@andmetoo @SEAPUNK still didn't have enough time to look into this re #26 (comment). Might consider depending on andmetoo/copy-to-clipboard#hotfix-line-endings from package.json for time being

@andmetoo
Copy link
Author

@SEAPUNK U can use directly from my fork
npm install --save https://github.com/andmetoo/copy-to-clipboard/tarball/hotfix-line-endings

@SEAPUNK
Copy link

SEAPUNK commented Oct 23, 2016

I am, but I prefer that I install from releases.

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.

4 participants