Re: [pgAdmin][RM4195] Tab navigation is broken in Query tool (Accessibility)

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgAdmin][RM4195] Tab navigation is broken in Query tool (Accessibility)
Дата
Msg-id CA+OCxoyGY0SVN8CQBkXRjh+Tx=+Z8ekZfaFH2+eUY1uC5fQj9Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM4195] Tab navigation is broken in Query tool (Accessibility)  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Ответы Re: [pgAdmin][RM4195] Tab navigation is broken in Query tool (Accessibility)  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Список pgadmin-hackers
Hi,

On further testing, this doesn't seem to be working as expected. Alt+Shift+] or [ in the query tool are typing ’ and ” and not changing the tab. If I switch to a non-input tab (e.g. History), they still don't work. 

Alt+Shift+Tab changes the panel successfully though, and does cause the flash as expected. 

I'm using Chrome 74.0.3729.131 on Mojave.

On Wed, May 22, 2019 at 6:53 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,

Attached is the updated patch with doc changes.

On Tue, May 21, 2019 at 5:42 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi,

On Tue, May 21, 2019 at 5:32 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Tue, May 21, 2019 at 10:01 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix the accessibility issue in query tool.

The query tool can be completely restructured unlike previously where query tool used to stick at top and other tabs in the bottom. 
- I have added a new shortcut to switch between frames if in query tool the tab positions are changed. On switching the frames, the frame will blink to show it is focussed. 
- In the frame, you can switch tab same as previous. The tabs in SQL editor are cyclic now, which means if you execute shortcut to go right on the last tab then it will cycle to first tab.
- I have also changed the default shortcut to switch between tabs from Shift+Ctrl+ArrowKeys to Shift+Ctrl+[ and ] similar to what we have in browser tabs.
- Similar changes done for debugger also.

Shouldn't there be documentation changes to go with this? 
Yes, definitely :P
Will send the updated patch. 

Otherwise, it seems to work :-)

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

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


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


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

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

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin][RM4263] "Not connected to the server or the connectionto the server has been closed." returned with some queries
Следующее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Feature test stabilisation.