merge proposal diff only shows once per page load
Bug #741440 reported by
Robert Collins
This bug affects 4 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Henning Eggers | ||
Bug Description
Visit a bug with an attached branch that has been proposed for merging (or merged).
Click on the link to bring up the merge proposal.
Close it.
Click on the link again... fooled you, you can't click on it.
(Chromium if that matters)
Related branches
lp:~deryck/launchpad/mp-diff-reload-741440
- j.c.sackett (community): Approve
-
Diff: 210 lines (+123/-17)3 files modifiedlib/lp/code/javascript/branchmergeproposal.diff.js (+22/-17)
lib/lp/code/javascript/tests/test_branchdiff.html (+31/-0)
lib/lp/code/javascript/tests/test_branchdiff.js (+70/-0)
| Changed in launchpad: | |
| assignee: | nobody → Deryck Hodge (deryck) |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.05 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Martin Pool (mbp) wrote | #2 |
| Changed in launchpad: | |
| milestone: | 11.05 → none |
| status: | Fix Released → Triaged |
| Changed in launchpad: | |
| status: | Triaged → Fix Committed |
| assignee: | Deryck Hodge (deryck) → Henning Eggers (henninge) |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r12994 <http://