----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5660/#review8275 -----------------------------------------------------------
I'm not the maintainer of the applet, so I'm sorry that I can only comment on the code itself, not on whether it should go into the applet. I noticed a few issues: - The includes are still in the header. Since you've removed the member variable, you can remove them as well. - The user visible text "File:" you're setting as main text needs to go through i18n. - You're setting the tooltip content twice (the first call to ToolTipManager::setContent is unnecessary IMHO). - Ingomar On 2010-10-20 15:47:47, Sujith H wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5660/ > ----------------------------------------------------------- > > (Updated 2010-10-20 15:47:47) > > > Review request for Plasma and Anne-Marie Mahfouf. > > > Summary > ------- > > Added the filename as tooltip for the frame applet. The filename will be > displayed as the > tooltip when the user moves the moves on the frame. > > > Diffs > ----- > > trunk/KDE/kdeplasma-addons/applets/frame/frame.cpp 1187544 > > Diff: http://svn.reviewboard.kde.org/r/5660/diff > > > Testing > ------- > > Tested on the set of pictures I had in my laptop. > > > Thanks, > > Sujith > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel