From 2bac80215f49e90e889e279f95bbb0a03ac7336f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Thu, 11 Aug 2022 14:30:55 +0200 Subject: [PATCH] Add permissions hint for visually impaired people We were displaying an icon showing that certain actions can't be performed. However, people who can't see the icons were hearing that they _can_ perform certain actions while the opposite is true. We've considered other options to solve this problem. One was to split the list in two: actions that can be performed and actions that can't be performed. It was tricky because in some cases we're listing that actions that can be performed now and in other cases we're displaying the actions that people will be able to perform once they verify their account. Another option was to include the word "Cannot" as a prefix instead of "Additional verification needed". We haven't done so because, while in English we say "cannot do this thing", in other languages they say "this thing cannot do". So we've gone with a solution where people hearing what's on the screen know what's going on and we don't have to make big changes in the code. --- app/components/account/permissions_list_component.html.erb | 4 ++++ config/locales/en/verification.yml | 1 + config/locales/es/verification.yml | 1 + spec/components/account/permissions_list_component_spec.rb | 7 +++++++ 4 files changed, 13 insertions(+) diff --git a/app/components/account/permissions_list_component.html.erb b/app/components/account/permissions_list_component.html.erb index 055c9a922..028976b03 100644 --- a/app/components/account/permissions_list_component.html.erb +++ b/app/components/account/permissions_list_component.html.erb @@ -1,6 +1,10 @@