From c8e7ce33f5b02b309ab3339e9cc067ce863f8c4a Mon Sep 17 00:00:00 2001 From: Mikey O'Toole Date: Fri, 30 Jun 2023 14:39:07 +0100 Subject: [PATCH] fix(theme): only set classname on ul elements if they have an existing class (#9099) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Sébastien Lorber Co-authored-by: sebastienlorber --- .../src/theme/MDXComponents/Ul/index.tsx | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packages/docusaurus-theme-classic/src/theme/MDXComponents/Ul/index.tsx b/packages/docusaurus-theme-classic/src/theme/MDXComponents/Ul/index.tsx index d426298f6a..4c76343602 100644 --- a/packages/docusaurus-theme-classic/src/theme/MDXComponents/Ul/index.tsx +++ b/packages/docusaurus-theme-classic/src/theme/MDXComponents/Ul/index.tsx @@ -11,7 +11,11 @@ import type {Props} from '@theme/MDXComponents/Ul'; import styles from './styles.module.css'; -function transformUlClassName(className?: string): string { +function transformUlClassName(className?: string): string | undefined { + // Fix https://github.com/facebook/docusaurus/issues/9098 + if (typeof className === 'undefined') { + return undefined; + } return clsx( className, // This class is set globally by GitHub/MDX. We keep the global class, and