Review Request: Plasmate:add a new label at the startpage

Giorgos Tsiapaliwkas terietor at gmail.com
Sat Nov 12 19:29:42 UTC 2011


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

(Updated Nov. 12, 2011, 7:29 p.m.)


Review request for Plasma.


Changes
-------

I updated the diff according to Aaron's observations.

I noticed that plasmate even with this patch that it doesn't handle the corrupted metadata.desktop files.
Do you want me to write another patch for that?(of course it will an other review)

thanks


Description
-------

Hi,

in order someone to load a project using the plasmate,he has to give a path like this,"$my_plasmoid/$my_plasmoid/contents".
If someone wasn't aware of that he would have an issue.So i add a label which will inform the user about that.

thanks in advance


Diffs (updated)
-----

  startpage.cpp 329355c 
  startpage.ui 942e6ab 

Diff: http://git.reviewboard.kde.org/r/102991/diff/diff


Testing
-------

it's ok


Thanks,

Giorgos Tsiapaliwkas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111112/76fa501d/attachment.html>


More information about the Plasma-devel mailing list