Skip to content

London | 26-ITP-Sep | Ebrahim Moqbel | Sprint 1 | Form Controls #1462

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Ebrahim-Moqbel wants to merge 13 commits into
base: main
Choose a base branch
from
+58 −7
Open

London | 26-ITP-Sep | Ebrahim Moqbel | Sprint 1 | Form Controls #1462

Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
4032809
A reason for me to join CYF
Ebrahim-Moqbel Jan 13, 2026
50fcc71
Reson
Ebrahim-Moqbel Jan 13, 2026
749a886
What is Git and why is it being used?
Ebrahim-Moqbel Jan 13, 2026
0173fea
Article on the purpos of git
Ebrahim-Moqbel Jan 13, 2026
47925f5
Merge branch 'CodeYourFuture:main' into main
Ebrahim-Moqbel Jan 25, 2026
dc50152
Merge branch 'CodeYourFuture:main' into main
Ebrahim-Moqbel Feb 2, 2026
84147a6
form control
Ebrahim-Moqbel Feb 2, 2026
280c8fd
size div
Ebrahim-Moqbel Feb 2, 2026
270dbee
unwanted article deleted
Ebrahim-Moqbel Feb 2, 2026
697f95f
labelling the T-shirt size and chaninging the color section to be rec…
Ebrahim-Moqbel Feb 7, 2026
a008f20
I placed the radio buttom name after the radio button input fot the …
Ebrahim-Moqbel Feb 8, 2026
c849df2
Merge branch 'main' into Features/Form-Controls
Ebrahim-Moqbel Jun 6, 2026
4338fc8
Merge branch 'CodeYourFuture:main' into Features/Form-Controls
Ebrahim-Moqbel Sep 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 58 additions & 7 deletions Form-Controls/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,67 @@
<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>
<section>

<div>
<label for="name"> Name*: </label>
<input type="text" id="name" name="name" required minlength="2">

@Luro91 Luro91 Sep 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Min length is a good idea but also allows 2 space characters to be entered and passing the validaiton. How can you ensure the characaters are not spaces?

</div>

<div>
<label for="email">Email*:</label>
<input type="email" id="email" name="email" required>
</div>


<fieldset>
<legend >T-shirt Color</legend>
<div>
<label for="black">
Comment on lines +31 to +32

@Luro91 Luro91 Sep 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The standard for child elements is to make them more visible in the code by using indentation. How can you ensure your code is formatted according to standards?

<input type="radio" id="black" name="t-shirt-color" value="black" required>
Black

</label>

</div>
<div>
<label for="red">
<input type="radio" id="red" name="t-shirt-color" value="red">
Red
</label>

</div>
<div>
<label for="green">
<input type="radio" id="green" name="t-shirt-color" value="green">
Green
</label>

</div>
</fieldset>


<div>
<label for="t-shirt-size"> T-Shirt Size</label>
<select name="T-shirtSize" id="t-shirt-size" required>
<option value="XS">XS</option>
<option label="S" value="S">S</option>
<option label="M" value="M">M</option>
<option label="L" value="L">L</option>
<option label="XL" value="XL">XL</option>
<option label="XXL" value="XXL">XXL</option>
</select>
</div>
<div>
<button type="submit">Place your order</button>
</div>
</section>
</form>
</main>
<footer>
<!-- change to your name-->
<p>By HOMEWORK SOLUTION</p>

<p>By Ebraim Moqbel</p>
</footer>
</body>
</html>
Loading