From kde-panel-devel Tue Apr 01 20:09:39 2008 From: "Ismael Asensio" Date: Tue, 01 Apr 2008 20:09:39 +0000 To: kde-panel-devel Subject: Re: [PATCH] tooltip: add update parametr to updateToolTip function Message-Id: <80c786080804011309v50ac4ebap1c8fafb093d4e462 () mail ! gmail ! com> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=120716457801499 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============1044346728==" --===============1044346728== Content-Type: multipart/alternative; boundary="----=_Part_16210_1042146.1207080579292" ------=_Part_16210_1042146.1207080579292 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline > > > > To make clear why I need this for digital clock: > > > > In digital clock we use Plasma::DataEngine and connectSource(...) with > > interval 1sec or 60secs (depending on user's configuration). > > DataEngine calls dataUpdated(...) every that interval and inside this > > function we update displayed time in panel + displayed time in tooltip > > - even if it's not displayed (tooltip). That causes a lot of unneeded > > updates when tooltip is not visible. > > > > Problem will be even bigger when we introduce multizones support (I > > have patch 90% ready). For each timezone we will use single query and > > we will add response to tooltip. That will cause horribly big amount > > on unneeded operations. > Sorry if I say something stupid here but to solve this issue couldn't the tooltip just get updated before it is shown (updateToolTip), that is, retrieving the time from all the required timezones and then, for the seconds, get synced with the seconds info provided by the plasmoid itself (If I understood it right, via data Updated). That way the "expensive" retrieval of data would be the same either the tooltip is shown or not. Maybe is too hackish that way? ------=_Part_16210_1042146.1207080579292 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline

> To make clear why I need this for digital clock:
>
> In digital clock we use Plasma::DataEngine and connectSource(...) with
> interval 1sec or 60secs (depending on user's configuration).
> DataEngine calls dataUpdated(...) every that interval and inside this
> function we update displayed time in panel + displayed time in tooltip
> - even if it's not displayed (tooltip). That causes a lot of unneeded
> updates when tooltip is not visible.
>
> Problem will be even bigger when we introduce multizones support (I
> have patch 90% ready). For each timezone we will use single query and
> we will add response to tooltip. That will cause horribly big amount
> on unneeded operations.

Sorry if I say something stupid here but to solve this issue couldn't the tooltip just get updated before it is shown (updateToolTip), that is, retrieving the time from all the required timezones and then, for the seconds, get synced with the seconds info provided by the plasmoid itself (If I understood it right, via data Updated).

That way the "expensive" retrieval of data would be the same either the tooltip is shown or not.
Maybe is too hackish that way?
------=_Part_16210_1042146.1207080579292-- --===============1044346728== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Panel-devel mailing list Panel-devel@kde.org https://mail.kde.org/mailman/listinfo/panel-devel --===============1044346728==--