odoo
/
odoo
2
0
Fork 0
Code Pull Requests Releases Activity

[FIX] clicking on checkbox (boolean field) of list view should correctly trigger whatever the row action is 

replaced @disabled checkboxes for boolean fields by @readonly + alpha
(50%), as disabled input do not generate *any* event. Not default
action (of course) but not e.g. a click either, whether delegated or
bound directly on the element, there's no way at all to see if e.g. a
user has clicked on a disabled checkbox or radio.

lp bug: https://launchpad.net/bugs/1013569 fixed

bzr revid: xmo@openerp.com-20121113131806-7fhog1f96gxnjr67
This commit is contained in:
Xavier Morel 2012-11-13 14:18:06 +01:00
parent a379a62d88
commit 9fcde05168
3 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
addons/web/static/src/css/base.css
View File

@ -2731,6 +2731,10 @@
.openerp .oe_list_content > tbody > tr > td.oe_list_checkbox:first-child:after, .openerp .oe_list_content > tbody > tr th.oe_list_checkbox:first-child:after {
border-width: 0;
}
.openerp .oe_list_content > tbody > tr > td.oe_list_field_boolean input {
filter: alpha(opacity=50);
opacity: 0.5;
}
.openerp .oe_list_content > tbody > tr:nth-child(odd) {
background-color: #f0f0fa;
background-color: #f0f0fa;

2
addons/web/static/src/css/base.sass
View File

@ -2165,6 +2165,8 @@ $sheet-padding: 16px
width: 17px
&:after
border-width: 0
> td.oe_list_field_boolean input
@include opacity()
> tr:nth-child(odd)
background-color: #f0f0fa
@include vertical-gradient(#f0f0fa, #eeeef6)

2
addons/web/static/src/js/view_list.js
View File

@ -2156,7 +2156,7 @@ instance.web.list.Boolean = instance.web.list.Column.extend({
* @private
*/
_format: function (row_data, options) {
return _.str.sprintf('<input type="checkbox" %s disabled="disabled"/>',
return _.str.sprintf('<input type="checkbox" %s readonly="readonly"/>',
row_data[this.id].value ? 'checked="checked"' : '');
}
});