----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106680/#review21135 -----------------------------------------------------------
plasmate/editors/editpage.cpp <http://git.reviewboard.kde.org/r/106680/#comment16556> I think that we need the entire "[plasmate]/themeImageDialog/$foo because we do if (mimetype == i.key()) { ... }. No? - Giorgos Tsiapaliokas On Oct. 4, 2012, 10:49 a.m., Giorgos Tsiapaliokas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106680/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2012, 10:49 a.m.) > > > Review request for Plasma. > > > Description > ------- > > create a new theme package->click on the new > > a file dialog should appear but instead a simple edit box appears requesting > a new filename. > > This patch solves the issue > > > Diffs > ----- > > plasmate/editors/editpage.h 8cc5fab > plasmate/editors/editpage.cpp 91f6bce > plasmate/packagemodel.cpp 67cc0f4 > > Diff: http://git.reviewboard.kde.org/r/106680/diff/ > > > Testing > ------- > > > Thanks, > > Giorgos Tsiapaliokas > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
