Show a loading spinner in album views so sourceinfopages etc don't look white for a few seconds

src/libtomahawk/playlist/albummodel.cpp

@@ -254,6 +254,8 @@ AlbumModel::addCollection( const collection_ptr& collection, bool overwrite )
     Database::instance()->enqueue( QSharedPointer<DatabaseCommand>( cmd ) );
 
     m_title = tr( "All albums from %1" ).arg( collection->source()->friendlyName() );
+
+    emit loadingStarted();
 }
 
 
@@ -280,6 +282,8 @@ AlbumModel::addFilteredCollection( const collection_ptr& collection, unsigned in
         m_title = tr( "All albums from %1" ).arg( collection->source()->friendlyName() );
     else
         m_title = tr( "All albums" );
+
+    emit loadingStarted();
 }
 
 
@@ -292,6 +296,8 @@ AlbumModel::addAlbums( const QList<Tomahawk::album_ptr>& albums )
     if ( m_overwriteOnAdd )
         clear();
 
+    emit loadingFinished();
+
     int c = rowCount( QModelIndex() );
     QPair< int, int > crows;
     crows.first = c;

src/libtomahawk/playlist/albummodel.h

@@ -94,6 +94,8 @@ signals:
 
     void trackCountChanged( unsigned int tracks );
 
+    void loadingStarted();
+    void loadingFinished();
 private slots:
     void onDataChanged();
 

src/libtomahawk/playlist/albumview.cpp

@@ -32,6 +32,7 @@
 #include "albummodel.h"
 #include "viewmanager.h"
 #include "utils/logger.h"
+#include "dynamic/widgets/LoadingSpinner.h"
 
 #define SCROLL_TIMEOUT 280
 
@@ -42,7 +43,7 @@ AlbumView::AlbumView( QWidget* parent )
     : QListView( parent )
     , m_model( 0 )
     , m_proxyModel( 0 )
-//    , m_delegate( 0 )
+    , m_loadingSpinner( new LoadingSpinner( this ) )
 {
     setDragEnabled( true );
     setDropIndicatorShown( false );
@@ -109,6 +110,9 @@ AlbumView::setAlbumModel( AlbumModel* model )
     connect( m_proxyModel, SIGNAL( filterChanged( QString ) ), SLOT( onFilterChanged( QString ) ) );
     connect( m_proxyModel, SIGNAL( rowsInserted( QModelIndex, int, int ) ), SLOT( onViewChanged() ) );
 
+    connect( m_model, SIGNAL( loadingStarted() ), m_loadingSpinner, SLOT( fadeIn() ) );
+    connect( m_model, SIGNAL( loadingFinished() ), m_loadingSpinner, SLOT( fadeOut() ) );
+
     setAcceptDrops( false );
     onViewChanged(); // Fetch covers if albums were added to model before model was attached to view
 }

src/libtomahawk/playlist/albumview.h

@@ -28,6 +28,7 @@
 #include "albumproxymodel.h"
 
 class AlbumModel;
+class LoadingSpinner;
 
 class DLLEXPORT AlbumView : public QListView, public Tomahawk::ViewPage
 {
@@ -74,7 +75,7 @@ private slots:
 private:
     AlbumModel* m_model;
     AlbumProxyModel* m_proxyModel;
-//    PlaylistItemDelegate* m_delegate;
+    LoadingSpinner* m_loadingSpinner;
 
     QTimer m_timer;
 };

src/libtomahawk/playlist/dynamic/widgets/LoadingSpinner.cpp

@@ -1,6 +1,6 @@
 /* === This file is part of Tomahawk Player - <http://tomahawk-player.org> ===
  *
- *   Copyright 2010-2011, Christian Muehlhaeuser <muesli@tomahawk-player.org>
+ *   Copyright 2010 Leo Franchi <lfranchi@kde.org>
  *
  *   Tomahawk is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -58,6 +58,9 @@ LoadingSpinner::~LoadingSpinner()
 void
 LoadingSpinner::fadeIn()
 {
+    if ( isVisible() )
+        return;
+
     show();
 
     m_anim->start();
@@ -71,6 +74,9 @@ LoadingSpinner::fadeIn()
 void
 LoadingSpinner::fadeOut()
 {
+    if ( !isVisible() )
+        return;
+
     m_showHide->setDirection( QTimeLine::Backward );
 
     if ( m_showHide->state() != QTimeLine::Running )

src/libtomahawk/playlist/dynamic/widgets/LoadingSpinner.h

@@ -1,6 +1,6 @@
 /* === This file is part of Tomahawk Player - <http://tomahawk-player.org> ===
  *
- *   Copyright 2010-2011, Christian Muehlhaeuser <muesli@tomahawk-player.org>
+ *   Copyright 2010 Leo Franchi <lfranchi@kde.org>
  *
  *   Tomahawk is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by