Merge lp://staging/~dtwood/launchpad/fix_508722 into lp://staging/launchpad

Proposed by David Wood
Status: Merged
Approved by: Deryck Hodge
Approved revision: no longer in the source branch.
Merged at revision: 11010
Proposed branch: lp://staging/~dtwood/launchpad/fix_508722
Merge into: lp://staging/launchpad
Diff against target: 27 lines (+5/-5)
1 file modified
lib/lp/bugs/templates/bugtarget-macros-filebug.pt (+5/-5)
To merge this branch: bzr merge lp://staging/~dtwood/launchpad/fix_508722
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Deryck Hodge Pending
Review via email: mp+20236@code.staging.launchpad.net

Commit message

Better formatting in tal to fix spacing issues in a title attribute on filebug template.

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

Looks fine to me.

review: Approve
Revision history for this message
Deryck Hodge (deryck) wrote :

I can get this landed for David once PQM reopens, which will, I assume be Thursday or Friday.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/bugs/templates/bugtarget-macros-filebug.pt'
--- lib/lp/bugs/templates/bugtarget-macros-filebug.pt 2009-12-04 12:12:35 +0000
+++ lib/lp/bugs/templates/bugtarget-macros-filebug.pt 2010-02-26 17:19:21 +0000
@@ -156,9 +156,9 @@
156 <tal:defines define="bugtracker product_or_distro/getExternalBugTracker">156 <tal:defines define="bugtracker product_or_distro/getExternalBugTracker">
157 <h3>157 <h3>
158 Are you sure this bug is in <acronym class="explain"158 Are you sure this bug is in <acronym class="explain"
159 style="color: inherit" title="the original developer of159 style="color: inherit"
160 the software">upstream</acronym> <span160 title="the original developer of the software">upstream</acronym>
161 tal:replace="product_or_distro/displayname" />?161 <span tal:replace="product_or_distro/displayname" />?
162 </h3>162 </h3>
163 <tal:distro-source-pacakges163 <tal:distro-source-pacakges
164 condition="product_or_distro/distrosourcepackages">164 condition="product_or_distro/distrosourcepackages">
@@ -198,8 +198,8 @@
198 </tal:not-distro-source-packages>198 </tal:not-distro-source-packages>
199 <tal:has-bugtracker condition="bugtracker">199 <tal:has-bugtracker condition="bugtracker">
200 <p id="bugtarget-upstream-bugtracker-info">200 <p id="bugtarget-upstream-bugtracker-info">
201 Bugs in <abbr title="the original developer of the201 Bugs in <abbr title="the original developer of the software"
202 software" class="explain">upstream</abbr>202 class="explain">upstream</abbr>
203 <span tal:replace="product_or_distro/displayname">203 <span tal:replace="product_or_distro/displayname">
204 Alsa Utils204 Alsa Utils
205 </span>205 </span>