From 893e4aae9fd6bb1be29b7ee036d608102cef4cb6 Mon Sep 17 00:00:00 2001 From: Denis Ledoux Date: Thu, 19 Jun 2014 14:58:44 +0200 Subject: [PATCH] [FIX] web: focused field in view editable list When adding an item to the editable list, the focused field was no longer the first visible field This is related to revision 4a508885ac043b25465aa5741955082aa66ce949 visible_columns list is not ordered --- addons/web/static/src/js/view_list_editable.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/addons/web/static/src/js/view_list_editable.js b/addons/web/static/src/js/view_list_editable.js index b97aef19815..7d2e44263ce 100644 --- a/addons/web/static/src/js/view_list_editable.js +++ b/addons/web/static/src/js/view_list_editable.js @@ -251,7 +251,10 @@ openerp.web.list_editable = function (instance) { }, options).then(function () { $recordRow.addClass('oe_edition'); self.resize_fields(); - var focus_field = options && options.focus_field ? options.focus_field : (self.visible_columns.length ? self.visible_columns[0].name : undefined); + var focus_field = options && options.focus_field ? options.focus_field : undefined; + if (!focus_field){ + focus_field = _.find(self.editor.form.fields_order, function(field){ return fields[field] && fields[field].$el.is(':visible:has(input)'); }); + } if (focus_field) fields[focus_field].$el.find('input').select(); return record.attributes; });