Glasgow | May 2026 | Craig Stoddart | Sprint 1 | Form Control

[CraigStodd]

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

HTML

1. Changed header titles to: h1 "T-Shirt Order," h2 "Verify your order and details below"
2. Included Requirements in comments: Added each requirement as a separate comment, with the corresponding code below each comment.
3. Placed form content in a container.
4. Requirement 1: Added "Name" label with input type, id and name specification.
5. Requirement 2: Added "Email label" using the same code from requirement 1 as a template. Changed input type, id and name specifications to "email," "mail" and "User Email, respectively.
6. Requirement 3: created dropdown-select menu using . Only 1 T-shirt colour can be selected.
7. Added Select a colour . This becomes the default title in the dropdown-select menu and cannot be selected as an option.
8. Requirement 4: Created size selector using . Only 1 size can be selected at a time
9. Added "Submit" button
10. Placed every text element in

    tags within each

    to allow further styling via CSS

CSS

1. Added stylesheet "mystyle.css"
2. Added font family specification for

   ,

   elements

3. Added font weight specification for

   elements

4. Added font family + font size specification for

   elements

5. Added container styling: "border-style; border-width; padding; max-width"

Questions

• Have I forked/branched this correctly?

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

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

• One of the requirements has not yet been met: "All fields are required" -- it means the user should not be able to submit the form when any of the input field is missing a value.

• Can you rename your branch to feature/form-controls? (We missed specified the expected branch name in the backlog and I just updated it)

  • Suggestion: Ask AI what's the naming convention for branch name.

• About the Changelist section of the PR description

  • Some of the contents are not formatted properly
  • Because the reviewers already know what the Form-Control exercise is, you can just state in this section that you have implemented an HTML form that meets all the requirements in the Form-Controls exercise. That is, you don't have to list the individual changes you made.

[cjyuan]

Why introduce id attribute to each radio button input?

[CraigStodd]

Hi, I was following the tutorials linked in the curriculum which led me to believe this was necessary.