]> git.sesse.net Git - vlc/blobdiff - modules/gui/qt4/dialogs/plugins.cpp
Qt: ExtensionsModel: use data abstraction in delegate
[vlc] / modules / gui / qt4 / dialogs / plugins.cpp
index 6bee71c38e0d437bc1f0dd3d81b5343ad88fcf89..701341f0acb1fafcb0eb11f123c907fb29f56aaa 100644 (file)
@@ -1,5 +1,5 @@
 /*****************************************************************************
- * plugins.hpp : Plug-ins and extensions listing
+ * plugins.cpp : Plug-ins and extensions listing
  ****************************************************************************
  * Copyright (C) 2008-2010 the VideoLAN team
  * $Id$
 
 #include "plugins.hpp"
 
-#include "util/customwidgets.hpp"
+#include "util/searchlineedit.hpp"
 #include "extensions_manager.hpp"
 
 #include <assert.h>
 
-//#include <vlc_modules.h>
+#include <vlc_modules.h>
 
 #include <QTreeWidget>
 #include <QStringList>
@@ -44,7 +44,6 @@
 #include <QLabel>
 #include <QVBoxLayout>
 #include <QComboBox>
-#include <QTextBrowser>
 #include <QHBoxLayout>
 #include <QVBoxLayout>
 #include <QSpacerItem>
 #include <QPainter>
 #include <QStyleOptionViewItem>
 #include <QKeyEvent>
+#include <QPushButton>
+#include <QPixmap>
+
+static QPixmap *loadPixmapFromData( char *, int size );
 
 
 PluginDialog::PluginDialog( intf_thread_t *_p_intf ) : QVLCFrame( _p_intf )
@@ -69,19 +72,21 @@ PluginDialog::PluginDialog( intf_thread_t *_p_intf ) : QVLCFrame( _p_intf )
 
     QDialogButtonBox *box = new QDialogButtonBox;
     QPushButton *okButton = new QPushButton( qtr( "&Close" ), this );
-    box->addButton( okButton, QDialogButtonBox::AcceptRole );
+    box->addButton( okButton, QDialogButtonBox::RejectRole );
     layout->addWidget( box );
     BUTTONACT( okButton, close() );
+    restoreWidgetPosition( "PluginsDialog", QSize( 435, 280 ) );
 }
 
 PluginDialog::~PluginDialog()
 {
+    saveWidgetPosition( "PluginsDialog" );
 }
 
 /* Plugins tab */
 
-PluginTab::PluginTab( intf_thread_t *p_intf )
-        : QVLCFrame( p_intf )
+PluginTab::PluginTab( intf_thread_t *p_intf_ )
+        : QVLCFrame( p_intf_ )
 {
     QGridLayout *layout = new QGridLayout( this );
 
@@ -120,16 +125,18 @@ PluginTab::PluginTab( intf_thread_t *p_intf )
             this, search( const QString& ) );
 
     setMinimumSize( 500, 300 );
-    readSettings( "Plugins", QSize( 540, 400 ) );
+    restoreWidgetPosition( "Plugins", QSize( 540, 400 ) );
 }
 
 inline void PluginTab::FillTree()
 {
-    module_t **p_list = module_list_get( NULL );
-    module_t *p_module;
+    size_t count;
+    module_t **p_list = module_list_get( &count );
 
-    for( unsigned int i = 0; (p_module = p_list[i] ) != NULL; i++ )
+    for( unsigned int i = 0; i < count; i++ )
     {
+        module_t *p_module = p_list[i];
+
         QStringList qs_item;
         qs_item << qfu( module_get_name( p_module, true ) )
                 << qfu( module_get_capability( p_module ) )
@@ -141,6 +148,7 @@ inline void PluginTab::FillTree()
         QTreeWidgetItem *item = new PluginTreeItem( qs_item );
         treePlugins->addTopLevelItem( item );
     }
+    module_list_free( p_list );
 }
 
 void PluginTab::search( const QString& qs )
@@ -158,28 +166,52 @@ void PluginTab::search( const QString& qs )
 
 PluginTab::~PluginTab()
 {
-    writeSettings( "Plugins" );
+    saveWidgetPosition( "Plugins" );
     getSettings()->setValue( "Plugins/Header-State",
                              treePlugins->header()->saveState() );
 }
 
