| author | zecke <zecke> | 2004-10-08 21:34:32 (UTC) |
|---|---|---|
| committer | zecke <zecke> | 2004-10-08 21:34:32 (UTC) |
| commit | 16afa90172c9b1facaa8888780ed8bb8bd93598c (patch) (side-by-side diff) | |
| tree | 3642810e51ceaacff2f6fba499e78d0cff0121b9 | |
| parent | 0c3966672847b3b81c96c942727178368ac70e65 (diff) | |
| download | opie-16afa90172c9b1facaa8888780ed8bb8bd93598c.zip opie-16afa90172c9b1facaa8888780ed8bb8bd93598c.tar.gz opie-16afa90172c9b1facaa8888780ed8bb8bd93598c.tar.bz2 | |
Use KeyPressed instead of KeyReleased for the Accelerator
Key in the Image Display component. This fixes the
double 'next' problem when going from the browser to the viewer/display
| -rw-r--r-- | noncore/graphics/opie-eye/gui/imageview.cpp | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/noncore/graphics/opie-eye/gui/imageview.cpp b/noncore/graphics/opie-eye/gui/imageview.cpp index b4f3110..5522131 100644 --- a/noncore/graphics/opie-eye/gui/imageview.cpp +++ b/noncore/graphics/opie-eye/gui/imageview.cpp @@ -27,5 +27,3 @@ ImageView::~ImageView() odebug << "Delete Imageview" << oendl; - if (m_viewManager) { - delete m_viewManager; - } + delete m_viewManager; } @@ -57,2 +55,10 @@ void ImageView::initKeys() + /** + * Handle KeyEvents when they're pressed. This avoids problems + * with 'double next' on Return. + * The Return press would switch to this view and the return + * release would emit the dispNext Signal. + */ + m_viewManager->setEventMask( Opie::Core::OKeyConfigManager::MaskPressed ); + m_viewManager->addKeyConfig( Opie::Core::OKeyConfigItem(tr("View Image Info"), "imageviewinfo", @@ -148,3 +154,3 @@ void ImageView::focusInEvent(QFocusEvent *) -void ImageView::hide() +void ImageView::hide() { |