Manchester | 26-ITP-May| Abid Akhtar| Sprint 1| Form Controls#1358
Manchester | 26-ITP-May| Abid Akhtar| Sprint 1| Form Controls#1358AbidAkhtar-tech wants to merge 6 commits into
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
AbidAkhtar-tech
added
the
Needs Review
| <label for="username">Name</label> | ||
| <input type="text" id="username" minlength="2" required autocomplete="name"> |
There was a problem hiding this comment.
Can you update this input element so that it can reject name containing only space characters? For example, " " (a string made up of two space characters)?
Note:
Your branch is one commit behind the upstream repo (CYF's main). If you use the "Sync fork" feature on your repo on GitHub to update your branch, you will find in the README.md of this exercise a regular expression that can be used to enforce "name contains at least two non-space characters".
| <legend>Choose a colour</legend> | ||
|
|
||
| <label for="red">Red</label> | ||
| <input type="radio" id="red" name="colour" value="red" required> |
There was a problem hiding this comment.
Could consider enclosing <input> within <label> as:
<label>Red
<input type="radio" name="colour" value="red" required>
</label>
This way, you don't have to introduce id to every radio button.
| <!-- | ||
| try writing out the requirements first as comments | ||
| this will also help you fill in your PR message later--> |
There was a problem hiding this comment.
Feel free to delete any unnecessary code to keep your code base clean.
cjyuan
added
Reviewed
|
Thank You for the feedback! I will fix the typo. |
2 participants
Self-Checklist
Changes
This PR adds an accessible T-shirt order form using semantic HTML and CSS.