Re: Fix for issue RM1336 [pgadmin4]

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: Fix for issue RM1336 [pgadmin4]
Дата
Msg-id CA+OCxoy20eBO-z9jK9eNJQvoEYOCOQrB4hs-9rgLUc1V=hwN9w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Fix for issue RM1336 [pgadmin4]  (Harshal Dhumal <harshal.dhumal@enterprisedb.com>)
Ответы Re: Fix for issue RM1336 [pgadmin4]
Список pgadmin-hackers
Hi Harshal,

On Thu, Jun 30, 2016 at 7:59 AM, Harshal Dhumal
<harshal.dhumal@enterprisedb.com> wrote:
> Hi Dave,
>
> Somehow control key is mapped to Command key in Mac. So on Mac shortcuts are
> Shift+Command+E, Shift+Command+X, Shift+Command+A

OK, those work, but I think we need to take a step back here (partly
because Cmd+Shift+A seems to be Select All in CodeMirror on Mac). I've
committed the patch now, but changed Ctrl+Shift+A to Ctrl+Shift+N.

Can you please work on the following:

1) Create a table of shortcuts from our runtime, our web app, and
CodeMirror. This should list:

Component (e.g. runtime, CodeMirror, Query Tool)
Standard shortcut (e.g. Ctrl+Shift+A)
Mac shortcut (e.g. Cmd+Shift+A)
Function (e.g. Explain Analyze or Select All)

Let's make this as complete and accurate as possible, so it can be
included in the documentation, and used by us to select or
de-duplicate shortcuts.

2) Confirm that the shortcuts we're using in our runtime and web
application don't conflict with any in CodeMirror (or web browsers, in
the case of the web app).

3) Update the web application so the shortcuts are correctly displayed
on Mac automatically - e.g. the tooltips and menus should show
Cmd+Shift not Ctrl+Shift

4) Investigate #1360, and ensure that the CodeMirror shortcuts work
consistently between the runtime and browsers on all platforms.

5) Investigate any remaining shortcuts that don't work as expected.

Thanks, Dave.

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

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


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

Предыдущее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Fix query execution keys in the Mac runtime. Needs mo
Следующее
От: Neel Patel
Дата:
Сообщение: [pgAdmin4][runtime]: Download feature in runtime