extragear/multimedia/amarok/src

Seb Ruiz ruiz at kde.org
Fri Sep 7 02:32:33 UTC 2007


SVN commit 709281 by seb:

Experiment by placing the statusbar under the ContextView. I find that there is barely enough space under the playlist to read anything, eg: "building collection"
Just wondering if we really even need a statusbar. We could use some of the space that we have in the big huge toolbar to display action based information, such as "Building collection" or "Submitting to last.fm"
Thoughts?
CCMAIL: amarok at kde.org


 M  +3 -3      MainWindow.cpp  


--- trunk/extragear/multimedia/amarok/src/MainWindow.cpp #709280:709281
@@ -228,8 +228,6 @@
 
     this->toolBars().clear();
 
-    Amarok::StatusBar *statusbar = new Amarok::StatusBar( playlistwindow );
-    QAction* repeatAction = Amarok::actionCollection()->action( "repeat" );
 
     createMenus();
 
@@ -241,9 +239,11 @@
         QVBoxLayout* layout = new QVBoxLayout( contextWidget );
         layout->setContentsMargins( 0, 0, 0, 0 );
         layout->addWidget( Context::ContextView::self() );
+        Amarok::StatusBar *statusbar = new Amarok::StatusBar( contextWidget );
+        layout->addWidget( statusbar );
 
         if( AmarokConfig::useCoverBling() && QGLFormat::hasOpenGL() )
-             layout->addWidget( new CoverBling( this ) );
+            layout->addWidget( new CoverBling( this ) );
 
         ControlBox* controlBox = new ControlBox( contextWidget );
         controlBox->show();



More information about the Amarok mailing list