diff options
| author | Alexandre Kühn <alexkuhn@users.noreply.github.com> | 2018-01-03 11:42:39 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-01-03 11:42:39 +0100 |
| commit | 8bc7cebd9c19ae8f77378d115a8401d7b7de56b0 (patch) | |
| tree | 4cc13e4a51b11e1da1f30996cab01d3d6876a451 /odoo | |
| parent | c8fa2d581caa13e881188552e6e2e22f8e858f7f (diff) | |
[IMP] web: many2many editable list with X remove icon (#20978)
In many2many lists, the remove record from the widget is a trash bin.
It is confusing, as people feel they are deleting it when they are
only unlinking it.
So in many2many, we propose to change the trash icon by a 'X'.
This icon appears in many2many relations and in many2many widgets:
even though hr.expense.sheet defines the expense lines as a one2many,
it uses a many2many widget, so we use a 'X' instead of the trash bin.
To avoid ambiguity, we decided to rename 'delete' to 'remove'.
Diffstat (limited to 'odoo')
| -rw-r--r-- | odoo/addons/test_new_api/static/tests/x2many.js | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/odoo/addons/test_new_api/static/tests/x2many.js b/odoo/addons/test_new_api/static/tests/x2many.js index ea976330eb9..b18cb574970 100644 --- a/odoo/addons/test_new_api/static/tests/x2many.js +++ b/odoo/addons/test_new_api/static/tests/x2many.js @@ -246,11 +246,11 @@ odoo.define('web.test.x2many', function (require) { run: function () {}, // don't change texarea content }, { // remove content: "remove b", - trigger: '.tab-pane:eq(0) .o_field_widget tbody tr:has(td:containsExact(bbb)) .o_list_record_delete', + trigger: '.tab-pane:eq(0) .o_field_widget tbody tr:has(td:containsExact(bbb)) .o_list_record_remove', extra_trigger: '.tab-pane:eq(0) .o_field_widget tbody tr td:containsExact(aaa)', }, { content: "remove e", - trigger: 'tr:has(td:containsExact(e)) .o_list_record_delete', + trigger: 'tr:has(td:containsExact(e)) .o_list_record_remove', extra_trigger: 'body:not(:has(tr:has(td:containsExact(bbb))))', }, { // save content: "save discussion", @@ -258,7 +258,7 @@ odoo.define('web.test.x2many', function (require) { extra_trigger: 'body:not(:has(tr:has(td:containsExact(e))))', }, { // check saved data content: "check data 4", - trigger: '.o_content:not(:has(.tab-pane:eq(0) .o_field_widget tbody tr:has(.o_list_record_delete):eq(4)))', + trigger: '.o_content:not(:has(.tab-pane:eq(0) .o_field_widget tbody tr:has(.o_list_record_remove):eq(4)))', run: function () {}, // it's a check }, { content: "check data 5", @@ -408,7 +408,7 @@ odoo.define('web.test.x2many', function (require) { trigger: '.ui-autocomplete a:first', }, { // remove record content: "delete the last item in the editable list", - trigger: '.o_list_view .o_data_row td.o_list_record_delete span:visible:last', + trigger: '.o_list_view .o_data_row td.o_list_record_remove i:visible:last', }, { content: "test one2many onchange after delete", trigger: '.o_content:not(:has(textarea[name="message_concat"]:propValueContains(Administrator:d)))', |