| Issue 348: | Comment on a patchset should be linkable/bookmarkable | |
| 36 people starred this issue and may be notified of changes. | Back to list |
There is no way to link to a specific comment on a patchset outside of Gerrit and discuss it in a mailing list, for instance, by providing a direct URL to the comment in question.
Dec 1, 2009
#1
sop@google.com
Status:
Accepted
Aug 12, 2010
Or perhaps you could make a link to a comment simply display that comment with some context around it, but not the rest of the file. In other words, do not scroll, simply display a reduced diff.
May 19, 2011
(No comment was entered for this change.)
Blocking:
93
May 19, 2011
Issue 740 has been merged into this issue.
Oct 7, 2011
Seconded, espcecially the laconic: > Patch Set 2: (3 inline comments) In the main view is annoying, it'd be really nice to have: file foo: comment on lines <a href=...>nnn</a>, <a href=...>mmm</a> file bar: comment on line <a href=...>ttt</a> Right now having to guess which comments are "new" is mildly annoying, especially for really large patches (especially since "marking" a comment as done is done through ... generating a new one…).
Sep 20, 2013
See comment 1 above, the diff view code at [1] (diffChunkNav() is an example of programatically scrolling to a line), and the code for dispatching based on anchor on line 619 of [2] if you'd like to work on this. [1] https://gerrit.googlesource.com/gerrit/+/8e0f6d06/gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/SideBySide2.java [2] https://gerrit.googlesource.com/gerrit/+/master/gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java
Dec 13, 2013
(No comment was entered for this change.)
Labels:
Blocking-2.9
Dec 18, 2013
https://gerrit-review.googlesource.com/53311
Status:
ChangeUnderReview
Owner: sop@google.com
Dec 20, 2013
(No comment was entered for this change.)
Status:
Submitted
Labels: -Blocking-2.9 FixedIn-2.9
Jul 18, 2014
(No comment was entered for this change.)
Status:
Released
|
|
| ► Sign in to add a comment |