Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 3728 by stuart.c...@gmail.com: Copy/paste code diff doesn't work in Firefox 35
https://code.google.com/p/reviewboard/issues/detail?id=3728

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
*** rejected.
***
*** If you have a security issue to report, please send it confidentially
to
*** secur...@reviewboard.org. Posting security-related issues to this bug
*** tracker causes us to have to do an emergency release.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** This bug tracker is public. Please check that any logs or other
information
*** that you include has been stripped of confidential information.


What version are you running?

2.0.12

What's the URL of the page containing the problem?

Any code diff page.

What steps will reproduce the problem?
1. Highlight code in one panel, either by dragging over the text, or ctrl+drag selecting table cells.
2. Press Ctrl+C in browser or right-click and choose "Copy" from menu.
3. Press Ctrl+V in a text editor.

What is the expected output? What do you see instead?

Expected output is all the highlighted text.

Actual output is the first line of the highlighted text, alone.

If selecting using ctrl+drag to select table cells, the output is two blank lines followed by the text of the first highlighted cell.

What operating system are you using? What browser?

Firefox 35.0, Ubuntu 14.04

Please provide any additional information below.

The issue does not occur in Chromium 39, which is the other browser I tested.

Just upgraded from 2.0.8 to 2.0.12 and disappointed that not only does issue 1094 appear to not work, it also breaks the workaround, which was to ctrl+drag select table cells.

The copy/paste issue could be solved by HTML layout alone if the diffs were structured as one table row per chunk, each column containing an embedded table with all the lines of the chunk, but I can understand if this would cause a lot of code rewriting and make it difficult to select ranges of code that cross chunk boundaries.

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.

Reply via email to