[IMP] use semantic css classes instead of element names

[RFR] and being on it, make reacting to changes in overrides simple

web_widget_x2many_2d_matrix/static/src/js/web_widget_x2many_2d_matrix.js

@@ -308,47 +308,48 @@ openerp.web_widget_x2many_2d_matrix = function(instance)
         start: function()
         {
             var self = this;
-            this.$el.find('input').on(
-                'change',
-                function()
+            this.$el.find('.edit').on(
+                'change', self.proxy(this.xy_value_change));
+            this.compute_totals();
+            this.setup_many2one_axes();
+            this.on("change:effective_readonly",
+                    this, this.proxy(this.effective_readonly_change));
+            this.effective_readonly_change();
+            return this._super.apply(this, arguments);
+        },
+
+        xy_value_change: function(e)
         {
-                    var $this = jQuery(this),
-                        val = $this.val()
-                    if(self.validate_xy_value(val))
+            var $this = jQuery(e.currentTarget),
+                val = $this.val();
+            if(this.validate_xy_value(val))
             {
-                        var data = {}, value = self.parse_xy_value(val);
-                        data[self.field_value] = value;
+                var data = {}, value = this.parse_xy_value(val);
+                data[this.field_value] = value;
 
-                        $this.siblings('span').text(self.format_xy_value(value));
-                        $this.val(self.format_xy_value(value));
+                $this.siblings('.read').text(this.format_xy_value(value));
+                $this.val(this.format_xy_value(value));
 
-                        self.dataset.write($this.data('id'), data);
+                this.dataset.write($this.data('id'), data);
                 $this.parent().removeClass('oe_form_invalid');
-                        self.compute_totals();
+                this.compute_totals();
             }
             else
             {
                 $this.parent().addClass('oe_form_invalid');
             }
 
-                });
-            this.compute_totals();
-            this.setup_many2one_axes();
-            this.on("change:effective_readonly",
-                    this, this.proxy(this.effective_readonly_change));
-            this.effective_readonly_change();
-            return this._super.apply(this, arguments);
         },
 
         effective_readonly_change: function()
         {
             this.$el
-            .find('tbody td.oe_list_field_cell span.oe_form_field>input')
+            .find('tbody td.oe_list_field_cell span.oe_form_field .edit')
             .toggle(!this.get('effective_readonly'));
             this.$el
-            .find('tbody td.oe_list_field_cell span.oe_form_field>span')
+            .find('tbody td.oe_list_field_cell span.oe_form_field .read')
             .toggle(this.get('effective_readonly'));
-            this.$el.find('input').first().focus();
+            this.$el.find('.edit').first().focus();
         },
 
         is_syntax_valid: function()

web_widget_x2many_2d_matrix/static/src/xml/web_widget_x2many_2d_matrix.xml

@@ -16,8 +16,8 @@
                         <th><t t-esc="widget.get_y_axis_label(y)" /></th>
                         <td t-foreach="widget.get_x_axis_values()" t-as="x" t-att-class="'oe_list_field_cell' + (widget.is_numeric ? ' oe_number' : '')" t-att-data-x="x">
                             <span t-att-class="widget.get_xy_value_class()">
-                                <input style="display: none" t-att-data-id="widget.get_xy_id(x, y)" t-att-value="widget.format_xy_value(widget.get_xy_value(x, y))" />
-                                <span style="display: none"><t t-esc="widget.format_xy_value(widget.get_xy_value(x, y))" /></span>
+                                <input class="edit" t-att-data-id="widget.get_xy_id(x, y)" t-att-value="widget.format_xy_value(widget.get_xy_value(x, y))" />
+                                <span class="read"><t t-esc="widget.format_xy_value(widget.get_xy_value(x, y))" /></span>
                             </span>
                         </td>
                         <td t-if="widget.show_row_totals" class="row_total oe_number" t-att-data-y="y"/>