Launchpad itself

Merge lp:~wallyworld/launchpad/link-bugs-in-merge-proposal into lp:launchpad

  1. link-bugs-in-merge-proposal
  2. Merge into devel
Proposed by Ian Booth
Status: Merged
Approved by: Ian Booth
Approved revision: no longer in the source branch.
Merged at revision: 11563
Proposed branch: lp:~wallyworld/launchpad/link-bugs-in-merge-proposal
Merge into: lp:launchpad
Diff against target: 94 lines (+29/-31)
2 files modified
lib/lp/code/stories/branches/xx-branchmergeproposals.txt (+0/-1)
lib/lp/code/templates/branchmergeproposal-index.pt (+29/-30)
To merge this branch: bzr merge lp:~wallyworld/launchpad/link-bugs-in-merge-proposal
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Ian Booth (community) ui Approve
Paul Hummer (community) Approve
Review via email: mp+34826@code.launchpad.net

Commit message

Move the linked bugs section on the branch merge proposal page to be just below the branch summary section, above the review comments.

Description of the change

Move the linked bugs section on the branch merge proposal page to be just below the branch summary section, above the review comments. I looked at putting the linked bugs in the unused space to the right of the branch summary details but this caused the branch and bug links in both sections to wrap. I added some horizontal white space just above the newly placed linked bugs section to make the page look better. The white space was added by wrapping the linked bugs div inside a <div class="first"> but I'm expecting there may be a better way - if so, I'll change it.

Tests:
    - no changes or new tests required.
    - ran the doc and windmill and unit tests for "branchmergeproposal" and
      "branchmergeproposals" (bin/test -t ...)

To post a comment you must log in.
Revision history for this message
Ian Booth (wallyworld) wrote :

Screenshot of new merge proposal screen:

https://devpad.canonical.com/~ianb/linkedbug.png

Revision history for this message
Ian Booth (wallyworld) wrote :

Oops wrong server - use this one:

https://people.canonical.com/~ianb/linkedbug.png

Revision history for this message
Ian Booth (wallyworld) wrote :

UI review suggested moving Linked Bugs section to below the Description, above the Comments

This has been done. I think it needs a horizontal separator line though, between the Linked Bugs and the "Add a Comment" button.

See https://devpad.canonical.com/~ianb/linkedbug2.png

Revision history for this message
Paul Hummer (rockstar) wrote :

Much better! Thanks for the changes!

review: Approve (ui)
Revision history for this message
Ian Booth (wallyworld) wrote :

This was a duplicate review request. Paul Hummer has approved the changes.

review: Approve (ui)
Revision history for this message
Tim Penhey (thumper) :
review: Approve
Revision history for this message
Tim Penhey (thumper) wrote :

The tal:condition="view/has_bug_or_spec" expression on the div before the title should be moved to the out-most div that has the class yui-g. That way we don't get extra whitespace if there are no bugs or blueprints.

Revision history for this message
Ian Booth (wallyworld) wrote :

As per discussion with Tim, the related bugs section has been made to line up with the description text and the section header has been removed.

See screenshot: http://people.canonical.com/~ianb/linkedbug3.png

Revision history for this message
Tim Penhey (thumper) wrote :

Picture?

Revision history for this message
Tim Penhey (thumper) wrote :

Sorry, didn't see the picture as the mail didn't thread properly. This looks good to me.

review: Approve
Revision history for this message
Tim Penhey (thumper) wrote :

Ian, did Paul get back to you about this?

Revision history for this message
Ian Booth (wallyworld) wrote :

Yes. I pinged him this morning and he said it was all ok. I've already
submitted the job to ec2

On 15/09/10 14:31, Tim Penhey wrote:
> Ian, did Paul get back to you about this?

Revision history for this message
Ian Booth (wallyworld) wrote :

