Re: New key shortcuts for query window

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Thom Brown <thom(at)linux(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: New key shortcuts for query window
Date: 2011-07-12 09:59:14
Message-ID: CA+OCxoyKfVRH-pYHrcOVBNRqiRvnewUmEUmuf9rjuQqiBZ0V5g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Sun, Jul 10, 2011 at 1:45 PM, Thom Brown <thom(at)linux(dot)com> wrote:
> Hi all,
>
> I think the SQL query window should have a File > Close menu item,
> which maps to Ctrl+W (or Apple+W in the case of a Mac).  This is easy
> to add.

We already have File -> Exit, which uses Ctrl+W. On the Mac, that
currently gets mapped to pgAdmin3 -> Quit pgAdmin3, which is Cmd+Q.

That could certainly be cleaned up on frmQuery, but should also be
done on other forms at the same time.

> But something else I want to change isn't so easy.  I'd like
> Execute to be mapped to Ctrl+R (as it's much easier to use in OSX
> since F keys are far less used since they need to be used in
> combination with the fn key).  This is already mapped to F5, and it's
> easy to make it work with both, but I can't get the menu item to
> display both mappings because of how wxMenuItem::Append works.  It
> appears you can only specify one accelerator per menu item.  You can
> add another menu item for the additional key binding, but that looks
> rubbish.
>
> Any idea if it's possible to show both key bindings on the same item?
>
> e.g. Execute   F5/Ctrl+R

No, I don't believe you can, because wxWidgets menus try to interpret
the string after \t in the text to auto-create the accelerator keys.

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

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

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Thom Brown 2011-07-12 10:03:41 Re: New key shortcuts for query window
Previous Message Thom Brown 2011-07-12 09:55:25 Re: Prevent duplicate attributes