* [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