Merge pull request #248 from lvcabral/fix/menu-alignment

Fixed: The submenus are not aligned with the title menu #247
AlexTorresDev · Dec 19, 2023 · 1c62abe · 1c62abe
2 parents 338f1b6 + 2a7ffc5
commit 1c62abe
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/menubar/index.ts b/src/menubar/index.ts
@@ -1004,15 +1004,15 @@ export class MenuBar extends Disposable {
 		const actualMenuIndex = menuIndex >= this.numMenusShown ? MenuBar.OVERFLOW_INDEX : menuIndex
 		const customMenu = actualMenuIndex === MenuBar.OVERFLOW_INDEX ? this.overflowMenu : this.menus[actualMenuIndex]
 
-		if (!customMenu.actions || !customMenu.buttonElement || !customMenu.titleElement) {
+		if (!customMenu.actions || !customMenu.buttonElement) {
 			return
 		}
 
 		const menuHolder = $('.cet-menubar-menu-container', { title: '' })
 		customMenu.buttonElement.classList.add('open')
 
-		const titleBoundingRect = customMenu.titleElement.getBoundingClientRect()
-		const titleBoundingRectZoom = DOM.getDomNodeZoomLevel(customMenu.titleElement)
+		const titleBoundingRect = customMenu.buttonElement.getBoundingClientRect()
+		const titleBoundingRectZoom = DOM.getDomNodeZoomLevel(customMenu.buttonElement)
 
 		if (this.options.compactMode === Direction.Right) {
 			menuHolder.style.top = `${titleBoundingRect.top}px`