Re: [pgAdmin4][Patch]: RM # 2895 - All the controls and displayelements in the main window to be accessible using the tab key in anappropriate order

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgAdmin4][Patch]: RM # 2895 - All the controls and displayelements in the main window to be accessible using the tab key in anappropriate order
Дата
Msg-id CA+OCxow=SfHAuquN=SsV8eLOOKWmxtH7mO+uDENK_yZ+K2E7Aw@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][Patch]: RM # 2895 - All the controls and display elementsin the main window to be accessible using the tab key in an appropriate order  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch]: RM # 2895 - All the controls and displayelements in the main window to be accessible using the tab key in anappropriate order  (Khushboo Vashi <khushboo.vashi@enterprisedb.com>)
Список pgadmin-hackers
Hi

On Mon, Jan 29, 2018 at 9:00 AM, Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,

Please find the attached patch for RM #2895: As a keyboard user who has difficulty using a mouse, I need all the controls and display elements in the main window to be accessible using the tab key in an appropriate order.

Key Notes:
1. While loading the application, all the focusable objects are accessible (Main menu, Right Panel, Left Panel)
1. Left hand side tree view is accessible via Alt+Shift+B
2. Right side tabbed panels are accessible via Alt+shift+[  and Alt+shift+]
3. Main menu navigation
    File Menu - Alt+Shift+F
    Object Menu - Alt+Shift+O
    Tools Menu - Alt+Shift+L
    Help Menu - Alt+Shift+H
4. Once the menu is opened, user can access the next/previous main menu through the left and the right arrow keys 
5. When the user is in the SQL Editor and wants to change the panel, use ESC key to move the cursor out from the text editor and then can use the default shortcut keys (Alt+Shift+[ / Alt+Shift+])

This seems to work quite well. I noticed a few issues in review thought:

- The image added to the docs is the wrong size/resolution. Please see Chethana to see how he generates them if needed.

- The table in the docs has some extra + symbols still present where I assume you copied/pasted an existing table and removed a column.

- Any reason why Alt+Shift+B isn't configurable?

- I think the description for Alt+Shift+B should be  "Focus the browser tree".

- I don't see any Jasmine tests for the new JS, or any feature tests. Whilst we needn't test every single option, I think we should at least test that changing a shortcut works as expected, and of course, unit test the JS.

Thanks!

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][RM#3063] Add 'pycodestyle ' Python PEP-8 checker module
Следующее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [pgAdmin4][RM#3063] Add 'pycodestyle ' Python PEP-8 checker module