Sheffield | 25-ITP-SEP | Declan Williams | Sprint 2 | Form controls

[CatchingKiyoko]

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

1. added fields for name, email and size and color options
2. added a min requirement of 2 characters for the name so can not be submitted blank
3. added email requirement so can't be submitted blank
4. implemented breaklines around elements so there not close together for accessibility reasons

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	69602b0
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/68ea5569072f5a0008202c73
😎 Deploy Preview	https://deploy-preview-865--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.

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Onboarding) doesn't match expected format (example: 'Sprint 2', without quotes)

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Onboarding) doesn't match expected format (example: 'Sprint 2', without quotes)

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Onboarding) doesn't match expected format (example: 'Sprint 2', without quotes)

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Sprint part (Onboarding) doesn't match expected format (example: 'Sprint 2', without quotes)

[github-actions]

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

[github-actions]

Your PR description contained template fields which weren't filled in.

Check you've ticked everything in the self checklist, and that any sections which prompt you to fill in an answer are either filled in or removed.

[cjyuan]

I noticed some minor improvement (which I missed in the previous review) you can make to further improve the code.

Can you try feeding your code to an AI tool and ask it to review the code?
Then take a look at the suggestions it gives you, and pick out the ones that actually make sense for your project. It's a great way to learn and improve your work.

[CatchingKiyoko]

i've gone through and refactored the code and made inprovements to name patterns also adding action and method attributes to tell the browser when the form is submitted and added aria labels for accessibility

[cjyuan]

Changes look good.