Skip to content
Open
6 changes: 3 additions & 3 deletions Form-Controls/README.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why are there changes in this file? How is it related to the task?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

sorry, could you make it clear. Did you need more explanation about the code.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You commited some change to Form-Controls/README.md which I don't understand.

Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ Let's write out our testable criteria. Check each one off as you complete it.

> Before you say you're done: Is your code readable? Does it run correctly? Does it look professional?

These practices reflect the level of quality expected in professional work.
These practices reflect the level of quality expected in professiona- [MDN: Form validation](https://developer.mozilla.org/en-US/docs/Learn/Forms/Form_validation)
l work.
They ensure your code is reliable, maintainable, and presents a polished, credible experience to users.

- [ ] My HTML code has no errors or warnings when validated using https://validator.w3.org/
Expand All @@ -61,7 +62,6 @@ They ensure your code is reliable, maintainable, and presents a polished, credib

## Resources
- [MDN: Form controls](https://developer.mozilla.org/en-US/docs/Learn/Forms)
- [MDN: Form validation](https://developer.mozilla.org/en-US/docs/Learn/Forms/Form_validation)
- [Lighthouse](https://developers.google.com/web/tools/lighthouse)
- [Lighthouse](https://developers.google.comcom/web/tools/lighthouse)
- [Lighthouse Guide](https://programming.codeyourfuture.io/guides/testing/lighthouse)
- [Format Code and Make Logical Commits in VS Code](../practical_guide.md)
65 changes: 58 additions & 7 deletions Form-Controls/index.html
Original file line number Diff line number Diff line change
@@ -1,27 +1,78 @@
<!DOCTYPE html>
<html lang="en">
<head>
<link rel="stylesheet" href="style.css" >
<meta charset="utf-8" />
<meta http-equiv="X-UA-Compatible" content="IE=edge" />
<title>My form exercise</title>
<meta name="description" content="" />
<meta name="description" content="The page Contains Form to submit customer information about T-shirts" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<style>
input:valid{
border:2px solid yellow;
}
input:invalid:not(:placeholder-shown){
border:2px solid red;
}
</style>
Comment thread
yonasdesbelegebre marked this conversation as resolved.
Outdated
</head>
<body>
<header>
<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-->
<!-- -->
<div>
<label for="customername">Customer name *: </label>
<input type="text" id="customername" name="customername" required
Comment thread
yonasdesbelegebre marked this conversation as resolved.
Outdated
pattern="^[A-Za-z]{2,}$"
placeholder="Customer name"/>
</div>

<div>
<label for="customeremail">Customer email: </label>
<input type="email" id="customeremail" name="customeremail" required
pattern="^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,}$"
Comment thread
yonasdesbelegebre marked this conversation as resolved.
Outdated
placeholder="Customer email" />
</div>

<div>
<label for="color">T-shirt color *:</label>
<div class="control">
<label class="t-color">
<input type="radio" name="color" value="Black" required/>
Black
</label>
<label class="t-color">
<input type="radio" name="color" value="Red" required/>
Red
</label>
<label class="t-color" disabled>
<input type="radio" name="color" value="Blue" required/>
Blue
</label>
</div>
</div>
<div>
<label for="sizes">T-shirt size *:</label>
<select name="sizes" id="sizes" required>
<option value="xs">XS</option>
<option value="s">S</option>
<option value="m">M</option>
<option value="l">L</option>
<option value="XL">XL</option>
<option vallue="XXL">XXL</optio>
Comment thread
yonasdesbelegebre marked this conversation as resolved.
Outdated
</select>
</div>
<div>
<button type="submit" id="submit">Submit</button>
</div>
</form>
</main>
</main>
<footer>
<!-- change to your name-->
<p>By HOMEWORK SOLUTION</p>
<p>By Yonas Gebre</p>
</footer>
</body>
</html>
68 changes: 68 additions & 0 deletions Form-Controls/style.css

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good job that you separated the CSS into its own file—that’s an excellent best practice and will make your code much easier to read, debug, and maintain as it grows.

Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
/* body style */
body {
background-color: lightwhite;
max-width: 800px;
display: flex;
flex-direction: column;
align-items: center;
padding: 25px;
}

header h1 {
color: #2c3e50;
margin-bottom: 15px;
}

form {
background: skyblue;
padding: 30px;
border-radius: 8px;
box-shadow: 0 4px 15px rgba(0, 0, 0, 0.1);
width: 100%;
max-width: 400px;
}
input:focus {
border-color: #3498db;
outline: none;
}
/* Labels and Inputs */
label {
display: block;
margin-top: 15px;
margin-bottom: 5px;
font-weight: 600;
font-size: 0.9rem;
}

input[type="text"],
input[type="email"],
select,button {
align-items: right;
width: 100%;
padding: 10px;
border: 2px solid #ddd;
border-radius: 6px;
box-sizing: border-box; /* Ensures padding doesn't break width */
transition: border-color 0.3s ease;
}

.radio {
display: flex;
align-items: center;
gap: 5px;
font-weight: normal;
}
#submit {
background-color: #3498db;
margin-top: 15px;
border: none;
cursor: pointer;
font-size: 1rem;
}

/* Footer */
footer {
margin-top: 20px;
font-size: 0.8rem;
color:black;
}
Loading