[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: KAction port: do we need to support multiple shortcuts per action?
From:       Hamish Rodda <rodda () kde ! org>
Date:       2005-11-29 17:28:55
Message-ID: 200511300428.58227.rodda () kde ! org
[Download RAW message or body]


On Wednesday 30 November 2005 03:58, Matthias Ettrich wrote:
> On Tuesday 29 November 2005 17:02, Hamish Rodda wrote:
> > If we were to use QAction as a base for KAction in KDE4, to
> > keep this feature we (or Trolltech) would have to add multiple sequence
> > support.
>
> ...which is trivial to do in a few lines of code.

Ok, not the impression I had before, but it's the best solution for KDE :)

> Wrt the API, is one 
> alternative shortcut enough or does it have to be lists? Is it OK that the
> shortcut context is the same for all shortcuts?

I don't think the KDE api allowed for different contexts in the one action 
previously, so I would think so (if someone can come up with a valid use case 
for having multiple Qt::ShortcutContexts, now would be the time).

> In that case, adding a 
> simple
>
>  void setAlternativeShortcuts(const QList<QKeySequence>&)
>  QList<QKeySequence> alternativeShortcuts() const;
>
> is all it takes. And sending a patch to qt-bugs.

Great :)

Cheers,
Hamish.

[Attachment #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic