From 9d2e2095ef4a0676c4daaa9ced69fad0f23d13b3 Mon Sep 17 00:00:00 2001
From: Alison Hall <alison.hall@publicissapient.com>
Date: Wed, 13 Mar 2024 17:12:05 -0400
Subject: [PATCH 1/3] Add notes and questions in comments

---
 content/body/accessible-text-svg.php                   | 1 +
 content/body/exposing-style-info-to-screen-readers.php | 1 +
 content/body/focus-styling.php                         | 1 +
 content/body/heading.php                               | 1 +
 content/body/skip-link.php                             | 1 +
 content/body/tooltip.php                               | 2 +-
 6 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/content/body/accessible-text-svg.php b/content/body/accessible-text-svg.php
index ab473ef3..816df1fb 100644
--- a/content/body/accessible-text-svg.php
+++ b/content/body/accessible-text-svg.php
@@ -51,6 +51,7 @@
     This text complies with text spacing and text resizing requirements. If you use the
     text-spacing bookmarklet above, the text spaces out and doesn't get cut off.
   </p>
+  <!-- There is no bookmarklet above, even though it's mentioned in the previous paragraph -->
 
   <figure>
     <img src="images/accessible-text-svg/rollercoaster-screenshot__text-spacing.jpg"
diff --git a/content/body/exposing-style-info-to-screen-readers.php b/content/body/exposing-style-info-to-screen-readers.php
index 961bb903..62902016 100644
--- a/content/body/exposing-style-info-to-screen-readers.php
+++ b/content/body/exposing-style-info-to-screen-readers.php
@@ -112,6 +112,7 @@
   </code></pre>
 </div>
 