Running through ec2 and a doc test failed (xx-branchmergeproposals.txt) due to the removal of the "Linked Bugs" header text from the page. Doc test now fixed (one line deletion).

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/code/stories/branches/xx-branchmergeproposals.txt'
--- lib/lp/code/stories/branches/xx-branchmergeproposals.txt 2010-05-27 02:19:27 +0000
+++ lib/lp/code/stories/branches/xx-branchmergeproposals.txt 2010-09-15 06:04:51 +0000
@@ -543,7 +543,6 @@
543543
544 >>> nopriv_browser.open(bmp_url)544 >>> nopriv_browser.open(bmp_url)
545 >>> print_bugs_and_specs(nopriv_browser)545 >>> print_bugs_and_specs(nopriv_browser)
546 Linked bug reports and blueprints
547 Bug #...: Bug for linking Undecided New546 Bug #...: Bug for linking Undecided New
548547
549548
550549
=== modified file 'lib/lp/code/templates/branchmergeproposal-index.pt'
--- lib/lp/code/templates/branchmergeproposal-index.pt 2010-07-15 13:57:39 +0000
+++ lib/lp/code/templates/branchmergeproposal-index.pt 2010-09-15 06:04:51 +0000
@@ -46,6 +46,10 @@
46 #proposal-summary td {46 #proposal-summary td {
47 padding-left: 0.5em;47 padding-left: 0.5em;
48 }48 }
49 .related-bugs-list {
50 padding-left: 20px;
51 padding-bottom: 10px;
52 }
49 /* A page-specific fix for inline text are editing to line up box. */53 /* A page-specific fix for inline text are editing to line up box. */
50 #edit-description .yui-ieditor-input { top: 0; }54 #edit-description .yui-ieditor-input { top: 0; }
51 #edit-commit_message .yui-ieditor-input { top: 0; }55 #edit-commit_message .yui-ieditor-input { top: 0; }
@@ -138,6 +142,18 @@
138 </tal:has-description>142 </tal:has-description>
139 </div>143 </div>
140144
145 <div class="yui-g" tal:condition="view/has_bug_or_spec">
146 <div id="related-bugs-and-blueprints" class="related-bugs-list"
147 tal:define="branch context/source_branch">
148 <div class="actions">
149 <metal:bug-branch-links use-macro="branch/@@+macros/bug-branch-links"/>
150 </div>
151 <div class="actions">
152 <metal:spec-branch-links use-macro="branch/@@+macros/spec-branch-links"/>
153 </div>
154 </div>
155 </div>
156
141 <div class="yui-g">157 <div class="yui-g">
142 <tal:not-logged-in condition="not: view/user">158 <tal:not-logged-in condition="not: view/user">
143 <div align="center" id="add-comment-login-first">159 <div align="center" id="add-comment-login-first">
@@ -176,36 +192,19 @@
176 </tal:logged-in>192 </tal:logged-in>
177193
178 <div class="yui-g">194 <div class="yui-g">
179 <div class="yui-u first">195 <div id="source-revisions"
180 <div id="source-revisions"196 tal:condition="not: context/queue_status/enumvalue:MERGED">
181 tal:condition="not: context/queue_status/enumvalue:MERGED">197
182198 <tal:history-available condition="context/source_branch/revision_count"
183 <tal:history-available condition="context/source_branch/revision_count"199 define="branch context/source_branch;
184 define="branch context/source_branch;200 revisions view/unlanded_revisions">
185 revisions view/unlanded_revisions">201 <h2>Unmerged revisions</h2>
186 <h2>Unmerged revisions</h2>202 <metal:landing-target use-macro="branch/@@+macros/branch-revisions"/>
187 <metal:landing-target use-macro="branch/@@+macros/branch-revisions"/>203 </tal:history-available>
188 </tal:history-available>204 <tal:remote-branch condition="context/source_branch/branch_type/enumvalue:REMOTE">
189205 <h2>Unmerged revisions</h2>
190 <tal:remote-branch condition="context/source_branch/branch_type/enumvalue:REMOTE">206 <p>Recent revisions are not available due to the source branch being remote.</p>
191 <h2>Unmerged revisions</h2>207 </tal:remote-branch>
192 <p>Recent revisions are not available due to the source branch being remote.</p>
193 </tal:remote-branch>
194 </div>
195 </div>
196
197 <div class="yui-u" tal:condition="view/has_bug_or_spec">
198 <div id="related-bugs-and-blueprints"
199 tal:define="branch context/source_branch">
200 <h2>Linked bug reports and blueprints</h2>
201 <div class="actions">
202
203 <metal:bug-branch-links use-macro="branch/@@+macros/bug-branch-links"/>
204 </div>
205 <div class="actions">
206 <metal:spec-branch-links use-macro="branch/@@+macros/spec-branch-links"/>
207 </div>
208 </div>
209 </div>208 </div>
210 </div>209 </div>
211210