extragear/multimedia/amarok/src
Nikolaj Hald Nielsen
nhnfreespirit at gmail.com
Wed Dec 13 18:25:34 UTC 2006
This absolutely rocks! Nice one!
- Nikolaj
On 12/13/06, Andrew Turner <andrewturner512 at googlemail.com> wrote:
> 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
> >
> _______________________________________________
> Amarok mailing list
> Amarok at kde.org
> https://mail.kde.org/mailman/listinfo/amarok
>
More information about the Amarok
mailing list