+<!-- It's unclear what this code walkthrough is of -->
 <?php includeShowcode("highlight-example", "", "", "", true, 2)?>
 <script type="application/json" id="highlight-example-props">
 {
diff --git a/content/body/focus-styling.php b/content/body/focus-styling.php
index b79581e6..67c9a52b 100644
--- a/content/body/focus-styling.php
+++ b/content/body/focus-styling.php
@@ -1,6 +1,7 @@
 <p>
   Focus states are used by keyboard users to know what interactive element they can currently manipulate. They are
   usually styled with
+  <!-- TODO: missing remaining content -->
 
 <h2>Focus Styling For Keyboard Users Only</h2>
 
diff --git a/content/body/heading.php b/content/body/heading.php
index aaefe784..0049ba3b 100644
--- a/content/body/heading.php
+++ b/content/body/heading.php
@@ -20,6 +20,7 @@
 </p>
 
 <div class="enable-example">
+  <!-- TODO: replace the placeholder content -->
   Insert video here
 </div>
 
diff --git a/content/body/skip-link.php b/content/body/skip-link.php
index f0c5a768..e0215370 100644
--- a/content/body/skip-link.php
+++ b/content/body/skip-link.php
@@ -304,6 +304,7 @@
     the skip links will remain visible unless they are clicked.</strong>
   This seems like a reasonable tradeoff (and can arguably be better for accessibility).
 </p>
+<!-- I don't see the skip links remaining visible like mentioned as an issue. Are there only certain scenarios when this occurs? -->
 
 <div id="mobile-example" class="enable-example">
   <div class="enable-mobile-visible-on-focus__container enable-skip-link--begin">
diff --git a/content/body/tooltip.php b/content/body/tooltip.php
index bc83a7cd..70c8e9d7 100644
--- a/content/body/tooltip.php
+++ b/content/body/tooltip.php
@@ -132,7 +132,7 @@
 </script>
 
 
-
+<!-- Should this be for the first example, instead of being under the second one? -->
 <?= includeNPMInstructions(
   'tooltip',
   array(),

From fedf6e96d88909957bd14fda6b9d4290175da27d Mon Sep 17 00:00:00 2001
From: Alison Hall <alison.hall@publicissapient.com>
Date: Fri, 15 Mar 2024 11:34:10 -0400
Subject: [PATCH 2/3] Add notes and comments for the Code Quality page

---
 content/body/code-quality.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/content/body/code-quality.php b/content/body/code-quality.php
index 2d5857be..cf967c14 100644
--- a/content/body/code-quality.php
+++ b/content/body/code-quality.php
@@ -106,6 +106,7 @@
 </ol>
 
 <h3>Before we start</h3>
+<!-- Are these Babel changes actually needed? I don't see them being used in this project. -->
 
 <p>
   Our unit testing examples use ES6 modules. In order to support ES6 Modules in Jest, you need to do the following
@@ -159,6 +160,8 @@
   </template>
 
   <?php includeShowcode("test-code-walkthrough", "", "", "", true, 4)?>
+  <!-- TODO: The step 1 note ends in an incomplete sentence -->
+  <!-- Should step 2 and 3 only highlight the 'describe' or 'it' lines, instead of almost the whole file? -->
   <script type="application/json" id="test-code-walkthrough-props">
   {
     "replaceHtmlRules": {},
@@ -212,6 +215,8 @@
   </template>
 
   <?php includeShowcode("switch-test-code-walkthrough", "", "", "", true, 4)?>
+  <!-- TODO: The step 1 note ends in an incomplete sentence -->
+  <!-- Should step 2 and 3 only highlight the 'describe' or 'it' lines, instead of almost the whole file? -->
   <script type="application/json" id="switch-test-code-walkthrough-props">
   {
     "replaceHtmlRules": {},
@@ -270,6 +275,9 @@
   </template>
 
   <?php includeShowcode("test-code-walkthrough2", "", "", "", true, 4)?>
+  <!-- TODO: The step 1 note ends in an incomplete sentence -->
+  <!-- Should step 2 and 3 only highlight the 'describe' or 'it' lines, instead of almost the whole file? -->
+  <!-- Step #8 seems to highlight a lot more than the page.evaluate function. Should the RegEx be modified? -->
   <script type="application/json" id="test-code-walkthrough2-props">
   {
     "replaceHtmlRules": {},

From df541f587d6934efd11829b59cffc03178bb115e Mon Sep 17 00:00:00 2001
From: Alison Hall <alison.hall@publicissapient.com>
Date: Wed, 29 May 2024 17:48:03 -0400
Subject: [PATCH 3/3] Add notes for the combobox and radiogroup pages

---
 content/body/combobox.php   | 4 +++-
 content/body/radiogroup.php | 8 ++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/content/body/combobox.php b/content/body/combobox.php
index 6d11bb3d..bc0672b9 100644
--- a/content/body/combobox.php
+++ b/content/body/combobox.php
@@ -9,7 +9,7 @@
   there. Because of this, I have spent a lot of time working on Enable's combobox implementation.
 </p>
 
-
+<!-- Example 3 is the first example on the page. This doesn't make sense. -->
 <h2>Example 3: Using HTML5 datalist</h2>
 
 <?php includeStats(array('doNot' => true, 'comment' => 'This does not work with assistive technologies in most web browsers.')) ?>
@@ -514,6 +514,7 @@
 
           <div role="listbox" id="aria-example-2__list" hidden>
             <div class="enable-combobox__group" role="group" aria-describedby="cat-1">
+            <!-- The "Communist States" heading has a permalink and is included in the TOC when it shouldn't be -->
               <h2 class="enable-combobox__group-header" role="presentation" id="cat-1">Communist States</h2>
 
               <div role="option">People's Republic of China</div>
@@ -529,6 +530,7 @@
             </div>
 
             <div class="enable-combobox__group" role="group" aria-describedby="cat-2">
+            <!-- The "Other States" heading has a permalink and is included in the TOC when it shouldn't be -->
               <h2 class="enable-combobox__group-header" role="presentation" id="cat-2">Other States</h2>
               <div role="option">Afghanistan</div>
               <div role="option">Albania</div>
diff --git a/content/body/radiogroup.php b/content/body/radiogroup.php
index 4b3253c8..525a0366 100644
--- a/content/body/radiogroup.php
+++ b/content/body/radiogroup.php
@@ -32,6 +32,7 @@
 
 
 <div id="example1" class="enable-example">
+    <!-- The "Set Alerts for your Account" heading has a permalink and is included in the TOC when it shouldn't be -->
   <h3 class="form-heading">Set Alerts for your Account</h3>
 
 
@@ -102,7 +103,8 @@
 </p>
 
 <div id="example1-styled" class="enable-example">
-  <div class="enable-radio">
+    <div class="enable-radio">
+      <!-- The "Set Alerts for your Account" heading has a permalink and is included in the TOC when it shouldn't be -->
     <h3 class="form-heading">Set Alerts for your Account</h3>
 
 
@@ -186,6 +188,7 @@
 </p>
 
 <div id="example2" class="enable-example">
+    <!-- The "Set Alerts for your Account" heading has a permalink and is included in the TOC when it shouldn't be -->
   <h3 class="form-heading">Set Alerts for your Account</h3>
 
 
@@ -287,6 +290,7 @@
 
 
 <div id="example4" class="enable-example">
+    <!-- The "Set Alerts for your Account" heading has a permalink and is included in the TOC when it shouldn't be -->
   <h3 class="form-heading">Set Alerts for your Account</h3>
   <div role="radiogroup" class="aria-form-group" aria-labelledby="html2-alert1">
     <p id="html2-alert1" class="legend">Send an alert when balance exceeds $ 3,000</p>
@@ -326,4 +330,4 @@
     "notes": "The radiogroup is like the <strong>fieldset</strong> tag. It should use the <strong>aria-labelledby</strong> to point to the element that labels the radiogroup (which acts as the legend tag)."
   }]
 }
-</script>
\ No newline at end of file
+</script>