+void PluginTab::keyPressEvent( QKeyEvent *keyEvent )
+{
+    if( keyEvent->key() == Qt::Key_Return ||
+        keyEvent->key() == Qt::Key_Enter )
+        keyEvent->accept();
+    else
+        keyEvent->ignore();
+}
+
 bool PluginTreeItem::operator< ( const QTreeWidgetItem & other ) const
 {
     int col = treeWidget()->sortColumn();
-    if( col == 2 )
+    if( col == PluginTab::SCORE )
         return text( col ).toInt() < other.text( col ).toInt();
+    else if ( col == PluginTab::CAPABILITY )
+    {
+        if ( text( PluginTab::CAPABILITY ) == other.text( PluginTab::CAPABILITY ) )
+            return text( PluginTab::NAME ) < other.text( PluginTab::NAME );
+        else
+            return text( PluginTab::CAPABILITY ) < other.text( PluginTab::CAPABILITY );
+    }
     return text( col ) < other.text( col );
 }
 
 /* Extensions tab */
-ExtensionTab::ExtensionTab( intf_thread_t *p_intf )
-        : QVLCFrame( p_intf )
+ExtensionTab::ExtensionTab( intf_thread_t *p_intf_ )
+        : QVLCFrame( p_intf_ )
 {
     // Layout
     QVBoxLayout *layout = new QVBoxLayout( this );
 
+    QLabel *notice = new QLabel( qtr("Get more extensions from")
+            + QString( " <a href=\"http://addons.videolan.org/\">"
+                       "addons.videolan.org</a>." ) );
+    notice->setOpenExternalLinks( true );
+    layout->addWidget( notice );
+
     // ListView
     extList = new QListView( this );
+    CONNECT( extList, activated( const QModelIndex& ),
+             this, moreInformation() );
     layout->addWidget( extList );
 
     // List item delegate
@@ -204,8 +236,7 @@ ExtensionTab::ExtensionTab( intf_thread_t *p_intf )
     butMoreInfo = new QPushButton( QIcon( ":/menu/info" ),
                                    qtr( "More information..." ),
                                    this );
-    CONNECT( butMoreInfo, clicked(),
-             this, moreInformation() );
+    CONNECT( butMoreInfo, clicked(), this, moreInformation() );
     hbox->addWidget( butMoreInfo );
 
     // Reload button
@@ -213,22 +244,48 @@ ExtensionTab::ExtensionTab( intf_thread_t *p_intf )
     QPushButton *reload = new QPushButton( QIcon( ":/update" ),
                                            qtr( "Reload extensions" ),
                                            this );
-    CONNECT( reload, clicked(),
-             EM, reloadExtensions() );
+    CONNECT( reload, clicked(), EM, reloadExtensions() );
+    CONNECT( reload, clicked(), this, updateButtons() );
+    CONNECT( extList->selectionModel(),
+             selectionChanged( const QItemSelection &, const QItemSelection & ),
+             this,
+             updateButtons() );
     hbox->addWidget( reload );
 
     // Add buttons hbox
     layout->addItem( hbox );
+    updateButtons();
 }
 
 ExtensionTab::~ExtensionTab()
 {
 }
 
+void ExtensionTab::updateButtons()
+{
+    butMoreInfo->setEnabled( extList->selectionModel()->hasSelection() );
+}
+
 // Do not close on ESC or ENTER
 void ExtensionTab::keyPressEvent( QKeyEvent *keyEvent )
 {
-    keyEvent->ignore();
+    if( keyEvent->key() == Qt::Key_Return ||
+        keyEvent->key() == Qt::Key_Enter )
+        keyEvent->accept();
+    else
+        keyEvent->ignore();
+}
+
+// Show more information
+void ExtensionTab::moreInformation()
+{
+    QModelIndex index = extList->selectionModel()->selectedIndexes().first();
+    ExtensionCopy *ext = (ExtensionCopy*) index.internalPointer();
+    if( !ext )
+        return;
+
+    ExtensionInfoDialog dlg( *ext, p_intf, this );
+    dlg.exec();
 }
 
 /* Safe copy of the extension_t struct */
@@ -239,14 +296,21 @@ public:
     {
         name = qfu( p_ext->psz_name );
         description = qfu( p_ext->psz_description );
+        shortdesc = qfu( p_ext->psz_shortdescription );
+        if( description.isEmpty() )
+            description = shortdesc;
+        if( shortdesc.isEmpty() && !description.isEmpty() )
+            shortdesc = description;
         title = qfu( p_ext->psz_title );
         author = qfu( p_ext->psz_author );
         version = qfu( p_ext->psz_version );
         url = qfu( p_ext->psz_url );
+        icon = loadPixmapFromData( p_ext->p_icondata, p_ext->i_icondata_size );
     }
