Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I needall buttons in the application to have appropriate tooltips for my screenreader to interpret - Mailing list pgadmin-hackers

From Khushboo Vashi
Subject Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I needall buttons in the application to have appropriate tooltips for my screenreader to interpret
Date
Msg-id CAFOhELfaGqh-5XxP-GXRfbt4c3Jt0ytfXD5_A6R_qJM4gJz7PQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I needall buttons in the application to have appropriate tooltips for my screenreader to interpret  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I needall buttons in the application to have appropriate tooltips for my screenreader to interpret
List pgadmin-hackers


On Mon, Jan 15, 2018 at 10:39 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Mon, Jan 15, 2018 at 10:23 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.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.

Thanks for the clarification.
Please find the attached updated patch. 
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Harshal Dhumal
Date:
Subject: Module-wise Keyboard preferences
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: RM #2904: As a visually impaired user I needall buttons in the application to have appropriate tooltips for my screenreader to interpret