From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Hao Wang <haowang(at)pivotal(dot)io>, Sarah McAlear <smcalear(at)pivotal(dot)io> |
Subject: | Re: [pgAdmin4][PATCH] Refactor and change of implementation of keyboard_shortcuts function dependencies |
Date: | 2017-08-04 12:07:19 |
Message-ID: | CAG7mmoxK34qfmmS8a89nmPqSApmiZ_r41rOXt3EPkkNpyDFs0g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Murtuza,
On Thu, Aug 3, 2017 at 3:38 PM, Sarah McAlear <smcalear(at)pivotal(dot)io> wrote:
> Hi Hackers!
>
> Attached is a patch that extracts the functions called by the
> keyboard_shortcuts function extracted earlier from sqlEditor. This
> includes
>
> - executeQuery
> - explainAnalyze
> - explain
> - download
> - commentBlockCode
> - commentLineCode
> - uncommentLineCode
>
>
> There is still more work to be done, but this is it for now.
>
> There is also an additional patch that changes the implementation of the
> commentLineCode, uncommentLineCode, and commentBlockCode functions. The
> shortcut
>
> - for commentLineCode is now cmd + / (ctrl + / for Windows)
> - for uncommentLineCode cmd + . (ctrl + . for Windows)
> - for comment and uncomment blockCode shift + cmd + / (shift + ctrl + /
> for Windows)
>
> This is consistent with other IDEs and the way commenting is implemented.
>
Please review it, and let us know your comments.
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company
<http://www.enterprisedb.com/>
*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>
>
> Thanks!
> Hao & Sarah
>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2017-08-04 12:08:27 | Re: [gpAdmin4][patch] query history updates |
Previous Message | pgAdmin 4 Jenkins | 2017-08-04 10:49:31 | Build failed in Jenkins: pgadmin4-master-python34 #266 |