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

List:       kde-commits
Subject:    KDE/kdepim/akregator/src
From:       Hugo Pereira Da Costa <hugo.pereira () free ! fr>
Date:       2010-05-06 18:55:21
Message-ID: 20100506185521.6FDAEAC8AC () svn ! kde ! org
[Download RAW message or body]

SVN commit 1123741 by hpereiradacosta:

layout improvement: 
- make ArticleViewer derive from QWidget and not QFrame (so that it has 0 margin)
- make the view contained in ArticleViewerPart sunken.

Screenshot at: http://www.flickr.com/photos/hugo-oxygen-icons-org/4577939521/



 M  +2 -2      articleviewer.cpp  
 M  +1 -1      articleviewer.h  


--- trunk/KDE/kdepim/akregator/src/articleviewer.cpp #1123740:1123741
@@ -76,7 +76,7 @@
 namespace Akregator {
 
 ArticleViewer::ArticleViewer(QWidget *parent)
-    : QFrame(parent),
+    : QWidget(parent),
       m_url(0),
       m_htmlFooter(),
       m_currentText(),
@@ -87,7 +87,6 @@
       m_normalViewFormatter( new DefaultNormalViewFormatter( m_imageDir, m_part->view() ) ),
       m_combinedViewFormatter( new DefaultCombinedViewFormatter( m_imageDir, m_part->view() ) )
 {
-    setFrameStyle(QFrame::StyledPanel | QFrame::Raised);
     QGridLayout* layout = new QGridLayout(this);
     layout->setMargin(0);
     layout->addWidget(m_part->widget(), 0, 0);
@@ -104,6 +103,7 @@
     m_part->setAutoloadImages(true);
     m_part->setStatusMessagesEnabled(false);
     m_part->view()->setAttribute(Qt::WA_InputMethodEnabled, true); //workaround to fix 216878
+    m_part->view()->setFrameStyle( QFrame::StyledPanel|QFrame::Sunken );
 
     // change the cursor when loading stuff...
     connect( m_part, SIGNAL(started(KIO::Job *)),
--- trunk/KDE/kdepim/akregator/src/articleviewer.h #1123740:1123741
@@ -51,7 +51,7 @@
 
 class ArticleViewerPart;
 
-class ArticleViewer : public QFrame
+class ArticleViewer : public QWidget
 {
     Q_OBJECT
     public:
[prev in list] [next in list] [prev in thread] [next in thread]