From 76ba56e60db87f2fa574b37974ee4f48b3fd94b6 Mon Sep 17 00:00:00 2001
From: Martin Geubelle <mge@odoo.com>
Date: Thu, 23 Nov 2017 18:15:15 +0100
Subject: [PATCH] [FIX] web: List: handle custom class on button

It should be possible to set a custom class on a button in the list view.

Closes #20620
---
 addons/web/static/src/js/views/list/list_renderer.js | 2 +-
 addons/web/static/tests/views/list_tests.js          | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/addons/web/static/src/js/views/list/list_renderer.js b/addons/web/static/src/js/views/list/list_renderer.js
index 33c075fc77f..b7ca5cc118b 100644
--- a/addons/web/static/src/js/views/list/list_renderer.js
+++ b/addons/web/static/src/js/views/list/list_renderer.js
@@ -307,7 +307,7 @@ var ListRenderer = BasicRenderer.extend({
         } else {
             $button.text(node.attrs.string);
         }
-
+        this._handleAttributes($button, node);
         this._registerModifiers(node, record, $button);
 
         if (record.res_id) {
diff --git a/addons/web/static/tests/views/list_tests.js b/addons/web/static/tests/views/list_tests.js
index 22399961285..5f003dfac61 100644
--- a/addons/web/static/tests/views/list_tests.js
+++ b/addons/web/static/tests/views/list_tests.js
@@ -1088,7 +1088,7 @@ QUnit.module('Views', {
     });
 
     QUnit.test('can display button in edit mode', function (assert) {
-        assert.expect(1);
+        assert.expect(2);
 
         var list = createView({
             View: ListView,
@@ -1096,10 +1096,11 @@ QUnit.module('Views', {
             data: this.data,
             arch: '<tree editable="bottom">' +
                     '<field name="foo"/>' +
-                    '<button name="notafield" type="object" icon="fa-asterisk"/>' +
+                    '<button name="notafield" type="object" icon="fa-asterisk" class="o_yeah"/>' +
                 '</tree>',
         });
         assert.ok(list.$('tbody button').length, "should have a button");
+        assert.ok(list.$('tbody button').hasClass('o_yeah'), "class should be set on the button");
         list.destroy();
     });