From 9e9400f369d9a2b31a3066b7bbbd1ea4245af651 Mon Sep 17 00:00:00 2001
From: anitahy73 <anita.hy73@gmail.com>
Date: Fri, 15 May 2026 21:51:22 +0100
Subject: [PATCH 1/2] eview issues fixed

---
 Form-Controls/index.html | 48 +++++++++++++++++++++++++++++++++-------
 1 file changed, 40 insertions(+), 8 deletions(-)

diff --git a/Form-Controls/index.html b/Form-Controls/index.html
index 74b591ffc..13dbff077 100644
--- a/Form-Controls/index.html
+++ b/Form-Controls/index.html
@@ -12,16 +12,48 @@
       <h1>Product Pick</h1>
     </header>
     <main>
-      <form>
-        <!-- write your html here-->
-        <!-- 
-          try writing out the requirements first as comments
-        this will also help you fill in your PR message later-->
-      </form>
+<form>
+
+            <label for="name">Name:</label>
+            <input type="text" id="name" name="name"><br>
+            <label for="email">Email:</label>
+            <input type="email" id="email" name="email"><br><br>
+
+            <fieldset>
+            <legend>colour:</legend>
+
+            <label>
+                <input type="radio" id="colour-red" name="colour" value="red">
+                Red
+            </label>
+            
+            <label >
+                <input type="radio" id="colour-blue" name="colour" value="blue">
+                Blue
+            </label>
+            
+            <label >
+                <input type="radio" id="colour-green" name="colour" value="green">
+                Green
+            </label>
+            
+            </fieldset>
+
+            <label for="size">choose your size</label>
+            
+
+            <select id="size">
+                <option value="s">small</option>
+                <option value="m">medium</option>
+                <option value="l">large</option>
+                <option value="xl">xlarge</option>
+                <option value="xl">xxlarge</option>
+
+            </select> 
+        </form>
     </main>
     <footer>
-      <!-- change to your name-->
-      <p>By HOMEWORK SOLUTION</p>
+      <p>Anita Amirhaeri</p>
     </footer>
   </body>
 </html>

From b0042cd6859cce0cc62eb12a0361b2620238105e Mon Sep 17 00:00:00 2001
From: anitahy73 <anita.hy73@gmail.com>
Date: Fri, 15 May 2026 22:15:32 +0100
Subject: [PATCH 2/2] Fixed Review changes

---
 Form-Controls/index.html | 65 ++++++++++++++++++++--------------------
 1 file changed, 32 insertions(+), 33 deletions(-)

diff --git a/Form-Controls/index.html b/Form-Controls/index.html
index 13dbff077..227f889c0 100644
--- a/Form-Controls/index.html
+++ b/Form-Controls/index.html
@@ -12,45 +12,44 @@
       <h1>Product Pick</h1>
     </header>
     <main>
-<form>
+      <form>
 
-            <label for="name">Name:</label>
-            <input type="text" id="name" name="name"><br>
-            <label for="email">Email:</label>
-            <input type="email" id="email" name="email"><br><br>
+        <label for="name">Name:</label>
+        <input type="text" id="name" name="name" minlength="2" required ><br>
+        <label for="email">Email:</label>
+        <input type="email" id="email" name="email"><br><br>
 
-            <fieldset>
-            <legend>colour:</legend>
+        <fieldset>
+          <legend>colour:</legend>
 
-            <label>
-                <input type="radio" id="colour-red" name="colour" value="red">
-                Red
-            </label>
-            
-            <label >
-                <input type="radio" id="colour-blue" name="colour" value="blue">
-                Blue
-            </label>
-            
-            <label >
-                <input type="radio" id="colour-green" name="colour" value="green">
-                Green
-            </label>
-            
-            </fieldset>
+          <label>
+              <input type="radio" id="colour-red" name="colour" value="red" required>
+              Red
+          </label>
+          
+          <label >
+              <input type="radio" id="colour-blue" name="colour" value="blue">
+              Blue
+          </label>
+          
+          <label >
+              <input type="radio" id="colour-green" name="colour" value="green">
+              Green
+          </label>
+        
+        </fieldset>
 
-            <label for="size">choose your size</label>
-            
+        <p for="size">Choose your size</p>
 
-            <select id="size">
-                <option value="s">small</option>
-                <option value="m">medium</option>
-                <option value="l">large</option>
-                <option value="xl">xlarge</option>
-                <option value="xl">xxlarge</option>
+        <select id="size" name="size" required>
+            <option value="s">Small</option>
+            <option value="m">Medium</option>
+            <option value="l">Large</option>
+            <option value="xl">XLarge</option>
+            <option value="xl">XXLarge</option>
 
-            </select> 
-        </form>
+        </select> 
+      </form>
     </main>
     <footer>
       <p>Anita Amirhaeri</p>