From 3edd24d6f8bff362f4176f6af9e8b97b6dacc8e0 Mon Sep 17 00:00:00 2001 From: tmusial99 Date: Fri, 7 Feb 2025 17:59:53 +0100 Subject: [PATCH] PLATFORM-10494 | updateState for toolbar dialogs with position `above` --- src/ui/ve.ui.Toolbar.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/ui/ve.ui.Toolbar.js b/src/ui/ve.ui.Toolbar.js index 900615a81c..b8eaa45b74 100644 --- a/src/ui/ve.ui.Toolbar.js +++ b/src/ui/ve.ui.Toolbar.js @@ -118,7 +118,9 @@ ve.ui.Toolbar.prototype.setup = function ( groups, surface ) { opening: 'onInspectorOrDialogOpeningOrClosing', closing: 'onInspectorOrDialogOpeningOrClosing' } ); - this.getSurface().getToolbarDialogs().connect( this, { + // FANDOM Change - for some reason by default the function returns only toolbar dialogs with position 'side' + // while all of toolbar dialogs have default position 'above' + this.getSurface().getToolbarDialogs('above').connect( this, { opening: 'onInspectorOrDialogOpeningOrClosing', closing: 'onInspectorOrDialogOpeningOrClosing' } ); @@ -213,7 +215,9 @@ ve.ui.Toolbar.prototype.updateToolState = function () { const activeDialogs = [ this.surface.getDialogs(), this.surface.getContext().getInspectors(), - this.surface.getToolbarDialogs() + // FANDOM Change - for some reason by default the function returns only toolbar dialogs with position 'side' + // while all of toolbar dialogs have default position 'above' + this.surface.getToolbarDialogs('above') ].map( ( windowManager ) => { if ( windowManager.getCurrentWindow() ) { return windowManager.getCurrentWindow().constructor.static.name;