Review Request: Plasmate:images can now be added to projects

Giorgos Tsiapaliwkas terietor at gmail.com
Mon Sep 12 23:10:30 UTC 2011



> On Sept. 12, 2011, 12:56 p.m., David Jarvie wrote:
> > editors/editpage.cpp, lines 105-106
> > <http://git.reviewboard.kde.org/r/102290/diff/1/?file=31661#file31661line105>
> >
> >     KFileDialog::getOpenUrls() is a static function, so it doesn't use 'imageDialog' which you have just constructed. You need to remove the previous line and call KFileDialog::getOpenUrls() with suitable parameters.

thanks,nice catch.

Maybe can you tell me why the kfiledialog doesn't open in the $HOME directory?


- Giorgos


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102290/#review6441
-----------------------------------------------------------


On Aug. 10, 2011, 6:29 p.m., Giorgos Tsiapaliwkas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102290/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2011, 6:29 p.m.)
> 
> 
> Review request for Plasma and Aaron J. Seigo.
> 
> 
> Summary
> -------
> 
> hello,
> 
> without this patch a user cannot add an image with plasmate.
> 
> reproduce,go to files-images-new,the plasmate will open a text editor instead of a dialog,which(the dialog) is able to open an image.
> 
> With the patch a dialog will open asking for an image.
> 
> 
> Diffs
> -----
> 
>   editors/editpage.h 7b5dca3 
>   editors/editpage.cpp d4b0082 
>   packagemodel.cpp 8c0907a 
> 
> Diff: http://git.reviewboard.kde.org/r/102290/diff
> 
> 
> Testing
> -------
> 
> the patch is not ready yet,i have noted some questions.
> Also the plasmate tries to open the image with a text editor.This have to be fixed,but how?Should we make plasmate able to preview images?
> 
> In addition,when you add something in the list of files(using the different options provided by the files qdockwidget) it names it as "new".This has to be fixed and the plasmate has to show the real name of the file.(different request,i just want an approval for this patch).
> 
> 
> Thanks,
> 
> Giorgos
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20110912/b25a57d1/attachment-0001.html>


More information about the Plasma-devel mailing list