Re: [pdm-devel] [PATCH proxmox-datacenter-manager 15/16] ui: sdn: add EvpnPanel to main menu

[Dominik Csapak <d.csapak@proxmox.com>]

i would it put above the remotes, since when one has many remotes, the 
evpn entry will be pushed downwards

we may want to redesign the navmenu in the future so that the list never
gets longer as the viewport/screen and only the remote list is
scrollable, but for now putting it above the remotes should be fine


On 8/27/25 1:40 PM, Stefan Hanreich wrote:
> Expose the new EVPN overview in the main menu. In the future this
> might move below a dedicated SDN top-level entry, but since we have
> only one view for now, display it as a top-level entry.
> 
> Signed-off-by: Stefan Hanreich <s.hanreich@proxmox.com>
> ---
>   ui/src/main_menu.rs | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/ui/src/main_menu.rs b/ui/src/main_menu.rs
> index 4f40d2c..490e1a1 100644
> --- a/ui/src/main_menu.rs
> +++ b/ui/src/main_menu.rs
> @@ -17,6 +17,7 @@ use proxmox_yew_comp::{NotesView, XTermJs};
>   use pdm_api_types::remotes::RemoteType;
>   
>   use crate::remotes::RemotesPanel;
> +use crate::sdn::evpn::EvpnPanel;
>   use crate::{
>       AccessControl, CertificatesPanel, Dashboard, RemoteList, ServerAdministration,
>       SystemConfiguration,
> @@ -286,6 +287,15 @@ impl Component for PdmMainMenu {
>               remote_submenu,
>           );
>   
> +        register_view(
> +            &mut menu,
> +            &mut content,
> +            tr!("EVPN"),
> +            "evpn",
> +            Some("fa fa-sitemap"),
> +            |_| EvpnPanel::new().into(),
> +        );
> +
>           let drawer = NavigationDrawer::new(menu)
>               .aria_label("Datacenter Manager")
>               .class("pwt-border-end")



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