Merge pull request #2375 from tf/iframe-size

Do not apply max-width to dynamically resized iframes

Author: tf

entry_types/scrolled/package/spec/contentElements/iframeEmbed/IframeEmbed-spec.js

@@ -0,0 +1,27 @@
+import React from 'react';
+import '@testing-library/jest-dom/extend-expect';
+
+import {renderInContentElement} from 'pageflow-scrolled/testHelpers';
+import {IframeEmbed} from 'contentElements/iframeEmbed/IframeEmbed';
+import fitViewportStyles from 'frontend/FitViewport.module.css';
+
+describe('IframeEmbed', () => {
+  it('renders FitViewport with aspect ratio by default', () => {
+    const {container} = renderInContentElement(
+      <IframeEmbed configuration={{source: 'https://example.com'}} />,
+      {seed: {}}
+    );
+
+    expect(container.querySelector(`.${fitViewportStyles.container}`)).not.toBeNull();
+  });
+
+  it('skips aspect ratio when autoResize is enabled', () => {
+    const {container} = renderInContentElement(
+      <IframeEmbed configuration={{source: 'https://example.com',
+                                   autoResize: true}} />,
+      {seed: {}}
+    );
+
+    expect(container.querySelector(`.${fitViewportStyles.container}`)).toBeNull();
+  });
+});

entry_types/scrolled/package/src/contentElements/iframeEmbed/IframeEmbed.js

@@ -55,7 +55,7 @@ export function IframeEmbed({configuration}) {
   return (
     <div className={styles.wrapper}
          style={{pointerEvents: isEditable && !isSelected ? 'none' : undefined}}>
-      <FitViewport aspectRatio={aspectRatios[aspectRatio || 'wide']}
+      <FitViewport aspectRatio={configuration.autoResize ? null : aspectRatios[aspectRatio || 'wide']}
                    opaque={utils.isBlank(configuration.source)}>
         <ContentElementBox>
           <ContentElementFigure configuration={configuration}>