> On May 18, 2011, 6:52 p.m., Ralf Engels wrote: > > Hi, > > nice, but I would not know what do do with it. > > It seems like a cover-bling applet (which by the way currently crashes for > > me) > > > > Could you maybe make a "cover-view" applet that can be switched between the > > different views?
Hi, It's definitely a good idea : a single applet enabling to toggle either grid or flow thing. But now that I think about it, it might be a bit of a hassle...As multiple QObject inheritance does not seem to be allowed, this would imply merging both main applet sources. Not sure we'd like to do that, actually. What's your opinion ? - Manu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100841/#review3388 ----------------------------------------------------------- On March 11, 2011, 9:47 p.m., Manu Wagner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100841/ > ----------------------------------------------------------- > > (Updated March 11, 2011, 9:47 p.m.) > > > Review request for Amarok. > > > Summary > ------- > > Added a new context applet, called CoverGrid. > It displays all albums of the collection by their cover art in a grid view. > Tooltips display album & artist names, mouse double click appends album to > playlist > > > Diffs > ----- > > src/context/applets/covergrid/AlbumItem.h PRE-CREATION > src/context/applets/covergrid/AlbumItem.cpp PRE-CREATION > src/context/applets/covergrid/CMakeLists.txt PRE-CREATION > src/context/applets/covergrid/CoverGridApplet.h PRE-CREATION > src/context/applets/covergrid/CoverGridApplet.cpp PRE-CREATION > src/context/applets/covergrid/amarok-context-applet-covergrid.desktop > PRE-CREATION > src/context/applets/covergrid/covergridSettings.ui PRE-CREATION > src/context/applets/covergrid/defaultcover.png PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/100841/diff > > > Testing > ------- > > basic functionalities seem to work > > > Screenshots > ----------- > > new applet in action > http://git.reviewboard.kde.org/r/100841/s/96/ > > > Thanks, > > Manu > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
