CYF_Glasgow | Mansoor Munawar | Module-Onboarding | WEEK 2 | Form control

[MansoorM11]

Self checklist

• [ ✔️] I have committed my files one by one, on purpose, and for a reason
• [ ✔️] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
• [ ✔️] I have tested my changes
• [ ✔️] My changes follow the style guide
• [ ✔️] My changes meet the requirements of this task

Changelist

Briefly explain your PR.
I created a form which asks for user name, email, size of t-shirt and color of t-shirt.
All of the form elements are required before the form can be submitted.
I have used semantic html elements such as header, main, footer.
My form meets 100% accessibility requirement.

Questions

Ask any questions you have for your reviewer.

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	bf1081d
🔍 Latest deploy log	https://app.netlify.com/sites/cyf-onboarding-module/deploys/67d6fd63cef4f90008c8c7a0
😎 Deploy Preview	https://deploy-preview-253--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: 85 (🔴 down 1 from production) PWA: - View the detailed breakdown and full score reports

---

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

[A-O-Emmanuel]

This is okay, I think you need to write better commit messages, consider reading through this article How to write better git commit messages, a step by step guide. Also I think your html indentation and spacing just needs to be improved a little bit, consider reading through this article How to indent HTML code and why it's important. Good job!