Skip to content

Manchester | Sep ITP 2026 | Salah Al-Sabhi | Sprint 1 | Form Controls - #1476

Open
salahalsabhi wants to merge 2 commits into
CodeYourFuture:mainfrom
salahalsabhi:feature/form-controls
Open

Manchester | Sep ITP 2026 | Salah Al-Sabhi | Sprint 1 | Form Controls#1476
salahalsabhi wants to merge 2 commits into
CodeYourFuture:mainfrom
salahalsabhi:feature/form-controls

Conversation

@salahalsabhi

@salahalsabhi salahalsabhi commented Sep 8, 2026

Copy link
Copy Markdown

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

Task code

CYF-1004

Changelist

Added a t-shirt order form with:

  • required name field with pattern validation
  • required email field
  • 3 colour options using radio buttons
  • 6 size options using a select menu
  • labels, fieldset, and legend for accessibility

Tested in the browser. Lighthouse Accessibility score is 100.

@netlify

netlify Bot commented Sep 8, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit 8a9cf27
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a9fe1d6c8a4df0008972f54
😎 Deploy Preview https://deploy-preview-1476--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
2 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 84 (🔴 down 2 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.

@github-actions

This comment has been minimized.

4 similar comments
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@salahalsabhi salahalsabhi changed the title Feature/form controls Manchester | Sep ITP 2026 | Salah Al-Sabhi | Sprint 1 | Form Controls Sep 8, 2026
@github-actions

This comment has been minimized.

4 similar comments
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@salahalsabhi salahalsabhi added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@salahalsabhi salahalsabhi added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@salahalsabhi

Copy link
Copy Markdown
Author

Hi, I’ve updated the PR title and description for CYF-1004 Form Controls.
Please could someone add the Needs Review label and review when possible?
Thanks!

@salahalsabhi salahalsabhi added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@github-actions

This comment has been minimized.

@github-actions github-actions Bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
@salahalsabhi salahalsabhi added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant