From ed4b9064a109e19a9c4969ebf0b563d3d53530b8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luis=20Gonz=C3=A1lez?= Date: Fri, 21 Apr 2017 17:54:03 +0000 Subject: [PATCH] [FIX] web: Properly label buttons which change views, with aria-label Closes #16576 Description of the issue/feature this PR addresses: According to HTML good practices, when an element is used but there's no text displayed on the screen, the attribute aria-label should be used to label that element. This is the case in the buttons to change view types (List, Kanban, etc) since there's no text displayed, but an icon instead. This HTML attribute is specially useful for screen readers, which usually are not able to figure out a button's functionality when that button is not properly labeled. Current behavior before PR: Buttons are not properly labeled Desired behavior after PR is merged: Appearance should be the same, but the produced HTML should be of better quality, because it would fit more HTML good conventions and it whould be more accessible Related issue: I had already reported this issue on the branch 10.0 at PR #16030. However, I decided to report it on this branch, because I believe this branch is the right place to be reported. --- addons/web/static/src/xml/base.xml | 1 + 1 file changed, 1 insertion(+) diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml index b9b7caeea7d..50e59713562 100644 --- a/addons/web/static/src/xml/base.xml +++ b/addons/web/static/src/xml/base.xml @@ -96,6 +96,7 @@