From 64ca1c90518917b37b8d5644835a36a40a1004d2 Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Mon, 18 Nov 2024 11:24:17 -0600
Subject: [PATCH] Add role=presentation to li elements in editor

---
 packages/block-library/src/social-link/edit.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/packages/block-library/src/social-link/edit.js b/packages/block-library/src/social-link/edit.js
index 42a39a23ada3b0..14b0f11334a091 100644
--- a/packages/block-library/src/social-link/edit.js
+++ b/packages/block-library/src/social-link/edit.js
@@ -182,7 +182,13 @@ const SocialLinkEdit = ( {
 					onChange={ ( value ) => setAttributes( { rel: value } ) }
 				/>
 			</InspectorControls>
+			{ /*
+			 * Because the `<ul>` element has a role=document, the `<li>` is
+			 * not semantically correct, so adding role=presentation is cleaner.
+			 * https://github.com/WordPress/gutenberg/pull/64883#issuecomment-2472874551
+			 */ }
 			<li
+				role="presentation"
 				className={ wrapperClasses }
 				style={ {
 					color: iconColorValue,