Merge lp:~openerp-dev/openerp-web/7.0-opw-603903-msh into lp:openerp-web/7.0

Proposed by Mohammed Shekha(Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-603903-msh
Merge into: lp:openerp-web/7.0
Diff against target: 14 lines (+4/-0)
1 file modified
addons/web/static/src/js/view_list_editable.js (+4/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-603903-msh
Reviewer Review Type Date Requested Status
Xavier (Open ERP) Pending
Martin Trigaux (OpenERP) Pending
Naresh(OpenERP) Pending
Review via email: mp+206491@code.launchpad.net

Description of the change

Hello,

Fixed the issue of default_focus not supported in editable listview.

demo: Go to any view and which having Editable listview -> set default_focus in some field and then go to that view and see default focus is still on first field in editable listview while in one2many popup or form view it is working as expected.

Thanks.

To post a comment you must log in.
4143. By Mohammed Shekha(OpenERP)<email address hidden>

[FIX]Fixed the issue of default_focus not supported in editable listview.

Unmerged revisions

4143. By Mohammed Shekha(OpenERP)<email address hidden>

[FIX]Fixed the issue of default_focus not supported in editable listview.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/static/src/js/view_list_editable.js'
--- addons/web/static/src/js/view_list_editable.js 2013-12-18 15:26:26 +0000
+++ addons/web/static/src/js/view_list_editable.js 2014-02-17 05:19:53 +0000
@@ -756,6 +756,10 @@
756 return;756 return;
757 }757 }
758758
759 if (form.default_focus_field) {
760 form.fields_order.unshift(form.default_focus_field.name);
761 }
762
759 _(form.fields_order).detect(function (name) {763 _(form.fields_order).detect(function (name) {
760 // look for first visible field in fields_order, focus it764 // look for first visible field in fields_order, focus it
761 var field = form.fields[name];765 var field = form.fields[name];