Merge lp:~wallyworld/launchpad/fix-enable-review-type into lp:launchpad/db-devel
Status: | Superseded |
---|---|
Proposed branch: | lp:~wallyworld/launchpad/fix-enable-review-type |
Merge into: | lp:launchpad/db-devel |
Diff against target: |
43 lines (+13/-2) 2 files modified
lib/lp/code/templates/branch-register-merge.pt (+1/-1) lib/lp/code/windmill/tests/test_branchmergeproposal_review.py (+12/-1) |
To merge this branch: | bzr merge lp:~wallyworld/launchpad/fix-enable-review-type |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Māris Fogels (community) | Approve | ||
Review via email: mp+40302@code.launchpad.net |
This proposal has been superseded by a proposal from 2010-11-18.
Commit message
Enable review type field using onblur instead of onchange - works better with popup selection wisget to choose reviewer.
Description of the change
= Summary =
As per bug 671665, when the popup selection widget is used to choose the reviewer for a mp, the review type field does not become enabled.
= Implementation =
There are 2 parts to the fix.
1. Enhance lazr-js so that focus is returned to the input field when the user finishes selecting the value from the popup
2. Change the mechanism to trigger the review_type field enable update from onchange to onblur
The enabled processing will occur when the user tabs out of the reviewer field, and will work in the same way whether the reviewer value is typed or entered using the popup.
This branch should be merged only once lazr-js is upgraded. The relevant branch there is lp:~wallyworld/lazr-js/popup-selection-focus-fix
= Tests =
The code.windmill.
bin/test -vvt test_branchmerg
= Lint =
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
The merge proposal for lazr-js branch lp:~wallyworld/lazr-js/popup-selection-focus-fix has now been approved