public inbox for [email protected]
help / color / mirror / Atom feedFrom: Surinder Kumar <[email protected]>
To: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgAdmin4][Patch]: RM#1478 - Make code mirror text editor keyboard shortcuts consistent irrespective of platform type
Date: Mon, 8 Aug 2016 17:53:42 +0530
Message-ID: <CAM5-9D9A_hcssJ62RUeihYfTMY4zopqRsKioih5w5xd-e9FuKw@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxowu8GSWiM=T3oAGZgcp7XVQG5pzuO=djU-zCDfV-ka=YA@mail.gmail.com>
References: <CAM5-9D8ExeecjY6Mk7iZ_ZFs+iyLr1-6QtJR0djYvSO_65NtEw@mail.gmail.com>
<CA+OCxoy+NzJ+LJj4hqunCXprx0XpK7xZX975QOwXj+o32XEHYA@mail.gmail.com>
<CAM5-9D_qJN6GfLCdgpDr=fv4Ss+M17WnC2qFvnJK-4EPNAyXww@mail.gmail.com>
<CA+OCxowu8GSWiM=T3oAGZgcp7XVQG5pzuO=djU-zCDfV-ka=YA@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
On Mon, Aug 8, 2016 at 5:52 PM, Dave Page <[email protected]> wrote:
> On Mon, Aug 8, 2016 at 1:15 PM, Surinder Kumar
> <[email protected]> wrote:
> > On Mon, Aug 8, 2016 at 4:43 PM, Dave Page <[email protected]> wrote:
> >>
> >> Hi Surinder,
> >>
> >> This looks very promising, but it only covers the query tool at
> >> present. We need the shortcuts to apply to all codemirror instances.
> >> Do the others work anyway, because we're invoking in a different way,
> >> or is more work needed there?
> >
> > To apply the same shortcuts to other codemirror instances(e.g:
> > SqlTabControl) we just need to set a parameter
> > extraKeys: pgAdmin.editor_shortcut_keys for every codemirror instance.
> > As of now, shortcuts are applicable to query tool & data filter.
> >
> > Also, we need to define pgAdmin.editor_shortcut_keys variable in
> server.js
> > instead of sqleditor.js to work it for
> > other code mirror instances.
>
> OK, please update the patch to do that.
>
> Can you get that to me in the next hour or so? I'd like to include it in
> beta 4.
>
Sure.
>
> >> On Fri, Aug 5, 2016 at 6:42 PM, Surinder Kumar
> >> <[email protected]> wrote:
> >> > Hi
> >> >
> >> > I found that Code-mirror library itself provides the List of keyboard
> >> > shortcuts for Mac OSX and other platform type.
> >> > We just need to pass them in extraKeys param while creating
> code-mirror
> >> > instance.
> >> >
> >> > Following is the keyboard shortcuts for various operations in text
> >> > editor:
> >> >
> >> > Copy - [Ctrl-C, Cmd-C]
> >> > Cut - [Ctrl-X, Cmd-X]
> >> > Select All - [Ctrl-A, Cmd-A]
> >> > Undo - [Ctrl-Z, Cmd-Z]
> >> > Redo - [Ctrl-Y, Cmd-Y]
> >> > Delete Line - [Ctrl-D, Cmd-D]
> >> > Move left/right(words) - [Alt-Left, Alt-Right]
> >> > Move start/end of line - [Cmd-Left, Cmd-Right] // Ctrl-Left/Right are
> >> > bound
> >> > to System shortcuts. so these keys cannot be used.
> >> >
> >> > These keyboard shortcuts are consistent in Web/Runtime applications in
> >> > all
> >> > platform types.
> >> >
> >> > Issue not fixed:
> >> > keyboard shortcut for Paste text doesn't work in Mac Runtime only. It
> is
> >> > working on Linux runtime & Windows Runtime.
> >> > An issue "Key shortcuts doesn't work on Mac OS for QWebView widget" is
> >> > also
> >> > reported.
> >> >
> >> > I also found that right click paste is working in Mac Runtime which is
> >> > the
> >> > feature of code-mirror, So I am looking into its code to figure out
> and
> >> > I
> >> > will send a patch with fix once it gets fixed.
> >> >
> >> > Please find attached patch and review.
> >> >
> >> >
> >> > Thanks,
> >> > Surinder Kumar
> >> >
> >> >
> >> >
> >> > --
> >> > Sent via pgadmin-hackers mailing list ([email protected]
> )
> >> > To make changes to your subscription:
> >> > http://www.postgresql.org/mailpref/pgadmin-hackers
> >> >
> >>
> >>
> >>
> >> --
> >> Dave Page
> >> Blog: http://pgsnake.blogspot.com
> >> Twitter: @pgsnake
> >>
> >> EnterpriseDB UK: http://www.enterprisedb.com
> >> The Enterprise PostgreSQL Company
> >
> >
>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
view thread (7+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected]
Subject: Re: [pgAdmin4][Patch]: RM#1478 - Make code mirror text editor keyboard shortcuts consistent irrespective of platform type
In-Reply-To: <CAM5-9D9A_hcssJ62RUeihYfTMY4zopqRsKioih5w5xd-e9FuKw@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox