[EuiMarkdownEditor] Add footer visibility toggle and toolbar right slot to MarkdownEditor

packages/eui/changelogs/upcoming/8889.md

@@ -0,0 +1 @@
+- Added `showFooter` and `toolbarProps.right` props to `EuiMarkdownEditor` for more flexible layout control. ([#8889](https://github.com/elastic/eui/pull/8889))

packages/eui/src/components/markdown_editor/__snapshots__/markdown_editor.test.tsx.snap

@@ -177,7 +177,7 @@ exports[`EuiMarkdownEditor is rendered 1`] = `
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-text"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       type="button"
     >
       <span
@@ -214,6 +214,7 @@ exports[`EuiMarkdownEditor is rendered 1`] = `
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"
@@ -424,7 +425,7 @@ exports[`EuiMarkdownEditor props autoExpandPreview is rendered with false 1`] =
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-text"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       type="button"
     >
       <span
@@ -460,6 +461,7 @@ exports[`EuiMarkdownEditor props autoExpandPreview is rendered with false 1`] =
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"
@@ -670,7 +672,7 @@ exports[`EuiMarkdownEditor props height is rendered in full mode 1`] = `
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-text"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       type="button"
     >
       <span
@@ -706,6 +708,7 @@ exports[`EuiMarkdownEditor props height is rendered in full mode 1`] = `
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"
@@ -916,7 +919,7 @@ exports[`EuiMarkdownEditor props height is rendered with a custom size 1`] = `
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-text"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       type="button"
     >
       <span
@@ -952,6 +955,7 @@ exports[`EuiMarkdownEditor props height is rendered with a custom size 1`] = `
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"
@@ -1162,7 +1166,7 @@ exports[`EuiMarkdownEditor props maxHeight is rendered with a custom size 1`] =
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-text"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       type="button"
     >
       <span
@@ -1198,6 +1202,7 @@ exports[`EuiMarkdownEditor props maxHeight is rendered with a custom size 1`] =
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"
@@ -1417,7 +1422,7 @@ exports[`EuiMarkdownEditor props readOnly is set to true 1`] = `
     </div>
     <button
       class="euiButtonEmpty emotion-euiButtonDisplay-euiButtonEmpty-s-empty-disabled-isDisabled"
-      data-test-subj="markdown_editor_preview_button"
+      data-test-subj="markdown_editor_edit_button"
       disabled=""
       type="button"
     >
@@ -1454,6 +1459,7 @@ exports[`EuiMarkdownEditor props readOnly is set to true 1`] = `
       />
       <div
         class="euiMarkdownEditorFooter emotion-euiMarkdownEditorFooter"
+        data-test-subj="euiMarkdownEditorFooter"
       >
         <div
           class="euiMarkdownEditorFooter__actions emotion-euiMarkdownEditorFooter__actions"

packages/eui/src/components/markdown_editor/index.ts

@@ -6,8 +6,12 @@
  * Side Public License, v 1.
  */
 
-export type { EuiMarkdownEditorProps } from './markdown_editor';
+export type {
+  EuiMarkdownEditorProps,
+  EuiMarkdownEditorRef,
+} from './markdown_editor';
 export { EuiMarkdownEditor } from './markdown_editor';
+export { EuiMarkdownEditorHelpButton } from './markdown_editor_help_button';
 export {
   getDefaultEuiMarkdownParsingPlugins,
   getDefaultEuiMarkdownProcessingPlugins,

packages/eui/src/components/markdown_editor/markdown_editor.stories.tsx

@@ -6,10 +6,13 @@
  * Side Public License, v 1.
  */
 
+import React from 'react';
 import type { Meta, StoryObj } from '@storybook/react';
+import { action } from '@storybook/addon-actions';
 
 import { EuiMarkdownEditor, EuiMarkdownEditorProps } from './markdown_editor';
 import { MODE_EDITING, MODE_VIEWING } from './markdown_modes';
+import { EuiButton } from '../button';
 
 const initialContent = `## Hello world!
 
@@ -62,3 +65,25 @@ export const Errors: Story = {
     errors: ['An error happened.', 'Woops, another error happened.'],
   },
 };
+
+export const CustomToolbarContent: Story = {
+  parameters: {
+    controls: {
+      include: ['toolbarProps'],
+    },
+  },
+  args: {
+    value: initialContent,
+    toolbarProps: {
+      right: (
+        <EuiButton
+          iconType="check"
+          size="s"
+          onClick={() => action('onClick')('Validated!')}
+        >
+          Validate
+        </EuiButton>
+      ),
+    },
+  },
+};

packages/eui/src/components/markdown_editor/markdown_editor.test.tsx

@@ -7,7 +7,7 @@
  */
 
 import React from 'react';
-import { fireEvent } from '@testing-library/react';
+import { cleanup, fireEvent } from '@testing-library/react';
 import { shouldRenderCustomStyles } from '../../test/internal';
 import { requiredProps } from '../../test/required_props';
 import { render, screen } from '../../test/rtl';
@@ -124,22 +124,22 @@ describe('EuiMarkdownEditor', () => {
   });
 
   test('is preview rendered', () => {
-    const { container, getByTestSubject } = render(
+    const { container, getByRole } = render(
       <EuiMarkdownEditor
         editorId="editorId"
         value="## Hello world"
         onChange={() => null}
         {...requiredProps}
       />
     );
-    fireEvent.click(getByTestSubject('markdown_editor_preview_button'));
+    fireEvent.click(getByRole('button', { name: 'Preview' }));
     expect(
       container.querySelector('.euiText.euiMarkdownFormat')?.querySelector('h2')
     ).toHaveTextContent('Hello world');
   });
 
   test('modal with help syntax is rendered', () => {
-    const { baseElement, getByLabelText } = render(
+    const { baseElement } = render(
       <EuiMarkdownEditor
         editorId="editorId"
         value=""
@@ -149,7 +149,7 @@ describe('EuiMarkdownEditor', () => {
     );
     expect(baseElement.querySelector('.euiModal')).not.toBeInTheDocument();
 
-    fireEvent.click(getByLabelText('Show markdown help'));
+    fireEvent.click(screen.getByLabelText('Show markdown help'));
 
     expect(baseElement.querySelector('.euiModal')).toBeInTheDocument();
   });
@@ -158,7 +158,7 @@ describe('EuiMarkdownEditor', () => {
     const { parsingPlugins, processingPlugins, uiPlugins } =
       getDefaultEuiMarkdownPlugins({ exclude: ['tooltip'] });
 
-    const { baseElement, getByLabelText } = render(
+    const { baseElement } = render(
       <EuiMarkdownEditor
         editorId="editorId"
         value=""
@@ -169,7 +169,7 @@ describe('EuiMarkdownEditor', () => {
         {...requiredProps}
       />
     );
-    fireEvent.click(getByLabelText('Show markdown help'));
+    fireEvent.click(screen.getByLabelText('Show markdown help'));
 
     expect(baseElement.querySelector('.euiModal')).not.toBeInTheDocument();
     expect(baseElement.querySelector('.euiPopover')).toBeInTheDocument();
@@ -182,12 +182,13 @@ describe('EuiMarkdownEditor', () => {
       onChange: jest.fn(),
     };
 
-    const { getByTestSubject } = render(
-      <EuiMarkdownEditor {...testProps} {...requiredProps} />
-    );
+    render(<EuiMarkdownEditor {...testProps} {...requiredProps} />);
 
     const event = { target: { value: 'sometext' } };
-    fireEvent.change(getByTestSubject('euiMarkdownEditorTextArea'), event);
+    fireEvent.change(
+      screen.getByTestSubject('euiMarkdownEditorTextArea'),
+      event
+    );
 
     expect(testProps.onChange).toHaveBeenCalledTimes(1);
     expect(testProps.onChange).toHaveBeenLastCalledWith(event.target.value);
@@ -239,11 +240,9 @@ describe('EuiMarkdownEditor', () => {
         errors: testMessage,
       };
 
-      const { getByLabelText } = render(
-        <EuiMarkdownEditor {...testProps} {...requiredProps} />
-      );
+      render(<EuiMarkdownEditor {...testProps} {...requiredProps} />);
 
-      expect(getByLabelText('Show errors')).toBeInTheDocument();
+      expect(screen.getByLabelText('Show errors')).toBeInTheDocument();
     });
 
     test('does not render error if error messages are empty', () => {
@@ -254,11 +253,9 @@ describe('EuiMarkdownEditor', () => {
         errors: [],
       };
 
-      const { queryByTestSubject } = render(
-        <EuiMarkdownEditor {...testProps} {...requiredProps} />
-      );
+      render(<EuiMarkdownEditor {...testProps} {...requiredProps} />);
 
-      expect(queryByTestSubject('Show errors')).not.toBeInTheDocument();
+      expect(screen.queryByTestSubject('Show errors')).not.toBeInTheDocument();
     });
   });
 
@@ -286,11 +283,9 @@ describe('EuiMarkdownEditor', () => {
     document.execCommand = execCommandMock;
 
     beforeEach(() => {
-      const { getByTestSubject } = render(
-        <EuiMarkdownEditor {...testProps} {...requiredProps} />
-      );
+      render(<EuiMarkdownEditor {...testProps} {...requiredProps} />);
 
-      const textarea = getByTestSubject(
+      const textarea = screen.getByTestSubject(
         'euiMarkdownEditorTextArea'
       ) as HTMLTextAreaElement;
       textarea.setSelectionRange(0, 5);
@@ -380,4 +375,61 @@ describe('EuiMarkdownEditor', () => {
       );
     });
   });
+  describe('toolbar props', () => {
+    it('shows the custom toolbar component when passed', () => {
+      const CustomToolbarComponent = () => <div>Custom toolbar</div>;
+      render(
+        <EuiMarkdownEditor
+          onChange={() => null}
+          value="markdown test"
+          {...requiredProps}
+          toolbarProps={{ right: <CustomToolbarComponent /> }}
+        />
+      );
+      expect(screen.getByText('Custom toolbar')).toBeInTheDocument();
+      expect(
+        screen.queryByRole('button', { name: 'Preview' })
+      ).not.toBeInTheDocument();
+      cleanup();
+      render(
+        <EuiMarkdownEditor
+          onChange={() => null}
+          value="markdown test"
+          {...requiredProps}
+        />
+      );
+      expect(screen.queryByText('Custom toolbar')).not.toBeInTheDocument();
+      expect(
+        screen.getByRole('button', { name: 'Preview' })
+      ).toBeInTheDocument();
+      fireEvent.click(screen.getByRole('button', { name: 'Preview' }));
+      expect(
+        screen.getByRole('button', { name: 'Editor' })
+      ).toBeInTheDocument();
+    });
+  });
+  it('should show footer by default and hide when showFooter is false', () => {
+    render(
+      <EuiMarkdownEditor
+        onChange={() => null}
+        value="markdown test"
+        {...requiredProps}
+      />
+    );
+    expect(
+      screen.getByTestSubject('euiMarkdownEditorFooter')
+    ).toBeInTheDocument();
+    cleanup();
+    render(
+      <EuiMarkdownEditor
+        onChange={() => null}
+        value="markdown test"
+        {...requiredProps}
+        showFooter={false}
+      />
+    );
+    expect(
+      screen.queryByTestSubject('euiMarkdownEditorFooter')
+    ).not.toBeInTheDocument();
+  });
 });