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 033631FF183 for ; Wed, 27 Aug 2025 11:37:44 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id ED27D15B94; Wed, 27 Aug 2025 11:37:44 +0200 (CEST) Mime-Version: 1.0 Date: Wed, 27 Aug 2025 11:37:41 +0200 Message-Id: From: "Lukas Wagner" To: "Proxmox Datacenter Manager development discussion" , "Dominik Csapak" X-Mailer: aerc 0.20.1-0-g2ecb8770224a References: <20250826123336.3970108-1-d.csapak@proxmox.com> <20250826123336.3970108-6-d.csapak@proxmox.com> In-Reply-To: <20250826123336.3970108-6-d.csapak@proxmox.com> X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1756287456373 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.028 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: Re: [pdm-devel] [PATCH datacenter-manager v3 5/9] ui: dashboard: remotes panel: open search on click 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 Cc: pdm-devel Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" I think this is a nice feature, but I think the discoverability is not too great here; it seems like a feature that users could easily miss if they don't read about it in the documentation or stumble across it by accident. I think the main issue is that the 'card' does not look like an interactive UI element at all; unless you hover it with the cursor you get no hint that it is indeed clickable. Maybe an actual button with some text or an icon (although I'm not too sure what the text should say or the icon should look like) would be better? What do you think? On Tue Aug 26, 2025 at 2:31 PM CEST, Dominik Csapak wrote: > using the SearchProvider, insert a search term into the search box to > find and show either all remotes (when healthy) or the failed ones. > > Signed-off-by: Dominik Csapak > --- > ui/src/dashboard/remote_panel.rs | 32 +++++++++++++++++++++++++++++++- > 1 file changed, 31 insertions(+), 1 deletion(-) > > diff --git a/ui/src/dashboard/remote_panel.rs b/ui/src/dashboard/remote_panel.rs > index 7471fb6..849b4d3 100644 > --- a/ui/src/dashboard/remote_panel.rs > +++ b/ui/src/dashboard/remote_panel.rs > @@ -1,5 +1,6 @@ > use std::rc::Rc; > > +use pdm_search::{Search, SearchTerm}; > use proxmox_yew_comp::Status; > use pwt::{ > css, > @@ -14,6 +15,8 @@ use yew::{ > > use pdm_api_types::resource::ResourcesStatus; > > +use crate::search_provider::get_search_provider; > + > #[derive(Properties, PartialEq)] > /// A panel for showing the overall remotes status > pub struct RemotePanel { > @@ -38,13 +41,20 @@ impl From for VNode { > struct PdmRemotePanel {} > > impl Component for PdmRemotePanel { > - type Message = &'static str; > + type Message = Search; > type Properties = RemotePanel; > > fn create(_ctx: &yew::Context) -> Self { > Self {} > } > > + fn update(&mut self, ctx: &Context, msg: Self::Message) -> bool { > + if let Some(search) = get_search_provider(ctx) { > + search.search(msg); > + } > + false > + } > + > fn view(&self, ctx: &yew::Context) -> yew::Html { > let props = ctx.props(); > if props.status.is_none() { > @@ -77,6 +87,13 @@ impl Component for PdmRemotePanel { > }; > Column::new() > .tabindex(if failure { 0 } else { -1 }) > + .onclick(ctx.link().callback(move |_| create_search_term(failure))) > + .onkeydown(ctx.link().batch_callback(move |event: KeyboardEvent| { > + match event.key().as_str() { > + "Enter" | " " => Some(create_search_term(failure)), > + _ => None, > + } > + })) > .padding(4) > .class(css::FlexFit) > .class(css::AlignItems::Center) > @@ -88,3 +105,16 @@ impl Component for PdmRemotePanel { > .into() > } > } > + > +fn create_search_term(failure: bool) -> Search { > + if failure { > + Search::with_terms(vec![ > + SearchTerm::new("remote").category(Some("type")), > + SearchTerm::new("offline").category(Some("status")), > + ]) > + } else { > + Search::with_terms(vec![SearchTerm::new("remote") > + .optional(true) > + .category(Some("type"))]) > + } > +} _______________________________________________ pdm-devel mailing list pdm-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel