Move bug link on merge proposal page above the reviewer block
Bug #436663 reported by
Barry Warsaw
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Ivo Kracht | ||
Bug Description
Merge proposals include a link to any bug their branch is linked to, but the information is somewhat hidden. This information should be shown above the Reviewer block, where a long merge proposal cover letter won't hide it.
Related branches
lp:~ivo-kracht/launchpad/bug-436663
- Curtis Hovey (community): Approve (ui)
- Jelmer Vernooij (community): Approve (code)
-
Diff: 82 lines (+14/-16)3 files modifiedlib/lp/code/stories/branches/xx-branchmergeproposals.txt (+7/-4)
lib/lp/code/templates/branchmergeproposal-index.pt (+0/-12)
lib/lp/code/templates/branchmergeproposal-pagelet-summary.pt (+7/-0)
| summary: |
- Show bug link on merge proposal page + Move bug link on merge proposal page above the reviewer block |
| description: | updated |
| affects: | launchpad → launchpad-code |
| tags: | added: code-review |
| Changed in launchpad-code: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| tags: | added: ui-easy |
| tags: |
added: easy removed: ui-easy |
| Changed in launchpad: | |
| importance: | Medium → Low |
| Changed in launchpad: | |
| assignee: | nobody → Ivo Kracht (ivo-kracht) |
| status: | Triaged → In Progress |
Revision history for this message
| Ivo Kracht (ivo-kracht) wrote | #1 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
here is a screenshot of how it will look after the change