[pdm-devel] [PATCH datacenter-manager v2 0/1] enable top nav bar's documentation button

[pdm-devel] [PATCH datacenter-manager v2 0/1] enable top nav bar's documentation button

[Shannon Sterz]

this patch enables pdm's documentation buton in the top nav bar and
opens the beta documentation when clicked.

changes since v1 (thanks @ Dominik Csapak):

- use `on_activate` instead of `onclick`
- use `open_with_url_and_target` instead of creating an a tag

Shannon Sterz (1):
  ui: enable the top navigation bar's documentation button

 ui/src/top_nav_bar.rs | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

--
2.47.3



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

[pdm-devel] [PATCH datacenter-manager v2 1/1] ui: enable the top navigation bar's documentation button

[Shannon Sterz]

and make it open the beta documentation in a new tab for now.

Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---
 ui/src/top_nav_bar.rs | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/ui/src/top_nav_bar.rs b/ui/src/top_nav_bar.rs
index 9b1e37e..9aeeb73 100644
--- a/ui/src/top_nav_bar.rs
+++ b/ui/src/top_nav_bar.rs
@@ -3,6 +3,7 @@ use std::rc::Rc;
 use anyhow::Error;
 use pwt::css::ColorScheme;
 use serde::Deserialize;
+use wasm_bindgen::UnwrapThrowExt;
 
 use pwt::prelude::*;
 use pwt::widget::menu::{Menu, MenuButton, MenuEntry, MenuEvent, MenuItem};
@@ -181,10 +182,17 @@ impl Component for PdmTopNavBar {
                 Tooltip::new(
                     Button::new(tr!("Documentation"))
                         .icon_class("fa fa-book")
-                        .disabled(true)
-                        .class(ColorScheme::Neutral),
+                        .class(ColorScheme::Neutral)
+                        .on_activate(|_| {
+                            gloo_utils::window()
+                                .open_with_url_and_target(
+                                    "https://pve.proxmox.com/wiki/Proxmox_Datacenter_Manager_Beta_Documentation",
+                                    "_blank"
+                                )
+                                .expect_throw("could not open documentation in a new window");
+                        }),
                 )
-                .tip(tr!("Coming soon")),
+                .tip(tr!("Open the Beta documentation in a new tab.")),
             );
 
         if let Some(username) = &props.username {
-- 
2.47.3



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

[pdm-devel] applied: [PATCH datacenter-manager v2 0/1] enable top nav bar's documentation button

[Thomas Lamprecht]

On Tue, 16 Sep 2025 16:48:03 +0200, Shannon Sterz wrote:
> this patch enables pdm's documentation buton in the top nav bar and
> opens the beta documentation when clicked.
> 
> changes since v1 (thanks @ Dominik Csapak):
> 
> - use `on_activate` instead of `onclick`
> - use `open_with_url_and_target` instead of creating an a tag
> 
> [...]

Applied, thanks!

[1/1] ui: enable the top navigation bar's documentation button
      commit: 84199f59ba88ac90e027e154b6e515088e281521


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