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 /noncore/graphics/opie-eye/gui/imageview.cpp | |
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
Diffstat (limited to 'noncore/graphics/opie-eye/gui/imageview.cpp') (more/less context) (ignore whitespace changes)
-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 @@ -26,7 +26,5 @@ ImageView::~ImageView() { odebug << "Delete Imageview" << oendl; - if (m_viewManager) { - delete m_viewManager; - } + delete m_viewManager; } @@ -56,4 +54,12 @@ void ImageView::initKeys() lst, false,this, "image_view_keys" ); + /** + * 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", Resource::loadPixmap("1to1"), ViewInfo, @@ -147,5 +153,5 @@ void ImageView::focusInEvent(QFocusEvent *) } -void ImageView::hide() +void ImageView::hide() { if (fullScreen()) { |