Re: [pgAdmin4][PATCH] Refactor and change of implementation of keyboard_shortcuts function dependencies

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
>

In response to

Responses

Browse pgadmin-hackers by date

  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