Skip to content

Manchester | 26-ITP-Sep | Aklilu Mihtsun | Sprint 1 | Form Controls - #1499

Open
AkliluMihtsun wants to merge 7 commits into
CodeYourFuture:mainfrom
AkliluMihtsun:feature/form-controls
Open

Manchester | 26-ITP-Sep | Aklilu Mihtsun | Sprint 1 | Form Controls#1499
AkliluMihtsun wants to merge 7 commits into
CodeYourFuture:mainfrom
AkliluMihtsun:feature/form-controls

Conversation

@AkliluMihtsun

Copy link
Copy Markdown

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

Task code

CYF-1004

Changelist

-Used semantic HTML elements to improve the structure and accessibility of the page
-Added a structured customer/product form using

and multiple sections
-Added elements to describe each fieldset
-Added required validation to fields that must be completed
-Added a pattern to ensure the customer's name contains at least two non-space characters
-Added styling for the page layout and form
-Checked the page with Lighthouse and achieved an Accessibility score of 100

Questions

-The "for" attribute in labels does it point to the id or name of the input?
-Why does the required attribute does not have value? Can we put value?

@netlify

netlify Bot commented Sep 11, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-onboarding-module ready!

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant