Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-574684-cpa into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-574684-cpa/+merge/105471

Hello,

Editable list view wrong class assignment.

Demo: Warehouse > Traceability > Stoke Move
1) Edit record.

Observed: 'oe_forms' class not add in <tr>(editable row not show field in 
proper manners).
Expected: 'oe_forms' class should be add in <tr>.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-574684-cpa/+merge/105471
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-574684-cpa.
=== modified file 'addons/web/static/src/js/view_list_editable.js'
--- addons/web/static/src/js/view_list_editable.js	2012-02-09 15:48:44 +0000
+++ addons/web/static/src/js/view_list_editable.js	2012-05-11 12:14:28 +0000
@@ -202,7 +202,7 @@
                 var $new_row = $('<tr>', {
                         id: _.uniqueId('oe-editable-row-'),
                         'data-id': record_id,
-                        'class': row ? $(row).attr('class') : '' + ' oe_forms',
+                        'class': (row ? $(row).attr('class') : '') + ' oe_forms',
                         click: function (e) {e.stopPropagation();}
                     })
                     .delegate('button.oe-edit-row-save', 'click', function () {

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to