Issue 506 in rietveld: [Feature request] Better copy pasting support

1 view
Skip to first unread message

riet...@googlecode.com

unread,
Sep 30, 2014, 10:26:34 AM9/30/14
to coderev...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium Component-UI

New issue 506 by lpro...@chromium.org: [Feature request] Better copy
pasting support
https://code.google.com/p/rietveld/issues/detail?id=506

STEPS TO REPRODUCE
1- Open a Rietveld issue. For example
https://codereview.chromium.org/120753002/.
2- View a unified diff of a file. For example:
https://codereview.chromium.org/120753002/diff/370001/base/metrics/statistics_recorder.h
3- Try to copy the right part of the comment that has a diff.

WHAT HAPPENS
https://screenshot.googleplex.com/AiiTOGmA7U
Selection spans across both sides, selecting something like:

// GetSnapshot copies some of the pointers to registered histograms into the
73 // caller supplied vector (Histograms). Only histograms with names
matching 73 // caller supplied vector (Histograms). Only histograms
which have |query| as
74 // query are returned. The query must be a substring of histogram
name for its 74 // a substring are copied (an empty string will process
all registered
75 // pointer to be copied. 75 // histograms).

WHAT SHOULD HAPPEN
Selection only selects the left or the right text, and also doesn't select
the numbers or any other spacing.

Selection should be:
// GetSnapshot copies some of the pointers to registered histograms into the
// caller supplied vector (Histograms). Only histograms which have |
query| as
// a substring are copied (an empty string will process all registered
// histograms).



--
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
Reply all
Reply to author
Forward
0 new messages