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

List:       kdevelop-devel
Subject:    Re: Shortcut problem: Ctrl-Enter
From:       Hamish Rodda <rodda () kde ! org>
Date:       2006-04-20 7:36:14
Message-ID: 200604201736.16541.rodda () kde ! org
[Download RAW message or body]

On Thursday 20 April 2006 16:56, Vladimir Prus wrote:
> Hi,
>
> I was trying to improve variable widget a bit, and run into a problem. The
> bottom of the variable widget has a input line where user can type an
> expression. To evaluate that expression, user can either just click Enter,
> or click "Evaluate button", but to watch the expression, he has to click
> the "Watch" button -- there's no keyboard access.
>
> This is somewhat inconvenient, so I wanted to make "Ctrl-Enter" watch
> expression. However, this is already a global shortcut for "show context
> menu", set in editorproxy.cpp.
>
> I've a number of questions:
>
> 1. Is there are way to override shortcut for variablewidget? I'm trying to
> add keyPressEvent, but shortcut handling is done in event filter, so my
> method is not even called.

Perhaps your own event filter? Or is this the X11 event filter you're talking 
about.

> 2. Failing (1), does it make sense to have "Ctrl-Enter" shortcut work
> globally. If I type "Ctrl-Enter" in variablewidget, then certainly showing
> popup menu in *editor* is unexpected. Any approach to limiting the shortcut
> only to editor windows?

I think that should be the case, how to technically achieve it with KAccel I 
don't know about.  In KDE4 it's already solved with 
QAction::setShortcutContext().

> 3. Do we need this shortcut for popup menu at all?

I'd say no, but then I didn't know about it...
From my POV, just remove it.

Cheers,
Hamish.

[Attachment #3 (application/pgp-signature)]
_______________________________________________
KDevelop-devel mailing list
KDevelop-devel@barney.cs.uni-potsdam.de
http://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel

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