extragear/multimedia/amarok/src

Andrew Turner andrewturner512 at googlemail.com
Wed Dec 13 17:58:51 UTC 2006


http://www.srcf.ucam.org/~ajt60/amarok/covercollection.png

On 13/12/06, Seb Ruiz <me at sebruiz.net> wrote:
> awesome. anybody want to send me a screen shot of this baby?
>
> cheers,
> seb
>
> On 13/12/06, Andrew Turner <andrewturner512 at googlemail.com> wrote:
> > Fixed in 613259. Wouldn't want to spoil your holiday.
> >
> > andrewt512
> >
> > On 13/12/06, Seb Ruiz <me at sebruiz.net> wrote:
> > > Should really use a query builder statement here - raw sql statements
> > > outside of the collectiondb is bad! (i know, cat calling the kettle
> > > black :)
> > >
> > > cya!
> > >
> > > seb, in turkey!
> > >
> > > On 12/12/06, Mark Kretschmann <markey at web.de> wrote:
> > > > SVN commit 612787 by markey:
> > > >
> > > > Show album covers in collection browser. Experimental patch by Trever Fischer <wm161 at wm161.net>.
> > > >
> > > > I think this is pretty cool. Maybe we should limit size of the images though?
> > > >
> > > > BUG: 91044
> > > > CCMAIL: amarok at kde.org
> > > >
> > > >
> > > >  M  +19 -0     collectionbrowser.cpp
> > > >  M  +2 -0      collectionbrowser.h
> > > >
> > > >
> > > > --- trunk/extragear/multimedia/amarok/src/collectionbrowser.cpp #612786:612787
> > > > @@ -1490,7 +1490,26 @@
> > > >     }
> > > >  }
> > > >
> > > > +void
> > > > +CollectionItem::setPixmap(int column, const QPixmap & pix)
> > > > +{
> > > > +    if ( m_cat == IdAlbum ) {
> > > > +        QString album = text(0);
> > > > +        QStringList values =
> > > > +            CollectionDB::instance()->query ( QString (
> > > > +                "SELECT DISTINCT artist.name FROM artist, album, tags "
> > > > +                "WHERE artist.id = tags.artist AND tags.album = album.id "
> > > > +                "AND album.name = '%1';" )
> > > > +                .arg( CollectionDB::instance()->escapeString( album ) ) );
> > > > +        if ( !values.isEmpty() )
> > > > +            QListViewItem::setPixmap( column, QPixmap( CollectionDB::instance()->albumImage( values[0], album ) ) );
> > > > +        else
> > > > +            QListViewItem::setPixmap( column, QPixmap( CollectionDB::instance()->notAvailCover() ) );
> > > > +    } else
> > > > +        QListViewItem::setPixmap( column, pix );
> > > > +}
> > > >
> > > > +
> > > >  void
> > > >  CollectionView::fetchCover() //SLOT
> > > >  {
> > > > --- trunk/extragear/multimedia/amarok/src/collectionbrowser.h #612786:612787
> > > > @@ -165,6 +165,8 @@
> > > >         inline bool isUnknown() {return m_isUnknown;}
> > > >         inline bool isSampler() {return m_isSampler;}
> > > >
> > > > +        virtual void setPixmap(int column, const QPixmap & pix);
> > > > +
> > > >         static QPixmap *star();
> > > >         static QPixmap *smallStar();
> > > >
> > > > _______________________________________________
> > > > Amarok mailing list
> > > > Amarok at kde.org
> > > > https://mail.kde.org/mailman/listinfo/amarok
> > > >
> > >
> > >
> > > --
> > > http://www.sebruiz.net/
> > > _______________________________________________
> > > Amarok mailing list
> > > Amarok at kde.org
> > > https://mail.kde.org/mailman/listinfo/amarok
> > >
> > _______________________________________________
> > Amarok mailing list
> > Amarok at kde.org
> > https://mail.kde.org/mailman/listinfo/amarok
> >
>
>
> --
> http://www.sebruiz.net/
> _______________________________________________
> Amarok mailing list
> Amarok at kde.org
> https://mail.kde.org/mailman/listinfo/amarok
>



More information about the Amarok mailing list