Merge lp:~openerp-dev/openerp-web/7.0-opw-602138-acl into lp:openerp-web/7.0

Proposed by Anaël Closson (openerp)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-602138-acl
Merge into: lp:openerp-web/7.0
Diff against target: 13 lines (+2/-2)
1 file modified
addons/web/static/src/js/view_list_editable.js (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-602138-acl
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+200527@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

4105. By Anaël Closson (openerp)

[FIX] OPW 602138 : clicking on certain row of an editable list sometime focus the field before being positioned.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_list_editable.js'
2--- addons/web/static/src/js/view_list_editable.js 2013-12-18 15:26:26 +0000
3+++ addons/web/static/src/js/view_list_editable.js 2014-01-06 13:03:07 +0000
4@@ -751,8 +751,8 @@
5 && (field = form.fields[focus_field])
6 // And is visible
7 && field.$el.is(':visible')) {
8- // focus it
9- field.focus();
10+ // focus it, w/ timeout to avoid triggering focus before field placement
11+ setTimeout(function () { field.focus() }, 1);
12 return;
13 }
14