London | 26-ITP-May | Remi M. B. | Module-Onboarding Sprint 1 | Form Controls

[remimarcelle]

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
• I have only used HTML, no CSS or JavaScript
• My form is semantic HTML
• All inputs have associated labels
• My Lighthouse Accessibility score is 100
• I require a valid name (minimum 2 characters)
• I require a valid email
• I require one colour from a defined set of 3 colours
• I require one size from a defined set of 6 sizes
• My code is consistently formatted
• My page content is free of typos
• I commit often and push regularly to GitHub
• My HTML code has no errors or warnings (https://validator.w3.org/)

Changelist

Built a form to collect t-shirt orders using HTML only. It asks for the
customer's full name, email address, t-shirt colour (radio buttons) and
size (dropdown). Wrapped inputs and radio button labels in p tags to add
spacing between touch targets, bringing the Lighthouse accessibility
score from 92 to 100.

Questions

Do I have to screenshot that I used devtools to refactor before I changed the code in vscode?

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	37cfdea
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a065d250271d30008f31c51
😎 Deploy Preview	https://deploy-preview-1258--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.

[remimarcelle]

It seems I accidently committed code from the wireframe task in this pull request? Idk how

[remimarcelle]

remimarcelle/My-Coursework-Planner#16

[Luro91]

Looks good. I have a comment about the code formatting

[Luro91]

Well done.
Thanks for formatting the code. (FYI I also answered your comment here: #1258 (comment))

[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 (Module-Onboarding Sprint 1) doesn't match expected format (example: 'Sprint 2', without quotes)

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above.