Lars Gullik Bjønnes wrote:
[EMAIL PROTECTED] (Lars Gullik Bjønnes) writes:
| Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
|
| | >>>>> "Peter" == Peter Kümmel <[EMAIL PROTECTED]> writes:
| |
| | Peter> I've reproduced the error and fixed it. The problem was, not
| | Peter> only page up/down keys were dropped. This code does not work
| | Peter> (because of the implicit casts?)
| |
| | Peter> static const int delayed_keys = Qt::Key_PageDown |
| | Peter> Qt::Key_PageUp; if (e->key() & delayed_keys) {
| |
| | I know this stuff is very difficult to get right, but I do not like at
| | all the idea of testing explicitely qt::pageup/down. We should not
| | depend on these hardcoded keys. For example, your patch will fail for
| | people who use the emacs bindings and use Ctrl+v for page down. I am
| | not asking for adding another case for Ctrl+v (!) but I'd prefer a
| | solution which does not take this into account (especially since I do
| | not understand why only PageDown would be a problem (why is paragraph
| | down not a problem?)
|
| What is the problem that you are trying to solve here?
Is it my old pet? "Countinued scorrling after key-release"?
What was wrong with my patch from months back?
Works for me!
