remove milestone from bug has too many icons
Bug #514190 reported by
John A Meinel
This bug affects 3 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| LAZR Javascript Library |
Fix Released
|
Low
|
Deryck Hodge | ||
| Launchpad itself |
Fix Released
|
Low
|
Deryck Hodge | ||
Bug Description
See attached.
When selecting the ajax "target to milestone", I see the "Remove milestone" link. However, it has 5 red remove buttons next to it. I assume that was meant to have just 1.
Related branches
lp:~deryck/lazr-js/stop-all-those-null-choice-edit-icons
- Paul Hummer (community): Approve
-
Diff: 15 lines (+4/-1)1 file modifiedsrc-js/lazrjs/choiceedit/choiceedit.js (+4/-1)
lp:~deryck/launchpad/rockstar-js-refresh
- Deryck Hodge (community): Approve
-
Diff: 1680 lines (+555/-140)65 files modifiedbuildout-templates/bin/test.in (+1/-1)
lib/canonical/launchpad/doc/lazr-js-widgets.txt (+6/-6)
lib/canonical/launchpad/doc/location-widget.txt (+1/-1)
lib/canonical/launchpad/javascript/client/client.js (+7/-2)
lib/canonical/launchpad/windmill/jstests/initialize.js (+6/-1)
lib/canonical/launchpad/windmill/testing/widgets.py (+13/-13)
lib/canonical/widgets/lazrjs.py (+3/-3)
lib/canonical/widgets/location.py (+1/-1)
lib/canonical/widgets/templates/inline-picker.pt (+4/-4)
lib/canonical/widgets/templates/license.pt (+1/-1)
lib/canonical/widgets/templates/vocabulary-picker.js.template (+1/-1)
lib/lp/app/browser/tests/base-layout.txt (+2/-2)
lib/lp/app/browser/tests/test_base_layout.py (+4/-4)
lib/lp/app/configure.zcml (+14/-0)
lib/lp/app/javascript/lp-mochi.js (+1/-1)
lib/lp/app/javascript/lp.js (+2/-2)
lib/lp/app/javascript/picker.js (+3/-3)
lib/lp/app/javascript/tests/test_lp_collapsibles.html (+1/-1)
lib/lp/app/javascript/tests/test_lp_collapsibles.js (+2/-1)
lib/lp/app/templates/base-layout-macros.pt (+10/-1)
lib/lp/app/templates/base-layout.pt (+1/-1)
lib/lp/app/templates/launchpad-widget-macros.pt (+1/-1)
lib/lp/bugs/browser/bugalsoaffects.py (+1/-1)
lib/lp/bugs/javascript/bugtask_index.js (+2/-2)
lib/lp/bugs/javascript/official_bug_tags.js (+1/-1)
lib/lp/bugs/javascript/tests/test_bug_subscription_widget.html (+1/-1)
lib/lp/bugs/javascript/tests/test_bug_subscription_widget.js (+2/-1)
lib/lp/bugs/javascript/tests/test_me_too.html (+1/-1)
lib/lp/bugs/javascript/tests/test_me_too.js (+2/-17)
lib/lp/bugs/javascript/tests/test_subscriber.html (+1/-1)
lib/lp/bugs/javascript/tests/test_subscriber.js (+1/-1)
lib/lp/bugs/templates/bugtarget-patches.pt (+2/-2)
lib/lp/bugs/templates/bugtask-tasks-and-nominations-table-row.pt (+6/-6)
lib/lp/bugs/windmill/tests/test_bug_inline_subscriber.py (+2/-1)
lib/lp/bugs/windmill/tests/test_mark_duplicate.py (+19/-7)
lib/lp/bugs/windmill/tests/test_official_bug_tags_management.py (+12/-8)
lib/lp/code/interfaces/branchmergequeue.py (+129/-0)
lib/lp/code/javascript/branchmergeproposal.diff.js (+2/-2)
lib/lp/code/javascript/tests/test_productseries-setbranch.html (+2/-2)
lib/lp/code/javascript/tests/test_productseries_setbranch.js (+1/-1)
lib/lp/code/model/branchmergequeue.py (+88/-0)
lib/lp/code/model/tests/test_branchmergequeue.py (+155/-0)
lib/lp/code/windmill/tests/test_branch_popupdiff.py (+1/-0)
lib/lp/code/windmill/tests/test_branchmergeproposal_review.py (+2/-0)
lib/lp/registry/browser/__init__.py (+1/-1)
lib/lp/registry/browser/person.py (+1/-1)
lib/lp/registry/browser/team.py (+2/-2)
lib/lp/registry/browser/tests/productseries-views.txt (+1/-1)
lib/lp/registry/javascript/tests/test_milestone_table.html (+1/-1)
lib/lp/registry/javascript/tests/test_milestone_table.js (+1/-1)
lib/lp/registry/javascript/tests/timeline-iframe.html (+1/-1)
lib/lp/registry/javascript/tests/timeline.js (+6/-5)
lib/lp/registry/templates/distributionsourcepackage-index.pt (+1/-4)
lib/lp/registry/templates/product-index.pt (+1/-1)
lib/lp/registry/templates/productseries-setbranch.pt (+1/-1)
lib/lp/registry/templates/team-portlet-membership.pt (+1/-1)
lib/lp/soyuz/javascript/tests/archivesubscribers_index.js (+2/-1)
lib/lp/soyuz/javascript/tests/lp_dynamic_dom_updater.js (+2/-1)
lib/lp/soyuz/javascript/tests/test_archivesubscribers_index.html (+1/-1)
lib/lp/soyuz/javascript/tests/test_lp_dynamic_dom_updater.html (+1/-1)
lib/lp/soyuz/templates/archive-index.pt (+1/-4)
lib/lp/testing/__init__.py (+4/-3)
lib/lp/translations/templates/languageset-index.pt (+1/-1)
utilities/yui-deps.py (+6/-1)
versions.cfg (+1/-1)
Revision history for this message
| John A Meinel (jameinel) wrote | #1 |
| Changed in malone: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| tags: | added: bug-page javascript sprite ui |
| summary: |
- remove milestone from bug has incorrect icons + remove milestone from bug has too many icons |
Revision history for this message
| Deryck Hodge (deryck) wrote | #2 |
| Changed in lazr-js: | |
| status: | New → In Progress |
| importance: | Undecided → Low |
| Changed in malone: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Deryck Hodge (deryck) |
| Changed in lazr-js: | |
| assignee: | nobody → Deryck Hodge (deryck) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #3 |
| tags: | added: qa-needstesting |
| Changed in lazr-js: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| Changed in malone: | |
| milestone: | none → 10.12 |
| tags: | added: qa-needstesting |
| Changed in malone: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in malone: | |
| status: | Fix Committed → Fix Released |
| Changed in lazr-js: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Adding a lazr-js task because this has to be fixed there.