From 982a77974581ad9614b12b1848bd192f2e794f3e Mon Sep 17 00:00:00 2001
From: Chris Lindgren <chris.a.lindgren@gmail.com>
Date: Mon, 19 Feb 2024 13:24:37 -0500
Subject: [PATCH] Feedback for Joey

---
 mahoney/index.html | 13 +++++++++++++
 1 file changed, 13 insertions(+)
diff --git a/mahoney/index.html b/mahoney/index.html
index 996b4b9..eb18299 100644
--- a/mahoney/index.html
+++ b/mahoney/index.html
@@ -5,6 +5,16 @@
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
     <title>Document</title>
 </head>
+<!--
+    Overall Feedback
+
+    - It's tough to give feedback without content at this point. Be sure to catch up this week before the midterm. LMK what questions you have. Here's some advice to help you move forward:
+
+    1. Note you you have open and closed your body. All of your content goes inside of the body element.
+    2. the closing tag for the html root element will always be the last closing tag on the page, since it contains ALLTHETHINGS.
+    3. You have a lot going on with no content, so I think you've led yourself astray a bit. I would just restart for simplicity. Start with the first thing that needs to be in the body: the skip link. Then, is it your header? Then your nav? Then your main? Then, what landmarks are children of the main?
+    4. You've missnamed numerous elements below. The browser won't tell you that there's something wrong, but your screenreader testing will. Be sure to use my provided resources and review the readings. For example, there's an HTML5 element resource sheet that would have helped you catch lots of those misnamed elements. I think your missign attendance early on might have made it difficult for you, so be sure to dedicate some time this week to this class. It will be super important, as we move forward quite quickly. I want you to understand HTML fundamentals before we startup CSS next week.
+-->
 <body>
     
 </body>
@@ -16,17 +26,20 @@
     </head>
 </nav>
 <main>
+    <!-- the head is a very specific element. You're probably thinking about header elements -->
     <head>
         <h1></h1>
     </head>
     <body1>
         <h2></h2>
+        <!-- there are no paragraph numbering. It's always just <p> -->
         <p1></p1>
     </body1>
     <body2>
         <h3></h3>
     </body2>
 </main>
+<!-- Really, really be sure to review the readings on these elements, and also use the handout resources, so you can use the appropriate names that fit the HTML5 specifications. -->
 <foot>
     <p2></p2>
     <p3></p3>
