launchpad-reviewers team mailing list archive

[Guruprasad <mp+490693@xxxxxxxxxxxxxxxxxx>]

Guruprasad has proposed merging ~lgp171188/launchpad:remove-bzr-codebrowse-link-branch-listing-page into launchpad:master.

Commit message:
Remove the codebrowse links from the branch listing page

This was missed when we removed the other codebrowse links for bzr shutdown

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/490693
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:remove-bzr-codebrowse-link-branch-listing-page into launchpad:master.

diff --git a/lib/lp/code/templates/branch-listing.pt b/lib/lp/code/templates/branch-listing.pt
index ed8a65c..f90cd2d 100644
--- a/lib/lp/code/templates/branch-listing.pt
+++ b/lib/lp/code/templates/branch-listing.pt
@@ -154,7 +154,10 @@ replace='structure string:<script type="text/javascript">
                               onmouseout string:hide_commit(${branch/id});">
             <div class="lastCommit">
               <a tal:attributes="href branch/revision_codebrowse_link"
-                 tal:content="branch/revision_count">1234</a>.
+                 tal:content="branch/revision_count"
+                 tal:condition="branch/code_is_browsable">1234</a>
+              <tal:revno condition="not: branch/code_is_browsable"
+                         replace="branch/revision_count">1234</tal:revno>.
               <tal:revision-log replace="branch/revision_log/fmt:shorten/40"/>
             </div>
             <div class="popupTitle"