> On 2010-10-20 09:57:56, Ingomar Wesp wrote: > > Since the ToolTipContent is copied in ToolTipManager::setContent anyway, I > > wouldn't declare it as a member variable, but just allocate a > > ToolTipContent on the stack as needed. This way, you wouldn't have to > > introduce a new member variable and could keep the header unchanged. > > Otherwise, it looks good to me.
Sounds cool. I had modified the patch. Hope this is what meant by you. - Sujith ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5660/#review8253 ----------------------------------------------------------- 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