Hi ! Just, a small reminder about that patch. :-)
I read in an another thread that you want to change the calendar popup and the tootip, so maybe that patch is not necessary anymore. But please in the new solution, keep the seconds visibles :-) Thanks. Luc. 2011/2/5 Iamluc <l...@vieillescazes.net> > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6336/ > Review request for Plasma. > By Iamluc. > > *Updated Feb. 5, 2011, 2:26 p.m.* > Changes > > Hi, > > I followed Aaron comments : > > * rename ClockTimer to something clearer -> ClockToolTip? > ---> Done > > * have ClockToolTip subclass both QObject and Plasma::ToolTipContent > --> Done > > * move updateToolTip into ClockToolTip, renamed as dataUpdated(const QString > &source, const Plasma::DataEngine &data) ;) > --> Done, and move function addTzToTipText() too. > > * connect to the dataengine on creation of ClockToolTip > --> Done > > * create a ClockToolTip object when toolTipAboutToShow is called > * delete it in toolTipHidden > --> Both done > > * pass in the ClockApplet as the parent to ClockToolTip and then in > ClockToolTip::dataUpdated, set the mainText and then call > Plasma::ToolTipManager::self()->setContent(clockApplet, this) > --> Done > > Description > > Hi, > > When using a clock applet, seconds are often not visibles. But sometimes you > need them. > This patch shows them in the tooltip of the applet. > > Analog-clock has been updated to refresh the tooltip every seconds. > If this change is accepted, I could change digital-clock > > Thanks ! > > Testing > > It Works on plasmoidviewer and with a real session > > Diffs (updated) > > - /trunk/KDE/kdebase/workspace/libs/plasmaclock/CMakeLists.txt > (1217808) > - /trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.h (1217808) > - /trunk/KDE/kdebase/workspace/libs/plasmaclock/clockapplet.cpp > (1217808) > - /trunk/KDE/kdebase/workspace/libs/plasmaclock/clocktooltip.h > (PRE-CREATION) > - /trunk/KDE/kdebase/workspace/libs/plasmaclock/clocktooltip.cpp > (PRE-CREATION) > > View Diff <http://svn.reviewboard.kde.org/r/6336/diff/> >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
