On Sunday 12 April 2009, Emdek wrote: > On Sunday 12-04-2009 18:18:01 Aaron J. Seigo wrote: > >> - show titles of windows when showing multiple previews; > > > > will this fit nicely? look nice? could we do it with QToolTips on the > > previews? > > I'm using it already in my applet and it looks good I think: > > http://kde-look.org/CONTENT/content-pre1/99737-1.png
and for really long titles? i suppose you just elide the text?
> And what about giving possibility to center tool tip over widget?
yes, this is fine imho.
> And still I don't know which object should emit these signals.
how about:
ToolTipSignaller *ToolTipContent::signallerFor(QGraphicsWidget *target)
{
ToolTipSignaller *signaller = d->signallers.value(target);
if (!signaller) {
signaller = new ToolTipSignaller(target);
d->signallers.insert(target, signaller);
}
return signaller;
}
the signals would belong to this ToolTipSignals class; in fact, that's all it
would contain: a handful of signals.
when ToolTipManager::currentWidget is changing, disconnect the old (if any)
signaller for the widget and connect the new signaller (if any) for the new
widget to signals from ToolTip. this would cause signals from ToolTip to end
up at the correct destination object, via the signaller.
then in ToolTipContentPrivate::onWidgetDestroyed() remove the ToolTipSignaller
*from the d->signallers collection.
how's that sound?
heck, we could even replace the fugly tooltipAboutToShow and
tooltipAboutToHide nonsense with this stuff .. why didn't i think of this last
year? *sigh*
> So now we need clear list of needed signals and then start to work. ;-)
hoverEnter(const KUrl &)
hoverLeave(const KUrl &)
activated(const KUrl &)
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Software
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
