> On March 11, 2012, 10:36 p.m., Marco Martin wrote: > > looks fine, but i would prefer the thumbnail plugins to be configurable if > > more than images are going to be used > > Shantanu Tushar Jha wrote: > I didn't understand the "more than images are going to be used" part :( > > (KIO includes three plugins by default "directorythumbnail", > "imagethumbnail", "jpegthumbnail")
i mean, support for different types of thumbnail plugins configurable. but since they are generated in a quite lazy way (no preview is generated until asked for) you can go for it, then we'll see if it impacts performancein any way - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104229/#review11302 ----------------------------------------------------------- On March 11, 2012, 4:51 p.m., Shantanu Tushar Jha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104229/ > ----------------------------------------------------------- > > (Updated March 11, 2012, 4:51 p.m.) > > > Review request for Plasma and Marco Martin. > > > Description > ------- > > This patch makes two additions- > 1. Thumbnail size is hardcoded into metadatamodel, this patch adds a property > to make it configurable > 2. KIO Thumbnailer uses a default set of plugins (from global config, which > can't be set from PA) which doesn't include video thumbnailers for example. > This makes metadatamodel fail when its asked for a thumbnail for a video > file. This patch explicitly tells KIO to use all plugins. > > > Diffs > ----- > > components/metadatamodel/metadatamodel.h 85750fe > components/metadatamodel/metadatamodel.cpp 74ad09c > > Diff: http://git.reviewboard.kde.org/r/104229/diff/ > > > Testing > ------- > > Both features work when tested from QML and don't seem to have any > side-effects. > > > Thanks, > > Shantanu Tushar Jha > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
