From f6b12d447ddd75bb11793cb8b6e04544a105d54c Mon Sep 17 00:00:00 2001
From: Josiah Ivey <josiah.ivey@gmail.com>
Date: Mon, 6 May 2024 09:41:49 -0700
Subject: [PATCH] remove region role from answer (#71)

* remove region role from answer

* update snapshots

---------

Co-authored-by: Thomas Woodward <tom.g.woodward@gmail.com>
---
 src/components/Answer.tsx                      |  2 +-
 .../__snapshots__/Answer.spec.tsx.snap         |  8 --------
 .../__snapshots__/AnswersTable.spec.tsx.snap   | 12 ------------
 .../__snapshots__/Exercise.spec.tsx.snap       | 10 ----------
 .../ExerciseQuestion.spec.tsx.snap             | 18 ------------------
 .../__snapshots__/Question.spec.tsx.snap       | 14 --------------
 6 files changed, 1 insertion(+), 63 deletions(-)

diff --git a/src/components/Answer.tsx b/src/components/Answer.tsx
index 3e5561f..d90dadd 100644
--- a/src/components/Answer.tsx
+++ b/src/components/Answer.tsx
@@ -194,7 +194,7 @@ export const Answer = (props: AnswerProps) => {
 
   return (
     <div className="openstax-answer">
-      <section role="region" className={classes}>
+      <section className={classes}>
         {body}
       </section>
     </div>
diff --git a/src/components/__snapshots__/Answer.spec.tsx.snap b/src/components/__snapshots__/Answer.spec.tsx.snap
index 729e23a..0bec985 100644
--- a/src/components/__snapshots__/Answer.spec.tsx.snap
+++ b/src/components/__snapshots__/Answer.spec.tsx.snap
@@ -6,7 +6,6 @@ exports[`Answer matches snapshot 1`] = `
 >
   <section
     className="answers-answer"
-    role="region"
   >
     <input
       checked={false}
@@ -63,7 +62,6 @@ exports[`Answer renders a checked answer 1`] = `
 >
   <section
     className="answers-answer answer-selected"
-    role="region"
   >
     <input
       checked={true}
@@ -120,7 +118,6 @@ exports[`Answer renders a correct answer 1`] = `
 >
   <section
     className="answers-answer disabled answer-correct"
-    role="region"
   >
     <label
       className="answer-label"
@@ -174,7 +171,6 @@ exports[`Answer renders an incorrect answer 1`] = `
 >
   <section
     className="answers-answer answer-incorrect"
-    role="region"
   >
     <label
       className="answer-label"
@@ -228,7 +224,6 @@ exports[`Answer renders feedback 1`] = `
 >
   <section
     className="answers-answer"
-    role="region"
   >
     <input
       checked={false}
@@ -295,7 +290,6 @@ exports[`Answer renders teacher preview 1`] = `
 >
   <section
     className="answers-answer answer-correct"
-    role="region"
   >
     <div
       className="correct-incorrect"
@@ -356,7 +350,6 @@ exports[`Answer renders teacher review 1`] = `
 >
   <section
     className="answers-answer"
-    role="region"
   >
     <div
       className="review-wrapper"
@@ -404,7 +397,6 @@ exports[`Answer renders with a custom renderer if set 1`] = `
 >
   <section
     className="answers-answer"
-    role="region"
   >
     <input
       checked={false}
diff --git a/src/components/__snapshots__/AnswersTable.spec.tsx.snap b/src/components/__snapshots__/AnswersTable.spec.tsx.snap
index b82db0c..8e7aa02 100644
--- a/src/components/__snapshots__/AnswersTable.spec.tsx.snap
+++ b/src/components/__snapshots__/AnswersTable.spec.tsx.snap
@@ -9,7 +9,6 @@ exports[`AnswersTable matches tutor teacher-preview snapshot 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <div
         className="correct-incorrect"
@@ -57,7 +56,6 @@ exports[`AnswersTable matches tutor teacher-preview snapshot 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <div
         className="correct-incorrect"
@@ -112,7 +110,6 @@ exports[`AnswersTable renders all feedback 1`] = `
   >
     <section
       className="answers-answer disabled answer-selected answer-incorrect"
-      role="region"
     >
       <input
         checked={true}
@@ -184,7 +181,6 @@ exports[`AnswersTable renders all feedback 1`] = `
   >
     <section
       className="answers-answer disabled answer-correct"
-      role="region"
     >
       <input
         checked={false}
@@ -263,7 +259,6 @@ exports[`AnswersTable renders correct answer feedback 1`] = `
   >
     <section
       className="answers-answer disabled answer-selected answer-correct"
-      role="region"
     >
       <label
         className="answer-label"
@@ -326,7 +321,6 @@ exports[`AnswersTable renders correct answer feedback 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <label
         className="answer-label"
@@ -372,7 +366,6 @@ exports[`AnswersTable renders incorrect answer feedback 1`] = `
   >
     <section
       className="answers-answer disabled answer-selected answer-incorrect"
-      role="region"
     >
       <input
         checked={true}
@@ -444,7 +437,6 @@ exports[`AnswersTable renders incorrect answer feedback 1`] = `
   >
     <section
       className="answers-answer disabled answer-correct"
-      role="region"
     >
       <input
         checked={false}
@@ -508,7 +500,6 @@ exports[`AnswersTable renders instructions 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <input
         checked={false}
@@ -556,7 +547,6 @@ exports[`AnswersTable renders instructions 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <input
         checked={false}
@@ -611,7 +601,6 @@ exports[`AnswersTable sorts by given ID order 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <input
         checked={false}
@@ -659,7 +648,6 @@ exports[`AnswersTable sorts by given ID order 1`] = `
   >
     <section
       className="answers-answer disabled"
-      role="region"
     >
       <input
         checked={false}
diff --git a/src/components/__snapshots__/Exercise.spec.tsx.snap b/src/components/__snapshots__/Exercise.spec.tsx.snap
index 9eedd8b..5efa557 100644
--- a/src/components/__snapshots__/Exercise.spec.tsx.snap
+++ b/src/components/__snapshots__/Exercise.spec.tsx.snap
@@ -77,7 +77,6 @@ exports[`Exercise using step data matches snapshot 1`] = `
                 >
                   <section
                     className="answers-answer disabled answer-selected"
-                    role="region"
                   >
                     <input
                       checked={true}
@@ -125,7 +124,6 @@ exports[`Exercise using step data matches snapshot 1`] = `
                 >
                   <section
                     className="answers-answer disabled"
-                    role="region"
                   >
                     <input
                       checked={false}
@@ -275,7 +273,6 @@ exports[`Exercise with question state data matches snapshot 1`] = `
                 >
                   <section
                     className="answers-answer disabled answer-selected"
-                    role="region"
                   >
                     <input
                       checked={true}
@@ -323,7 +320,6 @@ exports[`Exercise with question state data matches snapshot 1`] = `
                 >
                   <section
                     className="answers-answer disabled"
-                    role="region"
                   >
                     <input
                       checked={false}
@@ -613,7 +609,6 @@ exports[`Exercise with question state data renders header icons with multiple ch
                 >
                   <section
                     className="answers-answer disabled answer-selected"
-                    role="region"
                   >
                     <input
                       checked={true}
@@ -661,7 +656,6 @@ exports[`Exercise with question state data renders header icons with multiple ch
                 >
                   <section
                     className="answers-answer disabled"
-                    role="region"
                   >
                     <input
                       checked={false}
@@ -991,7 +985,6 @@ exports[`Exercise with question state data renders header icons with two-step ex
                 >
                   <section
                     className="answers-answer disabled answer-selected"
-                    role="region"
                   >
                     <input
                       checked={true}
@@ -1039,7 +1032,6 @@ exports[`Exercise with question state data renders header icons with two-step ex
                 >
                   <section
                     className="answers-answer disabled"
-                    role="region"
                   >
                     <input
                       checked={false}
@@ -1189,7 +1181,6 @@ exports[`Exercise with question state data shows a detailed solution 1`] = `
                 >
                   <section
                     className="answers-answer disabled answer-selected"
-                    role="region"
                   >
                     <input
                       checked={true}
@@ -1237,7 +1228,6 @@ exports[`Exercise with question state data shows a detailed solution 1`] = `
                 >
                   <section
                     className="answers-answer disabled"
-                    role="region"
                   >
                     <input
                       checked={false}
diff --git a/src/components/__snapshots__/ExerciseQuestion.spec.tsx.snap b/src/components/__snapshots__/ExerciseQuestion.spec.tsx.snap
index 0f563b1..958280b 100644
--- a/src/components/__snapshots__/ExerciseQuestion.spec.tsx.snap
+++ b/src/components/__snapshots__/ExerciseQuestion.spec.tsx.snap
@@ -26,7 +26,6 @@ exports[`ExerciseQuestion matches snapshot 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -74,7 +73,6 @@ exports[`ExerciseQuestion matches snapshot 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -179,7 +177,6 @@ exports[`ExerciseQuestion renders Re-submit button 1`] = `
       >
         <section
           className="answers-answer answer-selected"
-          role="region"
         >
           <input
             checked={true}
@@ -227,7 +224,6 @@ exports[`ExerciseQuestion renders Re-submit button 1`] = `
       >
         <section
           className="answers-answer answer-incorrect"
-          role="region"
         >
           <input
             checked={false}
@@ -339,7 +335,6 @@ exports[`ExerciseQuestion renders Save button 1`] = `
       >
         <section
           className="answers-answer answer-selected"
-          role="region"
         >
           <input
             checked={true}
@@ -387,7 +382,6 @@ exports[`ExerciseQuestion renders Save button 1`] = `
       >
         <section
           className="answers-answer answer-incorrect"
-          role="region"
         >
           <input
             checked={false}
@@ -499,7 +493,6 @@ exports[`ExerciseQuestion renders all attempts remaining 1`] = `
       >
         <section
           className="answers-answer"
-          role="region"
         >
           <input
             checked={false}
@@ -547,7 +540,6 @@ exports[`ExerciseQuestion renders all attempts remaining 1`] = `
       >
         <section
           className="answers-answer"
-          role="region"
         >
           <input
             checked={false}
@@ -659,7 +651,6 @@ exports[`ExerciseQuestion renders continue button (unused?) 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -707,7 +698,6 @@ exports[`ExerciseQuestion renders continue button (unused?) 1`] = `
       >
         <section
           className="answers-answer disabled answer-incorrect"
-          role="region"
         >
           <input
             checked={false}
@@ -818,7 +808,6 @@ exports[`ExerciseQuestion renders detailed solution and published comments 1`] =
       >
         <section
           className="answers-answer disabled answer-correct"
-          role="region"
         >
           <label
             className="answer-label"
@@ -863,7 +852,6 @@ exports[`ExerciseQuestion renders detailed solution and published comments 1`] =
       >
         <section
           className="answers-answer disabled answer-incorrect"
-          role="region"
         >
           <label
             className="answer-label"
@@ -989,7 +977,6 @@ exports[`ExerciseQuestion renders free response 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -1037,7 +1024,6 @@ exports[`ExerciseQuestion renders free response 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -1142,7 +1128,6 @@ exports[`ExerciseQuestion renders no attempts remaining 1`] = `
       >
         <section
           className="answers-answer disabled"
-          role="region"
         >
           <input
             checked={false}
@@ -1190,7 +1175,6 @@ exports[`ExerciseQuestion renders no attempts remaining 1`] = `
       >
         <section
           className="answers-answer disabled answer-incorrect"
-          role="region"
         >
           <input
             checked={false}
@@ -1301,7 +1285,6 @@ exports[`ExerciseQuestion renders some attempts remaining 1`] = `
       >
         <section
           className="answers-answer"
-          role="region"
         >
           <input
             checked={false}
@@ -1349,7 +1332,6 @@ exports[`ExerciseQuestion renders some attempts remaining 1`] = `
       >
         <section
           className="answers-answer answer-incorrect"
-          role="region"
         >
           <input
             checked={false}
diff --git a/src/components/__snapshots__/Question.spec.tsx.snap b/src/components/__snapshots__/Question.spec.tsx.snap
index f6bfb17..7185197 100644
--- a/src/components/__snapshots__/Question.spec.tsx.snap
+++ b/src/components/__snapshots__/Question.spec.tsx.snap
@@ -23,7 +23,6 @@ exports[`Question defaults QuestionHtml html 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -71,7 +70,6 @@ exports[`Question defaults QuestionHtml html 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -141,7 +139,6 @@ exports[`Question defaults collaborator_solutions 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -189,7 +186,6 @@ exports[`Question defaults collaborator_solutions 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -259,7 +255,6 @@ exports[`Question defaults formats 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -307,7 +302,6 @@ exports[`Question defaults formats 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -386,7 +380,6 @@ exports[`Question matches snapshot 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -434,7 +427,6 @@ exports[`Question matches snapshot 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -504,7 +496,6 @@ exports[`Question renders exercise uid 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -552,7 +543,6 @@ exports[`Question renders exercise uid 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -627,7 +617,6 @@ exports[`Question renders formats 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -675,7 +664,6 @@ exports[`Question renders formats 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -757,7 +745,6 @@ exports[`Question renders solutions 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}
@@ -805,7 +792,6 @@ exports[`Question renders solutions 1`] = `
     >
       <section
         className="answers-answer disabled"
-        role="region"
       >
         <input
           checked={false}