From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id D173E1FF16B for ; Tue, 9 Sep 2025 09:38:04 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id B9D6AA4C; Tue, 9 Sep 2025 09:38:06 +0200 (CEST) From: Dominik Csapak To: pdm-devel@lists.proxmox.com Date: Tue, 9 Sep 2025 09:37:56 +0200 Message-ID: <20250909073801.661423-2-d.csapak@proxmox.com> X-Mailer: git-send-email 2.47.2 In-Reply-To: <20250909073801.661423-1-d.csapak@proxmox.com> References: <20250909073801.661423-1-d.csapak@proxmox.com> MIME-Version: 1.0 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.023 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pdm-devel] [PATCH datacenter-manager 2/2] ui: dashboard: render last refresh date/time instead of a seconds counter X-BeenThere: pdm-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Datacenter Manager development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox Datacenter Manager development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" this is less jarring, since it does not change the content of the page as often. Since we now only have to do things when we actually reload, we can use the actual configured interval for the internal one, and don't have to check every second if that elapsed. Signed-off-by: Dominik Csapak --- ui/src/dashboard/mod.rs | 4 +-- ui/src/dashboard/status_row.rs | 47 +++++++++++++--------------------- 2 files changed, 20 insertions(+), 31 deletions(-) diff --git a/ui/src/dashboard/mod.rs b/ui/src/dashboard/mod.rs index 8d68dc2..11b4e2d 100644 --- a/ui/src/dashboard/mod.rs +++ b/ui/src/dashboard/mod.rs @@ -71,7 +71,7 @@ pub const DEFAULT_MAX_AGE_S: u64 = 30; /// The default refresh interval, we poll more frequently than the default max-age to quicker show /// any new data that was gathered either by the backend polling tasks or by a manual update /// triggered by another user. -pub const DEFAULT_REFRESH_INTERVAL_S: u64 = 10; +pub const DEFAULT_REFRESH_INTERVAL_S: u32 = 10; /// The default hours to show for task summaries. Use 2 days to ensure that all tasks from yesterday /// are included independent from the time a user checks the dashboard on the current day. @@ -96,7 +96,7 @@ impl Default for Dashboard { #[serde(rename_all = "kebab-case")] pub struct DashboardConfig { #[serde(skip_serializing_if = "Option::is_none")] - refresh_interval: Option, + refresh_interval: Option, #[serde(skip_serializing_if = "Option::is_none")] max_age: Option, #[serde(skip_serializing_if = "Option::is_none")] diff --git a/ui/src/dashboard/status_row.rs b/ui/src/dashboard/status_row.rs index 85048fe..5e37741 100644 --- a/ui/src/dashboard/status_row.rs +++ b/ui/src/dashboard/status_row.rs @@ -1,5 +1,4 @@ use gloo_timers::callback::Interval; -use js_sys::Date; use yew::{Component, Properties}; use pwt::prelude::*; @@ -9,13 +8,13 @@ use pwt::{ }; use pwt_macros::widget; -use proxmox_yew_comp::utils::format_duration_human; +use proxmox_yew_comp::utils::render_epoch; #[widget(comp=PdmDashboardStatusRow)] #[derive(Properties, PartialEq, Clone)] pub struct DashboardStatusRow { last_refresh: Option, - reload_interval_s: u64, + reload_interval_s: u32, on_reload: Callback, @@ -25,7 +24,7 @@ pub struct DashboardStatusRow { impl DashboardStatusRow { pub fn new( last_refresh: Option, - reload_interval_s: u64, + reload_interval_s: u32, on_reload: impl Into>, on_settings_click: impl Into>, ) -> Self { @@ -41,24 +40,24 @@ impl DashboardStatusRow { pub enum Msg { /// The bool denotes if the reload comes from the click or the timer. Reload(bool), - CheckReload, } #[doc(hidden)] pub struct PdmDashboardStatusRow { - _interval: Option, + _interval: Interval, } impl PdmDashboardStatusRow { - fn update_interval(&mut self, ctx: &yew::Context) { + fn create_interval(ctx: &yew::Context) -> Interval { let link = ctx.link().clone(); - let _interval = ctx.props().last_refresh.map(|_| { - Interval::new(1000, move || { - link.send_message(Msg::CheckReload); - }) - }); + let _interval = Interval::new( + ctx.props().reload_interval_s.saturating_mul(1000), + move || { + link.send_message(Msg::Reload(false)); + }, + ); - self._interval = _interval; + _interval } } @@ -67,9 +66,9 @@ impl Component for PdmDashboardStatusRow { type Properties = DashboardStatusRow; fn create(ctx: &yew::Context) -> Self { - let mut this = Self { _interval: None }; - this.update_interval(ctx); - this + Self { + _interval: Self::create_interval(ctx), + } } fn update(&mut self, ctx: &Context, msg: Self::Message) -> bool { @@ -79,21 +78,11 @@ impl Component for PdmDashboardStatusRow { props.on_reload.emit(clicked); true } - Msg::CheckReload => match ctx.props().last_refresh { - Some(last_refresh) => { - let duration = Date::now() / 1000.0 - last_refresh; - if duration >= props.reload_interval_s as f64 { - ctx.link().send_message(Msg::Reload(false)); - } - true - } - None => false, - }, } } fn changed(&mut self, ctx: &Context, _old_props: &Self::Properties) -> bool { - self.update_interval(ctx); + self._interval = Self::create_interval(ctx); true } @@ -119,8 +108,8 @@ impl Component for PdmDashboardStatusRow { ) .with_child(Container::new().with_child(match ctx.props().last_refresh { Some(last_refresh) => { - let duration = Date::now() / 1000.0 - last_refresh; - tr!("Last refreshed: {0} ago", format_duration_human(duration)) + let date = render_epoch(last_refresh as i64); + tr!("Last refresh: {0}", date) } None => tr!("Now refreshing"), })) -- 2.47.2 _______________________________________________ pdm-devel mailing list pdm-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel