Locale Switcher: Fix JS error and update heading style

A JS error was thrown when logged out due to the button in the admin bar being missing. Also changes the modal title font from Garamond to Inter.
WordPress · Jul 9, 2024 · 19c679e · 19c679e
1 parent fa88c03
commit 19c679e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/wp-content/mu-plugins/pub/locale-switcher/src/index.js b/wp-content/mu-plugins/pub/locale-switcher/src/index.js
@@ -27,7 +27,7 @@ const LocaleSwitcher = ( props ) => {
 	const closeModal = () => setOpen( false );
 	const [ value, setValue ] = useState( false );
 
-	externalButton.addEventListener( 'click', ( event ) => {
+	externalButton?.addEventListener( 'click', ( event ) => {
 		event.preventDefault();
 		openModal();
 	} );

diff --git a/wp-content/themes/pub/wporg-learn-2024/src/style/_wp-components.scss b/wp-content/themes/pub/wporg-learn-2024/src/style/_wp-components.scss
@@ -9,4 +9,9 @@
 
 .components-modal__header {
 	padding: var(--wp--preset--spacing--20) var(--wp--preset--spacing--30) 0;
+
+	.components-modal__header-heading {
+		font-family: var(--wp--preset--font-family--inter);
+		line-height: var(--wp--custom--heading--level-1--typography--line-height);
+	}
 }