ereviewboard icon indicating copy to clipboard operation
ereviewboard copied to clipboard

Support Multiple Line Selection for Comments

Open RunnersReign opened this issue 13 years ago • 198 comments

When making a comment on a code diff using the Eclipse Plugin, I would normally select multiple lines that I want to make the comment on and then make the comment on that block of text. This is great for instances where you want to show the entire section of code that will cause the error. I finding however that the the comment only refers to the first line of the selection when creating the comment, not the entire highlighted selection.

Steps to Reproduce:

  • View a review in Eclipse
  • Select multiple lines to make a comment on.
  • Right click and select "Add Comment"
  • Enter in text and click "Ok"
  • Post comment to server.

If you open the review within the Review Board web application, you can see only the first line selected is shown as having a comment associated with it.

It would be a great if all lines selected would be commented on instead of just the first line.

RunnersReign avatar Feb 01 '12 17:02 RunnersReign

Thanks for raising this issue. We build on the support from the Mylyn Reviews project, so this might not be implemented upstream. I'll check and see if we can do something locally.

rombert avatar Feb 01 '12 22:02 rombert

Any news? During code review I'd like to comment on a method and not a specific line or the entire commit

ittaiz avatar Mar 25 '14 08:03 ittaiz

No, this still has not been implemented.

On Tue, Mar 25, 2014 at 10:00 AM, Ittai Zeidman [email protected]:

Any news? During code review I'd like to comment on a method and not a specific line or the entire commit

— Reply to this email directly or view it on GitHubhttps://github.com/rombert/ereviewboard/issues/92#issuecomment-38538660 .

http://robert.muntea.nu/

rombert avatar Mar 25 '14 08:03 rombert

This would be a great feature +1 from me.

brianmrock avatar Apr 08 '15 15:04 brianmrock

+1

bc-shusson avatar Apr 24 '15 01:04 bc-shusson

+1

JustDerb avatar Apr 26 '15 03:04 JustDerb

+1

codemedic avatar May 05 '15 10:05 codemedic

+1

rainulf avatar May 05 '15 17:05 rainulf

+1

bwplotka avatar Jun 16 '15 13:06 bwplotka

+1

TroldeJens avatar Jul 07 '15 14:07 TroldeJens

+1

tj-recess avatar Jul 21 '15 18:07 tj-recess

+1

ryanrolds avatar Sep 04 '15 20:09 ryanrolds

+1

bg-o avatar Sep 23 '15 10:09 bg-o

+1

kavirajk avatar Sep 29 '15 15:09 kavirajk

+1

ttddyy avatar Nov 08 '15 08:11 ttddyy

+1

yotamoron avatar Dec 06 '15 15:12 yotamoron

yes please

emckenna avatar Feb 17 '16 16:02 emckenna

+1

cybersam avatar Feb 24 '16 21:02 cybersam

+1

aarosil avatar Mar 15 '16 05:03 aarosil

+1

samuelmburu avatar Mar 16 '16 18:03 samuelmburu

+1

ulisesbocchio avatar Mar 23 '16 01:03 ulisesbocchio

+1

mktal avatar Mar 24 '16 18:03 mktal

+1

jpotte1d avatar May 06 '16 18:05 jpotte1d

+1

baras avatar May 22 '16 13:05 baras

+1

jeremy-techson avatar May 24 '16 06:05 jeremy-techson

+1

ebrause avatar May 26 '16 20:05 ebrause

+1

meanmina avatar Jun 09 '16 10:06 meanmina

+1

flopex avatar Jun 14 '16 21:06 flopex

+1

alexissan avatar Jun 15 '16 09:06 alexissan

+1

Stadler76 avatar Jul 09 '16 15:07 Stadler76