Launchpad itself

Merge lp:~thumper/launchpad/code-review-comment-tweak into lp:launchpad

  1. code-review-comment-tweak
  2. Merge into devel
Proposed by Tim Penhey on 2010-04-16
Status: Merged
Approved by: Tim Penhey on 2010-04-16
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~thumper/launchpad/code-review-comment-tweak
Merge into: lp:launchpad
Diff against target: 13 lines (+0/-3)
1 file modified
lib/lp/code/templates/codereviewcomment-body.pt (+0/-3)
To merge this branch: bzr merge lp:~thumper/launchpad/code-review-comment-tweak
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle 2010-04-16 Approve on 2010-04-16
Review via email: mp+23522@code.launchpad.net

Commit Message

Remove comment text from the code review comment rendering.

Description of the Change

Remove template text that allows the fixing of bug 562943

To post a comment you must log in.
Michael Hudson-Doyle (mwhudson) wrote :

That was a slightly strange feature I guess!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/code/templates/codereviewcomment-body.pt'
2--- lib/lp/code/templates/codereviewcomment-body.pt 2010-02-18 16:34:09 +0000
3+++ lib/lp/code/templates/codereviewcomment-body.pt 2010-04-16 04:23:49 +0000
4@@ -11,9 +11,6 @@
5 <div class="boardCommentBody attachmentBody" tal:content="structure attachment/diff_text/fmt:diff"/>
6 </div>
7 </tal:good-attachments>
8- <tal:other-attachments repeat="attachment view/other_attachments">
9- <tal:comment replace="structure string:&lt;!-- '${attachment/filename}' '${attachment/mimetype}' --&gt;"/>
10- </tal:other-attachments>
11
12 <div class="replyLink">
13 <tal:reply tal:define="link context/menu:context/reply"