diff options
| author | Luis González <lgonzalez@vauxoo.com> | 2018-03-22 07:18:35 -0400 |
|---|---|---|
| committer | kujiu <kujiu@users.noreply.github.com> | 2018-03-22 12:18:35 +0100 |
| commit | ef540eb006abac0d318f2a708384d125f6203662 (patch) | |
| tree | 0cf133f93d87cd6a2f2b6eb138f87e0b36ac8885 /odoo/addons/test_new_api/static | |
| parent | ac76a6b40cace06c63c049be0ae9e58721048473 (diff) | |
[FIX] web: Make accessible with keyboard the delete button on o2m fields (#23719)
* [FIX] web: Make accessible with keyboard the delete button on o2m fields
Currently, the delete button in one2many fields (trash icon) is not
reachable when using the keyboard, because it's neither a link, nor a
button or any other interactable HTML element, but a `<span>`.
This change causes that button to be an actual HTML button keeping the
same appearance, so it may be reached when using the keyboard.
* [FIX] web: Modify test to expect button instead of span as o2m trash icon
Since buttons to delete records in o2m fields (the trash icon) were
changed from `<span>` to `<button>` to be able to access them with the
keyboard, the test need to be modified so it expect the new element
type and doesn't break.
Diffstat (limited to 'odoo/addons/test_new_api/static')
| -rw-r--r-- | odoo/addons/test_new_api/static/tests/x2many.js | 2 |
1 files changed, 1 insertions, 1 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..30f351c21f6 100644 --- a/odoo/addons/test_new_api/static/tests/x2many.js +++ b/odoo/addons/test_new_api/static/tests/x2many.js @@ -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_delete button:visible:last', }, { content: "test one2many onchange after delete", trigger: '.o_content:not(:has(textarea[name="message_concat"]:propValueContains(Administrator:d)))', |