London | 26-ITP-Jan | Oussama Mouggal | Sprint 2 | Form controls

[Oussama-Mouggal]

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 completed form controls assignment using html only

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	d703714
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/698223278758bc0008a3e466
😎 Deploy Preview	https://deploy-preview-1117--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]

@Oussama-Mouggal - Your form meets a lot of the exercise requirements. However, there are few things that should be worked on:

• Can you run your deployed site (https://deploy-preview-1117--cyf-onboarding-module.netlify.app/form-controls/) through W3 validatorhttps://validator.w3.org/? I see 2 errors that should be fixed.
• Can you use a different tag than <h2> in your footer? <h2> in the footer signals that there is a major section in the footer but really a name is more minor information.
• The comments from lines 56-67 can most likely be removed how that you have finished the exercise.
• I'm able to submit the form without selecting a tshirt colour.
• Can you find a way to make the tshirt size not automatically default to 'XS' when the page loads?

[Oussama-Mouggal]

Thank you very much Jenny for your valuable feedback. I made the required changes and sync the PR.

[jenny-alexander]

Nicely done 👏