public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pdm-devel] [PATCH datacenter-manager] ui: use RenderFn instead of TextRenderFn
@ 2025-12-11 10:10 Shannon Sterz
  2025-12-11 10:33 ` Dominik Csapak
  0 siblings, 1 reply; 2+ messages in thread
From: Shannon Sterz @ 2025-12-11 10:10 UTC (permalink / raw)
  To: pdm-devel

TextRenderFn has been removed from proxmox-yew-widget-toolkit
recently. use the generic RenderFn instead.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---

note that this should only be applied once proxmox-yew-widget-toolkit
has been bumped. this isn't yet an issue, but i didn't find a patch for
this on list yet. hence, i'm sending this now so that it isn't
forgotten.

 ui/src/main.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/ui/src/main.rs b/ui/src/main.rs
index 5f859db..2bd900e 100644
--- a/ui/src/main.rs
+++ b/ui/src/main.rs
@@ -7,7 +7,7 @@ use web_sys::HtmlElement;
 use yew::prelude::*;

 use pwt::prelude::*;
-use pwt::props::TextRenderFn;
+use pwt::props::RenderFn;
 use pwt::state::{Loader, PersistentState, SharedStateObserver};
 use pwt::widget::{Column, DesktopApp, Dialog, Mask};
 use pwt::AsyncPool;
@@ -346,9 +346,7 @@ impl Component for DatacenterManagerApp {
                 </ContextProvider<RemoteList>>
             </ContextProvider<SearchProvider>>
         })
-        .catalog_url_builder(TextRenderFn::new(|lang| {
-            format!("locale/catalog-{lang}.mo")
-        }))
+        .catalog_url_builder(RenderFn::new(|lang| format!("locale/catalog-{lang}.mo")))
         .into()
     }
 }
--
2.47.3



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [pdm-devel] [PATCH datacenter-manager] ui: use RenderFn instead of TextRenderFn
  2025-12-11 10:10 [pdm-devel] [PATCH datacenter-manager] ui: use RenderFn instead of TextRenderFn Shannon Sterz
@ 2025-12-11 10:33 ` Dominik Csapak
  0 siblings, 0 replies; 2+ messages in thread
From: Dominik Csapak @ 2025-12-11 10:33 UTC (permalink / raw)
  To: Proxmox Datacenter Manager development discussion, Shannon Sterz

thanks for sending this, wanted to do later, but did not get around to 
it yet!

Consider this

Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

(I'm purposely not applying this right now since pwt is not yet bumped)


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-12-11 10:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-12-11 10:10 [pdm-devel] [PATCH datacenter-manager] ui: use RenderFn instead of TextRenderFn Shannon Sterz
2025-12-11 10:33 ` Dominik Csapak

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal