----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104229/#review11302 -----------------------------------------------------------
looks fine, but i would prefer the thumbnail plugins to be configurable if more than images are going to be used - Marco Martin 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
