Manchester | 26-ITP-Jan | Mehroz Munir | Sprint 2 | Form Controls

[MehrozMunir]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Changelist

I have created a form using html to get inputs from the user including their name, email, color and size of the shirt they want. And a submit button.

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	49e0184
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6973cd965a40db0008d76261
😎 Deploy Preview	https://deploy-preview-1030--cyf-onboarding-module.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	2 paths audited Performance: 100 (no change from production) Accessibility: 100 (no change from production) Best Practices: 100 (no change from production) SEO: 86 (no change from production) PWA: - View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[jenny-alexander]

Hi @MehrozMunir, your form controls exercise is done very well. The form validation works nicely and you have used semantic HTML. I think the use of <p> is a little bit excessive but you most likely did this for styling purposes. Since CSS can't be used for styling, I'll let this pass. 🙂

I left a few comments in the code. Please have a look.

[MehrozMunir]

Hi @jenny-alexander
Thank you for reviewing my code and providing valuable feedback.

[jenny-alexander]

Hi @MehrozMunir - I changed the PR back to 'reviewed' since you have not addressed my comments on code changes. The work isn't complete until the reviewer changes the label to 'Complete'!
Here is the CYF guide (check out the Completing a PR section): https://programming.codeyourfuture.io/guides/reviewing/trainee-pr-guide/#addressing-reviewer-feedback