dbeaver/pro#7810 fix: improve submenu visibility handling for lazy-lo… (#3993)

* dbeaver/pro#7810 fix: improve submenu visibility handling for lazy-loaded items

* dbeaver/pro#7810 fix: return back isDisabled for handler

---------

Co-authored-by: Daria Marutkina <125263541+dariamarutkina@users.noreply.github.com>

Author: SychevAndrey

webapp/packages/core-ui/src/ContextMenu/SubMenuElement.tsx

@@ -68,8 +68,18 @@ export const SubMenuElement = observer<ISubMenuElementProps>(function SubMenuEle
 
   const handler = subMenuData.handler;
 
-  // TODO: it's better to remove this expensive check to allow lazy loading of menu items
-  const hidden = getComputed(() => subMenuData.items.every(item => item.hidden) || handler?.isHidden?.(subMenuData.context));
+  // NOTE: some menus rely on lazy loading. When items are not loaded yet `items` may be empty,
+  // but the submenu still must be rendered to allow opening it and triggering loaders.
+  const hidden = getComputed(() => {
+    if (handler?.isHidden?.(subMenuData.context)) {
+      return true;
+    }
+
+    const hasVisibleItems = subMenuData.items.some(item => !item.hidden);
+    const canLazyLoadItems = subMenuData.loaders.length > 0;
+
+    return !hasVisibleItems && !canLazyLoadItems;
+  });
   const IconComponent = handler?.iconComponent?.() ?? subMenu.iconComponent?.();
   const extraProps = handler?.getExtraProps?.() ?? (subMenu.getExtraProps?.() as any);
   // TODO: fix, this triggers `Cannot update a component (`SubMenuElement`) while rendering a different component`

webapp/packages/plugin-sql-generator/src/SqlGenerators/SqlGeneratorsBootstrap.ts

@@ -1,6 +1,6 @@
 /*
  * CloudBeaver - Cloud Database Manager
- * Copyright (C) 2020-2024 DBeaver Corp and others
+ * Copyright (C) 2020-2025 DBeaver Corp and others
  *
  * Licensed under the Apache License, Version 2.0.
  * you may not use this file except in compliance with the License.