Servoy / svyNavigation

Navigation extension for Servoy applications
MIT License
0 stars 0 forks source link

template svyNavigationUX$SidenavMobile function onMenuItemSelected jump twice #10

Open juanetec opened 2 years ago

juanetec commented 2 years ago

Hi, using the new template svyNavigationUX$SidenavMobile the process onMenuItemSelected which is in the onMenuItemSelectedHandler is beeing call twice at that process. It give me a problem with the "elements.sidenav.open". If you use in a menu that not have children no problem, but if you have children it close the sidenav when you are clicking in the parent. In the svyNavigationUX$Sidenav template it don't have that problem I control the "elements.sidenav.open" at my extended onMenuItemSelected process.

`function onMenuItemSelectedHandler(menuItemId, event) {

if (onMenuItemSelected(menuItemId, event) === true) {
    elements.sidenav.open = false;
}

if(menuItemId === DEFAULT_SIDENAV_ACTIONS.LOGOUT){
    onLogout();
    return false;
}

if (onMenuItemSelected(menuItemId, event) === false) {
    return false;
}

// form to navigate too
var formName = getMenuItemFormName(menuItemId)
var form = forms[formName];

// open the selected navigation item
if (menuItemId && formName && form) {
    var menuItem = elements.sidenav.getMenuItem(menuItemId);

    var item = new scopes.svyNavigation.NavigationItem(formName, menuItem.text);
    return scopes.svyNavigation.open(item);
}

return true;

}` image

Un Saludo

Juan Ramírez

paronne commented 2 years ago

Thanks for your feeback Juan. Created follow up case https://support.servoy.com/browse/SVYX-463