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

List:       kde-commits
Subject:    KDE/kdepim/akregator/src
From:       Jakob Petsovits <jpetso () gmx ! at>
Date:       2008-06-01 20:22:40
Message-ID: 1212351760.512507.31472.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 815441 by jpetso:

Assign the article view as focus proxy of the article list view.
That fully takes away focus from the article list, fixing stuff
like PgDn/PgUp not working when the list has been clicked.

Fixes part of bug 124486, whereas Ctrl+arrow for the
per-feed navigation actions still doesn't work,
this commit doesn't change anything in that regard.

CCBUG: 124486


 M  +0 -5      articlelistview.cpp  
 M  +0 -3      articlelistview.h  
 M  +2 -0      articleviewer.cpp  
 M  +1 -0      mainwidget.cpp  


--- trunk/KDE/kdepim/akregator/src/articlelistview.cpp #815440:815441
@@ -342,11 +342,6 @@
 }
 #endif
 
-void Akregator::ArticleListView::keyPressEvent(QKeyEvent* e)
-{
-    e->ignore();
-}
-
 void Akregator::ArticleListView::contextMenuEvent( QContextMenuEvent* event )
 {
     QWidget* w = ActionManager::getInstance()->container( "article_popup" );
--- trunk/KDE/kdepim/akregator/src/articlelistview.h #815440:815441
@@ -137,9 +137,6 @@
     void paintEvent( QPaintEvent* e );
 
     //reimpl
-    void keyPressEvent( QKeyEvent* e );
-
-    //reimpl
     void contextMenuEvent( QContextMenuEvent* event );
 
     void selectIndex( const QModelIndex& index );
--- trunk/KDE/kdepim/akregator/src/articleviewer.cpp #815440:815441
@@ -89,6 +89,8 @@
     layout->setMargin(0);
     layout->addWidget(m_part->widget(), 0, 0);
 
+    setFocusProxy( m_part->widget() );
+
     m_part->setZoomFactor(100);
     m_part->setJScriptEnabled(false);
     m_part->setJavaEnabled(false);
--- trunk/KDE/kdepim/akregator/src/mainwidget.cpp #815440:815441
@@ -221,6 +221,7 @@
 
     m_articleViewer = new ArticleViewer(m_articleSplitter);
     m_actionManager->initArticleViewer(m_articleViewer);
+    m_articleListView->setFocusProxy(m_articleViewer);
 
     connect( m_articleViewer, SIGNAL(signalOpenUrlRequest(Akregator::OpenUrlRequest& )),
              Kernel::self()->frameManager(), SLOT(slotOpenUrlRequest( Akregator::OpenUrlRequest& )) );
[prev in list] [next in list] [prev in thread] [next in thread]