loggerhead

Merge lp:~mwhudson/loggerhead/line-numbers-back-in-annotate into lp:loggerhead

  1. line-numbers-back-in-annotate
  2. Merge into trunk-rich
Proposed by Michael Hudson-Doyle
Status: Merged
Approved by: Paul Hummer
Approved revision: 262
Merged at revision: not available
Proposed branch: lp:~mwhudson/loggerhead/line-numbers-back-in-annotate
Merge into: lp:loggerhead
Diff against target: None lines
To merge this branch: bzr merge lp:~mwhudson/loggerhead/line-numbers-back-in-annotate
Reviewer Review Type Date Requested Status
Paul Hummer (community) Approve
Review via email: mp+3057@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

At some point we had
https://bugs.edge.launchpad.net/launchpad-bazaar/+bug/98828 ("codebrowse
source listings don't contain line number anchors") fixed on Launchpad,
and then it was broken again. This branch puts the line number anchors
back.

Cheers,
mwh

Revision history for this message
Paul Hummer (rockstar) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'loggerhead/templates/annotate.pt'
2--- loggerhead/templates/annotate.pt 2008-11-17 18:08:38 +0000
3+++ loggerhead/templates/annotate.pt 2009-01-22 22:14:47 +0000
4@@ -57,7 +57,7 @@
5
6 <tr tal:repeat="line contents"
7 tal:attributes="class string:blueRow${line/parity}">
8- <td class="annoLine" tal:content="line/lineno">1</td>
9+ <td class="annoLine"><a tal:attributes="id string:L${line/lineno}" tal:content="line/lineno">1</a></td>
10 <td class="annoRev">
11 <a tal:condition="python:line.status=='changed'"
12 tal:content="python:util.trunc(line.change.revno)"

Subscribers

People subscribed via source and target branches