Glasgow | 26-ITP-May | Francesco Romano Monda | Sprint 1 | Form Controls

[fromonda]

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

• Created a form to collect customer details for a T-shirt order
• Added a required name field with validation to ensure at least two non-space characters
• Added a required email field with built-in email validation
• Added a colour selection using radio buttons (Red, Blue, Green)
• Restricted colour input so only predefined options can be selected
• Added a size dropdown with 6 required options (XS to XXL)
• Ensured all fields are required and validated before submission

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	57b593c
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a184584d4f58f0008c5a8db
😎 Deploy Preview	https://deploy-preview-1353--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
🤖 Make changes	Run an agent on this branch

---

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

[cjyuan]

Can you format the "Changelist" section of the PR description so that it looks like this:

---

Changelist

• Created a form to collect customer details for a T-shirt order
• Added a required name field with validation to ensure at least two non-space characters
• Added a required email field with built-in email validation
• Added a colour selection using radio buttons (Red, Blue, Green)
• Restricted colour input so only predefined options can be selected
• Added a size dropdown with 6 required options (XS to XXL)
• Ensured all fields are required and validated before submission

[cjyuan]

Form looks good.

• Could you improve the Lighthouse Accessibility score (as reported by the browser's Lighthouse dev tool) to 100?

• Can you add a submit button so that we could use it to test the form?

• Your code is well formatted.

  If you haven't yet enabled "Format on save" on VS Code, please take a look at this guide. In long run, it could save you a lot of time.

[cjyuan]

Can you also address this comment: https://github.com/CodeYourFuture/Module-Onboarding/pull/1353#issuecomment-4553570004

[cjyuan]

Changes look good.

Can you also address this comment: https://github.com/CodeYourFuture/Module-Onboarding/pull/1353#issuecomment-4553570004

I will mark this PR as "Complete" first.

[fromonda]

When I click on comment #1353, I am taken directly to the comment you wrote yesterday about changelist formatting. I have followed your suggestion: I used bullet points and just highlighted titles in bold. GitHub does not support changing font size in comments.

[cjyuan]

Check out this: https://github.com/adam-p/markdown-here/wiki/markdown-cheatsheet

Anything you can produce with HTML (without CSS), you can also do with Markdown. You can even mix HTML with Markdown.