From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I need all buttons in the application to have appropriate tooltips for my screen reader to interpret |
Date: | 2018-01-15 17:09:42 |
Message-ID: | CA+OCxozi+Ok0AJercrMYiNEf4W__rcFHqjbP7XaxTt9MrjtQug@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Mon, Jan 15, 2018 at 10:23 AM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi,
>
> Please find the attached patch for RM # 2904: As a visually impaired user
> I need all buttons in the application to have appropriate tooltips for my
> screen reader to interpret.
>
Unfortunately I think there's been a misunderstanding with this patch, most
likely my fault for not being clearer;
- There is no need to add tooltips to buttons that already have text,
unless we want to add additional information. For example, on the OK
button, we might have a tooltip that says "Apply the changes and close the
dialog". That really is optional though, as screen readers could already
read the button text (but we should be consistent).
- We do need to add tooltips to buttons that don't have any text. For
example;
* dialogue title-bar buttons
* the "close tab" and "move tabs left"/"move tabs right" buttons that
may appear in tab bars
* the add row button at the top of a subnode control (e.g. to add a
column definition)
* the delete and expand row buttons in subnode grid rows
Sorry for not being clearer about the requirements.
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-01-15 17:39:11 | Re: RM2815: Relogin to pgAdmin from sqleditor/datadrid if session exprires |
Previous Message | Dave Page | 2018-01-15 17:02:12 | Re: [pgAdmin4][Patch#3013] Fixed minor UI issue on Dashboard |