Cape Town | 26-ITP-JAN | Shannon Davids | Sprint 2 | Form Controls #1126
Cape Town | 26-ITP-JAN | Shannon Davids | Sprint 2 | Form Controls #1126Shannon-Davids wants to merge 2 commits intoCodeYourFuture:mainfrom
Conversation
✅ Deploy Preview for cyf-onboarding-module ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Shannon-Davids
added
the
Needs Review
jenny-alexander
added
Review in progress
jenny-alexander
left a comment
jenny-alexander
left a comment
There was a problem hiding this comment.
Hi @Shannon-Davids - good job with the form control exercise 👏. Your form works as intended. I left a few comments for you to review.
💡 In VSCode, you can format your document to make it easier to read. Right-click within the code. This will bring up a menu. You should see 'Format Document'. Click on that to align the code.
| <footer> | ||
| <!-- change to your name--> | ||
| <h2>By HOMEWORK SOLUTION</h2> | ||
| <h2>By Shannon Davids</h2> |
There was a problem hiding this comment.
<h2> announces a major section in the footer. Since your name is the only thing in the footer, a different tag should be used. Can you think of a more appropriate tag?
| <br> | ||
| <div> | ||
| <label for="size">T-shirt Size</label> | ||
| <select name"size" id="size" required> |
There was a problem hiding this comment.
There is a small typo on line 39, can you find it?
|
|
||
| <div> | ||
| <label for="name">Customer Name:</label> | ||
| <input type="text" name="name" id="name" placeholder="Username" required minlength="2"> |
There was a problem hiding this comment.
The label on this input is 'Customer Name' but the placeholder is 'Username'. This might be a bit confusing for a user. Can you use the same wording for both label and placeholder?
jenny-alexander
added
Reviewed
2 participants
Learners, PR Template
Self checklist
Changelist
I have edited form as required with semantic Html
Questions
No Questions