From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id 80CB91FF165 for ; Thu, 28 Aug 2025 10:54:54 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 37F46B105; Thu, 28 Aug 2025 10:55:03 +0200 (CEST) Message-ID: <25effa68-2674-4aa9-94e0-b5e43ec9831c@proxmox.com> Date: Thu, 28 Aug 2025 10:54:27 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Beta To: Lukas Wagner , Proxmox Datacenter Manager development discussion References: <20250826123336.3970108-1-d.csapak@proxmox.com> <20250826123336.3970108-6-d.csapak@proxmox.com> Content-Language: en-US From: Dominik Csapak In-Reply-To: X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1756371261627 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.021 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 RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_RPBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_SAFE_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: pdm-devel-bounces@lists.proxmox.com Sender: "pdm-devel" On 8/27/25 11:37 AM, Lukas Wagner wrote: > 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? i get what you mean, and yeah maybe an icon to click on would be ok (a search icon for example) though i guess that won't work on all panels (e.g. in the remote panel i can't think of a good place right now) what i want to do later is to include a settings panel for the search bar, similar to what other tools (e.g. gmail do) that lets you configure the search parameters that get filled in the text box this should improve discoverability very much. i didn't do it yet simply because of time constraints > > 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