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 for copy-pasting formatted text to markdown area #7082

Merged
merged 1 commit into from Jun 24, 2014

Conversation

erbunao
Copy link
Contributor

@erbunao erbunao commented Jun 5, 2014

We (@nmcalabroso) fixed the bug for pasting a formatted text to the markdown area. This PR is a fix for #7032. Hoping for the feedbacks.

@erbunao
Copy link
Contributor Author

erbunao commented Jun 6, 2014

@randx @dosire This would be the fix for the clipboard bug :)

@jvanbaarsen
Copy link
Contributor

@randx Looks good

@erbunao
Copy link
Contributor Author

erbunao commented Jun 10, 2014

@jvanbaarsen hope there's no more hidden bugs.

@dzaporozhets
Copy link
Member

@erbunao I am a bit confused. We reverted #7032 because it breaks copy-paste. What do this fix do then?

@erbunao
Copy link
Contributor Author

erbunao commented Jun 11, 2014

@randx when I last checked our feature, it cannot copy formatted text, only the image and a plain text. So this fix is for fully supporting all types of text. The reason why our last PR can't copy a formatted text is because we override the printing of text to the text area but the copy paste function is working as is. But for this fix, we changed our approach to just paste any kind of text to the text area when it is not an image. Basically, it can now just copy paste literally any text just like any copy paste function. I hope this clears things out :)

@dosire
Copy link
Member

dosire commented Jun 18, 2014

@randx So this is #7032 with a fix for the problem.

dzaporozhets added a commit that referenced this pull request Jun 24, 2014
Fix for copy-pasting formatted text to markdown area
@dzaporozhets dzaporozhets merged commit d7eb6e5 into gitlabhq:master Jun 24, 2014
@dzaporozhets
Copy link
Member

@dosire ok thanks for explaining

@dosire
Copy link
Member

dosire commented Jun 24, 2014

@randx Thanks for merging.

@erbunao Thanks for fixing, it is great that you contribute awesome features and provide fixes for them as well.

@sanand0
Copy link

sanand0 commented Jul 27, 2014

@erbunao I tried this on Windows 8.1 + Chrome 36 via the Gitlab Omnibus 7.1 package. The following did not work:

  1. Copying a file from File Explorer and pasting it into a new issue description. No file was uploaded, nor was there a console error.
  2. Copying formatted HTML from this page (with and without images) and pasting it into a new issue description. The plain text was pasted, but no image was uploaded, nor a console error.

Am I doing something wrong?

@dosire
Copy link
Member

dosire commented Jul 27, 2014

@sanand0 I don't know, but please open a new issue referencing this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants