Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor
Date
Msg-id CAKKotZRZT=5kT8cb58GVBy3POV6gbHQR57nfz5s1HTPpvfN5UQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor
List pgadmin-hackers
Hi Dave,

On Mon, Jul 17, 2017 at 2:33 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Jul 12, 2017 at 1:16 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch which will add functionality to allow user to comment/uncomment code in query editor.
RM#2456

This is cool, but I'm not sure it's right as-is:

* I prefer SQL style commenting, e.g.

-- This is a comment

Should we make that a config option if CodeMirror can do it? Or a different hotkey?
 
I'll check the extension code and update you accordingly, and It will be good idea to keep the both the options because with large code block current style works the best.
* You've added it as an option to the Clear XXX dropdown, which really isn't the right place in my opinion. Should we add a new drop down for this, and include some/all of the other Editing options on there? E.g. tab/shift-tab.

I thought that is misc options dropdown for our editor, but I don't see any point adding new drop down for one single option, Can we add new button instead? 
* I think the docs should say Ctrl+Shift+/ rather than Shift+Ctrl+/, and be ordered in the table to reflect that. It seems more natural to me.

Initially I wrote ctrl + shift + /only but when I saw all other shortcuts starts with Shift , then I changed it to shift + ctrl + / :)
Thoughts?

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

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

pgadmin-hackers by date:

Previous
From: pgAdmin 4 Jenkins
Date:
Subject: Build failed in Jenkins: pgadmin4-master-python36 #238
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: Allow user to cancel long running queries from dashboard