MC-5383: Option Menus, Heading Inline Editor & tinyMCE Inline Editor Are Cut Off on Full Screen View

Change wysiwygfactory signature to accommodate new stageId primitive param

Author: danmooney2

app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/banner/preview.js

@@ -177,12 +177,13 @@ export default class Preview extends BasePreview {
         config.adapter.settings.fixed_toolbar_container = "#" + this.parent.id + " .pagebuilder-banner-text-content";
 
         WysiwygFactory(
-            this.parent,
+            this.parent.id,
             element.id,
             this.config.name,
             config,
             this.parent.dataStore,
             "message",
+            this.parent.stageId,
         ).then((wysiwyg: WysiwygInterface): void => {
             this.wysiwyg = wysiwyg;
         });

app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/slide/preview.js

@@ -268,12 +268,13 @@ export default class Preview extends BasePreview {
         }
 
         return WysiwygFactory(
-            this.parent,
+            this.parent.id,
             this.element.id,
             this.config.name,
             wysiwygConfig,
             this.parent.dataStore,
             "content",
+            this.parent.stageId,
         ).then((wysiwyg: WysiwygInterface): void => {
             this.wysiwyg = wysiwyg;
         });

app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/text/preview.js

@@ -73,12 +73,13 @@ export default class Preview extends BasePreview {
         wysiwygConfig.adapter.settings.auto_focus = this.parent.dropped ? element.id : null;
 
         WysiwygFactory(
-            this.parent,
+            this.parent.id,
             element.id,
             this.config.name,
             wysiwygConfig,
             this.parent.dataStore,
             "content",
+            this.parent.stageId,
         ).then((wysiwyg: WysiwygInterface): void => {
             this.wysiwyg = wysiwyg;
         });

app/code/Magento/PageBuilder/view/adminhtml/web/js/wysiwyg/factory.js

@@ -9,24 +9,25 @@ import {AdditionalDataConfigInterface} from "../content-type-config";
 import DataStore from "../data-store";
 import Wysiwyg from "./tinymce4";
 import WysiwygInterface, {WysiwygConstructorInterface} from "./wysiwyg-interface";
-import ContentTypeInterface from "../content-type.d";
 
 /**
- * @param {ContentTypeInterface} contentType The content type in the registry.
+ * @param {String} contentTypeId The ID in the registry of the content type.
  * @param {String} elementId The ID of the editor element in the DOM.
  * @param {String} contentTypeName The type of content type this editor will be used in. E.g. "banner".
  * @param {AdditionalDataConfigInterface} config The configuration for the wysiwyg.
  * @param {DataStore} dataStore The datastore to store the content in.
  * @param {String} fieldName The key in the provided datastore to set the data.
+ * @param {String} stageId The ID in the registry of the stage containing the content type.
  * @returns {Wysiwyg}
  */
 export default function create(
-    contentType: ContentTypeInterface,
+    contentTypeId: string,
     elementId: string,
     contentTypeName: string,
     config: AdditionalDataConfigInterface,
     dataStore: DataStore,
     fieldName: string,
+    stageId: string,
 ): Promise<WysiwygInterface> {
     config = $.extend(true, {}, config);
 
@@ -41,7 +42,7 @@ export default function create(
                         (ConfigModifierType: any) => {
                             const modifier = new ConfigModifierType();
                             // Allow dynamic settings to be set before editor is initialized
-                            modifier.modify(contentType.id, config);
+                            modifier.modify(contentTypeId, config);
                             configResolve();
                         },
                     );
@@ -52,12 +53,12 @@ export default function create(
             }).then(() => {
                 // Instantiate the component
                 const wysiwyg = new WysiwygInstance(
-                    contentType.id,
+                    contentTypeId,
                     elementId,
                     config,
                     dataStore,
                     fieldName,
-                    contentType.stageId
+                    stageId,
                 );
 
                 if (config.adapter_config.component_initializers

app/code/Magento/PageBuilder/view/adminhtml/web/ts/js/content-type/banner/preview.ts

@@ -34,7 +34,7 @@ export interface WysiwygConstructorInterface {
  */
 export interface WysiwygInterface {
     contentTypeId: string;
-    stageId: string,
+    stageId: string;
     elementId: string;
     config: AdditionalDataConfigInterface;