-    ~ExtensionCopy() {}
+    ~ExtensionCopy() { delete icon; }
 
-    QString name, title, description, author, version, url;
+    QString name, title, description, shortdesc, author, version, url;
+    QPixmap *icon;
 };
 
 /* Extensions list model for the QListView */
@@ -264,6 +328,9 @@ ExtensionListModel::ExtensionListModel( QListView *view, intf_thread_t *intf )
 
 ExtensionListModel::~ExtensionListModel()
 {
+    // Clear extensions list
+    while( !extensions.isEmpty() )
+        delete extensions.takeLast();
 }
 
 void ExtensionListModel::updateList()
@@ -288,7 +355,7 @@ void ExtensionListModel::updateList()
     FOREACH_ARRAY( p_ext, p_mgr->extensions )
     {
         ext = new ExtensionCopy( p_ext );
-        extensions.push_back( ext );
+        extensions.append( ext );
     }
     FOREACH_END()
     vlc_mutex_unlock( &p_mgr->lock );
@@ -297,7 +364,7 @@ void ExtensionListModel::updateList()
     emit dataChanged( index( 0 ), index( rowCount() - 1 ) );
 }
 
-int ExtensionListModel::rowCount( const QModelIndex& parent ) const
+int ExtensionListModel::rowCount( const QModelIndex& ) const
 {
     int count = 0;
     ExtensionsManager *EM = ExtensionsManager::getInstance( p_intf );
@@ -320,17 +387,23 @@ QVariant ExtensionListModel::data( const QModelIndex& index, int role ) const
 
     switch( role )
     {
+    case Qt::DisplayRole:
+        return ((ExtensionCopy *)index.internalPointer())->title;
+    case Qt::DecorationRole:
+        return *((ExtensionCopy *)index.internalPointer())->icon;
+    case DescriptionRole:
+        return ((ExtensionCopy *)index.internalPointer())->shortdesc;
     default:
         return QVariant();
     }
 }
 
 QModelIndex ExtensionListModel::index( int row, int column,
-                                       const QModelIndex& parent ) const
+                                       const QModelIndex& ) const
 {
     if( column != 0 )
         return QModelIndex();
-    if( row < 0 || row >= extensions.size() )
+    if( row < 0 || row >= extensions.count() )
         return QModelIndex();
 
     return createIndex( row, 0, extensions.at( row ) );
@@ -352,11 +425,7 @@ void ExtensionItemDelegate::paint( QPainter *painter,
                                    const QStyleOptionViewItem &option,
                                    const QModelIndex &index ) const
 {
-    ExtensionCopy *ext = ( ExtensionCopy* ) index.internalPointer();
-    assert( ext != NULL );
-
     int width = option.rect.width();
-    int height = option.rect.height();
 
     // Pixmap: buffer where to draw
     QPixmap pix(option.rect.size());
@@ -384,24 +453,32 @@ void ExtensionItemDelegate::paint( QPainter *painter,
         pen.setBrush( option.palette.text() );
     }
     pixpaint->setPen( pen );
+    QFontMetrics metrics = option.fontMetrics;
+
+    // Icon
+    QPixmap icon = index.data( Qt::DecorationRole ).value<QPixmap>();
+    if( !icon.isNull() )
+    {
+        pixpaint->drawPixmap( 7, 7, 2*metrics.height(), 2*metrics.height(),
+                              icon );
+    }
 
     // Title: bold
+    pixpaint->setRenderHint( QPainter::TextAntialiasing );
     font.setBold( true );
     pixpaint->setFont( font );
-    pixpaint->drawText( QRect( 10, 5, width - 70, 20 ),
-                        Qt::AlignLeft, ext->title );
+    pixpaint->drawText( QRect( 17 + 2 * metrics.height(), 7,
+                               width - 40 - 2 * metrics.height(),
+                               metrics.height() ),
+                        Qt::AlignLeft, index.data( Qt::DisplayRole ).toString() );
 
     // Short description: normal
     font.setBold( false );
     pixpaint->setFont( font );
-    pixpaint->drawText( QRect( 10, 30, width - 40, 20 ),
-                        Qt::AlignLeft, ext->description );
-
-    // Version: italic
-    font.setItalic( true );
-    pixpaint->setFont( font );
-    pixpaint->drawText( QRect( width - 50, 5, 20, 20 ),
-                        Qt::AlignLeft, ext->version );
+    pixpaint->drawText( QRect( 17 + 2 * metrics.height(),
+                               7 + metrics.height(), width - 40,
+                               metrics.height() ),
+                        Qt::AlignLeft, index.data( ExtensionListModel::DescriptionRole ).toString() );
 
     // Flush paint operations
     delete pixpaint;
@@ -416,8 +493,110 @@ QSize ExtensionItemDelegate::sizeHint( const QStyleOptionViewItem &option,
     if (index.isValid() && index.column() == 0)
     {
         QFontMetrics metrics = option.fontMetrics;
-        return QSize( 200, 20 + 2 * metrics.height() );
+        return QSize( 200, 14 + 2 * metrics.height() );
     }
     else
         return QSize();
 }
+
+/* "More information" dialog */
+
+ExtensionInfoDialog::ExtensionInfoDialog( const ExtensionCopy& extension,
+                                          intf_thread_t *p_intf,
+                                          QWidget *parent )
+       : QVLCDialog( parent, p_intf )
+{
+    // Let's be a modal dialog
+    setWindowModality( Qt::WindowModal );
+
+    // Window title
+    setWindowTitle( qtr( "About" ) + " " + extension.title );
+
+    // Layout
+    QGridLayout *layout = new QGridLayout( this );
+
+    // Icon
+    QLabel *icon = new QLabel( this );
+    if( !extension.icon )
+    {
+        QPixmap pix( ":/logo/vlc48.png" );
+        icon->setPixmap( pix );
+    }
+    else
+    {
+        icon->setPixmap( *extension.icon );
+    }
+    icon->setAlignment( Qt::AlignCenter );
+    icon->setFixedSize( 48, 48 );
+    layout->addWidget( icon, 1, 0, 2, 1 );
+
+    // Title
+    QLabel *label = new QLabel( extension.title, this );
+    QFont font = label->font();
+    font.setBold( true );
+    font.setPointSizeF( font.pointSizeF() * 1.3f );
+    label->setFont( font );
+    layout->addWidget( label, 0, 0, 1, -1 );
+
+    // Version
+    label = new QLabel( "<b>" + qtr( "Version" ) + ":</b>", this );
+    layout->addWidget( label, 1, 1, 1, 1, Qt::AlignBottom );
+    label = new QLabel( extension.version, this );
+    layout->addWidget( label, 1, 2, 1, 2, Qt::AlignBottom );
+
+    // Author
+    label = new QLabel( "<b>" + qtr( "Author" ) + ":</b>", this );
+    layout->addWidget( label, 2, 1, 1, 1, Qt::AlignTop );
+    label = new QLabel( extension.author, this );
+    layout->addWidget( label, 2, 2, 1, 2, Qt::AlignTop );
+
+
+    // Description
+    label = new QLabel( this );
+    label->setText( extension.description );
+    label->setWordWrap( true );
+    label->setOpenExternalLinks( true );
+    layout->addWidget( label, 4, 0, 1, -1 );
+
+    // URL
+    label = new QLabel( "<b>" + qtr( "Website" ) + ":</b>", this );
+    layout->addWidget( label, 5, 0, 1, 2 );
+    label = new QLabel( QString("<a href=\"%1\">%2</a>")
+                        .arg( extension.url ).arg( extension.url )
+                        , this );
+    label->setOpenExternalLinks( true );
+    layout->addWidget( label, 5, 2, 1, -1 );
+
+    // Script file
+    label = new QLabel( "<b>" + qtr( "File" ) + ":</b>", this );
+    layout->addWidget( label, 6, 0, 1, 2 );
+    QLineEdit *line = new QLineEdit( extension.name, this );
+    line->setReadOnly( true );
+    layout->addWidget( line, 6, 2, 1, -1 );
+
+    // Close button
+    QDialogButtonBox *group = new QDialogButtonBox( this );
+    QPushButton *closeButton = new QPushButton( qtr( "&Close" ) );
+    group->addButton( closeButton, QDialogButtonBox::RejectRole );
+    BUTTONACT( closeButton, close() );
+
+    layout->addWidget( group, 7, 0, 1, -1 );
+
+    // Fix layout
+    layout->setColumnStretch( 2, 1 );
+    layout->setRowStretch( 4, 1 );
+    setMinimumSize( 450, 350 );
+}
+
+static QPixmap *loadPixmapFromData( char *data, int size )
+{
+    if( !data || size <= 0 )
+        return NULL;
+    QPixmap *pixmap = new QPixmap();
+    if( !pixmap->loadFromData( (const uchar*) data, (uint) size ) )
+    {
+        delete pixmap;
+        return NULL;
+    }
+    return pixmap;
+}